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-includes/theme-compat/sidebar.php

    r42343 r43571  
    2121              /* Widgetized sidebar, if you have the plugin installed. */
    2222            if ( ! function_exists( 'dynamic_sidebar' ) || ! dynamic_sidebar() ) :
    23                     ?>
     23                ?>
    2424            <li>
    2525                <?php get_search_form(); ?>
     
    3232            -->
    3333
    34             <?php
    35             if ( is_404() || is_category() || is_day() || is_month() ||
     34                <?php
     35                if ( is_404() || is_category() || is_day() || is_month() ||
    3636                is_year() || is_search() || is_paged() ) :
    37             ?>
     37                    ?>
    3838             <li>
    3939
    40             <?php if ( is_404() ) : /* If this is a 404 page */ ?>
     40                    <?php if ( is_404() ) : /* If this is a 404 page */ ?>
    4141            <?php elseif ( is_category() ) : /* If this is a category archive */ ?>
    4242                <p>
     
    112112
    113113            </li>
    114         <?php endif; ?>
     114            <?php endif; ?>
    115115        </ul>
    116116        <ul role="navigation">
    117             <?php wp_list_pages( 'title_li=<h2>' . __( 'Pages' ) . '</h2>' ); ?>
     117                <?php wp_list_pages( 'title_li=<h2>' . __( 'Pages' ) . '</h2>' ); ?>
    118118
    119119            <li><h2><?php _e( 'Archives' ); ?></h2>
     
    123123            </li>
    124124
    125             <?php
    126             wp_list_categories(
    127                 array(
    128                     'show_count' => 1,
    129                     'title_li'   => '<h2>' . __( 'Categories' ) . '</h2>',
    130                 )
    131             );
    132 ?>
     125                <?php
     126                wp_list_categories(
     127                    array(
     128                        'show_count' => 1,
     129                        'title_li'   => '<h2>' . __( 'Categories' ) . '</h2>',
     130                    )
     131                );
     132                ?>
    133133        </ul>
    134134        <ul>
    135             <?php if ( is_home() || is_page() ) { /* If this is the frontpage */ ?>
    136                 <?php wp_list_bookmarks(); ?>
     135                <?php if ( is_home() || is_page() ) { /* If this is the frontpage */ ?>
     136                    <?php wp_list_bookmarks(); ?>
    137137
    138138                <li><h2><?php _e( 'Meta' ); ?></h2>
Note: See TracChangeset for help on using the changeset viewer.