get_children() should not be considered internal

[SergeyBiryukov]

get_children()​ is currently marked as @internal in the inline docs (since [8203]).

I think that is why it's missing from the developer reference (originally reported by ​@gmazzap).

I don't see a reason for that, it has a ​Codex article and should not be considered internal.

[DrewAPicture]

This is an issue with the parser, and should be handled on meta trac/GitHub. The perser simply looks for @interal tags of any kind, and the usage here is actually valid. We should really be using @ignore to skip generation of refernece pages.

[DrewAPicture]

Opened a ​pull request on the parser repo, and ​#meta812.

[DrewAPicture]

After talking to @SergeyBiryukov a bit about this, we should simply remove the @internal tag and re-parse the reference for 4.1.1. We can always do an audit later to convert other uses of @internal in core to @ignore as they apply.

30987.diff​

[DrewAPicture]

In 31169:

Remove an erroneous @internal tag from the DocBlock for get_children().

Use of the @internal tag caused the reference article for this function to be skipped during the parsing process.

Fixes #30987 for trunk.

[DrewAPicture]

Reopening for 4.1.1 consideration.

[DrewAPicture]

In 31170:

Ensure we're using the correct @ignore phpDocumentor tag to mark elements that should be skipped when parsing.

Up to this point, various core elements' DocBlocks incorrectly included an @internal tag as a means for skipping the parsing process. When paired with a description (inline or otherwise), @internal is a valid tag meant to provide internal-only context, but not necessarily to skip parsing the entire element.

See #30987.

[DrewAPicture]

Once the source is re-parsed and the parser changes merged, references will then be generated for (the currently-missing) get_children(), force_balance_tags(), and the_meta().