Make WordPress Core

#26089 closed defect (bug) (fixed)

Hook Docs (22): wp-admin/includes/template.php

Reported by: shelob9 Owned by: kpdesign
Milestone: 3.9 Priority: normal
Severity: normal Version:
Component: General Keywords: has-patch commit
Focuses: docs, administration Cc:


Will do this today at WordCamp Orlando contributor day.

Attachments (6)

26089.diff (4.4 KB) - added by shelob9 17 months ago.
Inline docs for includes/template.php
26089.2.diff (10.4 KB) - added by shelob9 17 months ago.
Inline docs for all filters and hooks in admin/includes/template.php
26089.3.diff (9.8 KB) - added by kpdesign 16 months ago.
First pass on 26089.2.diff
26089.4.diff (9.1 KB) - added by DrewAPicture 15 months ago.
2nd pass
26089.5.diff (9.2 KB) - added by kpdesign 15 months ago.
Final pass
26089.6.diff (9.7 KB) - added by DrewAPicture 14 months ago.

Download all attachments as: .zip

Change History (16)

@shelob917 months ago

Inline docs for includes/template.php

comment:1 @shelob917 months ago

  • Keywords has-patch added

All actions documented. Filters documented up to line 90. I will continue working on this soon.

@shelob917 months ago

Inline docs for all filters and hooks in admin/includes/template.php

comment:2 @shelob917 months ago

All filters and hooks now have inline docs. Ready for review.

comment:3 @DrewAPicture17 months ago

  • Keywords needs-patch added; has-patch removed
  • Owner set to kpdesign
  • Status changed from new to reviewing
  • Summary changed from Hook Docs: wp-admin/includes/template.php to Hook Docs (22): wp-admin/includes/template.php

Hi, thanks for the patch. It looks like you've mixed in functional docs changes which should be removed from this patch. If you wish to improve on those functional docs, feel to open a separate ticket containing those changes.

From looking at 26089.2.diff, looks like there are some spacing and wrapping issues, missing parameter docs, etc. Could do with a full review and a new patch.

@kpdesign16 months ago

First pass on 26089.2.diff

comment:4 @kpdesign16 months ago

  • Keywords has-patch docs-feedback added; needs-patch removed

26089.3.diff removes the functional docs changes, fixes formatting issues, documents all duplicate hooks, and fully documents other blocks.

comment:5 @jeremyfelt15 months ago

  • Component changed from Inline Docs to Template
  • Focuses docs added

comment:6 @nacin15 months ago

  • Component changed from Template to General
  • Focuses administration added

Sorry for the shuffling.

@DrewAPicture15 months ago

2nd pass

comment:7 @DrewAPicture15 months ago

  • Keywords docs-feedback removed

26089.4.diff is a second pass. Clarifies some language and cleans up spacing in a few places. One more once-over should do it them we can check this one in. Needs a review and recommendation.

@kpdesign15 months ago

Final pass

comment:8 @kpdesign15 months ago

  • Keywords commit added

26089.5.diff contains a few minor spacing fixes for a couple blocks in 26089.4.diff.

This one's ready to go in. Recommend commit.

@DrewAPicture14 months ago


comment:9 @DrewAPicture14 months ago

  • Milestone changed from Awaiting Review to 3.9

26089.6.diff should do it.

comment:10 @DrewAPicture14 months ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 27237:

Inline documentation for hooks in wp-admin/includes/template.php.

Props shelob9 for the initial patch. Props kpdesign.
Fixes #26089.

Note: See TracTickets for help on using tickets.