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)
Change History (11)
#4
@
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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Also, I believe the type of theme_supports could be narrowed from
string|arraytostring|string[];)