WordPress.org

Make WordPress Core

Opened 7 weeks ago

Last modified 6 weeks ago

#48348 new defect (bug)

WP_Customize_Color_Control::__construct should link to parent constructor for $args

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

Description

Currently, the fields of the $args argument are undocumented in WP_Customize_Color_Control::__construct(). I believe there should be a @see to the parent Ctor as per the handbook.

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

https://make.wordpress.org/core/handbook/best-practices/inline-documentation-standards/php/#1-1-parameters-that-are-arrays

Attachments (2)

48348.patch (5.9 KB) - added by Marcio Zebedeu 7 weeks ago.
adding documentation
48348.1.patch (6.1 KB) - added by Marcio Zebedeu 7 weeks ago.
adding documentation

Download all attachments as: .zip

Change History (7)

#1 @marekdedic
7 weeks ago

From what I can tell, this also hold for other controls, such as WP_Customize_Image_Control, WP_Customize_Media_Control, or WP_Customize_Upload_Control...

#2 @johnbillion
7 weeks ago

  • Keywords needs-patch good-first-bug added

#3 @desrosj
7 weeks ago

  • Milestone changed from Awaiting Review to 5.4

@Marcio Zebedeu
7 weeks ago

adding documentation

@Marcio Zebedeu
7 weeks ago

adding documentation

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


6 weeks ago

#5 @SergeyBiryukov
6 weeks ago

  • Keywords has-patch added; needs-patch removed
Note: See TracTickets for help on using tickets.