[wp-trac] [WordPress Trac] #55647: Coding Standards fixes for WP 6.1

WordPress Trac noreply at wordpress.org
Mon Sep 5 17:17:36 UTC 2022


#55647: Coding Standards fixes for WP 6.1
----------------------------+-------------------------------
 Reporter:  costdev         |       Owner:  (none)
     Type:  task (blessed)  |      Status:  new
 Priority:  normal          |   Milestone:  6.1
Component:  General         |     Version:
 Severity:  normal          |  Resolution:
 Keywords:  has-patch       |     Focuses:  coding-standards
----------------------------+-------------------------------

Comment (by SergeyBiryukov):

 In [changeset:"54071" 54071]:
 {{{
 #!CommitTicketReference repository="" revision="54071"
 Coding Standards: Use more meaningful variable names for output in the
 admin.

 This renames some variables for clarity, per the
 [https://developer.wordpress.org/coding-standards/wordpress-coding-
 standards/php/#naming-conventions Naming Conventions]:
 > Don’t abbreviate variable names unnecessarily; let the code be
 unambiguous and self-documenting.

 * `$out` is renamed to `$output` in various list table methods and admin
 functions.
 * `$sep` is renamed to `$separator` in various list table methods and
 admin functions.

 This affects:
 * `WP_Comments_List_Table::handle_row_actions()`
 * `WP_List_Table::row_actions()`
 * `WP_Media_List_Table::column_default()`
 * `WP_MS_Sites_List_Table::site_states()`
 * `WP_MS_Users_List_Table::column_blogs()`
 * `WP_Terms_List_Table::column_name()`
 * `_wp_dashboard_recent_comments_row()`
 * `image_align_input_fields()`
 * `image_size_input_fields()`
 * `wp_doc_link_parse()`
 * `_post_states()`
 * `_media_states()`

 Follow-up to [8653], [8692], [8864], [8910], [8911], [8916], [9103],
 [9153], [10607], [15491], [17793], [32644], [54070].

 Props mukesh27, costdev.
 See #56448, #55647.
 }}}

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


More information about the wp-trac mailing list