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/term/wpUpdateTerm.php

    r42343 r43571  
    2828
    2929        $found = wp_update_term(
    30             $t, 'wptests_tax', array(
     30            $t,
     31            'wptests_tax',
     32            array(
    3133                'name' => 'Let\\\'s all say \\"Hooray\\" for WordPress taxonomy',
    3234            )
    … …  
    4850
    4951        $found = wp_update_term(
    50             $t, 'wptests_tax', array(
     52            $t,
     53            'wptests_tax',
     54            array(
    5155                'description' => 'Let\\\'s all say \\"Hooray\\" for WordPress taxonomy',
    5256            )
    … …  
    6872
    6973        $found = wp_update_term(
    70             $t, 'wptests_tax', array(
     74            $t,
     75            'wptests_tax',
     76            array(
    7177                'name' => '',
    7278            )
    … …  
    8389    function test_wp_update_term_parent_does_not_exist() {
    8490        register_taxonomy(
    85             'wptests_tax', array(
     91            'wptests_tax',
     92            array(
    8693                'hierarchical' => true,
    8794            )
    … …  
    98105
    99106        $found = wp_update_term(
    100             $t, 'wptests_tax', array(
     107            $t,
     108            'wptests_tax',
     109            array(
    101110                'parent' => $fake_term_id,
    102111            )
    … …  
    121130
    122131        $found = wp_update_term(
    123             $t, 'wptests_tax', array(
     132            $t,
     133            'wptests_tax',
     134            array(
    124135                'slug' => '',
    125136            )
    … …  
    140151
    141152        $found = wp_update_term(
    142             $t, 'wptests_tax', array(
     153            $t,
     154            'wptests_tax',
     155            array(
    143156                'name' => 'Foo Bar',
    144157                'slug' => '',
    … …  
    160173
    161174        $found = wp_update_term(
    162             $t, 'wptests_tax', array(
     175            $t,
     176            'wptests_tax',
     177            array(
    163178                'slug' => 'foo-bar',
    164179            )
    … …  
    193208
    194209        $updated = wp_update_term(
    195             $t2, 'wptests_tax', array(
     210            $t2,
     211            'wptests_tax',
     212            array(
    196213                'slug' => 'foo',
    197214            )
    … …  
    226243
    227244        $updated = wp_update_term(
    228             $t2, 'wptests_tax_2', array(
     245            $t2,
     246            'wptests_tax_2',
     247            array(
    229248                'slug' => 'foo',
    230249            )
    … …  
    262281
    263282        $updated = wp_update_term(
    264             $t2, 'wptests_tax_2', array(
     283            $t2,
     284            'wptests_tax_2',
     285            array(
    265286                'name' => 'Foo',
    266287            )
    … …  
    278299    public function test_wp_update_term_should_allow_duplicate_names_at_different_levels_of_the_same_taxonomy() {
    279300        register_taxonomy(
    280             'wptests_tax', 'post', array(
     301            'wptests_tax',
     302            'post',
     303            array(
    281304                'hierarchical' => true,
    282305            )
    … …  
    310333
    311334        $updated = wp_update_term(
    312             $t3, 'wptests_tax', array(
     335            $t3,
     336            'wptests_tax',
     337            array(
    313338                'name' => 'Bar',
    314339            )
    … …  
    352377
    353378        wp_update_term(
    354             $t2_terms[0]->term_id, 'wptests_tax_2', array(
     379            $t2_terms[0]->term_id,
     380            'wptests_tax_2',
     381            array(
    355382                'name' => 'New Foo',
    356383            )
    … …  
    373400        $created_term_ids = wp_insert_term( 'Foo', 'wptests_tax' );
    374401        wp_update_term(
    375             $created_term_ids['term_id'], 'wptests_tax', array(
     402            $created_term_ids['term_id'],
     403            'wptests_tax',
     404            array(
    376405                'alias_of' => $term_1->slug,
    377406            )
    … …  
    406435        $created_term_ids = wp_insert_term( 'Foo', 'wptests_tax' );
    407436        wp_update_term(
    408             $created_term_ids['term_id'], 'wptests_tax', array(
     437            $created_term_ids['term_id'],
     438            'wptests_tax',
     439            array(
    409440                'alias_of' => $term_2->slug,
    410441            )
    … …  
    421452        $created_term_ids = wp_insert_term( 'Foo', 'wptests_tax' );
    422453        wp_update_term(
    423             $created_term_ids['term_id'], 'wptests_tax', array(
     454            $created_term_ids['term_id'],
     455            'wptests_tax',
     456            array(
    424457                'alias_of' => 'bar',
    425458            )
    … …  
    441474
    442475        $found = wp_update_term(
    443             $t, 'wptests_tax', array(
     476            $t,
     477            'wptests_tax',
     478            array(
    444479                'slug' => 'foo',
    445480            )
    … …  
    469504
    470505        $found = wp_update_term(
    471             $t2, 'wptests_tax', array(
     506            $t2,
     507            'wptests_tax',
     508            array(
    472509                'slug' => '',
    473510                'name' => 'Foo? Bar!', // Will sanitize to 'foo-bar'.
    … …  
    484521    public function test_wp_update_term_duplicate_slug_with_changed_parent() {
    485522        register_taxonomy(
    486             'wptests_tax', 'post', array(
     523            'wptests_tax',
     524            'post',
     525            array(
    487526                'hierarchical' => true,
    488527            )
    … …  
    506545
    507546        $found = wp_update_term(
    508             $t2, 'wptests_tax', array(
     547            $t2,
     548            'wptests_tax',
     549            array(
    509550                'parent' => $p,
    510551                'slug'   => 'foo-bar',
    … …  
    536577
    537578        $found = wp_update_term(
    538             $t2, 'wptests_tax', array(
     579            $t2,
     580            'wptests_tax',
     581            array(
    539582                'slug' => 'foo-bar',
    540583            )
    … …  
    557600        );
    558601        $found = wp_update_term(
    559             $t, 'wptests_tax', array(
     602            $t,
     603            'wptests_tax',
     604            array(
    560605                'slug' => 'foo',
    561606            )
    … …  
    587632        );
    588633        $found = wp_update_term(
    589             $t, 'wptests_tax', array(
     634            $t,
     635            'wptests_tax',
     636            array(
    590637                'slug' => 'foo',
    591638            )
    … …  
    598645    public function test_wp_update_term_should_clean_term_cache() {
    599646        register_taxonomy(
    600             'wptests_tax', 'post', array(
     647            'wptests_tax',
     648            'post',
     649            array(
    601650                'hierarchical' => true,
    602651            )
    … …  
    624673
    625674        $found = wp_update_term(
    626             $t1, 'wptests_tax', array(
     675            $t1,
     676            'wptests_tax',
     677            array(
    627678                'parent' => $t2,
    628679            )
    … …  
    643694    public function test_wp_update_term_should_assign_new_slug_when_reassigning_parent_as_long_as_there_is_no_other_term_with_the_same_slug() {
    644695        register_taxonomy(
    645             'wptests_tax', 'post', array(
     696            'wptests_tax',
     697            'post',
     698            array(
    646699                'hierarchical' => true,
    647700            )
    648701        );
    649702        register_taxonomy(
    650             'wptests_tax_2', 'post', array(
     703            'wptests_tax_2',
     704            'post',
     705            array(
    651706                'hierarchical' => true,
    652707            )
    … …  
    668723
    669724        wp_update_term(
    670             $t2, 'wptests_tax', array(
     725            $t2,
     726            'wptests_tax',
     727            array(
    671728                'parent' => $t1,
    672729            )
    … …  
    685742    public function test_wp_update_term_should_not_assign_new_slug_when_reassigning_parent_as_long_as_there_is_no_other_slug_conflict_within_the_taxonomy() {
    686743        register_taxonomy(
    687             'wptests_tax', 'post', array(
     744            'wptests_tax',
     745            'post',
     746            array(
    688747                'hierarchical' => true,
    689748            )
    690749        );
    691750        register_taxonomy(
    692             'wptests_tax_2', 'post', array(
     751            'wptests_tax_2',
     752            'post',
     753            array(
    693754                'hierarchical' => true,
    694755            )
    … …  
    718779
    719780        wp_update_term(
    720             $t3, 'wptests_tax', array(
     781            $t3,
     782            'wptests_tax',
     783            array(
    721784                'parent' => $t1,
    722785            )
Note: See TracChangeset for help on using the changeset viewer.