Make WordPress Core


Ignore:
Timestamp:
08/17/2018 01:50:26 AM (6 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/tests/phpunit/tests/user.php

    r43390 r43571  
    907907    public function test_wp_insert_user_should_not_truncate_to_a_duplicate_user_nicename_when_suffix_has_more_than_one_character() {
    908908        $user_ids = self::factory()->user->create_many(
    909             4, array(
     909            4,
     910            array(
    910911                'user_nicename' => str_repeat( 'a', 50 ),
    911912            )
     
    15571558        $_POST['email']    = 'subscriber@subscriber.test';
    15581559        $_POST['nickname'] = 'subscriber';
    1559         $this->assertSame(  $administrator, edit_user( $administrator ) );
     1560        $this->assertSame( $administrator, edit_user( $administrator ) );
    15601561
    15611562        // Should still have the old role.
     
    15721573        $_POST['email']    = 'administrator@administrator.test';
    15731574        $_POST['nickname'] = 'administrator';
    1574         $this->assertSame(  $editor, edit_user( $editor ) );
     1575        $this->assertSame( $editor, edit_user( $editor ) );
    15751576
    15761577        // Should have the new role.
Note: See TracChangeset for help on using the changeset viewer.