Make WordPress Core

Opened 11 years ago

Closed 10 years ago

Last modified 10 years ago

#26196 closed defect (bug) (fixed)

Hook Docs (2): wp-includes/class-wp-customize-section.php

Reported by: leewillis77's profile leewillis77 Owned by: kpdesign's profile kpdesign
Milestone: 3.9 Priority: normal
Severity: normal Version:
Component: Customize Keywords: has-patch
Focuses: docs Cc:

Description

Patch in development

Attachments (4)

26196.diff (2.7 KB) - added by leewillis77 11 years ago.
Hook docs patch
26196.2.diff (944 bytes) - added by leewillis77 11 years ago.
Updated to remove non-hook docs, and other changes suggested.
26196.3.diff (945 bytes) - added by kpdesign 11 years ago.
Minor tweaks
26196.4.diff (924 bytes) - added by DrewAPicture 10 years ago.
Final pass

Download all attachments as: .zip

Change History (13)

@leewillis77
11 years ago

Hook docs patch

#1 @leewillis77
11 years ago

  • Keywords has-patch added

#2 @kpdesign
11 years ago

  • Keywords needs-patch added; has-patch removed
  • Owner set to kpdesign
  • Status changed from new to reviewing

@leewillis77: Thanks for the patch. :)

Just a few things:

General:

  • Your patch contains functional doc and hook doc changes. We are only doing hook docs in these tickets. Please remove the functional doc changes from this patch, and submit those changes in a separate patch on a new ticket, with the subject "PHPDoc updates for wp-includes/class-wp-customize-section.php".

customize_render_section:

  • Change the short description to read: "Fires before rendering the supported Customizer sections."
  • Add a blank line between the @since and the @param line, per the example doc block in the documentation standard.

'customize_render_section_' . $this->id:

  • Change the short description to read: "Fires before rendering each supported Customizer section."
  • Add a long description, such as: "The dynamic portion of the hook name, $this->id, refers to the ID of the supported Customizer section to be rendered."
  • Remove the @param line. $this->id is part of the hook name.
  • Change the do_action line to: do_action( "customize_render_section_{$this->id}" );

Could you make the above changes, and submit a new patch please? :)

Note: Please don't overwrite the original patch when uploading a new one – it is preferred to have copies of all attachments submitted to preserve the history of the ticket.

@leewillis77
11 years ago

Updated to remove non-hook docs, and other changes suggested.

#3 @leewillis77
11 years ago

  • Keywords has-patch added; needs-patch removed

#4 @DrewAPicture
11 years ago

  • Summary changed from Hook docs: wp-includes/class-wp-customize-section.php to Hook Docs (2): wp-includes/class-wp-customize-section.php

26196.2.diff looks to be in order. We'll get this reviewed today and hopefully wrap it up.

@kpdesign
11 years ago

Minor tweaks

#5 @kpdesign
11 years ago

26196.3.diff contains a few minor tweaks.

Ready for a second review and a recommendation.

#6 @DrewAPicture
10 years ago

  • Milestone changed from Awaiting Review to 3.9

@DrewAPicture
10 years ago

Final pass

#7 @DrewAPicture
10 years ago

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

In 26892:

Inline documentation for hooks in wp-includes/class-wp-customize-section.php.

Props leewillis77, kpdesign.
Fixes #26196.

#8 @DrewAPicture
10 years ago

  • Focuses docs added

#9 @DrewAPicture
10 years ago

  • Component changed from Inline Docs to Appearance
Note: See TracTickets for help on using tickets.