Make WordPress Core

Changeset 53195


Ignore:
Timestamp:
04/17/2022 02:39:15 PM (2 years ago)
Author:
SergeyBiryukov
Message:

Coding Standards: Use more descriptive variable names in add_menu_classes().

Follow-up to [9578], [53193], [53194].

See #54728.

File:
1 edited

Legend:

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

    r53194 r53195  
    215215 */
    216216function add_menu_classes( $menu ) {
    217     $first     = false;
    218     $lastorder = false;
    219     $i         = 0;
    220     $mc        = count( $menu );
     217    $first_item  = false;
     218    $last_order  = false;
     219    $items_count = count( $menu );
     220    $i           = 0;
     221
    221222    foreach ( $menu as $order => $top ) {
    222223        $i++;
     
    224225        if ( 0 == $order ) { // Dashboard is always shown/single.
    225226            $menu[0][4] = add_cssclass( 'menu-top-first', $top[4] );
    226             $lastorder = 0;
     227            $last_order = 0;
    227228            continue;
    228229        }
    229230
    230         if ( 0 === strpos( $top[2], 'separator' ) && false !== $lastorder ) { // If separator.
    231             $first                 = true;
    232             $c                     = $menu[ $lastorder ][4];
    233             $menu[ $lastorder ][4] = add_cssclass( 'menu-top-last', $c );
     231        if ( 0 === strpos( $top[2], 'separator' ) && false !== $last_order ) { // If separator.
     232            $first_item             = true;
     233            $classes                = $menu[ $last_order ][4];
     234            $menu[ $last_order ][4] = add_cssclass( 'menu-top-last', $classes );
    234235            continue;
    235236        }
    236237
    237         if ( $first ) {
    238             $c                 = $menu[ $order ][4];
    239             $menu[ $order ][4] = add_cssclass( 'menu-top-first', $c );
    240             $first             = false;
    241         }
    242 
    243         if ( $mc == $i ) { // Last item.
    244             $c                 = $menu[ $order ][4];
    245             $menu[ $order ][4] = add_cssclass( 'menu-top-last', $c );
    246         }
    247 
    248         $lastorder = $order;
     238        if ( $first_item ) {
     239            $classes           = $menu[ $order ][4];
     240            $menu[ $order ][4] = add_cssclass( 'menu-top-first', $classes );
     241            $first_item        = false;
     242        }
     243
     244        if ( $i == items_count ) { // Last item.
     245            $classes           = $menu[ $order ][4];
     246            $menu[ $order ][4] = add_cssclass( 'menu-top-last', $classes );
     247        }
     248
     249        $last_order = $order;
    249250    }
    250251
Note: See TracChangeset for help on using the changeset viewer.