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-screen.php

    r42871 r43571  
    807807                                </a>
    808808                            </li>
    809                         <?php
     809                            <?php
    810810                            $class = '';
    811811                        endforeach;
     
    838838                                ?>
    839839                            </div>
    840                         <?php
     840                            <?php
    841841                            $classes = 'help-tab-content';
    842842                        endforeach;
     
    891891            <button type="button" id="contextual-help-link" class="button show-settings" aria-controls="contextual-help-wrap" aria-expanded="false"><?php _e( 'Help' ); ?></button>
    892892            </div>
    893         <?php
     893            <?php
    894894        endif;
    895895if ( $this->show_screen_options() ) :
    896         ?>
     896    ?>
    897897            <div id="screen-options-link-wrap" class="hide-if-no-js screen-meta-toggle">
    898898            <button type="button" id="show-settings-link" class="button show-settings" aria-controls="screen-options-wrap" aria-expanded="false"><?php _e( 'Screen Options' ); ?></button>
     
    969969    public function render_screen_options( $options = array() ) {
    970970        $options = wp_parse_args(
    971             $options, array(
     971            $options,
     972            array(
    972973                'wrap' => true,
    973974            )
     
    11221123                                                    <label class="columns-prefs-<?php echo $i; ?>">
    11231124                <input type='radio' name='screen_columns' value='<?php echo esc_attr( $i ); ?>'
    1124                     <?php checked( $screen_layout_columns, $i ); ?> />
    1125                 <?php printf( _n( '%s column', '%s columns', $i ), number_format_i18n( $i ) ); ?>
     1125                                                    <?php checked( $screen_layout_columns, $i ); ?> />
     1126                                                    <?php printf( _n( '%s column', '%s columns', $i ), number_format_i18n( $i ) ); ?>
    11261127                </label>
    1127                 <?php
     1128                                                    <?php
    11281129            endfor;
    1129             ?>
     1130                                                ?>
    11301131        </fieldset>
    11311132        <?php
     
    12361237        // This needs a submit button
    12371238        add_filter( 'screen_options_show_submit', '__return_true' );
    1238 ?>
     1239        ?>
    12391240        <fieldset class="metabox-prefs view-mode">
    12401241        <legend><?php _e( 'View Mode' ); ?></legend>
     
    12481249                </label>
    12491250        </fieldset>
    1250 <?php
     1251        <?php
    12511252    }
    12521253
Note: See TracChangeset for help on using the changeset viewer.