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-content/themes/twentyeleven/functions.php

    r42343 r43571  
    108108        // Add support for custom backgrounds.
    109109        add_theme_support(
    110             'custom-background', array(
     110            'custom-background',
     111            array(
    111112                /*
    112113                * Let WordPress know what our default background color is.
     
    259260        // Has the text been hidden?
    260261        if ( 'blank' == $text_color ) :
    261     ?>
     262            ?>
    262263        #site-title,
    263264        #site-description {
     
    266267            clip: rect(1px, 1px, 1px, 1px);
    267268        }
    268     <?php
    269         // If the user has set a custom color for the text use that
     269            <?php
     270            // If the user has set a custom color for the text use that
    270271        else :
    271     ?>
     272            ?>
    272273        #site-title a,
    273274        #site-description {
     
    276277    <?php endif; ?>
    277278    </style>
    278     <?php
     279        <?php
    279280    }
    280281endif; // twentyeleven_header_style
     
    289290     */
    290291    function twentyeleven_admin_header_style() {
    291     ?>
     292        ?>
    292293    <style type="text/css" id="twentyeleven-admin-header-css">
    293294    .appearance_page_custom-header #headimg {
     
    311312        padding: 0 0 3em;
    312313    }
    313     <?php
     314        <?php
    314315        // If the user has set a custom color for the text use that
    315     if ( get_header_textcolor() != HEADER_TEXTCOLOR ) :
    316     ?>
     316        if ( get_header_textcolor() != HEADER_TEXTCOLOR ) :
     317            ?>
    317318    #site-title a,
    318319    #site-description {
     
    326327    }
    327328    </style>
    328     <?php
     329        <?php
    329330    }
    330331endif; // twentyeleven_admin_header_style
     
    340341    function twentyeleven_admin_header_image() {
    341342
    342 ?>
     343        ?>
    343344        <div id="headimg">
    344345            <?php
     
    356357        <?php endif; ?>
    357358        </div>
    358     <?php
     359        <?php
    359360    }
    360361endif; // twentyeleven_admin_header_image
     
    530531
    531532        if ( $wp_query->max_num_pages > 1 ) :
    532     ?>
     533            ?>
    533534            <nav id="<?php echo esc_attr( $html_id ); ?>">
    534535                <h3 class="assistive-text"><?php _e( 'Post navigation', 'twentyeleven' ); ?></h3>
     
    536537                <div class="nav-next"><?php previous_posts_link( __( 'Newer posts <span class="meta-nav">&rarr;</span>', 'twentyeleven' ) ); ?></div>
    537538            </nav><!-- #nav-above -->
    538         <?php
     539            <?php
    539540    endif;
    540541    }
     
    637638            case 'pingback':
    638639            case 'trackback':
    639         ?>
     640                ?>
    640641        <li class="post pingback">
    641642        <p><?php _e( 'Pingback:', 'twentyeleven' ); ?> <?php comment_author_link(); ?><?php edit_comment_link( __( 'Edit', 'twentyeleven' ), '<span class="edit-link">', '</span>' ); ?></p>
    642     <?php
     643                <?php
    643644                break;
    644645            default:
    645         ?>
     646                ?>
    646647        <li <?php comment_class(); ?> id="li-comment-<?php comment_ID(); ?>">
    647648        <article id="comment-<?php comment_ID(); ?>" class="comment">
     
    668669                            )
    669670                        );
    670                         ?>
     671                    ?>
    671672
    672673                        <?php edit_comment_link( __( 'Edit', 'twentyeleven' ), '<span class="edit-link">', '</span>' ); ?>
     
    686687                    comment_reply_link(
    687688                        array_merge(
    688                             $args, array(
     689                            $args,
     690                            array(
    689691                                'reply_text' => __( 'Reply <span>&darr;</span>', 'twentyeleven' ),
    690692                                'depth'      => $depth,
     
    693695                        )
    694696                    );
    695 ?>
     697                    ?>
    696698                </div><!-- .reply -->
    697699            </article><!-- #comment-## -->
    698700
    699         <?php
     701                <?php
    700702                break;
    701703        endswitch;
Note: See TracChangeset for help on using the changeset viewer.