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/includes/class-wp-list-table.php

    r43019 r43571  
    138138    public function __construct( $args = array() ) {
    139139        $args = wp_parse_args(
    140             $args, array(
     140            $args,
     141            array(
    141142                'plural'   => '',
    142143                'singular' => '',
     
    275276    protected function set_pagination_args( $args ) {
    276277        $args = wp_parse_args(
    277             $args, array(
     278            $args,
     279            array(
    278280                'total_items' => 0,
    279281                'total_pages' => 0,
     
    361363            echo '<input type="hidden" name="detached" value="' . esc_attr( $_REQUEST['detached'] ) . '" />';
    362364        }
    363 ?>
     365        ?>
    364366<p class="search-box">
    365367    <label class="screen-reader-text" for="<?php echo esc_attr( $input_id ); ?>"><?php echo $text; ?>:</label>
    366368    <input type="search" id="<?php echo esc_attr( $input_id ); ?>" name="s" value="<?php _admin_search_query(); ?>" />
    367     <?php submit_button( $text, '', '', false, array( 'id' => 'search-submit' ) ); ?>
     369        <?php submit_button( $text, '', '', false, array( 'id' => 'search-submit' ) ); ?>
    368370</p>
    369 <?php
     371        <?php
    370372    }
    371373
     
    569571            $extra_checks
    570572            ORDER BY post_date DESC
    571         ", $post_type
     573        ",
     574                $post_type
    572575            )
    573576        );
     
    590593
    591594        $m = isset( $_GET['m'] ) ? (int) $_GET['m'] : 0;
    592 ?>
     595        ?>
    593596        <label for="filter-by-date" class="screen-reader-text"><?php _e( 'Filter by date' ); ?></label>
    594597        <select name="m" id="filter-by-date">
    595598            <option<?php selected( $m, 0 ); ?> value="0"><?php _e( 'All dates' ); ?></option>
    596 <?php
    597 foreach ( $months as $arc_row ) {
    598     if ( 0 == $arc_row->year ) {
    599         continue;
    600     }
    601 
    602     $month = zeroise( $arc_row->month, 2 );
    603     $year  = $arc_row->year;
    604 
    605     printf(
    606         "<option %s value='%s'>%s</option>\n",
    607         selected( $m, $year . $month, false ),
    608         esc_attr( $arc_row->year . $month ),
    609         /* translators: 1: month name, 2: 4-digit year */
    610         sprintf( __( '%1$s %2$d' ), $wp_locale->get_month( $month ), $year )
    611     );
    612 }
    613 ?>
     599        <?php
     600        foreach ( $months as $arc_row ) {
     601            if ( 0 == $arc_row->year ) {
     602                continue;
     603            }
     604
     605            $month = zeroise( $arc_row->month, 2 );
     606            $year  = $arc_row->year;
     607
     608            printf(
     609                "<option %s value='%s'>%s</option>\n",
     610                selected( $m, $year . $month, false ),
     611                esc_attr( $arc_row->year . $month ),
     612                /* translators: 1: month name, 2: 4-digit year */
     613                sprintf( __( '%1$s %2$d' ), $wp_locale->get_month( $month ), $year )
     614            );
     615        }
     616        ?>
    614617        </select>
    615 <?php
     618        <?php
    616619    }
    617620
     
    624627     */
    625628    protected function view_switcher( $current_mode ) {
    626 ?>
     629        ?>
    627630        <input type="hidden" name="mode" value="<?php echo esc_attr( $current_mode ); ?>" />
    628631        <div class="view-switch">
    629 <?php
    630 foreach ( $this->modes as $mode => $title ) {
    631     $classes = array( 'view-' . $mode );
    632     if ( $current_mode === $mode ) {
    633         $classes[] = 'current';
    634     }
    635     printf(
    636         "<a href='%s' class='%s' id='view-switch-$mode'><span class='screen-reader-text'>%s</span></a>\n",
    637         esc_url( add_query_arg( 'mode', $mode ) ),
    638         implode( ' ', $classes ),
    639         $title
    640     );
    641 }
     632        <?php
     633        foreach ( $this->modes as $mode => $title ) {
     634            $classes = array( 'view-' . $mode );
     635            if ( $current_mode === $mode ) {
     636                $classes[] = 'current';
     637            }
     638            printf(
     639                "<a href='%s' class='%s' id='view-switch-$mode'><span class='screen-reader-text'>%s</span></a>\n",
     640                esc_url( add_query_arg( 'mode', $mode ) ),
     641                implode( ' ', $classes ),
     642                $title
     643            );
     644        }
    642645        ?>
    643646        </div>
    644 <?php
     647        <?php
    645648    }
    646649
     
    678681                            'p'              => $post_id,
    679682                            'comment_status' => 'approved',
    680                         ), admin_url( 'edit-comments.php' )
     683                        ),
     684                        admin_url( 'edit-comments.php' )
    681685                    )
    682686                ),
     
    700704                            'p'              => $post_id,
    701705                            'comment_status' => 'moderated',
    702                         ), admin_url( 'edit-comments.php' )
     706                        ),
     707                        admin_url( 'edit-comments.php' )
    703708                    )
    704709                ),
     
    11531158
    11541159        $this->screen->render_screen_reader_content( 'heading_list' );
    1155 ?>
     1160        ?>
    11561161<table class="wp-list-table <?php echo implode( ' ', $this->get_table_classes() ); ?>">
    11571162    <thead>
     
    11621167
    11631168    <tbody id="the-list"
    1164     <?php
    1165     if ( $singular ) {
    1166         echo " data-wp-lists='list:$singular'";
    1167     }
     1169        <?php
     1170        if ( $singular ) {
     1171            echo " data-wp-lists='list:$singular'";
     1172        }
    11681173        ?>
    11691174        >
     
    11781183
    11791184</table>
    1180 <?php
     1185        <?php
    11811186        $this->display_tablenav( 'bottom' );
    11821187    }
     
    12101215            <?php $this->bulk_actions( $which ); ?>
    12111216        </div>
    1212         <?php
     1217            <?php
    12131218        endif;
    12141219        $this->extra_tablenav( $which );
     
    12181223        <br class="clear" />
    12191224    </div>
    1220 <?php
     1225        <?php
    12211226    }
    12221227
Note: See TracChangeset for help on using the changeset viewer.