Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#27715 closed defect (bug) (fixed)

Hook Docs (35): wp-includes/functions.php

Reported by: GaryJ Owned by: DrewAPicture
Milestone: 3.9 Priority: normal
Severity: normal Version:
Component: General Keywords: has-patch dev-reviewed
Focuses: docs Cc:


Documentation needed for the following hooks:



Attachments (5)

27715.diff (1.5 KB) - added by jesin 7 years ago.
Docs for robots_txt, wp_checkdate and wp_auth_check_load
27715.2.diff (1.5 KB) - added by jesin 7 years ago.
Corrected version of 27715.diff
27715-filters.diff (14.6 KB) - added by GaryJ 7 years ago.
Remaining filters
27715.3.diff (16.8 KB) - added by GaryJ 7 years ago.
Remaining filters and actions
27715.4.diff (18.9 KB) - added by DrewAPicture 7 years ago.
2nd pass.

Download all attachments as: .zip

Change History (14)

7 years ago

Docs for robots_txt, wp_checkdate and wp_auth_check_load

7 years ago

Corrected version of 27715.diff

#1 @GaryJ
7 years ago

I'm working on the other hooks within this file.

#2 @DrewAPicture
7 years ago

  • Owner set to GaryJ
  • Status changed from new to assigned

7 years ago

Remaining filters

#3 @GaryJ
7 years ago

  • Keywords dev-feedback added

27715-filters.diff is first pass at remaining filters.

(First diff created via git, so let me know if it's not compatible.)

7 years ago

Remaining filters and actions

#4 @GaryJ
7 years ago

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

27715.3.diff added which replaces my 27715-filters.diff file as it adds in the action docs as well. Assuming all is well, you'd need to apply 27715.3.diff and jesin's 27715.2.diff to document the all of the hooks in the file.

7 years ago

2nd pass.

#5 @DrewAPicture
7 years ago

  • Milestone changed from Awaiting Review to 3.9
  • Owner changed from GaryJ to DrewAPicture
  • Reporter changed from DrewAPicture to GaryJ
  • Status changed from assigned to reviewing

Hi Gary, thanks for the patches here. I've iterated on your last patch with 27715.4.diff.

Notable changes included standardizing some of the language across similar hooks, removing some jargon, one set of hook docs had a missing parameter, there was a missed filter, and a few other things.

#6 @GaryJ
7 years ago

+1 on those changes Drew.

Line 3330 of your patch has an indentation issue (which came from my patch). Probably accidental spaces instead of a tab in there.

#7 @nacin
7 years ago

I noticed line 3330 as well.

In wp_die(), extra line breaks are added before the start of the documentation blocks. These are not needed if the doc block is indented an additional level beyond the line preceding it (namely, the start of a function or in this case inside if conditions).

That's it, though — good for commit.

#8 @nacin
7 years ago

  • Keywords dev-reviewed added

#9 @DrewAPicture
7 years ago

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

In 28109:

Inline documentation for hooks in wp-includes/functions.php.

Props jesin, GaryJ.
Fixes #27715.

Note: See TracTickets for help on using tickets.