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-admin/includes/user.php

    r43486 r43571  
    538538});
    539539</script>
    540 <?php
     540    <?php
    541541}
    542542
     
    551551 */
    552552function use_ssl_preference( $user ) {
    553 ?>
     553    ?>
    554554    <tr class="user-use-ssl-wrap">
    555555        <th scope="row"><?php _e( 'Use https' ); ?></th>
    556556        <td><label for="use_ssl"><input name="use_ssl" type="checkbox" id="use_ssl" value="1" <?php checked( '1', $user->use_ssl ); ?> /> <?php _e( 'Always use https when visiting the admin' ); ?></label></td>
    557557    </tr>
    558 <?php
     558    <?php
    559559}
    560560
     
    577577Please click the following link to activate your user account:
    578578%%s'
    579         ), wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES ), home_url(), wp_specialchars_decode( translate_user_role( $role['name'] ) )
     579        ),
     580        wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES ),
     581        home_url(),
     582        wp_specialchars_decode( translate_user_role( $role['name'] ) )
    580583    );
    581584}
     
    628631    update_post_meta( $request_id, '_wp_user_request_completed_timestamp', time() );
    629632
    630     $request = wp_update_post( array(
    631         'ID'          => $request_id,
    632         'post_status' => 'request-completed',
    633     ) );
     633    $request = wp_update_post(
     634        array(
     635            'ID'          => $request_id,
     636            'post_status' => 'request-completed',
     637        )
     638    );
    634639
    635640    return $request;
     
    754759function _wp_personal_data_cleanup_requests() {
    755760    /** This filter is documented in wp-includes/user.php */
    756     $expires        = (int) apply_filters( 'user_request_key_expiration', DAY_IN_SECONDS );
    757 
    758     $requests_query = new WP_Query( array(
    759         'post_type'      => 'user_request',
    760         'posts_per_page' => -1,
    761         'post_status'    => 'request-pending',
    762         'fields'         => 'ids',
    763         'date_query'     => array(
     761    $expires = (int) apply_filters( 'user_request_key_expiration', DAY_IN_SECONDS );
     762
     763    $requests_query = new WP_Query(
     764        array(
     765            'post_type'      => 'user_request',
     766            'posts_per_page' => -1,
     767            'post_status'    => 'request-pending',
     768            'fields'         => 'ids',
     769            'date_query'     => array(
     770                array(
     771                    'column' => 'post_modified_gmt',
     772                    'before' => $expires . ' seconds ago',
     773                ),
     774            ),
     775        )
     776    );
     777
     778    $request_ids = $requests_query->posts;
     779
     780    foreach ( $request_ids as $request_id ) {
     781        wp_update_post(
    764782            array(
    765                 'column' => 'post_modified_gmt',
    766                 'before' => $expires . ' seconds ago',
    767             ),
    768         ),
    769     ) );
    770 
    771     $request_ids = $requests_query->posts;
    772 
    773     foreach ( $request_ids as $request_id ) {
    774         wp_update_post( array(
    775             'ID'            => $request_id,
    776             'post_status'   => 'request-failed',
    777             'post_password' => '',
    778         ) );
     783                'ID'            => $request_id,
     784                'post_status'   => 'request-failed',
     785                'post_password' => '',
     786            )
     787        );
    779788    }
    780789}
     
    797806    wp_enqueue_script( 'xfn' );
    798807
    799     $requests_table = new WP_Privacy_Data_Export_Requests_Table( array(
    800         'plural'   => 'privacy_requests',
    801         'singular' => 'privacy_request',
    802     ) );
     808    $requests_table = new WP_Privacy_Data_Export_Requests_Table(
     809        array(
     810            'plural'   => 'privacy_requests',
     811            'singular' => 'privacy_request',
     812        )
     813    );
    803814    $requests_table->process_bulk_action();
    804815    $requests_table->prepare_items();
     
    868879    wp_enqueue_script( 'xfn' );
    869880
    870     $requests_table = new WP_Privacy_Data_Removal_Requests_Table( array(
    871         'plural'   => 'privacy_requests',
    872         'singular' => 'privacy_request',
    873     ) );
     881    $requests_table = new WP_Privacy_Data_Removal_Requests_Table(
     882        array(
     883            'plural'   => 'privacy_requests',
     884            'singular' => 'privacy_request',
     885        )
     886    );
    874887
    875888    $requests_table->process_bulk_action();
     
    10141027function _wp_privacy_requests_screen_options() {
    10151028    $args = array(
    1016         'option'  => str_replace( 'tools_page_', '', get_current_screen()->id ) . '_requests_per_page',
     1029        'option' => str_replace( 'tools_page_', '', get_current_screen()->id ) . '_requests_per_page',
    10171030    );
    10181031    add_screen_option( 'per_page', $args );
     
    11761189        $action      = $this->current_action();
    11771190        $request_ids = isset( $_REQUEST['request_id'] ) ? wp_parse_id_list( wp_unslash( $_REQUEST['request_id'] ) ) : array();
    1178        
    1179         $count       = 0;
     1191
     1192        $count = 0;
    11801193
    11811194        if ( $request_ids ) {
     
    15031516                break;
    15041517            case 'request-completed':
    1505                 echo '<a href="' . esc_url( wp_nonce_url( add_query_arg( array(
    1506                     'action'     => 'delete',
    1507                     'request_id' => array( $item->ID ),
    1508                 ), admin_url( 'tools.php?page=export_personal_data' ) ), 'bulk-privacy_requests' ) ) . '" class="button">' . esc_html__( 'Remove request' ) . '</a>';
     1518                echo '<a href="' . esc_url(
     1519                    wp_nonce_url(
     1520                        add_query_arg(
     1521                            array(
     1522                                'action'     => 'delete',
     1523                                'request_id' => array( $item->ID ),
     1524                            ),
     1525                            admin_url( 'tools.php?page=export_personal_data' )
     1526                        ),
     1527                        'bulk-privacy_requests'
     1528                    )
     1529                ) . '" class="button">' . esc_html__( 'Remove request' ) . '</a>';
    15091530                break;
    15101531        }
     
    16171638                break;
    16181639            case 'request-completed':
    1619                 echo '<a href="' . esc_url( wp_nonce_url( add_query_arg( array(
    1620                     'action'     => 'delete',
    1621                     'request_id' => array( $item->ID ),
    1622                 ), admin_url( 'tools.php?page=remove_personal_data' ) ), 'bulk-privacy_requests' ) ) . '" class="button">' . esc_html__( 'Remove request' ) . '</a>';
     1640                echo '<a href="' . esc_url(
     1641                    wp_nonce_url(
     1642                        add_query_arg(
     1643                            array(
     1644                                'action'     => 'delete',
     1645                                'request_id' => array( $item->ID ),
     1646                            ),
     1647                            admin_url( 'tools.php?page=remove_personal_data' )
     1648                        ),
     1649                        'bulk-privacy_requests'
     1650                    )
     1651                ) . '" class="button">' . esc_html__( 'Remove request' ) . '</a>';
    16231652                break;
    16241653        }
Note: See TracChangeset for help on using the changeset viewer.