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/widgets/media-image-widget.php

    r42343 r43571  
    5353                'url',
    5454                'width',
    55             ), array_keys( $schema )
     55            ),
     56            array_keys( $schema )
    5657        );
    5758    }
    … …  
    8182                'add_media',
    8283                'unsupported_file_type',
    83             ), array_keys( $widget->l10n )
     84            ),
     85            array_keys( $widget->l10n )
    8486        );
    8587    }
    … …  
    105107            array(
    106108                'attachment_id' => 'media',
    107             ), $instance
     109            ),
     110            $instance
    108111        );
    109112        $this->assertSame( $result, $instance );
    … …  
    120123            array(
    121124                'url' => 'not_a_url',
    122             ), $instance
     125            ),
     126            $instance
    123127        );
    124128        $this->assertNotSame( $result, $instance );
    … …  
    136140            array(
    137141                'title' => '<h1>W00t!</h1>',
    138             ), $instance
     142            ),
     143            $instance
    139144        );
    140145        $this->assertNotSame( $result, $instance );
    … …  
    151156            array(
    152157                'size' => 'big league',
    153             ), $instance
     158            ),
     159            $instance
    154160        );
    155161        $this->assertSame( $result, $instance );
    … …  
    166172            array(
    167173                'width' => 'wide',
    168             ), $instance
     174            ),
     175            $instance
    169176        );
    170177        $this->assertSame( $result, $instance );
    … …  
    181188            array(
    182189                'height' => 'high',
    183             ), $instance
     190            ),
     191            $instance
    184192        );
    185193        $this->assertSame( $result, $instance );
    … …  
    196204            array(
    197205                'caption' => '"><i onload="alert(\'hello\')" />',
    198             ), $instance
     206            ),
     207            $instance
    199208        );
    200209        $this->assertSame(
    201             $result, array(
     210            $result,
     211            array(
    202212                'caption' => '"&gt;<i />',
    203213            )
    … …  
    215225            array(
    216226                'alt' => '"><i onload="alert(\'hello\')" />',
    217             ), $instance
     227            ),
     228            $instance
    218229        );
    219230        $this->assertSame(
    220             $result, array(
     231            $result,
     232            array(
    221233                'alt' => '">',
    222234            )
    … …  
    234246            array(
    235247                'link_type' => 'interesting',
    236             ), $instance
     248            ),
     249            $instance
    237250        );
    238251        $this->assertSame( $result, $instance );
    … …  
    249262            array(
    250263                'link_url' => 'not_a_url',
    251             ), $instance
     264            ),
     265            $instance
    252266        );
    253267        $this->assertNotSame( $result, $instance );
    … …  
    265279            array(
    266280                'image_classes' => '"><i onload="alert(\'hello\')" />',
    267             ), $instance
     281            ),
     282            $instance
    268283        );
    269284        $this->assertSame(
    270             $result, array(
     285            $result,
     286            array(
    271287                'image_classes' => 'i onloadalerthello',
    272288            )
    … …  
    284300            array(
    285301                'link_classes' => '"><i onload="alert(\'hello\')" />',
    286             ), $instance
     302            ),
     303            $instance
    287304        );
    288305        $this->assertSame(
    289             $result, array(
     306            $result,
     307            array(
    290308                'link_classes' => 'i onloadalerthello',
    291309            )
    … …  
    303321            array(
    304322                'link_rel' => '"><i onload="alert(\'hello\')" />',
    305             ), $instance
     323            ),
     324            $instance
    306325        );
    307326        $this->assertSame(
    308             $result, array(
     327            $result,
     328            array(
    309329                'link_rel' => 'i onloadalerthello',
    310330            )
    … …  
    322342            array(
    323343                'link_target_blank' => 'top',
    324             ), $instance
     344            ),
     345            $instance
    325346        );
    326347        $this->assertSame( $result, $instance );
    … …  
    337358            array(
    338359                'image_title' => '<h1>W00t!</h1>',
    339             ), $instance
     360            ),
     361            $instance
    340362        );
    341363        $this->assertNotSame( $result, $instance );
    … …  
    345367            array(
    346368                'imaginary_key' => 'value',
    347             ), $instance
     369            ),
     370            $instance
    348371        );
    349372        $this->assertSame( $result, $instance );
Note: See TracChangeset for help on using the changeset viewer.