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

    r43087 r43571  
    174174                'status',
    175175                'type',
    176             ), $keys
     176            ),
     177            $keys
    177178        );
    178179    }
     
    726727            array(
    727728                'page' => 2,
    728             ), rest_url( '/wp/v2/comments' )
     729            ),
     730            rest_url( '/wp/v2/comments' )
    729731        );
    730732        $this->assertFalse( stripos( $headers['Link'], 'rel="prev"' ) );
     
    746748            array(
    747749                'page' => 2,
    748             ), rest_url( '/wp/v2/comments' )
     750            ),
     751            rest_url( '/wp/v2/comments' )
    749752        );
    750753        $this->assertContains( '<' . $prev_link . '>; rel="prev"', $headers['Link'] );
     
    752755            array(
    753756                'page' => 4,
    754             ), rest_url( '/wp/v2/comments' )
     757            ),
     758            rest_url( '/wp/v2/comments' )
    755759        );
    756760        $this->assertContains( '<' . $next_link . '>; rel="next"', $headers['Link'] );
     
    765769            array(
    766770                'page' => 5,
    767             ), rest_url( '/wp/v2/comments' )
     771            ),
     772            rest_url( '/wp/v2/comments' )
    768773        );
    769774        $this->assertContains( '<' . $prev_link . '>; rel="prev"', $headers['Link'] );
     
    779784            array(
    780785                'page' => 6,
    781             ), rest_url( '/wp/v2/comments' )
     786            ),
     787            rest_url( '/wp/v2/comments' )
    782788        );
    783789        $this->assertContains( '<' . $prev_link . '>; rel="prev"', $headers['Link'] );
     
    856862        $obj      = get_comment( self::$approved_id );
    857863        $response = $endpoint->prepare_item_for_response( $obj, $request );
    858         $this->assertEquals( array(
    859             'id',
    860             'status',
    861         ), array_keys( $response->get_data() ) );
     864        $this->assertEquals(
     865            array(
     866                'id',
     867                'status',
     868            ),
     869            array_keys( $response->get_data() )
     870        );
    862871    }
    863872
     
    26662675                'author_name'       => '\o/ ¯\_(ツ)_/¯',
    26672676                'author_user_agent' => '\o/ ¯\_(ツ)_/¯',
    2668             ), array(
     2677            ),
     2678            array(
    26692679                'content'           => array(
    26702680                    'raw'      => '\o/ ¯\_(ツ)_/¯',
     
    26862696                    'author_name'       => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    26872697                    'author_user_agent' => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    2688                 ), array(
     2698                ),
     2699                array(
    26892700                    'content'           => array(
    26902701                        'raw'      => 'div <strong>strong</strong> oh noes',
     
    27022713                    'author_name'       => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    27032714                    'author_user_agent' => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    2704                 ), array(
     2715                ),
     2716                array(
    27052717                    'content'           => array(
    27062718                        'raw'      => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
     
    27222734                'author_name'       => '\\\&\\\ &amp; &invalid; < &lt; &amp;lt;',
    27232735                'author_user_agent' => '\\\&\\\ &amp; &invalid; < &lt; &amp;lt;',
    2724             ), array(
     2736            ),
     2737            array(
    27252738                'content'           => array(
    27262739                    'raw'      => '\\\&\\\ &amp; &invalid; < &lt; &amp;lt;',
     
    27412754                'author_name'       => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    27422755                'author_user_agent' => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    2743             ), array(
     2756            ),
     2757            array(
    27442758                'content'           => array(
    27452759                    'raw'      => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
     
    29092923
    29102924        register_rest_field(
    2911             'comment', 'my_custom_int', array(
     2925            'comment',
     2926            'my_custom_int',
     2927            array(
    29122928                'schema'          => $schema,
    29132929                'get_callback'    => array( $this, 'additional_field_get_callback' ),
     
    29682984
    29692985        register_rest_field(
    2970             'comment', 'my_custom_int', array(
     2986            'comment',
     2987            'my_custom_int',
     2988            array(
    29712989                'schema'          => $schema,
    29722990                'get_callback'    => array( $this, 'additional_field_get_callback' ),
     
    30243042                'collection',
    30253043                'up',
    3026             ), array_keys( $links )
     3044            ),
     3045            array_keys( $links )
    30273046        );
    30283047
Note: See TracChangeset for help on using the changeset viewer.