[wp-trac] [WordPress Trac] #56448: Use meaningful variable name for $c in WP_Users_List_Table class

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


#56448: Use meaningful variable name for $c in WP_Users_List_Table class
-------------------------+---------------------
 Reporter:  burhandodhy  |       Owner:  (none)
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  6.1
Component:  Users        |     Version:
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |     Focuses:
-------------------------+---------------------

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/56448#comment:11>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list