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

    r42719 r43571  
    381381})(jQuery);
    382382</script>
    383 <?php
     383        <?php
    384384    }
    385385
    … …  
    391391    public function js_2() {
    392392
    393     ?>
     393        ?>
    394394<script type="text/javascript">
    395395    function onEndCrop( coords ) {
    … …  
    427427            <?php
    428428            if ( ! current_theme_supports( 'custom-header', 'flex-height' ) && ! current_theme_supports( 'custom-header', 'flex-width' ) ) {
    429             ?>
     429                ?>
    430430            aspectRatio: xinit + ':' + yinit,
    431             <?php
     431                <?php
    432432            }
    433433            if ( ! current_theme_supports( 'custom-header', 'flex-height' ) ) {
    434             ?>
     434                ?>
    435435            maxHeight: <?php echo get_theme_support( 'custom-header', 'height' ); ?>,
    436             <?php
     436                <?php
    437437            }
    438438            if ( ! current_theme_supports( 'custom-header', 'flex-width' ) ) {
    439             ?>
     439                ?>
    440440            maxWidth: <?php echo get_theme_support( 'custom-header', 'width' ); ?>,
    441             <?php
     441                <?php
    442442            }
    443443            ?>
    … …  
    455455    });
    456456</script>
    457 <?php
     457        <?php
    458458    }
    459459
    … …  
    465465    public function step_1() {
    466466        $this->process_default_headers();
    467 ?>
     467        ?>
    468468
    469469<div class="wrap">
    470470<h1><?php _e( 'Custom Header' ); ?></h1>
    471471
    472 <?php if ( current_user_can( 'customize' ) ) { ?>
     472        <?php if ( current_user_can( 'customize' ) ) { ?>
    473473<div class="notice notice-info hide-if-no-customize">
    474474    <p>
    475         <?php
    476         printf(
    477             __( 'You can now manage and live-preview Custom Header in the <a href="%1$s">Customizer</a>.' ),
    478             admin_url( 'customize.php?autofocus[control]=header_image' )
    479         );
    480         ?>
     475            <?php
     476                printf(
     477                    __( 'You can now manage and live-preview Custom Header in the <a href="%1$s">Customizer</a>.' ),
     478                    admin_url( 'customize.php?autofocus[control]=header_image' )
     479                );
     480            ?>
    481481    </p>
    482482</div>
    483 <?php } ?>
    484 
    485 <?php if ( ! empty( $this->updated ) ) { ?>
     483        <?php } ?>
     484
     485        <?php if ( ! empty( $this->updated ) ) { ?>
    486486<div id="message" class="updated">
    487487<p><?php printf( __( 'Header updated. <a href="%s">Visit your site</a> to see how it looks.' ), home_url( '/' ) ); ?></p>
    488488</div>
    489 <?php } ?>
     489        <?php } ?>
    490490
    491491<h3><?php _e( 'Header Image' ); ?></h3>
    … …  
    494494<tbody>
    495495
    496 <?php if ( get_custom_header() || display_header_text() ) : ?>
     496        <?php if ( get_custom_header() || display_header_text() ) : ?>
    497497<tr>
    498498<th scope="row"><?php _e( 'Preview' ); ?></th>
    499499<td>
    500     <?php
    501     if ( $this->admin_image_div_callback ) {
    502         call_user_func( $this->admin_image_div_callback );
    503     } else {
    504         $custom_header = get_custom_header();
    505         $header_image  = get_header_image();
    506 
    507         if ( $header_image ) {
    508             $header_image_style = 'background-image:url(' . esc_url( $header_image ) . ');';
    509         } else {
    510             $header_image_style = '';
    511         }
    512 
    513         if ( $custom_header->width ) {
    514             $header_image_style .= 'max-width:' . $custom_header->width . 'px;';
    515         }
    516         if ( $custom_header->height ) {
    517             $header_image_style .= 'height:' . $custom_header->height . 'px;';
    518         }
    519     ?>
     500            <?php
     501            if ( $this->admin_image_div_callback ) {
     502                call_user_func( $this->admin_image_div_callback );
     503            } else {
     504                $custom_header = get_custom_header();
     505                $header_image  = get_header_image();
     506
     507                if ( $header_image ) {
     508                    $header_image_style = 'background-image:url(' . esc_url( $header_image ) . ');';
     509                } else {
     510                    $header_image_style = '';
     511                }
     512
     513                if ( $custom_header->width ) {
     514                    $header_image_style .= 'max-width:' . $custom_header->width . 'px;';
     515                }
     516                if ( $custom_header->height ) {
     517                    $header_image_style .= 'height:' . $custom_header->height . 'px;';
     518                }
     519                ?>
    520520    <div id="headimg" style="<?php echo $header_image_style; ?>">
    521         <?php
    522         if ( display_header_text() ) {
    523             $style = ' style="color:#' . get_header_textcolor() . ';"';
    524         } else {
    525             $style = ' style="display:none;"';
    526         }
    527         ?>
     521                <?php
     522                if ( display_header_text() ) {
     523                    $style = ' style="color:#' . get_header_textcolor() . ';"';
     524                } else {
     525                    $style = ' style="display:none;"';
     526                }
     527                ?>
    528528        <h1><a id="name" class="displaying-header-text" <?php echo $style; ?> onclick="return false;" href="<?php bloginfo( 'url' ); ?>" tabindex="-1"><?php bloginfo( 'name' ); ?></a></h1>
    529529        <div id="desc" class="displaying-header-text" <?php echo $style; ?>><?php bloginfo( 'description' ); ?></div>
    530530    </div>
    531     <?php } ?>
     531            <?php } ?>
    532532</td>
    533533</tr>
    534 <?php endif; ?>
    535 
    536 <?php if ( current_user_can( 'upload_files' ) && current_theme_supports( 'custom-header', 'uploads' ) ) : ?>
     534        <?php endif; ?>
     535
     536        <?php if ( current_user_can( 'upload_files' ) && current_theme_supports( 'custom-header', 'uploads' ) ) : ?>
    537537<tr>
    538538<th scope="row"><?php _e( 'Select Image' ); ?></th>
    539539<td>
    540540    <p><?php _e( 'You can select an image to be shown at the top of your site by uploading from your computer or choosing from your media library. After selecting an image you will be able to crop it.' ); ?><br />
    541     <?php
    542     if ( ! current_theme_supports( 'custom-header', 'flex-height' ) && ! current_theme_supports( 'custom-header', 'flex-width' ) ) {
    543         printf( __( 'Images of exactly <strong>%1$d &times; %2$d pixels</strong> will be used as-is.' ) . '<br />', get_theme_support( 'custom-header', 'width' ), get_theme_support( 'custom-header', 'height' ) );
    544     } elseif ( current_theme_supports( 'custom-header', 'flex-height' ) ) {
    545         if ( ! current_theme_supports( 'custom-header', 'flex-width' ) ) {
    546             printf(
    547                 /* translators: %s: size in pixels */
    548                 __( 'Images should be at least %s wide.' ) . ' ',
    549                 sprintf(
    550                     /* translators: %d: custom header width */
    551                     '<strong>' . __( '%d pixels' ) . '</strong>',
    552                     get_theme_support( 'custom-header', 'width' )
    553                 )
    554             );
    555         }
    556     } elseif ( current_theme_supports( 'custom-header', 'flex-width' ) ) {
    557         if ( ! current_theme_supports( 'custom-header', 'flex-height' ) ) {
    558             printf(
    559                 /* translators: %s: size in pixels */
    560                 __( 'Images should be at least %s tall.' ) . ' ',
    561                 sprintf(
    562                     /* translators: %d: custom header height */
    563                     '<strong>' . __( '%d pixels' ) . '</strong>',
    564                     get_theme_support( 'custom-header', 'height' )
    565                 )
    566             );
    567         }
    568     }
    569     if ( current_theme_supports( 'custom-header', 'flex-height' ) || current_theme_supports( 'custom-header', 'flex-width' ) ) {
    570         if ( current_theme_supports( 'custom-header', 'width' ) ) {
    571             printf(
    572                 /* translators: %s: size in pixels */
    573                 __( 'Suggested width is %s.' ) . ' ',
    574                 sprintf(
    575                     /* translators: %d: custom header width */
    576                     '<strong>' . __( '%d pixels' ) . '</strong>',
    577                     get_theme_support( 'custom-header', 'width' )
    578                 )
    579             );
    580         }
    581         if ( current_theme_supports( 'custom-header', 'height' ) ) {
    582             printf(
    583                 /* translators: %s: size in pixels */
    584                 __( 'Suggested height is %s.' ) . ' ',
    585                 sprintf(
    586                     /* translators: %d: custom header height */
    587                     '<strong>' . __( '%d pixels' ) . '</strong>',
    588                     get_theme_support( 'custom-header', 'height' )
    589                 )
    590             );
    591         }
    592     }
    593     ?>
     541            <?php
     542            if ( ! current_theme_supports( 'custom-header', 'flex-height' ) && ! current_theme_supports( 'custom-header', 'flex-width' ) ) {
     543                printf( __( 'Images of exactly <strong>%1$d &times; %2$d pixels</strong> will be used as-is.' ) . '<br />', get_theme_support( 'custom-header', 'width' ), get_theme_support( 'custom-header', 'height' ) );
     544            } elseif ( current_theme_supports( 'custom-header', 'flex-height' ) ) {
     545                if ( ! current_theme_supports( 'custom-header', 'flex-width' ) ) {
     546                    printf(
     547                        /* translators: %s: size in pixels */
     548                        __( 'Images should be at least %s wide.' ) . ' ',
     549                        sprintf(
     550                            /* translators: %d: custom header width */
     551                            '<strong>' . __( '%d pixels' ) . '</strong>',
     552                            get_theme_support( 'custom-header', 'width' )
     553                        )
     554                    );
     555                }
     556            } elseif ( current_theme_supports( 'custom-header', 'flex-width' ) ) {
     557                if ( ! current_theme_supports( 'custom-header', 'flex-height' ) ) {
     558                    printf(
     559                        /* translators: %s: size in pixels */
     560                        __( 'Images should be at least %s tall.' ) . ' ',
     561                        sprintf(
     562                            /* translators: %d: custom header height */
     563                            '<strong>' . __( '%d pixels' ) . '</strong>',
     564                            get_theme_support( 'custom-header', 'height' )
     565                        )
     566                    );
     567                }
     568            }
     569            if ( current_theme_supports( 'custom-header', 'flex-height' ) || current_theme_supports( 'custom-header', 'flex-width' ) ) {
     570                if ( current_theme_supports( 'custom-header', 'width' ) ) {
     571                    printf(
     572                        /* translators: %s: size in pixels */
     573                        __( 'Suggested width is %s.' ) . ' ',
     574                        sprintf(
     575                            /* translators: %d: custom header width */
     576                            '<strong>' . __( '%d pixels' ) . '</strong>',
     577                            get_theme_support( 'custom-header', 'width' )
     578                        )
     579                    );
     580                }
     581                if ( current_theme_supports( 'custom-header', 'height' ) ) {
     582                    printf(
     583                        /* translators: %s: size in pixels */
     584                        __( 'Suggested height is %s.' ) . ' ',
     585                        sprintf(
     586                            /* translators: %d: custom header height */
     587                            '<strong>' . __( '%d pixels' ) . '</strong>',
     588                            get_theme_support( 'custom-header', 'height' )
     589                        )
     590                    );
     591                }
     592            }
     593            ?>
    594594    </p>
    595595    <form enctype="multipart/form-data" id="upload-form" class="wp-upload-form" method="post" action="<?php echo esc_url( add_query_arg( 'step', 2 ) ); ?>">
    … …  
    598598        <input type="file" id="upload" name="import" />
    599599        <input type="hidden" name="action" value="save" />
    600         <?php wp_nonce_field( 'custom-header-upload', '_wpnonce-custom-header-upload' ); ?>
    601         <?php submit_button( __( 'Upload' ), '', 'submit', false ); ?>
     600            <?php wp_nonce_field( 'custom-header-upload', '_wpnonce-custom-header-upload' ); ?>
     601            <?php submit_button( __( 'Upload' ), '', 'submit', false ); ?>
    602602    </p>
    603     <?php
    604         $modal_update_href = esc_url(
    605             add_query_arg(
    606                 array(
    607                     'page'                          => 'custom-header',
    608                     'step'                          => 2,
    609                     '_wpnonce-custom-header-upload' => wp_create_nonce( 'custom-header-upload' ),
    610                 ), admin_url( 'themes.php' )
    611             )
    612         );
    613     ?>
     603            <?php
     604                $modal_update_href = esc_url(
     605                    add_query_arg(
     606                        array(
     607                            'page' => 'custom-header',
     608                            'step' => 2,
     609                            '_wpnonce-custom-header-upload' => wp_create_nonce( 'custom-header-upload' ),
     610                        ),
     611                        admin_url( 'themes.php' )
     612                    )
     613                );
     614            ?>
    614615    <p>
    615616        <label for="choose-from-library-link"><?php _e( 'Or choose an image from your media library:' ); ?></label><br />
    … …  
    622623</td>
    623624</tr>
    624 <?php endif; ?>
     625        <?php endif; ?>
    625626</tbody>
    626627</table>
    627628
    628629<form method="post" action="<?php echo esc_url( add_query_arg( 'step', 1 ) ); ?>">
    629 <?php submit_button( null, 'screen-reader-text', 'save-header-options', false ); ?>
     630        <?php submit_button( null, 'screen-reader-text', 'save-header-options', false ); ?>
    630631<table class="form-table">
    631632<tbody>
    632     <?php if ( get_uploaded_header_images() ) : ?>
     633        <?php if ( get_uploaded_header_images() ) : ?>
    633634<tr>
    634635<th scope="row"><?php _e( 'Uploaded Images' ); ?></th>
    635636<td>
    636637    <p><?php _e( 'You can choose one of your previously uploaded headers, or show a random one.' ); ?></p>
    637     <?php
    638         $this->show_header_selector( 'uploaded' );
    639     ?>
     638            <?php
     639            $this->show_header_selector( 'uploaded' );
     640            ?>
    640641</td>
    641642</tr>
    642     <?php
     643            <?php
    643644    endif;
    644645if ( ! empty( $this->default_headers ) ) :
    … …  
    647648<th scope="row"><?php _e( 'Default Images' ); ?></th>
    648649<td>
    649 <?php if ( current_theme_supports( 'custom-header', 'uploads' ) ) : ?>
     650    <?php if ( current_theme_supports( 'custom-header', 'uploads' ) ) : ?>
    650651    <p><?php _e( 'If you don&lsquo;t want to upload your own image, you can use one of these cool headers, or show a random one.' ); ?></p>
    651 <?php else : ?>
     652    <?php else : ?>
    652653    <p><?php _e( 'You can use one of these cool headers or show a random one on each page.' ); ?></p>
    653 <?php endif; ?>
     654    <?php endif; ?>
    654655    <?php
    655656    $this->show_header_selector( 'default' );
    … …  
    685686</table>
    686687
    687 <?php if ( current_theme_supports( 'custom-header', 'header-text' ) ) : ?>
     688        <?php if ( current_theme_supports( 'custom-header', 'header-text' ) ) : ?>
    688689
    689690<h3><?php _e( 'Header Text' ); ?></h3>
    … …  
    704705<td>
    705706    <p>
    706     <?php
    707     $default_color = '';
    708     if ( current_theme_supports( 'custom-header', 'default-text-color' ) ) {
    709         $default_color = get_theme_support( 'custom-header', 'default-text-color' );
    710         if ( $default_color && false === strpos( $default_color, '#' ) ) {
    711             $default_color = '#' . $default_color;
    712         }
    713     }
    714 
    715     $default_color_attr = $default_color ? ' data-default-color="' . esc_attr( $default_color ) . '"' : '';
    716 
    717     $header_textcolor = display_header_text() ? get_header_textcolor() : get_theme_support( 'custom-header', 'default-text-color' );
    718     if ( $header_textcolor && false === strpos( $header_textcolor, '#' ) ) {
    719         $header_textcolor = '#' . $header_textcolor;
    720     }
    721 
    722     echo '<input type="text" name="text-color" id="text-color" value="' . esc_attr( $header_textcolor ) . '"' . $default_color_attr . ' />';
    723     if ( $default_color ) {
    724         echo ' <span class="description hide-if-js">' . sprintf( _x( 'Default: %s', 'color' ), esc_html( $default_color ) ) . '</span>';
    725     }
    726     ?>
     707            <?php
     708            $default_color = '';
     709            if ( current_theme_supports( 'custom-header', 'default-text-color' ) ) {
     710                $default_color = get_theme_support( 'custom-header', 'default-text-color' );
     711                if ( $default_color && false === strpos( $default_color, '#' ) ) {
     712                    $default_color = '#' . $default_color;
     713                }
     714            }
     715
     716            $default_color_attr = $default_color ? ' data-default-color="' . esc_attr( $default_color ) . '"' : '';
     717
     718            $header_textcolor = display_header_text() ? get_header_textcolor() : get_theme_support( 'custom-header', 'default-text-color' );
     719            if ( $header_textcolor && false === strpos( $header_textcolor, '#' ) ) {
     720                $header_textcolor = '#' . $header_textcolor;
     721            }
     722
     723            echo '<input type="text" name="text-color" id="text-color" value="' . esc_attr( $header_textcolor ) . '"' . $default_color_attr . ' />';
     724            if ( $default_color ) {
     725                echo ' <span class="description hide-if-js">' . sprintf( _x( 'Default: %s', 'color' ), esc_html( $default_color ) ) . '</span>';
     726            }
     727            ?>
    727728    </p>
    728729</td>
    … …  
    730731</tbody>
    731732</table>
    732 <?php
     733            <?php
    733734endif;
    734735
    735 /**
    736  * Fires just before the submit button in the custom header options form.
    737  *
    738  * @since 3.1.0
    739  */
    740 do_action( 'custom_header_options' );
    741 
    742 wp_nonce_field( 'custom-header-options', '_wpnonce-custom-header-options' );
     736        /**
     737        * Fires just before the submit button in the custom header options form.
     738        *
     739        * @since 3.1.0
     740        */
     741        do_action( 'custom_header_options' );
     742
     743        wp_nonce_field( 'custom-header-options', '_wpnonce-custom-header-options' );
    743744?>
    744745
    745 <?php submit_button( null, 'primary', 'save-header-options' ); ?>
     746        <?php submit_button( null, 'primary', 'save-header-options' ); ?>
    746747</form>
    747748</div>
    748749
    749 <?php
     750        <?php
    750751    }
    751752
    … …  
    853854    <input type="hidden" name="attachment_id" id="attachment_id" value="<?php echo esc_attr( $attachment_id ); ?>" />
    854855    <input type="hidden" name="oitar" id="oitar" value="<?php echo esc_attr( $oitar ); ?>" />
    855     <?php if ( empty( $_POST ) && isset( $_GET['file'] ) ) { ?>
     856        <?php if ( empty( $_POST ) && isset( $_GET['file'] ) ) { ?>
    856857    <input type="hidden" name="create-new-attachment" value="true" />
    857858    <?php } ?>
    858     <?php wp_nonce_field( 'custom-header-crop-image' ); ?>
     859        <?php wp_nonce_field( 'custom-header-crop-image' ); ?>
    859860
    860861    <p class="submit">
    861     <?php submit_button( __( 'Crop and Publish' ), 'primary', 'submit', false ); ?>
    862     <?php
    863     if ( isset( $oitar ) && 1 == $oitar && ( current_theme_supports( 'custom-header', 'flex-height' ) || current_theme_supports( 'custom-header', 'flex-width' ) ) ) {
    864         submit_button( __( 'Skip Cropping, Publish Image as Is' ), '', 'skip-cropping', false );
    865     }
    866     ?>
     862        <?php submit_button( __( 'Crop and Publish' ), 'primary', 'submit', false ); ?>
     863        <?php
     864        if ( isset( $oitar ) && 1 == $oitar && ( current_theme_supports( 'custom-header', 'flex-height' ) || current_theme_supports( 'custom-header', 'flex-width' ) ) ) {
     865            submit_button( __( 'Skip Cropping, Publish Image as Is' ), '', 'skip-cropping', false );
     866        }
     867        ?>
    867868    </p>
    868869</form>
Note: See TracChangeset for help on using the changeset viewer.