WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (7 years ago)
Author:
pento
Message:

Code is Poetry.
WordPress' code just... wasn't.
This is now dealt with.

Props jrf, pento, netweb, GaryJ, jdgrimes, westonruter, Greg Sherwood from PHPCS, and everyone who's ever contributed to WPCS and PHPCS.
Fixes #41057.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-admin/includes/class-walker-nav-menu-edit.php

    r41688 r42343  
    6161
    6262        ob_start();
    63         $item_id = esc_attr( $item->ID );
     63        $item_id      = esc_attr( $item->ID );
    6464        $removed_args = array(
    6565            'action',
    … …  
    7474        if ( 'taxonomy' == $item->type ) {
    7575            $original_title = get_term_field( 'name', $item->object_id, $item->object, 'raw' );
    76             if ( is_wp_error( $original_title ) )
     76            if ( is_wp_error( $original_title ) ) {
    7777                $original_title = false;
     78            }
    7879        } elseif ( 'post_type' == $item->type ) {
    7980            $original_object = get_post( $item->object_id );
    80             $original_title = get_the_title( $original_object->ID );
     81            $original_title  = get_the_title( $original_object->ID );
    8182        } elseif ( 'post_type_archive' == $item->type ) {
    8283            $original_object = get_post_type_object( $item->object );
    … …  
    8990            'menu-item menu-item-depth-' . $depth,
    9091            'menu-item-' . esc_attr( $item->object ),
    91             'menu-item-edit-' . ( ( isset( $_GET['edit-menu-item'] ) && $item_id == $_GET['edit-menu-item'] ) ? 'active' : 'inactive'),
     92            'menu-item-edit-' . ( ( isset( $_GET['edit-menu-item'] ) && $item_id == $_GET['edit-menu-item'] ) ? 'active' : 'inactive' ),
    9293        );
    9394
    … …  
    101102            $classes[] = 'pending';
    102103            /* translators: %s: title of menu item in draft status */
    103             $title = sprintf( __('%s (Pending)'), $item->title );
     104            $title = sprintf( __( '%s (Pending)' ), $item->title );
    104105        }
    105106
    … …  
    107108
    108109        $submenu_text = '';
    109         if ( 0 == $depth )
     110        if ( 0 == $depth ) {
    110111            $submenu_text = 'style="display: none;"';
     112        }
    111113
    112114        ?>
    113         <li id="menu-item-<?php echo $item_id; ?>" class="<?php echo implode(' ', $classes ); ?>">
     115        <li id="menu-item-<?php echo $item_id; ?>" class="<?php echo implode( ' ', $classes ); ?>">
    114116            <div class="menu-item-bar">
    115117                <div class="menu-item-handle">
    … …  
    118120                        <span class="item-type"><?php echo esc_html( $item->type_label ); ?></span>
    119121                        <span class="item-order hide-if-js">
    120                             <a href="<?php
     122                            <a href="
     123                            <?php
    121124                                echo wp_nonce_url(
    122125                                    add_query_arg(
    123126                                        array(
    124                                             'action' => 'move-up-menu-item',
     127                                            'action'    => 'move-up-menu-item',
    125128                                            'menu-item' => $item_id,
    126129                                        ),
    127                                         remove_query_arg($removed_args, admin_url( 'nav-menus.php' ) )
     130                                        remove_query_arg( $removed_args, admin_url( 'nav-menus.php' ) )
    128131                                    ),
    129132                                    'move-menu_item'
    130133                                );
    131                             ?>" class="item-move-up" aria-label="<?php esc_attr_e( 'Move up' ) ?>">&#8593;</a>
     134                            ?>
     135                            " class="item-move-up" aria-label="<?php esc_attr_e( 'Move up' ); ?>">&#8593;</a>
    132136                            |
    133                             <a href="<?php
     137                            <a href="
     138                            <?php
    134139                                echo wp_nonce_url(
    135140                                    add_query_arg(
    136141                                        array(
    137                                             'action' => 'move-down-menu-item',
     142                                            'action'    => 'move-down-menu-item',
    138143                                            'menu-item' => $item_id,
    139144                                        ),
    140                                         remove_query_arg($removed_args, admin_url( 'nav-menus.php' ) )
     145                                        remove_query_arg( $removed_args, admin_url( 'nav-menus.php' ) )
    141146                                    ),
    142147                                    'move-menu_item'
    143148                                );
    144                             ?>" class="item-move-down" aria-label="<?php esc_attr_e( 'Move down' ) ?>">&#8595;</a>
     149                            ?>
     150                            " class="item-move-down" aria-label="<?php esc_attr_e( 'Move down' ); ?>">&#8595;</a>
    145151                        </span>
    146                         <a class="item-edit" id="edit-<?php echo $item_id; ?>" href="<?php
    147                             echo ( isset( $_GET['edit-menu-item'] ) && $item_id == $_GET['edit-menu-item'] ) ? admin_url( 'nav-menus.php' ) : add_query_arg( 'edit-menu-item', $item_id, remove_query_arg( $removed_args, admin_url( 'nav-menus.php#menu-item-settings-' . $item_id ) ) );
    148                         ?>" aria-label="<?php esc_attr_e( 'Edit menu item' ); ?>"><span class="screen-reader-text"><?php _e( 'Edit' ); ?></span></a>
     152                        <a class="item-edit" id="edit-<?php echo $item_id; ?>" href="
     153                                                                    <?php
     154                                                                    echo ( isset( $_GET['edit-menu-item'] ) && $item_id == $_GET['edit-menu-item'] ) ? admin_url( 'nav-menus.php' ) : add_query_arg( 'edit-menu-item', $item_id, remove_query_arg( $removed_args, admin_url( 'nav-menus.php#menu-item-settings-' . $item_id ) ) );
     155                        ?>
     156                        " aria-label="<?php esc_attr_e( 'Edit menu item' ); ?>"><span class="screen-reader-text"><?php _e( 'Edit' ); ?></span></a>
    149157                    </span>
    150158                </div>
    … …  
    181189                    <label for="edit-menu-item-classes-<?php echo $item_id; ?>">
    182190                        <?php _e( 'CSS Classes (optional)' ); ?><br />
    183                         <input type="text" id="edit-menu-item-classes-<?php echo $item_id; ?>" class="widefat code edit-menu-item-classes" name="menu-item-classes[<?php echo $item_id; ?>]" value="<?php echo esc_attr( implode(' ', $item->classes ) ); ?>" />
     191                        <input type="text" id="edit-menu-item-classes-<?php echo $item_id; ?>" class="widefat code edit-menu-item-classes" name="menu-item-classes[<?php echo $item_id; ?>]" value="<?php echo esc_attr( implode( ' ', $item->classes ) ); ?>" />
    184192                    </label>
    185193                </p>
    … …  
    194202                        <?php _e( 'Description' ); ?><br />
    195203                        <textarea id="edit-menu-item-description-<?php echo $item_id; ?>" class="widefat edit-menu-item-description" rows="3" cols="20" name="menu-item-description[<?php echo $item_id; ?>]"><?php echo esc_html( $item->description ); // textarea_escaped ?></textarea>
    196                         <span class="description"><?php _e('The description will be displayed in the menu if the current theme supports it.'); ?></span>
     204                        <span class="description"><?php _e( 'The description will be displayed in the menu if the current theme supports it.' ); ?></span>
    197205                    </label>
    198206                </p>
    … …  
    210218                    <?php if ( 'custom' != $item->type && $original_title !== false ) : ?>
    211219                        <p class="link-to-original">
    212                             <?php printf( __('Original: %s'), '<a href="' . esc_attr( $item->url ) . '">' . esc_html( $original_title ) . '</a>' ); ?>
     220                            <?php printf( __( 'Original: %s' ), '<a href="' . esc_attr( $item->url ) . '">' . esc_html( $original_title ) . '</a>' ); ?>
    213221                        </p>
    214222                    <?php endif; ?>
    215                     <a class="item-delete submitdelete deletion" id="delete-<?php echo $item_id; ?>" href="<?php
    216                     echo wp_nonce_url(
     223                    <a class="item-delete submitdelete deletion" id="delete-<?php echo $item_id; ?>" href="
     224                                                                                        <?php
     225                                                                                        echo wp_nonce_url(
     226                                                                                            add_query_arg(
     227                                                                                                array(
     228                                                                                                    'action'    => 'delete-menu-item',
     229                                                                                                    'menu-item' => $item_id,
     230                                                                                                ),
     231                                                                                                admin_url( 'nav-menus.php' )
     232                                                                                            ),
     233                                                                                            'delete-menu_item_' . $item_id
     234                                                                                        );
     235                    ?>
     236                    "><?php _e( 'Remove' ); ?></a> <span class="meta-sep hide-if-no-js"> | </span> <a class="item-cancel submitcancel hide-if-no-js" id="cancel-<?php echo $item_id; ?>" href="
     237                    <?php
     238                    echo esc_url(
    217239                        add_query_arg(
    218240                            array(
    219                                 'action' => 'delete-menu-item',
    220                                 'menu-item' => $item_id,
    221                             ),
    222                             admin_url( 'nav-menus.php' )
    223                         ),
    224                         'delete-menu_item_' . $item_id
    225                     ); ?>"><?php _e( 'Remove' ); ?></a> <span class="meta-sep hide-if-no-js"> | </span> <a class="item-cancel submitcancel hide-if-no-js" id="cancel-<?php echo $item_id; ?>" href="<?php echo esc_url( add_query_arg( array( 'edit-menu-item' => $item_id, 'cancel' => time() ), admin_url( 'nav-menus.php' ) ) );
    226                         ?>#menu-item-settings-<?php echo $item_id; ?>"><?php _e('Cancel'); ?></a>
     241                                'edit-menu-item' => $item_id,
     242                                'cancel'         => time(),
     243                            ), admin_url( 'nav-menus.php' )
     244                        )
     245                    );
     246                        ?>
     247                        #menu-item-settings-<?php echo $item_id; ?>"><?php _e( 'Cancel' ); ?></a>
    227248                </div>
    228249
Note: See TracChangeset for help on using the changeset viewer.