WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
08/17/2018 01:50:26 AM (8 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/widgets/text-widget.php

    r42617 r43571  
    189189        );
    190190        $expected_instance              = array_merge(
    191             $instance, array(
     191            $instance,
     192            array(
    192193                'filter' => true,
    193194                'visual' => true,
    … …  
    335336        // Legacy Text Widget without wpautop.
    336337        $instance                     = array_merge(
    337             $base_instance, array(
     338            $base_instance,
     339            array(
    338340                'filter' => false,
    339341            )
    … …  
    351353        // Legacy Text Widget with wpautop.
    352354        $instance                     = array_merge(
    353             $base_instance, array(
     355            $base_instance,
     356            array(
    354357                'filter' => true,
    355358                'visual' => false,
    … …  
    379382
    380383        $instance = array_merge(
    381             $base_instance, array(
     384            $base_instance,
     385            array(
    382386                'filter' => true,
    383387                'visual' => true,
    … …  
    470474
    471475        $instance = array_merge(
    472             $base_instance, array(
     476            $base_instance,
     477            array(
    473478                'visual' => false,
    474479            )
    … …  
    477482
    478483        $instance = array_merge(
    479             $base_instance, array(
     484            $base_instance,
     485            array(
    480486                'visual' => true,
    481487            )
    … …  
    484490
    485491        $instance = array_merge(
    486             $base_instance, array(
     492            $base_instance,
     493            array(
    487494                'filter' => 'content',
    488495            )
    … …  
    491498
    492499        $instance = array_merge(
    493             $base_instance, array(
     500            $base_instance,
     501            array(
    494502                'text'   => '',
    495503                'filter' => true,
    … …  
    499507
    500508        $instance = array_merge(
    501             $base_instance, array(
     509            $base_instance,
     510            array(
    502511                'text'   => "\nOne line",
    503512                'filter' => false,
    … …  
    507516
    508517        $instance = array_merge(
    509             $base_instance, array(
     518            $base_instance,
     519            array(
    510520                'text'   => "\nOne line\n\n",
    511521                'filter' => false,
    … …  
    515525
    516526        $instance = array_merge(
    517             $base_instance, array(
     527            $base_instance,
     528            array(
    518529                'text'   => "One\nTwo",
    519530                'filter' => false,
    … …  
    523534
    524535        $instance = array_merge(
    525             $base_instance, array(
     536            $base_instance,
     537            array(
    526538                'text'   => "One\n\nTwo",
    527539                'filter' => false,
    … …  
    531543
    532544        $instance = array_merge(
    533             $base_instance, array(
     545            $base_instance,
     546            array(
    534547                'text'   => "One\nTwo",
    535548                'filter' => true,
    … …  
    539552
    540553        $instance = array_merge(
    541             $base_instance, array(
     554            $base_instance,
     555            array(
    542556                'text'   => "One\n\nTwo",
    543557                'filter' => true,
    … …  
    547561
    548562        $instance = array_merge(
    549             $base_instance, array(
     563            $base_instance,
     564            array(
    550565                'text'   => 'Test<!-- comment -->',
    551566                'filter' => true,
    … …  
    569584        foreach ( $legacy_text_examples as $legacy_text_example ) {
    570585            $instance = array_merge(
    571                 $base_instance, array(
     586                $base_instance,
     587                array(
    572588                    'text'   => $legacy_text_example,
    573589                    'filter' => true,
    … …  
    577593
    578594            $instance = array_merge(
    579                 $base_instance, array(
     595                $base_instance,
     596                array(
    580597                    'text'   => $legacy_text_example,
    581598                    'filter' => false,
    … …  
    598615        foreach ( $migratable_text_examples as $migratable_text_example ) {
    599616            $instance = array_merge(
    600                 $base_instance, array(
     617                $base_instance,
     618                array(
    601619                    'text'   => $migratable_text_example,
    602620                    'filter' => true,
    … …  
    765783        );
    766784        $old_instance = array_merge(
    767             $instance, array(
     785            $instance,
     786            array(
    768787                'filter' => false,
    769788            )
    770789        );
    771790        $expected     = array_merge(
    772             $instance, array(
     791            $instance,
     792            array(
    773793                'filter' => true,
    774794            )
    … …  
    784804        );
    785805        $old_instance = array_merge(
    786             $instance, array(
     806            $instance,
     807            array(
    787808                'visual' => true,
    788809            )
    … …  
    790811        $result       = $widget->update( $instance, $old_instance );
    791812        $expected     = array_merge(
    792             $instance, array(
     813            $instance,
     814            array(
    793815                'visual' => true,
    794816            )
    … …  
    802824        );
    803825        $old_instance = array_merge(
    804             $instance, array(
     826            $instance,
     827            array(
    805828                'visual' => true,
    806829            )
    … …  
    808831        $result       = $widget->update( $instance, $old_instance );
    809832        $expected     = array_merge(
    810             $instance, array(
     833            $instance,
     834            array(
    811835                'visual' => true,
    812836                'filter' => true,
    … …  
    822846        );
    823847        $expected = array_merge(
    824             $instance, array(
     848            $instance,
     849            array(
    825850                'filter' => true,
    826851            )
    … …  
    837862        $result   = $widget->update( $instance, array() );
    838863        $expected = array_merge(
    839             $instance, array(
     864            $instance,
     865            array(
    840866                'filter' => false,
    841867            )
    … …  
    850876        );
    851877        $old_instance = array_merge(
    852             $instance, array(
     878            $instance,
     879            array(
    853880                'visual' => false,
    854881                'filter' => true,
    … …  
    857884        $result       = $widget->update( $instance, $old_instance );
    858885        $expected     = array_merge(
    859             $instance, array(
     886            $instance,
     887            array(
    860888                'visual' => false,
    861889                'filter' => false,
    … …  
    872900        $result   = $widget->update( $instance, array() );
    873901        $expected = array_merge(
    874             $instance, array(
     902            $instance,
     903            array(
    875904                'filter' => true,
    876905                'visual' => true,
    … …  
    885914        );
    886915        $old_instance = array_merge(
    887             $instance, array(
     916            $instance,
     917            array(
    888918                'filter' => 'content',
    889919            )
    … …  
    891921        $result       = $widget->update( $instance, $old_instance );
    892922        $expected     = array_merge(
    893             $instance, array(
     923            $instance,
     924            array(
    894925                'visual' => true,
    895926                'filter' => true,
    … …  
    906937        $result   = $widget->update( $instance, array() );
    907938        $expected = array_merge(
    908             $instance, array(
     939            $instance,
     940            array(
    909941                'filter' => true,
    910942                'visual' => true,
Note: See TracChangeset for help on using the changeset viewer.