WordPress.org

Make WordPress Core


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

    r42527 r43571  
    4343    function test_filter_object_list_and() {
    4444        $list = wp_filter_object_list(
    45             $this->object_list, array(
     45            $this->object_list,
     46            array(
    4647                'field1' => true,
    4748                'field2' => true,
    48             ), 'AND'
     49            ),
     50            'AND'
    4951        );
    5052        $this->assertEquals( 2, count( $list ) );
     
    5557    function test_filter_object_list_or() {
    5658        $list = wp_filter_object_list(
    57             $this->object_list, array(
     59            $this->object_list,
     60            array(
    5861                'field1' => true,
    5962                'field2' => true,
    60             ), 'OR'
     63            ),
     64            'OR'
    6165        );
    6266        $this->assertEquals( 3, count( $list ) );
     
    6872    function test_filter_object_list_not() {
    6973        $list = wp_filter_object_list(
    70             $this->object_list, array(
    71                 'field2' => true,
    72                 'field3' => true,
    73             ), 'NOT'
     74            $this->object_list,
     75            array(
     76                'field2' => true,
     77                'field3' => true,
     78            ),
     79            'NOT'
    7480        );
    7581        $this->assertEquals( 1, count( $list ) );
     
    7985    function test_filter_object_list_and_field() {
    8086        $list = wp_filter_object_list(
    81             $this->object_list, array(
     87            $this->object_list,
     88            array(
    8289                'field1' => true,
    8390                'field2' => true,
    84             ), 'AND', 'name'
     91            ),
     92            'AND',
     93            'name'
    8594        );
    8695        $this->assertEquals(
     
    8897                'foo' => 'foo',
    8998                'bar' => 'bar',
    90             ), $list
     99            ),
     100            $list
    91101        );
    92102    }
     
    94104    function test_filter_object_list_or_field() {
    95105        $list = wp_filter_object_list(
    96             $this->object_list, array(
    97                 'field2' => true,
    98                 'field3' => true,
    99             ), 'OR', 'name'
     106            $this->object_list,
     107            array(
     108                'field2' => true,
     109                'field3' => true,
     110            ),
     111            'OR',
     112            'name'
    100113        );
    101114        $this->assertEquals(
     
    103116                'foo' => 'foo',
    104117                'bar' => 'bar',
    105             ), $list
     118            ),
     119            $list
    106120        );
    107121    }
     
    109123    function test_filter_object_list_not_field() {
    110124        $list = wp_filter_object_list(
    111             $this->object_list, array(
    112                 'field2' => true,
    113                 'field3' => true,
    114             ), 'NOT', 'name'
     125            $this->object_list,
     126            array(
     127                'field2' => true,
     128                'field3' => true,
     129            ),
     130            'NOT',
     131            'name'
    115132        );
    116133        $this->assertEquals( array( 'baz' => 'baz' ), $list );
     
    124141                'bar' => 'bar',
    125142                'baz' => 'baz',
    126             ), $list
     143            ),
     144            $list
    127145        );
    128146
     
    133151                'bar' => 'bar',
    134152                'baz' => 'baz',
    135             ), $list
     153            ),
     154            $list
    136155        );
    137156    }
     
    147166                'b' => 'bar',
    148167                'z' => 'baz',
    149             ), $list
     168            ),
     169            $list
    150170        );
    151171    }
     
    161181                'b' => 'bar',
    162182                'z' => 'baz',
    163             ), $list
     183            ),
     184            $list
    164185        );
    165186    }
     
    175196                1 => 'bar',
    176197                2 => 'baz',
    177             ), $list
     198            ),
     199            $list
    178200        );
    179201    }
     
    191213                0   => 'bar',
    192214                'z' => 'baz',
    193             ), $list
     215            ),
     216            $list
    194217        );
    195218    }
     
    207230                'b' => 'bar',
    208231                'z' => 'baz',
    209             ), $list
     232            ),
     233            $list
    210234        );
    211235    }
     
    276300    function test_filter_object_list_nested_array_or() {
    277301        $list = wp_filter_object_list(
    278             $this->object_list, array(
     302            $this->object_list,
     303            array(
    279304                'field3' => true,
    280305                'field4' => array( 'blue' ),
    281             ), 'OR'
     306            ),
     307            'OR'
    282308        );
    283309        $this->assertEquals( 2, count( $list ) );
     
    304330                'foo' => 'foo',
    305331                'baz' => 'baz',
    306             ), $list
     332            ),
     333            $list
    307334        );
    308335    }
     
    310337    function test_filter_object_list_nested_array_or_field() {
    311338        $list = wp_filter_object_list(
    312             $this->object_list, array(
     339            $this->object_list,
     340            array(
    313341                'field3' => true,
    314342                'field4' => array( 'blue' ),
    315             ), 'OR', 'name'
     343            ),
     344            'OR',
     345            'name'
    316346        );
    317347        $this->assertEquals(
     
    319349                'foo' => 'foo',
    320350                'baz' => 'baz',
    321             ), $list
     351            ),
     352            $list
    322353        );
    323354    }
Note: See TracChangeset for help on using the changeset viewer.