[wp-trac] [WordPress Trac] #53059: Coding Standards: Give some variables in WP_Widget_Pages a more meaningful name

WordPress Trac noreply at wordpress.org
Wed Apr 21 18:17:41 UTC 2021


#53059: Coding Standards: Give some variables in WP_Widget_Pages a more meaningful
name
------------------------------+-------------------------------
 Reporter:  mukesh27          |       Owner:  davidbaumwald
     Type:  enhancement       |      Status:  assigned
 Priority:  normal            |   Milestone:  5.8
Component:  Widgets           |     Version:
 Severity:  normal            |  Resolution:
 Keywords:  has-patch commit  |     Focuses:  coding-standards
------------------------------+-------------------------------

Comment (by desrosj):

 Thanks so much for this ticket and patch, @mukesh27!

 In the future when reporting coding standards issues like this, please try
 to review entire files at a time and detail which files you reviewed when
 you discovered the issue. For example, I just reviewed the other classes
 in `wp-includes/widgets` for this same issue, and it seems the only
 occurrence is in `WP_Widget_Pages`.

 Changes like this are also really great for adding the `good-first-bug`
 keyword to. It's important to leave some easier tickets for contributors
 just getting involved with WordPress so that there are tickets they can
 feel confident jumping into!

 Also, every release has a coding standards improvement task. For 5.8, it's
 #52627. If there are small, one off changes like this, they can be posted
 on the task ticket for the current release. It helps group all of the
 coding standards related changes in one place for future reference.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/53059#comment:5>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list