Opened 8 years ago
Closed 8 years ago
#30987 closed defect (bug) (fixed)
get_children() should not be considered internal
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 4.2 | Priority: | normal |
Severity: | normal | Version: | 2.6 |
Component: | Posts, Post Types | Keywords: | has-patch commit |
Focuses: | docs | Cc: |
Description
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.
Attachments (1)
Change History (13)
This ticket was mentioned in Slack in #meta-devhub by sergeybiryukov. View the logs.
8 years ago
#3
@
8 years ago
Opened a pull request on the parser repo, and #meta812.
This ticket was mentioned in Slack in #core by drew. View the logs.
8 years ago
#5
@
8 years ago
- Keywords has-patch commit added
- Milestone changed from WordPress.org to 4.1.1
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.
#6
@
8 years ago
- Owner set to DrewAPicture
- Resolution set to fixed
- Status changed from new to closed
In 31169:
#7
@
8 years ago
- Keywords fixed-major added
- Resolution fixed deleted
- Status changed from closed to reopened
Reopening for 4.1.1 consideration.
#9
@
8 years ago
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()
.
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.