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/tests/phpunit/tests/rest-api/rest-users-controller.php

    r43340 r43571  
    4444            )
    4545        );
    46         self::$subscriber    = $factory->user->create(
     46        self::$subscriber   = $factory->user->create(
    4747            array(
    4848                'role'         => 'subscriber',
     
    5656                $post_type_name = 'r_' . json_encode( $show_in_rest ) . '_p_' . json_encode( $public );
    5757                register_post_type(
    58                     $post_type_name, array(
     58                    $post_type_name,
     59                    array(
    5960                        'public'                   => $public,
    6061                        'show_in_rest'             => $show_in_rest,
     
    176177                'slug',
    177178                'who',
    178             ), $keys
     179            ),
     180            $keys
    179181        );
    180182    }
     
    295297            array(
    296298                'page' => 2,
    297             ), rest_url( 'wp/v2/users' )
     299            ),
     300            rest_url( 'wp/v2/users' )
    298301        );
    299302        $this->assertFalse( stripos( $headers['Link'], 'rel="prev"' ) );
     
    314317            array(
    315318                'page' => 2,
    316             ), rest_url( 'wp/v2/users' )
     319            ),
     320            rest_url( 'wp/v2/users' )
    317321        );
    318322        $this->assertContains( '<' . $prev_link . '>; rel="prev"', $headers['Link'] );
     
    320324            array(
    321325                'page' => 4,
    322             ), rest_url( 'wp/v2/users' )
     326            ),
     327            rest_url( 'wp/v2/users' )
    323328        );
    324329        $this->assertContains( '<' . $next_link . '>; rel="next"', $headers['Link'] );
     
    333338            array(
    334339                'page' => 5,
    335             ), rest_url( 'wp/v2/users' )
     340            ),
     341            rest_url( 'wp/v2/users' )
    336342        );
    337343        $this->assertContains( '<' . $prev_link . '>; rel="prev"', $headers['Link'] );
     
    347353            array(
    348354                'page' => 6,
    349             ), rest_url( 'wp/v2/users' )
     355            ),
     356            rest_url( 'wp/v2/users' )
    350357        );
    351358        $this->assertContains( '<' . $prev_link . '>; rel="prev"', $headers['Link'] );
     
    381388                'per_page' => 5,
    382389                'page'     => 1,
    383             ), rest_url( 'wp/v2/users' )
     390            ),
     391            rest_url( 'wp/v2/users' )
    384392        );
    385393        $headers   = $response->get_headers();
     
    683691        $request = new WP_REST_Request( 'GET', '/wp/v2/users' );
    684692        $request->set_param(
    685             'slug', array(
     693            'slug',
     694            array(
    686695                'taco',
    687696                'burrito',
     
    857866        $user     = get_user_by( 'id', get_current_user_id() );
    858867        $response = $this->endpoint->prepare_item_for_response( $user, $request );
    859         $this->assertEquals( array(
    860             'id',
    861             'name',
    862         ), array_keys( $response->get_data() ) );
     868        $this->assertEquals(
     869            array(
     870                'id',
     871                'name',
     872            ),
     873            array_keys( $response->get_data() )
     874        );
    863875    }
    864876
     
    19942006                'nickname'    => '\o/ ¯\_(ツ)_/¯',
    19952007                'password'    => 'o/ ¯_(ツ)_/¯ \'"',
    1996             ), array(
     2008            ),
     2009            array(
    19972010                'name'        => '\o/ ¯\_(ツ)_/¯',
    19982011                'first_name'  => '\o/ ¯\_(ツ)_/¯',
     
    20202033                    'nickname'    => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    20212034                    'password'    => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    2022                 ), array(
     2035                ),
     2036                array(
    20232037                    'name'        => 'div strong',
    20242038                    'first_name'  => 'div strong',
     
    20422056                    'nickname'    => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    20432057                    'password'    => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    2044                 ), array(
     2058                ),
     2059                array(
    20452060                    'name'        => 'div strong',
    20462061                    'first_name'  => 'div strong',
     
    20692084                'nickname'    => '\\\&\\\ &amp; &invalid; < &lt; &amp;lt;',
    20702085                'password'    => '& &amp; &invalid; < &lt; &amp;lt;',
    2071             ), array(
     2086            ),
     2087            array(
    20722088                'username'    => $valid_username,
    20732089                'name'        => '\\\&amp;\\\ &amp; &amp;invalid; &lt; &lt; &amp;lt;',
     
    20962112                'nickname'    => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    20972113                'password'    => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    2098             ), array(
     2114            ),
     2115            array(
    20992116                'username'    => $valid_username,
    21002117                'name'        => 'div strong',
     
    24842501
    24852502        register_rest_field(
    2486             'user', 'my_custom_int', array(
     2503            'user',
     2504            'my_custom_int',
     2505            array(
    24872506                'schema'          => $schema,
    24882507                'get_callback'    => array( $this, 'additional_field_get_callback' ),
     
    25472566
    25482567        register_rest_field(
    2549             'user', 'my_custom_int', array(
     2568            'user',
     2569            'my_custom_int',
     2570            array(
    25502571                'schema'          => $schema,
    25512572                'get_callback'    => array( $this, 'additional_field_get_callback' ),
     
    27582779                'self',
    27592780                'collection',
    2760             ), array_keys( $links )
     2781            ),
     2782            array_keys( $links )
    27612783        );
    27622784
Note: See TracChangeset for help on using the changeset viewer.