[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