Changeset 54071 for trunk/src/wp-admin/includes/class-wp-comments-list-table.php

Timestamp:

09/05/2022 05:17:21 PM (4 years ago)

Author:

SergeyBiryukov

Message:

Coding Standards: Use more meaningful variable names for output in the admin.

This renames some variables for clarity, per the ​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.

File:

• trunk/src/wp-admin/includes/class-wp-comments-list-table.php (modified) (4 diffs)

trunk/src/wp-admin/includes/class-wp-comments-list-table.php

@@ -678 +678 @@
                 $the_comment_status = wp_get_comment_status( $comment );
 
-                $out = '';
+                $output = '';
 
                 $del_nonce     = esc_html( '_wpnonce=' . wp_create_nonce( "delete-comment_$comment->comment_ID" ) );
@@ -833 +833 @@
                 }
 
-                $out .= '<div class="' . ( $always_visible ? 'row-actions visible' : 'row-actions' ) . '">';
+                $output .= '<div class="' . ( $always_visible ? 'row-actions visible' : 'row-actions' ) . '">';
 
                 $i = 0;
@@ -843 +843 @@
                                 || 1 === $i
                         ) {
-                                $sep = '';
+                                $separator = '';
                         } else {
-                                $sep = ' | ';
+                                $separator = ' | ';
                         }
 
@@ -861 +861 @@
                         }
 
-                        $out .= "<span class='$action'>$sep$link</span>";
-                }
-
-                $out .= '</div>';
-
-                $out .= '<button type="button" class="toggle-row"><span class="screen-reader-text">' . __( 'Show more details' ) . '</span></button>';
-
-                return $out;
+                        $output .= "<span class='$action'>{$separator}{$link}</span>";
+                }
+
+                $output .= '</div>';
+
+                $output .= '<button type="button" class="toggle-row"><span class="screen-reader-text">' . __( 'Show more details' ) . '</span></button>';
+
+                return $output;
         }