Make WordPress Core


Ignore:
Timestamp:
09/05/2022 05:17:21 PM (21 months 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:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-admin/includes/template.php

    r53877 r54071  
    21692169            ++$i;
    21702170
    2171             $sep = ( $i < $state_count ) ? ', ' : '';
    2172 
    2173             $post_states_string .= "<span class='post-state'>$state$sep</span>";
     2171            $separator = ( $i < $state_count ) ? ', ' : '';
     2172
     2173            $post_states_string .= "<span class='post-state'>{$state}{$separator}</span>";
    21742174        }
    21752175    }
     
    22832283            ++$i;
    22842284
    2285             $sep = ( $i < $state_count ) ? ', ' : '';
    2286 
    2287             $media_states_string .= "<span class='post-state'>$state$sep</span>";
     2285            $separator = ( $i < $state_count ) ? ', ' : '';
     2286
     2287            $media_states_string .= "<span class='post-state'>{$state}{$separator}</span>";
    22882288        }
    22892289    }
Note: See TracChangeset for help on using the changeset viewer.