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

    r42997 r43571  
    6464
    6565        register_rest_route(
    66             'test-ns', '/test', array(
     66            'test-ns',
     67            '/test',
     68            array(
    6769                'methods'  => array( 'GET' ),
    6870                'callback' => '__return_null',
    … …  
    8486
    8587        register_rest_route(
    86             'test-ns', '/test', array(
     88            'test-ns',
     89            '/test',
     90            array(
    8791                'methods'  => array( 'GET' ),
    8892                'callback' => '__return_null',
    … …  
    104108    public function test_optional_param() {
    105109        register_rest_route(
    106             'optional', '/test', array(
     110            'optional',
     111            '/test',
     112            array(
    107113                'methods'  => array( 'GET' ),
    108114                'callback' => '__return_null',
    … …  
    123129    public function test_no_zero_param() {
    124130        register_rest_route(
    125             'no-zero', '/test', array(
     131            'no-zero',
     132            '/test',
     133            array(
    126134                'methods'  => array( 'GET' ),
    127135                'callback' => '__return_null',
    … …  
    140148    public function test_head_request_handled_by_get() {
    141149        register_rest_route(
    142             'head-request', '/test', array(
     150            'head-request',
     151            '/test',
     152            array(
    143153                'methods'  => array( 'GET' ),
    144154                'callback' => '__return_true',
    … …  
    158168    public function test_explicit_head_callback() {
    159169        register_rest_route(
    160             'head-request', '/test', array(
     170            'head-request',
     171            '/test',
     172            array(
    161173                array(
    162174                    'methods'  => array( 'HEAD' ),
    … …  
    178190
    179191        rest_get_server()->register_route(
    180             'test', '/test/(?P<data>.*)', array(
     192            'test',
     193            '/test/(?P<data>.*)',
     194            array(
    181195                array(
    182196                    'methods'  => WP_REST_Server::READABLE,
    … …  
    200214    function test_rest_route_capability_authorization_fails() {
    201215        register_rest_route(
    202             'test-ns', '/test', array(
     216            'test-ns',
     217            '/test',
     218            array(
    203219                'methods'             => 'GET',
    204220                'callback'            => '__return_null',
    … …  
    220236    function test_rest_route_capability_authorization() {
    221237        register_rest_route(
    222             'test-ns', '/test', array(
     238            'test-ns',
     239            '/test',
     240            array(
    223241                'methods'             => 'GET',
    224242                'callback'            => '__return_null',
    … …  
    246264
    247265        register_rest_route(
    248             'test-ns', '/test', array(
     266            'test-ns',
     267            '/test',
     268            array(
    249269                'methods'      => 'GET',
    250270                'callback'     => '__return_null',
    … …  
    271291
    272292        register_rest_route(
    273             'test-ns', '/test', array(
     293            'test-ns',
     294            '/test',
     295            array(
    274296                'methods'      => 'GET',
    275297                'callback'     => '__return_null',
    … …  
    279301
    280302        register_rest_route(
    281             'test-ns', '/test', array(
     303            'test-ns',
     304            '/test',
     305            array(
    282306                'methods'      => 'POST',
    283307                'callback'     => '__return_null',
    … …  
    305329
    306330        register_rest_route(
    307             'test-ns', '/test', array(
     331            'test-ns',
     332            '/test',
     333            array(
    308334                'methods'             => 'GET',
    309335                'callback'            => '__return_null',
    … …  
    314340
    315341        register_rest_route(
    316             'test-ns', '/test', array(
     342            'test-ns',
     343            '/test',
     344            array(
    317345                'methods'      => 'POST',
    318346                'callback'     => '__return_null',
    … …  
    334362    public function test_allow_header_sent_on_options_request() {
    335363        register_rest_route(
    336             'test-ns', '/test', array(
     364            'test-ns',
     365            '/test',
     366            array(
    337367                array(
    338368                    'methods'  => array( 'GET' ),
    … …  
    469499        // Register our testing route.
    470500        rest_get_server()->register_route(
    471             'test', '/test/embeddable', array(
     501            'test',
     502            '/test/embeddable',
     503            array(
    472504                'methods'  => 'GET',
    473505                'callback' => array( $this, 'embedded_response_callback' ),
    … …  
    542574        // Register our testing route.
    543575        rest_get_server()->register_route(
    544             'test', '/test/embeddable', array(
     576            'test',
     577            '/test/embeddable',
     578            array(
    545579                'methods'  => 'GET',
    546580                'callback' => array( $this, 'embedded_response_callback' ),
    … …  
    563597        // Register our testing route.
    564598        rest_get_server()->register_route(
    565             'test', '/test/embeddable', array(
     599            'test',
     600            '/test/embeddable',
     601            array(
    566602                'methods'  => 'GET',
    567603                'callback' => array( $this, 'embedded_response_callback' ),
    … …  
    589625        // Register our testing route.
    590626        rest_get_server()->register_route(
    591             'test', '/test/embeddable', array(
     627            'test',
     628            '/test/embeddable',
     629            array(
    592630                'methods'  => 'GET',
    593631                'callback' => array( $this, 'embedded_response_callback' ),
    … …  
    684722        $server = new WP_REST_Server();
    685723        $server->register_route(
    686             'test/example', '/test/example/some-route', array(
     724            'test/example',
     725            '/test/example/some-route',
     726            array(
    687727                array(
    688728                    'methods'  => WP_REST_Server::READABLE,
    … …  
    726766        $server = new WP_REST_Server();
    727767        $server->register_route(
    728             'test/example', '/test/example/some-route', array(
     768            'test/example',
     769            '/test/example/some-route',
     770            array(
    729771                array(
    730772                    'methods'  => WP_REST_Server::READABLE,
    … …  
    738780        );
    739781        $server->register_route(
    740             'test/another', '/test/another/route', array(
     782            'test/another',
     783            '/test/another/route',
     784            array(
    741785                array(
    742786                    'methods'  => WP_REST_Server::READABLE,
    … …  
    765809        $server = new WP_REST_Server();
    766810        $server->register_route(
    767             'test/example', '/test/example/some-route', array(
     811            'test/example',
     812            '/test/example/some-route',
     813            array(
    768814                array(
    769815                    'methods'  => WP_REST_Server::READABLE,
    … …  
    773819        );
    774820        $server->register_route(
    775             'test/another', '/test/another/route', array(
     821            'test/another',
     822            '/test/another/route',
     823            array(
    776824                array(
    777825                    'methods'  => WP_REST_Server::READABLE,
    … …  
    856904
    857905        rest_get_server()->register_route(
    858             'test', '/test/(?P<data>.*)', array(
     906            'test',
     907            '/test/(?P<data>.*)',
     908            array(
    859909                array(
    860910                    'methods'  => WP_REST_Server::READABLE,
    … …  
    875925
    876926        rest_get_server()->register_route(
    877             'test', '/test', array(
     927            'test',
     928            '/test',
     929            array(
    878930                array(
    879931                    'methods'  => WP_REST_Server::READABLE,
    … …  
    901953
    902954        rest_get_server()->register_route(
    903             'test', '/test', array(
     955            'test',
     956            '/test',
     957            array(
    904958                array(
    905959                    'methods'  => WP_REST_Server::READABLE,
    … …  
    928982
    929983        rest_get_server()->register_route(
    930             'test', '/test', array(
     984            'test',
     985            '/test',
     986            array(
    931987                array(
    932988                    'methods'  => WP_REST_Server::READABLE,
    … …  
    9541010
    9551011        rest_get_server()->register_route(
    956             'test', '/test', array(
     1012            'test',
     1013            '/test',
     1014            array(
    9571015                array(
    9581016                    'methods'  => WP_REST_Server::READABLE,
    … …  
    9801038
    9811039        rest_get_server()->register_route(
    982             'test', '/test', array(
     1040            'test',
     1041            '/test',
     1042            array(
    9831043                array(
    9841044                    'methods'  => WP_REST_Server::READABLE,
    … …  
    10551115    public function test_rest_validate_before_sanitization() {
    10561116        register_rest_route(
    1057             'test-ns', '/test', array(
     1117            'test-ns',
     1118            '/test',
     1119            array(
    10581120                'methods'  => array( 'GET' ),
    10591121                'callback' => '__return_null',
Note: See TracChangeset for help on using the changeset viewer.