Opened 11 years ago
Closed 11 years ago
#25826 closed defect (bug) (fixed)
Hook Docs (26): wp-includes/meta.php
Reported by: | raamdev | Owned by: | DrewAPicture |
---|---|---|---|
Milestone: | 3.9 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Options, Meta APIs | Keywords: | has-patch commit |
Focuses: | docs | Cc: |
Description
Adding inline docs to hooks in wp-includes/meta.php.
Attachments (5)
Change History (19)
#4
@
11 years ago
I read somewhere that I should be using the --no-prefix
option when creating diffs with git, so I've updated the patch. Let me know if anything else needs changing.
#6
@
11 years ago
- Summary changed from Hook Docs: wp-includes/meta.php to Hook Docs (26): wp-includes/meta.php
I'll get a review done on 25826.2.diff during our sprint tomorrow.
#9
@
11 years ago
- Owner changed from kpdesign to DrewAPicture
25826.3.diff cleans up descriptions, fixes some @since values, and other changes.
@todo: Hash notation for get_meta_sql
needs to be completed.
#10
@
11 years ago
- Keywords commit added
- Milestone changed from Awaiting Review to 3.9
25826.4.diff should do it.
Note: See
TracTickets for help on using
tickets.
Added patch for
do_action()
andapply_filters()
hooks.There were a few things I wasn't sure about regarding the the
bool|null
@params. For example, on line 164, did I document the filter correctly?Finally, the
apply_filters_ref_array()
filter on line 1104 still needs to be documented. I need help with that one.