WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
08/17/2018 01:50:26 AM (6 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/custom-background.php

    r42343 r43571  
    222222     */
    223223    public function admin_page() {
    224 ?>
     224        ?>
    225225<div class="wrap" id="custom-background">
    226226<h1><?php _e( 'Custom Background' ); ?></h1>
    227227
    228 <?php if ( current_user_can( 'customize' ) ) { ?>
     228        <?php if ( current_user_can( 'customize' ) ) { ?>
    229229<div class="notice notice-info hide-if-no-customize">
    230230    <p>
    231         <?php
    232         printf(
    233             __( 'You can now manage and live-preview Custom Backgrounds in the <a href="%1$s">Customizer</a>.' ),
    234             admin_url( 'customize.php?autofocus[control]=background_image' )
    235         );
    236         ?>
     231            <?php
     232                printf(
     233                    __( 'You can now manage and live-preview Custom Backgrounds in the <a href="%1$s">Customizer</a>.' ),
     234                    admin_url( 'customize.php?autofocus[control]=background_image' )
     235                );
     236            ?>
    237237    </p>
    238238</div>
    239 <?php } ?>
    240 
    241 <?php if ( ! empty( $this->updated ) ) { ?>
     239        <?php } ?>
     240
     241        <?php if ( ! empty( $this->updated ) ) { ?>
    242242<div id="message" class="updated">
    243243<p><?php printf( __( 'Background updated. <a href="%s">Visit your site</a> to see how it looks.' ), home_url( '/' ) ); ?></p>
    244244</div>
    245 <?php } ?>
     245        <?php } ?>
    246246
    247247<h3><?php _e( 'Background Image' ); ?></h3>
    … …  
    252252<th scope="row"><?php _e( 'Preview' ); ?></th>
    253253<td>
    254     <?php
    255     if ( $this->admin_image_div_callback ) {
    256         call_user_func( $this->admin_image_div_callback );
    257     } else {
    258         $background_styles = '';
    259         if ( $bgcolor = get_background_color() ) {
    260             $background_styles .= 'background-color: #' . $bgcolor . ';';
    261         }
    262 
    263         $background_image_thumb = get_background_image();
    264         if ( $background_image_thumb ) {
    265             $background_image_thumb = esc_url( set_url_scheme( get_theme_mod( 'background_image_thumb', str_replace( '%', '%%', $background_image_thumb ) ) ) );
    266             $background_position_x  = get_theme_mod( 'background_position_x', get_theme_support( 'custom-background', 'default-position-x' ) );
    267             $background_position_y  = get_theme_mod( 'background_position_y', get_theme_support( 'custom-background', 'default-position-y' ) );
    268             $background_size        = get_theme_mod( 'background_size', get_theme_support( 'custom-background', 'default-size' ) );
    269             $background_repeat      = get_theme_mod( 'background_repeat', get_theme_support( 'custom-background', 'default-repeat' ) );
    270             $background_attachment  = get_theme_mod( 'background_attachment', get_theme_support( 'custom-background', 'default-attachment' ) );
    271 
    272             // Background-image URL must be single quote, see below.
    273             $background_styles .= " background-image: url('$background_image_thumb');"
     254        <?php
     255        if ( $this->admin_image_div_callback ) {
     256            call_user_func( $this->admin_image_div_callback );
     257        } else {
     258            $background_styles = '';
     259            if ( $bgcolor = get_background_color() ) {
     260                $background_styles .= 'background-color: #' . $bgcolor . ';';
     261            }
     262
     263            $background_image_thumb = get_background_image();
     264            if ( $background_image_thumb ) {
     265                $background_image_thumb = esc_url( set_url_scheme( get_theme_mod( 'background_image_thumb', str_replace( '%', '%%', $background_image_thumb ) ) ) );
     266                $background_position_x  = get_theme_mod( 'background_position_x', get_theme_support( 'custom-background', 'default-position-x' ) );
     267                $background_position_y  = get_theme_mod( 'background_position_y', get_theme_support( 'custom-background', 'default-position-y' ) );
     268                $background_size        = get_theme_mod( 'background_size', get_theme_support( 'custom-background', 'default-size' ) );
     269                $background_repeat      = get_theme_mod( 'background_repeat', get_theme_support( 'custom-background', 'default-repeat' ) );
     270                $background_attachment  = get_theme_mod( 'background_attachment', get_theme_support( 'custom-background', 'default-attachment' ) );
     271
     272                // Background-image URL must be single quote, see below.
     273                $background_styles .= " background-image: url('$background_image_thumb');"
    274274                . " background-size: $background_size;"
    275275                . " background-position: $background_position_x $background_position_y;"
    276276                . " background-repeat: $background_repeat;"
    277277                . " background-attachment: $background_attachment;";
    278         }
    279     ?>
     278            }
     279            ?>
    280280    <div id="custom-background-image" style="<?php echo $background_styles; ?>"><?php // must be double quote, see above ?>
    281         <?php if ( $background_image_thumb ) { ?>
     281            <?php if ( $background_image_thumb ) { ?>
    282282        <img class="custom-background-image" src="<?php echo $background_image_thumb; ?>" style="visibility:hidden;" alt="" /><br />
    283283        <img class="custom-background-image" src="<?php echo $background_image_thumb; ?>" style="visibility:hidden;" alt="" />
    … …  
    288288</tr>
    289289
    290 <?php if ( get_background_image() ) : ?>
     290        <?php if ( get_background_image() ) : ?>
    291291<tr>
    292292<th scope="row"><?php _e( 'Remove Image' ); ?></th>
    293293<td>
    294294<form method="post">
    295 <?php wp_nonce_field( 'custom-background-remove', '_wpnonce-custom-background-remove' ); ?>
    296 <?php submit_button( __( 'Remove Background Image' ), '', 'remove-background', false ); ?><br/>
    297 <?php _e( 'This will remove the background image. You will not be able to restore any customizations.' ); ?>
     295            <?php wp_nonce_field( 'custom-background-remove', '_wpnonce-custom-background-remove' ); ?>
     296            <?php submit_button( __( 'Remove Background Image' ), '', 'remove-background', false ); ?><br/>
     297            <?php _e( 'This will remove the background image. You will not be able to restore any customizations.' ); ?>
    298298</form>
    299299</td>
    300300</tr>
    301 <?php endif; ?>
    302 
    303 <?php $default_image = get_theme_support( 'custom-background', 'default-image' ); ?>
    304 <?php if ( $default_image && get_background_image() != $default_image ) : ?>
     301        <?php endif; ?>
     302
     303        <?php $default_image = get_theme_support( 'custom-background', 'default-image' ); ?>
     304        <?php if ( $default_image && get_background_image() != $default_image ) : ?>
    305305<tr>
    306306<th scope="row"><?php _e( 'Restore Original Image' ); ?></th>
    307307<td>
    308308<form method="post">
    309 <?php wp_nonce_field( 'custom-background-reset', '_wpnonce-custom-background-reset' ); ?>
    310 <?php submit_button( __( 'Restore Original Image' ), '', 'reset-background', false ); ?><br/>
    311 <?php _e( 'This will restore the original background image. You will not be able to restore any customizations.' ); ?>
     309            <?php wp_nonce_field( 'custom-background-reset', '_wpnonce-custom-background-reset' ); ?>
     310            <?php submit_button( __( 'Restore Original Image' ), '', 'reset-background', false ); ?><br/>
     311            <?php _e( 'This will restore the original background image. You will not be able to restore any customizations.' ); ?>
    312312</form>
    313313</td>
    314314</tr>
    315 <?php endif; ?>
    316 
    317 <?php if ( current_user_can( 'upload_files' ) ) : ?>
     315        <?php endif; ?>
     316
     317        <?php if ( current_user_can( 'upload_files' ) ) : ?>
    318318<tr>
    319319<th scope="row"><?php _e( 'Select Image' ); ?></th>
    … …  
    323323        <input type="file" id="upload" name="import" />
    324324        <input type="hidden" name="action" value="save" />
    325         <?php wp_nonce_field( 'custom-background-upload', '_wpnonce-custom-background-upload' ); ?>
    326         <?php submit_button( __( 'Upload' ), '', 'submit', false ); ?>
     325            <?php wp_nonce_field( 'custom-background-upload', '_wpnonce-custom-background-upload' ); ?>
     326            <?php submit_button( __( 'Upload' ), '', 'submit', false ); ?>
    327327    </p>
    328328    <p>
    … …  
    335335</td>
    336336</tr>
    337 <?php endif; ?>
     337        <?php endif; ?>
    338338</tbody>
    339339</table>
    … …  
    343343<table class="form-table">
    344344<tbody>
    345 <?php if ( get_background_image() ) : ?>
     345        <?php if ( get_background_image() ) : ?>
    346346<input name="background-preset" type="hidden" value="custom">
    347347
    348 <?php
    349 $background_position = sprintf(
    350     '%s %s',
    351     get_theme_mod( 'background_position_x', get_theme_support( 'custom-background', 'default-position-x' ) ),
    352     get_theme_mod( 'background_position_y', get_theme_support( 'custom-background', 'default-position-y' ) )
    353 );
    354 
    355 $background_position_options = array(
    356     array(
    357         'left top'   => array(
    358             'label' => __( 'Top Left' ),
    359             'icon'  => 'dashicons dashicons-arrow-left-alt',
    360         ),
    361         'center top' => array(
    362             'label' => __( 'Top' ),
    363             'icon'  => 'dashicons dashicons-arrow-up-alt',
    364         ),
    365         'right top'  => array(
    366             'label' => __( 'Top Right' ),
    367             'icon'  => 'dashicons dashicons-arrow-right-alt',
    368         ),
    369     ),
    370     array(
    371         'left center'   => array(
    372             'label' => __( 'Left' ),
    373             'icon'  => 'dashicons dashicons-arrow-left-alt',
    374         ),
    375         'center center' => array(
    376             'label' => __( 'Center' ),
    377             'icon'  => 'background-position-center-icon',
    378         ),
    379         'right center'  => array(
    380             'label' => __( 'Right' ),
    381             'icon'  => 'dashicons dashicons-arrow-right-alt',
    382         ),
    383     ),
    384     array(
    385         'left bottom'   => array(
    386             'label' => __( 'Bottom Left' ),
    387             'icon'  => 'dashicons dashicons-arrow-left-alt',
    388         ),
    389         'center bottom' => array(
    390             'label' => __( 'Bottom' ),
    391             'icon'  => 'dashicons dashicons-arrow-down-alt',
    392         ),
    393         'right bottom'  => array(
    394             'label' => __( 'Bottom Right' ),
    395             'icon'  => 'dashicons dashicons-arrow-right-alt',
    396         ),
    397     ),
    398 );
    399 ?>
     348            <?php
     349            $background_position = sprintf(
     350                '%s %s',
     351                get_theme_mod( 'background_position_x', get_theme_support( 'custom-background', 'default-position-x' ) ),
     352                get_theme_mod( 'background_position_y', get_theme_support( 'custom-background', 'default-position-y' ) )
     353            );
     354
     355            $background_position_options = array(
     356                array(
     357                    'left top'   => array(
     358                        'label' => __( 'Top Left' ),
     359                        'icon'  => 'dashicons dashicons-arrow-left-alt',
     360                    ),
     361                    'center top' => array(
     362                        'label' => __( 'Top' ),
     363                        'icon'  => 'dashicons dashicons-arrow-up-alt',
     364                    ),
     365                    'right top'  => array(
     366                        'label' => __( 'Top Right' ),
     367                        'icon'  => 'dashicons dashicons-arrow-right-alt',
     368                    ),
     369                ),
     370                array(
     371                    'left center'   => array(
     372                        'label' => __( 'Left' ),
     373                        'icon'  => 'dashicons dashicons-arrow-left-alt',
     374                    ),
     375                    'center center' => array(
     376                        'label' => __( 'Center' ),
     377                        'icon'  => 'background-position-center-icon',
     378                    ),
     379                    'right center'  => array(
     380                        'label' => __( 'Right' ),
     381                        'icon'  => 'dashicons dashicons-arrow-right-alt',
     382                    ),
     383                ),
     384                array(
     385                    'left bottom'   => array(
     386                        'label' => __( 'Bottom Left' ),
     387                        'icon'  => 'dashicons dashicons-arrow-left-alt',
     388                    ),
     389                    'center bottom' => array(
     390                        'label' => __( 'Bottom' ),
     391                        'icon'  => 'dashicons dashicons-arrow-down-alt',
     392                    ),
     393                    'right bottom'  => array(
     394                        'label' => __( 'Bottom Right' ),
     395                        'icon'  => 'dashicons dashicons-arrow-right-alt',
     396                    ),
     397                ),
     398            );
     399            ?>
    400400<tr>
    401401<th scope="row"><?php _e( 'Image Position' ); ?></th>
    402402<td><fieldset><legend class="screen-reader-text"><span><?php _e( 'Image Position' ); ?></span></legend>
    403403<div class="background-position-control">
    404 <?php foreach ( $background_position_options as $group ) : ?>
     404            <?php foreach ( $background_position_options as $group ) : ?>
    405405    <div class="button-group">
    406     <?php foreach ( $group as $value => $input ) : ?>
     406                <?php foreach ( $group as $value => $input ) : ?>
    407407        <label>
    408408            <input class="screen-reader-text" name="background-position" type="radio" value="<?php echo esc_attr( $value ); ?>"<?php checked( $value, $background_position ); ?>>
    … …  
    447447<th scope="row"><?php _e( 'Background Color' ); ?></th>
    448448<td><fieldset><legend class="screen-reader-text"><span><?php _e( 'Background Color' ); ?></span></legend>
    449 <?php
    450 $default_color = '';
    451 if ( current_theme_supports( 'custom-background', 'default-color' ) ) {
    452     $default_color = ' data-default-color="#' . esc_attr( get_theme_support( 'custom-background', 'default-color' ) ) . '"';
    453 }
    454 ?>
     449        <?php
     450        $default_color = '';
     451        if ( current_theme_supports( 'custom-background', 'default-color' ) ) {
     452            $default_color = ' data-default-color="#' . esc_attr( get_theme_support( 'custom-background', 'default-color' ) ) . '"';
     453        }
     454        ?>
    455455<input type="text" name="background-color" id="background-color" value="#<?php echo esc_attr( get_background_color() ); ?>"<?php echo $default_color; ?>>
    456456</fieldset></td>
    … …  
    459459</table>
    460460
    461 <?php wp_nonce_field( 'custom-background' ); ?>
    462 <?php submit_button( null, 'primary', 'save-background-options' ); ?>
     461        <?php wp_nonce_field( 'custom-background' ); ?>
     462        <?php submit_button( null, 'primary', 'save-background-options' ); ?>
    463463</form>
    464464
    465465</div>
    466 <?php
     466        <?php
    467467    }
    468468
    … …  
    582582        $sizes = array_keys(
    583583            apply_filters(
    584                 'image_size_names_choose', array(
     584                'image_size_names_choose',
     585                array(
    585586                    'thumbnail' => __( 'Thumbnail' ),
    586587                    'medium'    => __( 'Medium' ),
Note: See TracChangeset for help on using the changeset viewer.