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-includes/media-template.php

    r42624 r43571  
    1616function wp_underscore_audio_template() {
    1717    $audio_types = wp_get_audio_extensions();
    18 ?>
     18    ?>
    1919<audio style="visibility: hidden"
    2020    controls
    … …  
    2525    <?php
    2626    foreach ( array( 'autoplay', 'loop' ) as $attr ) :
    27     ?>
     27        ?>
    2828    if ( ! _.isUndefined( data.model.<?php echo $attr; ?> ) && data.model.<?php echo $attr; ?> ) {
    2929        #> <?php echo $attr; ?><#
    … …  
    3737    <?php
    3838    foreach ( $audio_types as $type ) :
    39     ?>
     39        ?>
    4040    <# if ( ! _.isEmpty( data.model.<?php echo $type; ?> ) ) { #>
    4141    <source src="{{ data.model.<?php echo $type; ?> }}" type="{{ wp.media.view.settings.embedMimes[ '<?php echo $type; ?>' ] }}" />
    4242    <# } #>
     43        <?php
     44    endforeach;
     45    ?>
     46</audio>
    4347    <?php
    44     endforeach;
    45 ?>
    46 </audio>
    47 <?php
    4848}
    4949
    … …  
    5656function wp_underscore_video_template() {
    5757    $video_types = wp_get_video_extensions();
    58 ?>
     58    ?>
    5959<#  var w_rule = '', classes = [],
    6060        w, h, settings = wp.media.view.settings,
    … …  
    9797    <# if ( h ) { #>height="{{ h }}"<# } #>
    9898    <?php
    99     $props       = array(
     99    $props = array(
    100100        'poster'  => '',
    101101        'preload' => 'metadata',
    … …  
    103103    foreach ( $props as $key => $value ) :
    104104        if ( empty( $value ) ) {
    105         ?>
     105            ?>
    106106        <#
    107107        if ( ! _.isUndefined( data.model.<?php echo $key; ?> ) && data.model.<?php echo $key; ?> ) {
    108108            #> <?php echo $key; ?>="{{ data.model.<?php echo $key; ?> }}"<#
    109109        } #>
    110         <?php
     110            <?php
    111111        } else {
    112112            echo $key
    … …  
    120120    <?php
    121121    foreach ( array( 'autoplay', 'loop' ) as $attr ) :
    122     ?>
     122        ?>
    123123    if ( ! _.isUndefined( data.model.<?php echo $attr; ?> ) && data.model.<?php echo $attr; ?> ) {
    124124        #> <?php echo $attr; ?><#
    … …  
    138138    <?php
    139139    foreach ( $video_types as $type ) :
    140     ?>
     140        ?>
    141141    <# if ( data.model.<?php echo $type; ?> ) { #>
    142142    <source src="{{ data.model.<?php echo $type; ?> }}" type="{{ settings.embedMimes[ '<?php echo $type; ?>' ] }}" />
    … …  
    146146</video>
    147147</div>
    148 <?php
     148    <?php
    149149}
    150150
    … …  
    414414                    'album'  => __( 'Album' ),
    415415                ) as $key => $label ) :
    416                 ?>
     416                    ?>
    417417                <label class="setting" data-setting="<?php echo esc_attr( $key ); ?>">
    418418                    <span class="name"><?php echo $label; ?></span>
    … …  
    597597            'album'  => __( 'Album' ),
    598598        ) as $key => $label ) :
    599         ?>
     599            ?>
    600600        <label class="setting" data-setting="<?php echo esc_attr( $key ); ?>">
    601601            <span class="name"><?php echo $label; ?></span>
    … …  
    720720                    /** This filter is documented in wp-admin/includes/media.php */
    721721                    $sizes = apply_filters(
    722                         'image_size_names_choose', array(
     722                        'image_size_names_choose',
     723                        array(
    723724                            'thumbnail' => __( 'Thumbnail' ),
    724725                            'medium'    => __( 'Medium' ),
    … …  
    729730
    730731                    foreach ( $sizes as $value => $name ) :
    731                     ?>
     732                        ?>
    732733                        <#
    733734                        var size = data.sizes['<?php echo esc_js( $value ); ?>'];
    … …  
    798799                /** This filter is documented in wp-admin/includes/media.php */
    799800                $size_names = apply_filters(
    800                     'image_size_names_choose', array(
     801                    'image_size_names_choose',
     802                    array(
    801803                        'thumbnail' => __( 'Thumbnail' ),
    802804                        'medium'    => __( 'Medium' ),
    … …  
    807809
    808810                foreach ( $size_names as $size => $label ) :
    809                 ?>
     811                    ?>
    810812                    <option value="<?php echo esc_attr( $size ); ?>">
    811813                        <?php echo esc_html( $label ); ?>
    … …  
    868870        /** This filter is documented in wp-admin/includes/media.php */
    869871        if ( ! apply_filters( 'disable_captions', '' ) ) :
    870         ?>
     872            ?>
    871873            <label class="setting caption">
    872874                <span><?php _e( 'Caption' ); ?></span>
    … …  
    934936                    /** This filter is documented in wp-admin/includes/media.php */
    935937                    if ( ! apply_filters( 'disable_captions', '' ) ) :
    936                     ?>
     938                        ?>
    937939                        <label class="setting caption">
    938940                            <span><?php _e( 'Caption' ); ?></span>
    … …  
    977979                                    /** This filter is documented in wp-admin/includes/media.php */
    978980                                    $sizes = apply_filters(
    979                                         'image_size_names_choose', array(
     981                                        'image_size_names_choose',
     982                                        array(
    980983                                            'thumbnail' => __( 'Thumbnail' ),
    981984                                            'medium'    => __( 'Medium' ),
    … …  
    986989
    987990                                    foreach ( $sizes as $value => $name ) :
    988                                     ?>
     991                                        ?>
    989992                                        <#
    990993                                        var size = data.sizes['<?php echo esc_js( $value ); ?>'];
    … …  
    10941097
    10951098                foreach ( $audio_types as $type ) :
    1096                 ?>
     1099                    ?>
    10971100                <# if ( ! _.isEmpty( data.model.<?php echo $type; ?> ) ) {
    10981101                    if ( ! _.isUndefined( html5types.<?php echo $type; ?> ) ) {
    … …  
    11771180                <?php
    11781181                foreach ( $video_types as $type ) :
    1179                 ?>
     1182                    ?>
    11801183                <# if ( ! _.isEmpty( data.model.<?php echo $type; ?> ) ) {
    11811184                    if ( ! _.isUndefined( html5types.<?php echo $type; ?> ) ) {
Note: See TracChangeset for help on using the changeset viewer.