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

    r43569 r43571  
    255255        // Don't allow interim logins to navigate away from the page.
    256256        if ( ! $interim_login ) :
    257         ?>
     257                ?>
    258258        <p id="backtoblog"><a href="<?php echo esc_url( home_url( '/' ) ); ?>">
    259         <?php
     259                <?php
    260260                /* translators: %s: site title */
    261261                printf( _x( '&larr; Back to %s', 'site' ), get_bloginfo( 'title', 'display' ) );
    262         ?>
     262                ?>
    263263        </a></p>
    264         <?php the_privacy_policy_link( '<div class="privacy-policy-page-link">', '</div>' ); ?>
     264                <?php the_privacy_policy_link( '<div class="privacy-policy-page-link">', '</div>' ); ?>
    265265        <?php endif; ?>
    266266
    … …  
    292292 */
    293293function wp_shake_js() {
    294 ?>
     294        ?>
    295295<script type="text/javascript">
    296296addLoadEvent = function(func){if(typeof jQuery!="undefined")jQuery(document).ready(func);else if(typeof wpOnload!='function'){wpOnload=func;}else{var oldonload=wpOnload;wpOnload=function(){oldonload();func();}}};
    … …  
    300300addLoadEvent(function(){ var p=new Array(15,30,15,0,-15,-30,-15,0);p=p.concat(p.concat(p));var i=document.forms[0].id;g(i).position='relative';shake(i,p,20);});
    301301</script>
    302 <?php
     302        <?php
    303303}
    304304
    … …  
    604604                }
    605605
    606         ?>
     606                ?>
    607607
    608608        <form name="lostpasswordform" id="lostpasswordform" action="<?php echo esc_url( network_site_url( 'wp-login.php?action=lostpassword', 'login_post' ) ); ?>" method="post">
    … …  
    611611                <input type="text" name="user_login" id="user_login" class="input" value="<?php echo esc_attr( $user_login ); ?>" size="20" autocapitalize="off" /></label>
    612612        </p>
    613         <?php
    614         /**
    615         * Fires inside the lostpassword form tags, before the hidden fields.
    616         *
    617         * @since 2.1.0
    618         */
    619         do_action( 'lostpassword_form' );
    620         ?>
     613                <?php
     614                /**
     615                * Fires inside the lostpassword form tags, before the hidden fields.
     616                *
     617                * @since 2.1.0
     618                */
     619                do_action( 'lostpassword_form' );
     620                ?>
    621621                <input type="hidden" name="redirect_to" value="<?php echo esc_attr( $redirect_to ); ?>" />
    622622                <p class="submit"><input type="submit" name="wp-submit" id="wp-submit" class="button button-primary button-large" value="<?php esc_attr_e( 'Get New Password' ); ?>" /></p>
    … …  
    625625        <p id="nav">
    626626        <a href="<?php echo esc_url( wp_login_url() ); ?>"><?php _e( 'Log in' ); ?></a>
    627         <?php
    628         if ( get_option( 'users_can_register' ) ) :
    629                 $registration_url = sprintf( '<a href="%s">%s</a>', esc_url( wp_registration_url() ), __( 'Register' ) );
    630 
    631                 echo esc_html( $login_link_separator );
    632 
    633                 /** This filter is documented in wp-includes/general-template.php */
    634                 echo apply_filters( 'register', $registration_url );
     627                <?php
     628                if ( get_option( 'users_can_register' ) ) :
     629                        $registration_url = sprintf( '<a href="%s">%s</a>', esc_url( wp_registration_url() ), __( 'Register' ) );
     630
     631                        echo esc_html( $login_link_separator );
     632
     633                        /** This filter is documented in wp-includes/general-template.php */
     634                        echo apply_filters( 'register', $registration_url );
    635635        endif;
    636         ?>
     636                ?>
    637637        </p>
    638638
    639         <?php
    640         login_footer( 'user_login' );
    641 
    642         if ( $switched_locale ) {
    643                 restore_previous_locale();
    644         }
     639                <?php
     640                login_footer( 'user_login' );
     641
     642                if ( $switched_locale ) {
     643                        restore_previous_locale();
     644                }
    645645
    646646                break;
    … …  
    706706                login_header( __( 'Reset Password' ), '<p class="message reset-pass">' . __( 'Enter your new password below.' ) . '</p>', $errors );
    707707
    708         ?>
     708                ?>
    709709        <form name="resetpassform" id="resetpassform" action="<?php echo esc_url( network_site_url( 'wp-login.php?action=resetpass', 'login_post' ) ); ?>" method="post" autocomplete="off">
    710710        <input type="hidden" id="user_login" value="<?php echo esc_attr( $rp_login ); ?>" autocomplete="off" />
    … …  
    740740        <br class="clear" />
    741741
    742         <?php
    743         /**
    744         * Fires following the 'Strength indicator' meter in the user password reset form.
    745         *
    746         * @since 3.9.0
    747         *
    748         * @param WP_User $user User object of the user whose password is being reset.
    749         */
    750         do_action( 'resetpass_form', $user );
    751         ?>
     742                <?php
     743                /**
     744                * Fires following the 'Strength indicator' meter in the user password reset form.
     745                *
     746                * @since 3.9.0
     747                *
     748                * @param WP_User $user User object of the user whose password is being reset.
     749                */
     750                do_action( 'resetpass_form', $user );
     751                ?>
    752752        <input type="hidden" name="rp_key" value="<?php echo esc_attr( $rp_key ); ?>" />
    753753        <p class="submit"><input type="submit" name="wp-submit" id="wp-submit" class="button button-primary button-large" value="<?php esc_attr_e( 'Reset Password' ); ?>" /></p>
    … …  
    756756        <p id="nav">
    757757        <a href="<?php echo esc_url( wp_login_url() ); ?>"><?php _e( 'Log in' ); ?></a>
    758         <?php
    759         if ( get_option( 'users_can_register' ) ) :
    760                 $registration_url = sprintf( '<a href="%s">%s</a>', esc_url( wp_registration_url() ), __( 'Register' ) );
    761 
    762                 echo esc_html( $login_link_separator );
    763 
    764                 /** This filter is documented in wp-includes/general-template.php */
    765                 echo apply_filters( 'register', $registration_url );
     758                <?php
     759                if ( get_option( 'users_can_register' ) ) :
     760                        $registration_url = sprintf( '<a href="%s">%s</a>', esc_url( wp_registration_url() ), __( 'Register' ) );
     761
     762                        echo esc_html( $login_link_separator );
     763
     764                        /** This filter is documented in wp-includes/general-template.php */
     765                        echo apply_filters( 'register', $registration_url );
    766766        endif;
    767         ?>
     767                ?>
    768768        </p>
    769769
    770         <?php
    771         login_footer( 'user_pass' );
    772 
    773         if ( $switched_locale ) {
    774                 restore_previous_locale();
    775         }
     770                <?php
     771                login_footer( 'user_pass' );
     772
     773                if ( $switched_locale ) {
     774                        restore_previous_locale();
     775                }
    776776
    777777                break;
    … …  
    825825                $redirect_to = apply_filters( 'registration_redirect', $registration_redirect );
    826826                login_header( __( 'Registration Form' ), '<p class="message register">' . __( 'Register For This Site' ) . '</p>', $errors );
    827         ?>
     827                ?>
    828828        <form name="registerform" id="registerform" action="<?php echo esc_url( site_url( 'wp-login.php?action=register', 'login_post' ) ); ?>" method="post" novalidate="novalidate">
    829829        <p>
    … …  
    835835                <input type="email" name="user_email" id="user_email" class="input" value="<?php echo esc_attr( wp_unslash( $user_email ) ); ?>" size="25" /></label>
    836836        </p>
    837         <?php
    838         /**
    839         * Fires following the 'Email' field in the user registration form.
    840         *
    841         * @since 2.1.0
    842         */
    843         do_action( 'register_form' );
    844         ?>
     837                <?php
     838                /**
     839                * Fires following the 'Email' field in the user registration form.
     840                *
     841                * @since 2.1.0
     842                */
     843                do_action( 'register_form' );
     844                ?>
    845845        <p id="reg_passmail"><?php _e( 'Registration confirmation will be emailed to you.' ); ?></p>
    846846        <br class="clear" />
    … …  
    851851        <p id="nav">
    852852        <a href="<?php echo esc_url( wp_login_url() ); ?>"><?php _e( 'Log in' ); ?></a>
    853         <?php echo esc_html( $login_link_separator ); ?>
     853                <?php echo esc_html( $login_link_separator ); ?>
    854854        <a href="<?php echo esc_url( wp_lostpassword_url() ); ?>"><?php _e( 'Lost your password?' ); ?></a>
    855855        </p>
    856856
    857         <?php
    858         login_footer( 'user_login' );
    859 
    860         if ( $switched_locale ) {
    861                 restore_previous_locale();
    862         }
     857                <?php
     858                login_footer( 'user_login' );
     859
     860                if ( $switched_locale ) {
     861                        restore_previous_locale();
     862                }
    863863
    864864                break;
    865865
    866         case 'confirmaction' :
     866        case 'confirmaction':
    867867                if ( ! isset( $_GET['request_id'] ) ) {
    868868                        wp_die( __( 'Invalid request.' ) );
    … …  
    944944                                /* translators: 1: Browser cookie documentation URL, 2: Support forums URL */
    945945                                $user = new WP_Error(
    946                                         'test_cookie', sprintf(
     946                                        'test_cookie',
     947                                        sprintf(
    947948                                                __( '<strong>ERROR</strong>: Cookies are blocked due to unexpected output. For help, please see <a href="%1$s">this documentation</a> or try the <a href="%2$s">support forums</a>.' ),
    948                                                 __( 'https://codex.wordpress.org/Cookies' ), __( 'https://wordpress.org/support/' )
     949                                                __( 'https://codex.wordpress.org/Cookies' ),
     950                                                __( 'https://wordpress.org/support/' )
    949951                                        )
    950952                                );
    … …  
    953955                                /* translators: %s: Browser cookie documentation URL */
    954956                                $user = new WP_Error(
    955                                         'test_cookie', sprintf(
     957                                        'test_cookie',
     958                                        sprintf(
    956959                                                __( '<strong>ERROR</strong>: Cookies are blocked or not supported by your browser. You must <a href="%s">enable cookies</a> to use WordPress.' ),
    957960                                                __( 'https://codex.wordpress.org/Cookies' )
    … …  
    978981                                $interim_login = 'success';
    979982                                login_header( '', $message );
    980                         ?>
     983                                ?>
    981984                                </div>
    982985                                <?php
    983986                                /** This action is documented in wp-login.php */
    984987                                do_action( 'login_footer' );
    985                         ?>
     988                                ?>
    986989                                <?php if ( $customize_login ) : ?>
    987990                                <script type="text/javascript">setTimeout( function(){ new wp.customize.Messenger({ url: '<?php echo wp_customize_url(); ?>', channel: 'login' }).send('login') }, 1000 );</script>
    988991                        <?php endif; ?>
    989992                                </body></html>
    990         <?php
    991           exit;
     993                                <?php
     994                                exit;
    992995                        }
    993996
    … …  
    10631066                        $aria_describedby_error = '';
    10641067                }
    1065         ?>
     1068                ?>
    10661069
    10671070        <form name="loginform" id="loginform" action="<?php echo esc_url( site_url( 'wp-login.php', 'login_post' ) ); ?>" method="post">
    … …  
    10741077                <input type="password" name="pwd" id="user_pass"<?php echo $aria_describedby_error; ?> class="input" value="" size="20" /></label>
    10751078        </p>
    1076         <?php
    1077         /**
    1078         * Fires following the 'Password' field in the login form.
    1079         *
    1080         * @since 2.1.0
    1081         */
    1082         do_action( 'login_form' );
    1083         ?>
     1079                <?php
     1080                /**
     1081                * Fires following the 'Password' field in the login form.
     1082                *
     1083                * @since 2.1.0
     1084                */
     1085                do_action( 'login_form' );
     1086                ?>
    10841087        <p class="forgetmenot"><label for="rememberme"><input name="rememberme" type="checkbox" id="rememberme" value="forever" <?php checked( $rememberme ); ?> /> <?php esc_html_e( 'Remember Me' ); ?></label></p>
    10851088        <p class="submit">
    10861089                <input type="submit" name="wp-submit" id="wp-submit" class="button button-primary button-large" value="<?php esc_attr_e( 'Log In' ); ?>" />
    1087         <?php   if ( $interim_login ) { ?>
     1090                <?php   if ( $interim_login ) { ?>
    10881091                <input type="hidden" name="interim-login" value="1" />
    10891092        <?php   } else { ?>
    10901093                <input type="hidden" name="redirect_to" value="<?php echo esc_attr( $redirect_to ); ?>" />
    10911094        <?php } ?>
    1092         <?php if ( $customize_login ) : ?>
     1095                <?php if ( $customize_login ) : ?>
    10931096                <input type="hidden" name="customize-login" value="1" />
    10941097        <?php endif; ?>
    … …  
    10971100        </form>
    10981101
    1099         <?php if ( ! $interim_login ) { ?>
     1102                <?php if ( ! $interim_login ) { ?>
    11001103        <p id="nav">
    1101         <?php
    1102         if ( ! isset( $_GET['checkemail'] ) || ! in_array( $_GET['checkemail'], array( 'confirm', 'newpass' ) ) ) :
    1103                 if ( get_option( 'users_can_register' ) ) :
    1104                         $registration_url = sprintf( '<a href="%s">%s</a>', esc_url( wp_registration_url() ), __( 'Register' ) );
    1105 
    1106                         /** This filter is documented in wp-includes/general-template.php */
    1107                         echo apply_filters( 'register', $registration_url );
    1108 
    1109                         echo esc_html( $login_link_separator );
    1110                 endif;
    1111                 ?>
     1104                        <?php
     1105                        if ( ! isset( $_GET['checkemail'] ) || ! in_array( $_GET['checkemail'], array( 'confirm', 'newpass' ) ) ) :
     1106                                if ( get_option( 'users_can_register' ) ) :
     1107                                        $registration_url = sprintf( '<a href="%s">%s</a>', esc_url( wp_registration_url() ), __( 'Register' ) );
     1108
     1109                                        /** This filter is documented in wp-includes/general-template.php */
     1110                                        echo apply_filters( 'register', $registration_url );
     1111
     1112                                        echo esc_html( $login_link_separator );
     1113                                endif;
     1114                                ?>
    11121115                <a href="<?php echo esc_url( wp_lostpassword_url() ); ?>"><?php _e( 'Lost your password?' ); ?></a>
    1113         <?php endif; ?>
     1116                        <?php endif; ?>
    11141117        </p>
    11151118        <?php } ?>
    … …  
    11181121        function wp_attempt_focus(){
    11191122        setTimeout( function(){ try{
    1120         <?php if ( $user_login ) { ?>
     1123                <?php if ( $user_login ) { ?>
    11211124        d = document.getElementById('user_pass');
    11221125        d.value = '';
    … …  
    11261129        if( d.value != '' )
    11271130        d.value = '';
    1128         <?php
     1131                <?php
    11291132}
    11301133}
    … …  
    11361139        }
    11371140
    1138         <?php
    1139         /**
    1140         * Filters whether to print the call to `wp_attempt_focus()` on the login screen.
    1141         *
    1142         * @since 4.8.0
    1143         *
    1144         * @param bool $print Whether to print the function call. Default true.
    1145         */
    1146         if ( apply_filters( 'enable_login_autofocus', true ) && ! $error ) {
    1147 ?>
     1141                <?php
     1142                /**
     1143                * Filters whether to print the call to `wp_attempt_focus()` on the login screen.
     1144                *
     1145                * @since 4.8.0
     1146                *
     1147                * @param bool $print Whether to print the function call. Default true.
     1148                */
     1149                if ( apply_filters( 'enable_login_autofocus', true ) && ! $error ) {
     1150                        ?>
    11481151        wp_attempt_focus();
    1149         <?php } ?>
     1152                <?php } ?>
    11501153        if(typeof wpOnload=='function')wpOnload();
    1151         <?php if ( $interim_login ) { ?>
     1154                <?php if ( $interim_login ) { ?>
    11521155        (function(){
    11531156        try {
    … …  
    11621165        </script>
    11631166
    1164         <?php
    1165         login_footer();
    1166 
    1167         if ( $switched_locale ) {
    1168                 restore_previous_locale();
    1169         }
     1167                <?php
     1168                login_footer();
     1169
     1170                if ( $switched_locale ) {
     1171                        restore_previous_locale();
     1172                }
    11701173
    11711174                break;
Note: See TracChangeset for help on using the changeset viewer.