WordPress.org
Get WordPress

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/rest-api.php

    r42343 r43571  
    7979    public function test_route_canonicalized() {
    8080        register_rest_route(
    81             'test-ns', '/test', array(
     81            'test-ns',
     82            '/test',
     83            array(
    8284                'methods'  => array( 'GET' ),
    8385                'callback' => '__return_null',
    … …  
    112114    public function test_route_canonicalized_multiple() {
    113115        register_rest_route(
    114             'test-ns', '/test', array(
     116            'test-ns',
     117            '/test',
     118            array(
    115119                array(
    116120                    'methods'  => array( 'GET' ),
    … …  
    151155    public function test_route_merge() {
    152156        register_rest_route(
    153             'test-ns', '/test', array(
     157            'test-ns',
     158            '/test',
     159            array(
    154160                'methods'  => array( 'GET' ),
    155161                'callback' => '__return_null',
    … …  
    157163        );
    158164        register_rest_route(
    159             'test-ns', '/test', array(
     165            'test-ns',
     166            '/test',
     167            array(
    160168                'methods'  => array( 'POST' ),
    161169                'callback' => '__return_null',
    … …  
    174182    public function test_route_override() {
    175183        register_rest_route(
    176             'test-ns', '/test', array(
     184            'test-ns',
     185            '/test',
     186            array(
    177187                'methods'      => array( 'GET' ),
    178188                'callback'     => '__return_null',
    … …  
    181191        );
    182192        register_rest_route(
    183             'test-ns', '/test', array(
     193            'test-ns',
     194            '/test',
     195            array(
    184196                'methods'      => array( 'POST' ),
    185197                'callback'     => '__return_null',
    186198                'should_exist' => true,
    187             ), true
     199            ),
     200            true
    188201        );
    189202
    … …  
    205218    public function test_route_reject_empty_namespace() {
    206219        register_rest_route(
    207             '', '/test-empty-namespace', array(
     220            '',
     221            '/test-empty-namespace',
     222            array(
    208223                'methods'  => array( 'POST' ),
    209224                'callback' => '__return_null',
    210             ), true
     225            ),
     226            true
    211227        );
    212228        $endpoints = $GLOBALS['wp_rest_server']->get_routes();
    … …  
    221237    public function test_route_reject_empty_route() {
    222238        register_rest_route(
    223             '/test-empty-route', '', array(
     239            '/test-empty-route',
     240            '',
     241            array(
    224242                'methods'  => array( 'POST' ),
    225243                'callback' => '__return_null',
    226             ), true
     244            ),
     245            true
    227246        );
    228247        $endpoints = $GLOBALS['wp_rest_server']->get_routes();
    … …  
    240259    public function test_route_method() {
    241260        register_rest_route(
    242             'test-ns', '/test', array(
     261            'test-ns',
     262            '/test',
     263            array(
    243264                'methods'  => array( 'GET' ),
    244265                'callback' => '__return_null',
    … …  
    256277    public function test_route_method_string() {
    257278        register_rest_route(
    258             'test-ns', '/test', array(
     279            'test-ns',
     280            '/test',
     281            array(
    259282                'methods'  => 'GET',
    260283                'callback' => '__return_null',
    … …  
    272295    public function test_route_method_array() {
    273296        register_rest_route(
    274             'test-ns', '/test', array(
     297            'test-ns',
     298            '/test',
     299            array(
    275300                'methods'  => array( 'GET', 'POST' ),
    276301                'callback' => '__return_null',
    … …  
    281306
    282307        $this->assertEquals(
    283             $routes['/test-ns/test'][0]['methods'], array(
     308            $routes['/test-ns/test'][0]['methods'],
     309            array(
    284310                'GET'  => true,
    285311                'POST' => true,
    … …  
    293319    public function test_route_method_comma_seperated() {
    294320        register_rest_route(
    295             'test-ns', '/test', array(
     321            'test-ns',
     322            '/test',
     323            array(
    296324                'methods'  => 'GET,POST',
    297325                'callback' => '__return_null',
    … …  
    302330
    303331        $this->assertEquals(
    304             $routes['/test-ns/test'][0]['methods'], array(
     332            $routes['/test-ns/test'][0]['methods'],
     333            array(
    305334                'GET'  => true,
    306335                'POST' => true,
    … …  
    311340    public function test_options_request() {
    312341        register_rest_route(
    313             'test-ns', '/test', array(
     342            'test-ns',
     343            '/test',
     344            array(
    314345                'methods'  => 'GET,POST',
    315346                'callback' => '__return_null',
    … …  
    331362    public function test_options_request_not_options() {
    332363        register_rest_route(
    333             'test-ns', '/test', array(
     364            'test-ns',
     365            '/test',
     366            array(
    334367                'methods'  => 'GET,POST',
    335368                'callback' => '__return_true',
    … …  
    400433                'c' => 2,
    401434                'e' => 4,
    402             ), $response->get_data()
     435            ),
     436            $response->get_data()
    403437        );
    404438    }
    … …  
    431465                'c' => 2,
    432466                'e' => 4,
    433             ), $response->get_data()
     467            ),
     468            $response->get_data()
    434469        );
    435470    }
    … …  
    478513                    'c' => 8,
    479514                ),
    480             ), $response->get_data()
     515            ),
     516            $response->get_data()
    481517        );
    482518    }
    … …  
    666702
    667703        register_rest_route(
    668             'test-ns', '/test', array(
     704            'test-ns',
     705            '/test',
     706            array(
    669707                'methods'  => array( 'GET' ),
    670708                'callback' => '__return_null',
Note: See TracChangeset for help on using the changeset viewer.