WordPress.org

Make WordPress Core


Ignore:
Timestamp:
08/17/2018 01:50:26 AM (3 years ago)
Author:
pento
Message:

Coding Standards: Upgrade WPCS to 1.0.0

WPCS 1.0.0 includes a bunch of new auto-fixers, which drops the number of coding standards issues across WordPress significantly. Prior to running the auto-fixers, there were 15,312 issues detected. With this commit, we now drop to 4,769 issues.

This change includes three notable additions:

  • Multiline function calls must now put each parameter on a new line.
  • Auto-formatting files is now part of the grunt precommit script.
  • Auto-fixable coding standards issues will now cause Travis failures.

Fixes #44600.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-admin/nav-menus.php

    r43569 r43571  
    612612                        array( 'autofocus' => $focus ),
    613613                        'return' => urlencode( remove_query_arg( wp_removable_query_args(), wp_unslash( $_SERVER['REQUEST_URI'] ) ) ),
    614                     ), admin_url( 'customize.php' )
     614                    ),
     615                    admin_url( 'customize.php' )
    615616                )
    616617            ),
     
    653654            echo '<p>' . sprintf( _n( 'Your theme supports %s menu. Select which menu appears in each location.', 'Your theme supports %s menus. Select which menu appears in each location.', $num_locations ), number_format_i18n( $num_locations ) ) . '</p>';
    654655        }
    655     ?>
     656        ?>
    656657    <div id="menu-locations-wrap">
    657658        <form method="post" action="<?php echo esc_url( add_query_arg( array( 'action' => 'locations' ), admin_url( 'nav-menus.php' ) ) ); ?>">
     
    693694                                                'action' => 'edit',
    694695                                                'menu'   => $menu_locations[ $_location ],
    695                                             ), admin_url( 'nav-menus.php' )
     696                                            ),
     697                                            admin_url( 'nav-menus.php' )
    696698                                        )
    697699                                    );
     
    711713                                                'menu'   => 0,
    712714                                                'use-location' => $_location,
    713                                             ), admin_url( 'nav-menus.php' )
     715                                            ),
     716                                            admin_url( 'nav-menus.php' )
    714717                                        )
    715718                                    );
     
    730733        </form>
    731734    </div><!-- #menu-locations-wrap -->
    732     <?php
    733     /**
    734     * Fires after the menu locations table is displayed.
    735     *
    736     * @since 3.6.0
    737     */
    738     do_action( 'after_menu_locations_table' );
    739     ?>
     735        <?php
     736        /**
     737        * Fires after the menu locations table is displayed.
     738        *
     739        * @since 3.6.0
     740        */
     741        do_action( 'after_menu_locations_table' );
     742        ?>
    740743    <?php else : ?>
    741744    <div class="manage-menus">
     
    744747            <?php
    745748            printf(
    746                 __( 'Edit your menu below, or <a href="%s">create a new menu</a>. Don&#8217;t forget to save your changes!' ), esc_url(
     749                __( 'Edit your menu below, or <a href="%s">create a new menu</a>. Don&#8217;t forget to save your changes!' ),
     750                esc_url(
    747751                    add_query_arg(
    748752                        array(
    749753                            'action' => 'edit',
    750754                            'menu'   => 0,
    751                         ), admin_url( 'nav-menus.php' )
     755                        ),
     756                        admin_url( 'nav-menus.php' )
    752757                    )
    753758                )
    754759            );
    755 ?>
     760            ?>
    756761            <span class="screen-reader-text"><?php _e( 'Click the Save Menu button to save your changes.' ); ?></span>
    757762        </span><!-- /add-edit-menu-action -->
     
    803808                <?php
    804809                printf(
    805                     __( 'or <a href="%s">create a new menu</a>. Don&#8217;t forget to save your changes!' ), esc_url(
     810                    __( 'or <a href="%s">create a new menu</a>. Don&#8217;t forget to save your changes!' ),
     811                    esc_url(
    806812                        add_query_arg(
    807813                            array(
    808814                                'action' => 'edit',
    809815                                'menu'   => 0,
    810                             ), admin_url( 'nav-menus.php' )
     816                            ),
     817                            admin_url( 'nav-menus.php' )
    811818                        )
    812819                    )
    813820                );
    814 ?>
     821                ?>
    815822                <span class="screen-reader-text"><?php _e( 'Click the Save Menu button to save your changes.' ); ?></span>
    816823            </span><!-- /add-new-menu-action -->
     
    820827
    821828        $metabox_holder_disabled_class = '';
    822         if ( isset( $_GET['menu'] ) && '0' == $_GET['menu'] ) {
    823             $metabox_holder_disabled_class = ' metabox-holder-disabled';
    824         }
    825     ?>
     829if ( isset( $_GET['menu'] ) && '0' == $_GET['menu'] ) {
     830    $metabox_holder_disabled_class = ' metabox-holder-disabled';
     831}
     832?>
    826833    </div><!-- /manage-menus -->
    827834    <div id="nav-menus-frame" class="wp-clearfix">
     
    844851            <?php
    845852                $new_screen_class = '';
    846                 if ( $add_new_screen ) {
    847                     $new_screen_class = 'blank-slate';
    848                 }
     853            if ( $add_new_screen ) {
     854                $new_screen_class = 'blank-slate';
     855            }
    849856            ?>
    850857                <h2><?php _e( 'Menu structure' ); ?></h2>
     
    860867                    if ( $one_theme_location_no_menus ) {
    861868                        $menu_name_val = 'value="' . esc_attr( 'Menu 1' ) . '"';
    862                     ?>
     869                        ?>
    863870                        <input type="hidden" name="zero-menu-state" value="true" />
    864                     <?php
     871                        <?php
    865872                    } else {
    866873                        $menu_name_val = 'value="' . esc_attr( $nav_menu_selected_title ) . '"';
     
    881888                        <div id="post-body-content" class="wp-clearfix">
    882889                            <?php if ( ! $add_new_screen ) : ?>
    883                             <?php
     890                                <?php
    884891                                $hide_style = '';
    885                             if ( isset( $menu_items ) && 0 == count( $menu_items ) ) {
    886                                 $hide_style = 'style="display: none;"';
    887                             }
     892                                if ( isset( $menu_items ) && 0 == count( $menu_items ) ) {
     893                                    $hide_style = 'style="display: none;"';
     894                                }
    888895                                $starter_copy = ( $one_theme_location_no_menus ) ? __( 'Edit your default menu by adding or removing items. Drag each item into the order you prefer. Click Create Menu to save your changes.' ) : __( 'Drag each item into the order you prefer. Click the arrow on the right of the item to reveal additional configuration options.' );
    889                             ?>
     896                                ?>
    890897                            <div class="drag-instructions post-body-plain" <?php echo $hide_style; ?>>
    891898                                <p><?php echo $starter_copy; ?></p>
    892899                            </div>
    893                             <?php
    894                             if ( isset( $edit_markup ) && ! is_wp_error( $edit_markup ) ) {
    895                                 echo $edit_markup;
    896                             } else {
    897                             ?>
     900                                <?php
     901                                if ( isset( $edit_markup ) && ! is_wp_error( $edit_markup ) ) {
     902                                    echo $edit_markup;
     903                                } else {
     904                                    ?>
    898905                            <ul class="menu" id="menu-to-edit"></ul>
    899                             <?php } ?>
     906                                <?php } ?>
    900907                            <?php endif; ?>
    901908                            <?php if ( $add_new_screen ) : ?>
     
    904911                                    <input type="hidden" name="use-location" value="<?php echo esc_attr( $_GET['use-location'] ); ?>" />
    905912                                <?php endif; ?>
    906                             <?php
     913                                <?php
    907914                                endif;
    908915
    909916                                $no_menus_style = '';
    910                                 if ( $one_theme_location_no_menus ) {
    911                                     $no_menus_style = 'style="display: none;"';
    912                                 }
    913                             ?>
     917if ( $one_theme_location_no_menus ) {
     918    $no_menus_style = 'style="display: none;"';
     919}
     920?>
    914921                            <div class="menu-settings" <?php echo $no_menus_style; ?>>
    915922                                <h3><?php _e( 'Menu Settings' ); ?></h3>
     
    974981                                                'action' => 'delete',
    975982                                                'menu'   => $nav_menu_selected_id,
    976                                             ), admin_url( 'nav-menus.php' )
    977                                         ), 'delete-nav_menu-' . $nav_menu_selected_id
     983                                            ),
     984                                            admin_url( 'nav-menus.php' )
     985                                        ),
     986                                        'delete-nav_menu-' . $nav_menu_selected_id
    978987                                    )
    979988                                );
Note: See TracChangeset for help on using the changeset viewer.