WordPress.org

Make WordPress Core


Ignore:
Timestamp:
08/17/2018 01:50:26 AM (16 months 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/ms-functions.php

    r43548 r43571  
    499499    if ( is_array( $limited_email_domains ) && ! empty( $limited_email_domains ) ) {
    500500        $limited_email_domains = array_map( 'strtolower', $limited_email_domains );
    501         $emaildomain = strtolower( substr( $user_email, 1 + strpos( $user_email, '@' ) ) );
     501        $emaildomain           = strtolower( substr( $user_email, 1 + strpos( $user_email, '@' ) ) );
    502502        if ( ! in_array( $emaildomain, $limited_email_domains, true ) ) {
    503503            $errors->add( 'user_email', __( 'Sorry, that email address is not allowed!' ) );
     
    693693        if ( $diff > 2 * DAY_IN_SECONDS ) {
    694694            $wpdb->delete(
    695                 $wpdb->signups, array(
     695                $wpdb->signups,
     696                array(
    696697                    'domain' => $mydomain,
    697698                    'path'   => $path,
     
    768769
    769770    $wpdb->insert(
    770         $wpdb->signups, array(
     771        $wpdb->signups,
     772        array(
    771773            'domain'         => $domain,
    772774            'path'           => $path,
     
    833835
    834836    $wpdb->insert(
    835         $wpdb->signups, array(
     837        $wpdb->signups,
     838        array(
    836839            'domain'         => '',
    837840            'path'           => '',
     
    939942            'wpmu_signup_blog_notification_email',
    940943            __( "To activate your blog, please click the following link:\n\n%1\$s\n\nAfter you activate, you will receive *another email* with your login.\n\nAfter you activate, you can visit your site here:\n\n%2\$s" ),
    941             $domain, $path, $title, $user_login, $user_email, $key, $meta
     944            $domain,
     945            $path,
     946            $title,
     947            $user_login,
     948            $user_email,
     949            $key,
     950            $meta
    942951        ),
    943952        $activate_url,
     
    965974            /* translators: New site notification email subject. 1: Network name, 2: New site URL */
    966975            _x( '[%1$s] Activate %2$s', 'New site notification email subject' ),
    967             $domain, $path, $title, $user_login, $user_email, $key, $meta
     976            $domain,
     977            $path,
     978            $title,
     979            $user_login,
     980            $user_email,
     981            $key,
     982            $meta
    968983        ),
    969984        $from_name,
     
    10431058            'wpmu_signup_user_notification_email',
    10441059            __( "To activate your user, please click the following link:\n\n%s\n\nAfter you activate, you will receive *another email* with your login." ),
    1045             $user_login, $user_email, $key, $meta
     1060            $user_login,
     1061            $user_email,
     1062            $key,
     1063            $meta
    10461064        ),
    10471065        site_url( "wp-activate.php?key=$key" )
     
    10641082            /* translators: New user notification email subject. 1: Network name, 2: New user login */
    10651083            _x( '[%1$s] Activate %2$s', 'New user notification email subject' ),
    1066             $user_login, $user_email, $key, $meta
     1084            $user_login,
     1085            $user_email,
     1086            $key,
     1087            $meta
    10671088        ),
    10681089        $from_name,
     
    11291150    if ( empty( $signup->domain ) ) {
    11301151        $wpdb->update(
    1131             $wpdb->signups, array(
     1152            $wpdb->signups,
     1153            array(
    11321154                'active'    => 1,
    11331155                'activated' => $now,
    1134             ), array( 'activation_key' => $key )
     1156            ),
     1157            array( 'activation_key' => $key )
    11351158        );
    11361159
     
    11651188            $blog_id->add_data( $signup );
    11661189            $wpdb->update(
    1167                 $wpdb->signups, array(
     1190                $wpdb->signups,
     1191                array(
    11681192                    'active'    => 1,
    11691193                    'activated' => $now,
    1170                 ), array( 'activation_key' => $key )
     1194                ),
     1195                array( 'activation_key' => $key )
    11711196            );
    11721197        }
     
    11751200
    11761201    $wpdb->update(
    1177         $wpdb->signups, array(
     1202        $wpdb->signups,
     1203        array(
    11781204            'active'    => 1,
    11791205            'activated' => $now,
    1180         ), array( 'activation_key' => $key )
     1206        ),
     1207        array( 'activation_key' => $key )
    11811208    );
    11821209    /**
     
    13861413
    13871414Disable these notifications: %4$s'
    1388         ), $blogname, $siteurl, wp_unslash( $_SERVER['REMOTE_ADDR'] ), $options_site_url
     1415        ),
     1416        $blogname,
     1417        $siteurl,
     1418        wp_unslash( $_SERVER['REMOTE_ADDR'] ),
     1419        $options_site_url
    13891420    );
    13901421    /**
     
    14341465
    14351466Disable these notifications: %3$s'
    1436         ), $user->user_login, wp_unslash( $_SERVER['REMOTE_ADDR'] ), $options_site_url
     1467        ),
     1468        $user->user_login,
     1469        wp_unslash( $_SERVER['REMOTE_ADDR'] ),
     1470        $options_site_url
    14371471    );
    14381472
     
    20022036    if ( $user ) {
    20032037        $wpdb->insert(
    2004             $wpdb->registration_log, array(
     2038            $wpdb->registration_log,
     2039            array(
    20052040                'email'           => $user->user_email,
    20062041                'IP'              => preg_replace( '/[^0-9., ]/', '', wp_unslash( $_SERVER['REMOTE_ADDR'] ) ),
     
    20512086        if ( null == $used_global_id ) {
    20522087            $wpdb->insert(
    2053                 $wpdb->sitecategories, array(
     2088                $wpdb->sitecategories,
     2089                array(
    20542090                    'cat_ID'            => $term_id,
    20552091                    'cat_name'          => $c->name,
     
    20662102            $new_global_id = max( $max_global_id, $max_local_id ) + mt_rand( 100, 400 );
    20672103            $wpdb->insert(
    2068                 $wpdb->sitecategories, array(
     2104                $wpdb->sitecategories,
     2105                array(
    20692106                    'cat_ID'            => $new_global_id,
    20702107                    'cat_name'          => $c->name,
     
    28912928
    28922929    wp_mail(
    2893         $email_change_email['to'], sprintf(
     2930        $email_change_email['to'],
     2931        sprintf(
    28942932            $email_change_email['subject'],
    28952933            $network_name
    2896         ), $email_change_email['message'], $email_change_email['headers']
     2934        ),
     2935        $email_change_email['message'],
     2936        $email_change_email['headers']
    28972937    );
    28982938}
Note: See TracChangeset for help on using the changeset viewer.