Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#48346 closed defect (bug) (fixed)

WP_Customize_Section::__construct() is missing `$args` documentation

Reported by: marekdedic Owned by: SergeyBiryukov
Priority: normal Milestone: 5.4
Component: Customize Version: 5.2
Severity: minor Keywords: good-first-bug has-patch needs-refresh
Cc: Focuses: docs

Description

Documentation for the fields of the $args argument is missing:

https://developer.wordpress.org/reference/classes/wp_customize_section/__construct/

I believe a good starting point would be the documentation of WP_Customize_Manager::add_section():

https://developer.wordpress.org/reference/classes/wp_customize_manager/add_section/

Although I am not sure about the correctness of what's in there. But from the implementation of add_panel, I'd say they should match.

As per https://make.wordpress.org/core/handbook/best-practices/inline-documentation-standards/php/#1-1-parameters-that-are-arrays, there should only be a @see comment in add_section and the documentation should be in the constructor I believe.

Attachments (2)

48346.patch (3.6 KB ) - added by amolv 7 years ago.
48346.1.patch (3.6 KB ) - added by amolv 7 years ago.

Download all attachments as: .zip

Change History (11)

#1 @marekdedic
7 years ago

Also, I believe the type of theme_supports could be narrowed from string|array to string|string[] ;)

#2 @johnbillion
7 years ago

  • Keywords needs-patch good-first-bug added

#3 @desrosj
7 years ago

  • Milestone Awaiting Review5.4

@amolv
7 years ago

@amolv
7 years ago

#4 @amolv
7 years ago

  • Keywords has-patch added; needs-patch removed

48346.1.patch to add documetation to WP_Customize_Section::__construct() and reference to WP_Customize_Manager::add_section.

Last edited 7 years ago by SergeyBiryukov (previous) (diff)

This ticket was mentioned in Slack in #core by sergey. View the logs.


7 years ago

#6 @SergeyBiryukov
7 years ago

  • Owner set to SergeyBiryukov
  • Status newreviewing

This ticket was mentioned in Slack in #core by mike. View the logs.


7 years ago

#8 @kirasong
7 years ago

  • Keywords needs-refresh added

We took a look at this in triage, and it looks like the patch no longer applies, so marking it as needs-refresh.

#9 @SergeyBiryukov
7 years ago

  • Resolutionfixed
  • Status reviewingclosed

In 47386:

Docs: Add a reference to WP_Customize_Section::__construct() for information on accepted arguments in WP_Customize_Manager::add_section().

Synchronize the documentation between two places, use WP_Customize_Section::__construct() as the canonical source.

Props amolv, marekdedic.
Fixes #48346.

Note: See TracTickets for help on using tickets.