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-attachments-controller.php

    r43529 r43571  
    134134        // Single
    135135        $attachment_id = $this->factory->attachment->create_object(
    136             $this->test_file, 0, array(
     136            $this->test_file,
     137            0,
     138            array(
    137139                'post_mime_type' => 'image/jpeg',
    138140                'post_excerpt'   => 'A sample caption',
    … …  
    173175                'slug',
    174176                'status',
    175             ), $keys
     177            ),
     178            $keys
    176179        );
    177180        $media_types = array(
    … …  
    189192    public function test_registered_get_item_params() {
    190193        $id1      = $this->factory->attachment->create_object(
    191             $this->test_file, 0, array(
     194            $this->test_file,
     195            0,
     196            array(
    192197                'post_mime_type' => 'image/jpeg',
    193198                'post_excerpt'   => 'A sample caption',
    … …  
    205210        wp_set_current_user( 0 );
    206211        $id1            = $this->factory->attachment->create_object(
    207             $this->test_file, 0, array(
     212            $this->test_file,
     213            0,
     214            array(
    208215                'post_mime_type' => 'image/jpeg',
    209216                'post_excerpt'   => 'A sample caption',
    … …  
    212219        $draft_post     = $this->factory->post->create( array( 'post_status' => 'draft' ) );
    213220        $id2            = $this->factory->attachment->create_object(
    214             $this->test_file, $draft_post, array(
     221            $this->test_file,
     222            $draft_post,
     223            array(
    215224                'post_mime_type' => 'image/jpeg',
    216225                'post_excerpt'   => 'A sample caption',
    … …  
    219228        $published_post = $this->factory->post->create( array( 'post_status' => 'publish' ) );
    220229        $id3            = $this->factory->attachment->create_object(
    221             $this->test_file, $published_post, array(
     230            $this->test_file,
     231            $published_post,
     232            array(
    222233                'post_mime_type' => 'image/jpeg',
    223234                'post_excerpt'   => 'A sample caption',
    … …  
    239250        wp_set_current_user( self::$editor_id );
    240251        $id1            = $this->factory->attachment->create_object(
    241             $this->test_file, 0, array(
     252            $this->test_file,
     253            0,
     254            array(
    242255                'post_mime_type' => 'image/jpeg',
    243256                'post_excerpt'   => 'A sample caption',
    … …  
    246259        $draft_post     = $this->factory->post->create( array( 'post_status' => 'draft' ) );
    247260        $id2            = $this->factory->attachment->create_object(
    248             $this->test_file, $draft_post, array(
     261            $this->test_file,
     262            $draft_post,
     263            array(
    249264                'post_mime_type' => 'image/jpeg',
    250265                'post_excerpt'   => 'A sample caption',
    … …  
    253268        $published_post = $this->factory->post->create( array( 'post_status' => 'publish' ) );
    254269        $id3            = $this->factory->attachment->create_object(
    255             $this->test_file, $published_post, array(
     270            $this->test_file,
     271            $published_post,
     272            array(
    256273                'post_mime_type' => 'image/jpeg',
    257274                'post_excerpt'   => 'A sample caption',
    … …  
    271288    public function test_get_items_media_type() {
    272289        $id1      = $this->factory->attachment->create_object(
    273             $this->test_file, 0, array(
     290            $this->test_file,
     291            0,
     292            array(
    274293                'post_mime_type' => 'image/jpeg',
    275294            )
    … …  
    292311    public function test_get_items_mime_type() {
    293312        $id1      = $this->factory->attachment->create_object(
    294             $this->test_file, 0, array(
     313            $this->test_file,
     314            0,
     315            array(
    295316                'post_mime_type' => 'image/jpeg',
    296317            )
    … …  
    314335        $post_id        = $this->factory->post->create( array( 'post_title' => 'Test Post' ) );
    315336        $attachment_id  = $this->factory->attachment->create_object(
    316             $this->test_file, $post_id, array(
     337            $this->test_file,
     338            $post_id,
     339            array(
    317340                'post_mime_type' => 'image/jpeg',
    318341                'post_excerpt'   => 'A sample caption',
    … …  
    320343        );
    321344        $attachment_id2 = $this->factory->attachment->create_object(
    322             $this->test_file, 0, array(
     345            $this->test_file,
     346            0,
     347            array(
    323348                'post_mime_type' => 'image/jpeg',
    324349                'post_excerpt'   => 'A sample caption',
    … …  
    354379        wp_set_current_user( self::$editor_id );
    355380        $this->factory->attachment->create_object(
    356             $this->test_file, 0, array(
     381            $this->test_file,
     382            0,
     383            array(
    357384                'post_mime_type' => 'image/jpeg',
    358385                'post_excerpt'   => 'A sample caption',
    … …  
    372399        wp_set_current_user( 0 );
    373400        $attachment_id1 = $this->factory->attachment->create_object(
    374             $this->test_file, 0, array(
     401            $this->test_file,
     402            0,
     403            array(
    375404                'post_mime_type' => 'image/jpeg',
    376405                'post_excerpt'   => 'A sample caption',
    … …  
    394423        wp_set_current_user( 0 );
    395424        $attachment_id1 = $this->factory->attachment->create_object(
    396             $this->test_file, 0, array(
     425            $this->test_file,
     426            0,
     427            array(
    397428                'post_mime_type' => 'image/jpeg',
    398429                'post_excerpt'   => 'A sample caption',
    … …  
    401432        );
    402433        $attachment_id2 = $this->factory->attachment->create_object(
    403             $this->test_file, 0, array(
     434            $this->test_file,
     435            0,
     436            array(
    404437                'post_mime_type' => 'image/jpeg',
    405438                'post_excerpt'   => 'A sample caption',
    … …  
    435468    public function test_get_items_valid_date() {
    436469        $id1     = $this->factory->attachment->create_object(
    437             $this->test_file, 0, array(
     470            $this->test_file,
     471            0,
     472            array(
    438473                'post_date'      => '2016-01-15T00:00:00Z',
    439474                'post_mime_type' => 'image/jpeg',
    … …  
    442477        );
    443478        $id2     = $this->factory->attachment->create_object(
    444             $this->test_file, 0, array(
     479            $this->test_file,
     480            0,
     481            array(
    445482                'post_date'      => '2016-01-16T00:00:00Z',
    446483                'post_mime_type' => 'image/jpeg',
    … …  
    449486        );
    450487        $id3     = $this->factory->attachment->create_object(
    451             $this->test_file, 0, array(
     488            $this->test_file,
     489            0,
     490            array(
    452491                'post_date'      => '2016-01-17T00:00:00Z',
    453492                'post_mime_type' => 'image/jpeg',
    … …  
    466505    public function test_get_item() {
    467506        $attachment_id = $this->factory->attachment->create_object(
    468             $this->test_file, 0, array(
     507            $this->test_file,
     508            0,
     509            array(
    469510                'post_mime_type' => 'image/jpeg',
    470511                'post_excerpt'   => 'A sample caption',
    … …  
    481522    public function test_get_item_sizes() {
    482523        $attachment_id = $this->factory->attachment->create_object(
    483             $this->test_file, 0, array(
    484                 'post_mime_type' => 'image/jpeg',
    485                 'post_excerpt'   => 'A sample caption',
    486             ), $this->test_file
     524            $this->test_file,
     525            0,
     526            array(
     527                'post_mime_type' => 'image/jpeg',
     528                'post_excerpt'   => 'A sample caption',
     529            ),
     530            $this->test_file
    487531        );
    488532
    … …  
    505549    public function test_get_item_sizes_with_no_url() {
    506550        $attachment_id = $this->factory->attachment->create_object(
    507             $this->test_file, 0, array(
    508                 'post_mime_type' => 'image/jpeg',
    509                 'post_excerpt'   => 'A sample caption',
    510             ), $this->test_file
     551            $this->test_file,
     552            0,
     553            array(
     554                'post_mime_type' => 'image/jpeg',
     555                'post_excerpt'   => 'A sample caption',
     556            ),
     557            $this->test_file
    511558        );
    512559
    … …  
    529576        $draft_post = $this->factory->post->create( array( 'post_status' => 'draft' ) );
    530577        $id1        = $this->factory->attachment->create_object(
    531             $this->test_file, $draft_post, array(
     578            $this->test_file,
     579            $draft_post,
     580            array(
    532581                'post_mime_type' => 'image/jpeg',
    533582                'post_excerpt'   => 'A sample caption',
    … …  
    541590    public function test_get_item_inherit_status_with_invalid_parent() {
    542591        $attachment_id = $this->factory->attachment->create_object(
    543             $this->test_file, REST_TESTS_IMPOSSIBLY_HIGH_NUMBER, array(
     592            $this->test_file,
     593            REST_TESTS_IMPOSSIBLY_HIGH_NUMBER,
     594            array(
    544595                'post_mime_type' => 'image/jpeg',
    545596                'post_excerpt'   => 'A sample caption',
    … …  
    556607    public function test_get_item_auto_status_with_invalid_parent_not_authenticated_returns_error() {
    557608        $attachment_id = $this->factory->attachment->create_object(
    558             $this->test_file, REST_TESTS_IMPOSSIBLY_HIGH_NUMBER, array(
     609            $this->test_file,
     610            REST_TESTS_IMPOSSIBLY_HIGH_NUMBER,
     611            array(
    559612                'post_mime_type' => 'image/jpeg',
    560613                'post_excerpt'   => 'A sample caption',
    … …  
    791844        wp_set_current_user( self::$editor_id );
    792845        $attachment_id = $this->factory->attachment->create_object(
    793             $this->test_file, 0, array(
     846            $this->test_file,
     847            0,
     848            array(
    794849                'post_mime_type' => 'image/jpeg',
    795850                'post_excerpt'   => 'A sample caption',
    … …  
    819874        $original_parent = $this->factory->post->create( array() );
    820875        $attachment_id   = $this->factory->attachment->create_object(
    821             $this->test_file, $original_parent, array(
     876            $this->test_file,
     877            $original_parent,
     878            array(
    822879                'post_mime_type' => 'image/jpeg',
    823880                'post_excerpt'   => 'A sample caption',
    … …  
    841898        wp_set_current_user( self::$author_id );
    842899        $attachment_id = $this->factory->attachment->create_object(
    843             $this->test_file, 0, array(
     900            $this->test_file,
     901            0,
     902            array(
    844903                'post_mime_type' => 'image/jpeg',
    845904                'post_excerpt'   => 'A sample caption',
    … …  
    863922        wp_set_current_user( self::$editor_id );
    864923        $attachment_id = $this->factory->attachment->create_object(
    865             $this->test_file, 0, array(
     924            $this->test_file,
     925            0,
     926            array(
    866927                'post_mime_type' => 'image/jpeg',
    867928                'post_excerpt'   => 'A sample caption',
    … …  
    10601121                    'description' => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    10611122                    'caption'     => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    1062                 ), array(
     1123                ),
     1124                array(
    10631125                    'title'       => array(
    10641126                        'raw'      => 'div <strong>strong</strong> oh noes',
    … …  
    10821144                    'description' => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    10831145                    'caption'     => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    1084                 ), array(
     1146                ),
     1147                array(
    10851148                    'title'       => array(
    10861149                        'raw'      => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    … …  
    11081171                'description' => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    11091172                'caption'     => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    1110             ), array(
     1173            ),
     1174            array(
    11111175                'title'       => array(
    11121176                    'raw'      => '<div>div</div> <strong>strong</strong> <script>oh noes</script>',
    … …  
    11281192        wp_set_current_user( self::$editor_id );
    11291193        $attachment_id    = $this->factory->attachment->create_object(
    1130             $this->test_file, 0, array(
     1194            $this->test_file,
     1195            0,
     1196            array(
    11311197                'post_mime_type' => 'image/jpeg',
    11321198                'post_excerpt'   => 'A sample caption',
    … …  
    11421208        wp_set_current_user( self::$editor_id );
    11431209        $attachment_id = $this->factory->attachment->create_object(
    1144             $this->test_file, 0, array(
     1210            $this->test_file,
     1211            0,
     1212            array(
    11451213                'post_mime_type' => 'image/jpeg',
    11461214                'post_excerpt'   => 'A sample caption',
    … …  
    11651233        wp_set_current_user( self::$author_id );
    11661234        $attachment_id = $this->factory->attachment->create_object(
    1167             $this->test_file, 0, array(
     1235            $this->test_file,
     1236            0,
     1237            array(
    11681238                'post_mime_type' => 'image/jpeg',
    11691239                'post_excerpt'   => 'A sample caption',
    … …  
    11781248    public function test_prepare_item() {
    11791249        $attachment_id = $this->factory->attachment->create_object(
    1180             $this->test_file, 0, array(
     1250            $this->test_file,
     1251            0,
     1252            array(
    11811253                'post_mime_type' => 'image/jpeg',
    11821254                'post_excerpt'   => 'A sample caption',
    … …  
    11951267    public function test_prepare_item_limit_fields() {
    11961268        $attachment_id = $this->factory->attachment->create_object(
    1197             $this->test_file, 0, array(
     1269            $this->test_file,
     1270            0,
     1271            array(
    11981272                'post_mime_type' => 'image/jpeg',
    11991273                'post_excerpt'   => 'A sample caption',
    … …  
    12081282        $obj      = get_post( $attachment_id );
    12091283        $response = $endpoint->prepare_item_for_response( $obj, $request );
    1210         $this->assertEquals( array(
    1211             'id',
    1212             'slug',
    1213         ), array_keys( $response->get_data() ) );
     1284        $this->assertEquals(
     1285            array(
     1286                'id',
     1287                'slug',
     1288            ),
     1289            array_keys( $response->get_data() )
     1290        );
    12141291    }
    12151292
    … …  
    12621339
    12631340        register_rest_field(
    1264             'attachment', 'my_custom_int', array(
     1341            'attachment',
     1342            'my_custom_int',
     1343            array(
    12651344                'schema'       => $schema,
    12661345                'get_callback' => array( $this, 'additional_field_get_callback' ),
    … …  
    12761355
    12771356        $attachment_id = $this->factory->attachment->create_object(
    1278             $this->test_file, 0, array(
     1357            $this->test_file,
     1358            0,
     1359            array(
    12791360                'post_mime_type' => 'image/jpeg',
    12801361                'post_excerpt'   => 'A sample caption',
    … …  
    13001381
    13011382        register_rest_field(
    1302             'attachment', 'my_custom_int', array(
     1383            'attachment',
     1384            'my_custom_int',
     1385            array(
    13031386                'schema'          => $schema,
    13041387                'get_callback'    => array( $this, 'additional_field_get_callback' ),
    … …  
    13091392        wp_set_current_user( self::$editor_id );
    13101393        $attachment_id = $this->factory->attachment->create_object(
    1311             $this->test_file, 0, array(
     1394            $this->test_file,
     1395            0,
     1396            array(
    13121397                'post_mime_type' => 'image/jpeg',
    13131398                'post_excerpt'   => 'A sample caption',
    … …  
    13331418    public function test_search_item_by_filename() {
    13341419        $id  = $this->factory->attachment->create_object(
    1335             $this->test_file, 0, array(
     1420            $this->test_file,
     1421            0,
     1422            array(
    13361423                'post_mime_type' => 'image/jpeg',
    13371424            )
    13381425        );
    13391426        $id2 = $this->factory->attachment->create_object(
    1340             $this->test_file2, 0, array(
     1427            $this->test_file2,
     1428            0,
     1429            array(
    13411430                'post_mime_type' => 'image/png',
    13421431            )
Note: See TracChangeset for help on using the changeset viewer.