WordPress.org
Get WordPress

Make WordPress Core


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

    r43545 r43571  
    227227        }
    228228
    229 ?>
     229        ?>
    230230<div id="dashboard-widgets" class="metabox-holder<?php echo $columns_css; ?>">
    231231        <div id="postbox-container-1" class="postbox-container">
    … …  
    243243</div>
    244244
    245 <?php
     245        <?php
    246246        wp_nonce_field( 'closedpostboxes', 'closedpostboxesnonce', false );
    247247        wp_nonce_field( 'meta-box-order', 'meta-box-order-nonce', false );
    … …  
    261261 */
    262262function wp_dashboard_right_now() {
    263 ?>
     263        ?>
    264264        <div class="main">
    265265        <ul>
    … …  
    389389
    390390        if ( ! empty( $actions ) ) :
    391         ?>
     391                ?>
    392392        <div class="sub">
    393393                <?php echo $actions; ?>
    394394        </div>
    395         <?php
     395                <?php
    396396        endif;
    397397}
    … …  
    428428                echo '</ul>';
    429429        }
    430 ?>
     430        ?>
    431431        <br class="clear" />
    432432
    … …  
    459459                </p>
    460460        </form>
    461 <?php
     461        <?php
    462462        /**
    463463         * Fires at the end of the 'Right Now' widget in the Network Admin dashboard.
    … …  
    511511
    512512        $post_ID = (int) $post->ID;
    513 ?>
     513        ?>
    514514
    515515        <form name="post" action="<?php echo esc_url( admin_url( 'post.php' ) ); ?>" method="post" id="quick-press" class="initial-form hide-if-no-js">
    … …  
    693693                }
    694694        }
    695 ?>
     695        ?>
    696696
    697697                <li id="comment-<?php echo $comment->comment_ID; ?>" <?php comment_class( array( 'comment-item', wp_get_comment_status( $comment ) ), $comment ); ?>>
    … …  
    703703                        <div class="dashboard-comment-wrap has-row-actions">
    704704                        <p class="comment-meta">
    705                         <?php
     705                                <?php
    706706                                // Comments might not have a post they relate to, e.g. programmatically created ones.
    707                         if ( $comment_post_link ) {
    708                                 printf(
    709                                         /* translators: 1: comment author, 2: post link, 3: notification if the comment is pending */
    710                                         __( 'From %1$s on %2$s %3$s' ),
    711                                         '<cite class="comment-author">' . get_comment_author_link( $comment ) . '</cite>',
    712                                         $comment_post_link,
    713                                         '<span class="approve">' . __( '[Pending]' ) . '</span>'
    714                                 );
    715                         } else {
    716                                 printf(
    717                                         /* translators: 1: comment author, 2: notification if the comment is pending */
    718                                         __( 'From %1$s %2$s' ),
    719                                         '<cite class="comment-author">' . get_comment_author_link( $comment ) . '</cite>',
    720                                         '<span class="approve">' . __( '[Pending]' ) . '</span>'
    721                                 );
    722                         }
    723                         ?>
     707                                if ( $comment_post_link ) {
     708                                        printf(
     709                                                /* translators: 1: comment author, 2: post link, 3: notification if the comment is pending */
     710                                                __( 'From %1$s on %2$s %3$s' ),
     711                                                '<cite class="comment-author">' . get_comment_author_link( $comment ) . '</cite>',
     712                                                $comment_post_link,
     713                                                '<span class="approve">' . __( '[Pending]' ) . '</span>'
     714                                        );
     715                                } else {
     716                                        printf(
     717                                                /* translators: 1: comment author, 2: notification if the comment is pending */
     718                                                __( 'From %1$s %2$s' ),
     719                                                '<cite class="comment-author">' . get_comment_author_link( $comment ) . '</cite>',
     720                                                '<span class="approve">' . __( '[Pending]' ) . '</span>'
     721                                        );
     722                                }
     723                                ?>
    724724                        </p>
    725725
    726                         <?php
     726                                <?php
    727727                        else :
    728728                                switch ( $comment->comment_type ) {
    … …  
    737737                                }
    738738                                $type = esc_html( $type );
    739                         ?>
     739                                ?>
    740740                        <div class="dashboard-comment-wrap has-row-actions">
    741741                        <p class="comment-meta">
    742                         <?php
     742                                <?php
    743743                                // Pingbacks, Trackbacks or custom comment types might not have a post they relate to, e.g. programmatically created ones.
    744                         if ( $comment_post_link ) {
    745                                 printf(
    746                                         /* translators: 1: type of comment, 2: post link, 3: notification if the comment is pending */
    747                                         _x( '%1$s on %2$s %3$s', 'dashboard' ),
    748                                         "<strong>$type</strong>",
    749                                         $comment_post_link,
    750                                         '<span class="approve">' . __( '[Pending]' ) . '</span>'
    751                                 );
    752                         } else {
    753                                 printf(
    754                                         /* translators: 1: type of comment, 2: notification if the comment is pending */
    755                                         _x( '%1$s %2$s', 'dashboard' ),
    756                                         "<strong>$type</strong>",
    757                                         '<span class="approve">' . __( '[Pending]' ) . '</span>'
    758                                 );
    759                         }
    760                         ?>
     744                                if ( $comment_post_link ) {
     745                                        printf(
     746                                                /* translators: 1: type of comment, 2: post link, 3: notification if the comment is pending */
     747                                                _x( '%1$s on %2$s %3$s', 'dashboard' ),
     748                                                "<strong>$type</strong>",
     749                                                $comment_post_link,
     750                                                '<span class="approve">' . __( '[Pending]' ) . '</span>'
     751                                        );
     752                                } else {
     753                                        printf(
     754                                                /* translators: 1: type of comment, 2: notification if the comment is pending */
     755                                                _x( '%1$s %2$s', 'dashboard' ),
     756                                                "<strong>$type</strong>",
     757                                                '<span class="approve">' . __( '[Pending]' ) . '</span>'
     758                                        );
     759                                }
     760                                ?>
    761761                        </p>
    762762                        <p class="comment-author"><?php comment_author_link( $comment ); ?></p>
    … …  
    769769                        </div>
    770770                </li>
    771 <?php
     771        <?php
    772772        $GLOBALS['comment'] = null;
    773773}
    … …  
    10501050        if ( is_scalar( $widget_control_id ) && $widget_control_id && isset( $wp_dashboard_control_callbacks[ $widget_control_id ] ) && is_callable( $wp_dashboard_control_callbacks[ $widget_control_id ] ) ) {
    10511051                call_user_func(
    1052                         $wp_dashboard_control_callbacks[ $widget_control_id ], '', array(
     1052                        $wp_dashboard_control_callbacks[ $widget_control_id ],
     1053                        '',
     1054                        array(
    10531055                                'id'       => $widget_control_id,
    10541056                                'callback' => $wp_dashboard_control_callbacks[ $widget_control_id ],
    … …  
    17611763                                printf(
    17621764                                        __( 'Manage <a href="%1$s">widgets</a> or <a href="%2$s">menus</a>' ),
    1763                                         admin_url( 'widgets.php' ), admin_url( 'nav-menus.php' )
     1765                                        admin_url( 'widgets.php' ),
     1766                                        admin_url( 'nav-menus.php' )
    17641767                                );
    17651768                        } elseif ( current_theme_supports( 'widgets' ) ) {
Note: See TracChangeset for help on using the changeset viewer.