WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (7 years ago)
Author:
pento
Message:

Code is Poetry.
WordPress' code just... wasn't.
This is now dealt with.

Props jrf, pento, netweb, GaryJ, jdgrimes, westonruter, Greg Sherwood from PHPCS, and everyone who's ever contributed to WPCS and PHPCS.
Fixes #41057.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/tests/phpunit/tests/term/wpSetObjectTerms.php

    r39174 r42343  
    55 */
    66class Tests_Term_WpSetObjectTerms extends WP_UnitTestCase {
    7     protected $taxonomy = 'category';
     7    protected $taxonomy        = 'category';
    88    protected static $post_ids = array();
    99
    … …  
    104104
    105105        $terms = array();
    106         for ($i=0; $i<3; $i++ ) {
    107             $term = "term_{$i}";
     106        for ( $i = 0; $i < 3; $i++ ) {
     107            $term   = "term_{$i}";
    108108            $result = wp_insert_term( $term, $this->taxonomy );
    109109            $this->assertInternalType( 'array', $result );
    110             $term_id[$term] = $result['term_id'];
    111         }
    112 
    113         foreach ($ids as $id) {
    114             $tt = wp_set_object_terms( $id, array_values($term_id), $this->taxonomy );
     110            $term_id[ $term ] = $result['term_id'];
     111        }
     112
     113        foreach ( $ids as $id ) {
     114            $tt = wp_set_object_terms( $id, array_values( $term_id ), $this->taxonomy );
    115115            // should return three term taxonomy ids
    116             $this->assertEquals( 3, count($tt) );
     116            $this->assertEquals( 3, count( $tt ) );
    117117        }
    118118
    119119        // each term should be associated with every post
    120         foreach ($term_id as $term=>$id) {
    121             $actual = get_objects_in_term($id, $this->taxonomy);
    122             $this->assertEquals( $ids, array_map('intval', $actual) );
     120        foreach ( $term_id as $term => $id ) {
     121            $actual = get_objects_in_term( $id, $this->taxonomy );
     122            $this->assertEquals( $ids, array_map( 'intval', $actual ) );
    123123        }
    124124
    125125        // each term should have a count of 5
    126         foreach (array_keys($term_id) as $term) {
    127             $t = get_term_by('name', $term, $this->taxonomy);
     126        foreach ( array_keys( $term_id ) as $term ) {
     127            $t = get_term_by( 'name', $term, $this->taxonomy );
    128128            $this->assertEquals( 5, $t->count );
    129129        }
    … …  
    136136            rand_str(),
    137137            rand_str(),
    138             rand_str()
    139         );
    140 
    141         foreach ($ids as $id) {
     138            rand_str(),
     139        );
     140
     141        foreach ( $ids as $id ) {
    142142            $tt = wp_set_object_terms( $id, $terms, $this->taxonomy );
    143143            // should return three term taxonomy ids
    144             $this->assertEquals( 3, count($tt) );
     144            $this->assertEquals( 3, count( $tt ) );
    145145            // remember which term has which term_id
    146             for ($i=0; $i<3; $i++) {
    147                 $term = get_term_by('name', $terms[$i], $this->taxonomy);
    148                 $term_id[$terms[$i]] = intval($term->term_id);
     146            for ( $i = 0; $i < 3; $i++ ) {
     147                $term                    = get_term_by( 'name', $terms[ $i ], $this->taxonomy );
     148                $term_id[ $terms[ $i ] ] = intval( $term->term_id );
    149149            }
    150150        }
    151151
    152152        // each term should be associated with every post
    153         foreach ($term_id as $term=>$id) {
    154             $actual = get_objects_in_term($id, $this->taxonomy);
    155             $this->assertEquals( $ids, array_map('intval', $actual) );
     153        foreach ( $term_id as $term => $id ) {
     154            $actual = get_objects_in_term( $id, $this->taxonomy );
     155            $this->assertEquals( $ids, array_map( 'intval', $actual ) );
    156156        }
    157157
    158158        // each term should have a count of 5
    159         foreach ($terms as $term) {
    160             $t = get_term_by('name', $term, $this->taxonomy);
     159        foreach ( $terms as $term ) {
     160            $t = get_term_by( 'name', $term, $this->taxonomy );
    161161            $this->assertEquals( 5, $t->count );
    162162        }
    … …  
    165165    function test_set_object_terms_invalid() {
    166166        // bogus taxonomy
    167         $result = wp_set_object_terms( self::$post_ids[0], array(rand_str()), rand_str() );
     167        $result = wp_set_object_terms( self::$post_ids[0], array( rand_str() ), rand_str() );
    168168        $this->assertWPError( $result );
    169169    }
    … …  
    171171    public function test_wp_set_object_terms_append_true() {
    172172        register_taxonomy( 'wptests_tax', 'post' );
    173         $p = self::$post_ids[0];
    174         $t1 = self::factory()->term->create( array(
    175             'taxonomy' => 'wptests_tax',
    176         ) );
    177         $t2 = self::factory()->term->create( array(
    178             'taxonomy' => 'wptests_tax',
    179         ) );
     173        $p  = self::$post_ids[0];
     174        $t1 = self::factory()->term->create(
     175            array(
     176                'taxonomy' => 'wptests_tax',
     177            )
     178        );
     179        $t2 = self::factory()->term->create(
     180            array(
     181                'taxonomy' => 'wptests_tax',
     182            )
     183        );
    180184
    181185        $added1 = wp_set_object_terms( $p, array( $t1 ), 'wptests_tax' );
    … …  
    192196    public function test_wp_set_object_terms_append_false() {
    193197        register_taxonomy( 'wptests_tax', 'post' );
    194         $p = self::$post_ids[0];
    195         $t1 = self::factory()->term->create( array(
    196             'taxonomy' => 'wptests_tax',
    197         ) );
    198         $t2 = self::factory()->term->create( array(
    199             'taxonomy' => 'wptests_tax',
    200         ) );
     198        $p  = self::$post_ids[0];
     199        $t1 = self::factory()->term->create(
     200            array(
     201                'taxonomy' => 'wptests_tax',
     202            )
     203        );
     204        $t2 = self::factory()->term->create(
     205            array(
     206                'taxonomy' => 'wptests_tax',
     207            )
     208        );
    201209
    202210        $added1 = wp_set_object_terms( $p, array( $t1 ), 'wptests_tax' );
    … …  
    213221    public function test_wp_set_object_terms_append_default_to_false() {
    214222        register_taxonomy( 'wptests_tax', 'post' );
    215         $p = self::$post_ids[0];
    216         $t1 = self::factory()->term->create( array(
    217             'taxonomy' => 'wptests_tax',
    218         ) );
    219         $t2 = self::factory()->term->create( array(
    220             'taxonomy' => 'wptests_tax',
    221         ) );
     223        $p  = self::$post_ids[0];
     224        $t1 = self::factory()->term->create(
     225            array(
     226                'taxonomy' => 'wptests_tax',
     227            )
     228        );
     229        $t2 = self::factory()->term->create(
     230            array(
     231                'taxonomy' => 'wptests_tax',
     232            )
     233        );
    222234
    223235        $added1 = wp_set_object_terms( $p, array( $t1 ), 'wptests_tax' );
    … …  
    239251        // first set: 3 terms
    240252        $terms_1 = array();
    241         for ($i=0; $i<3; $i++ ) {
    242             $term = "term_{$i}";
     253        for ( $i = 0; $i < 3; $i++ ) {
     254            $term   = "term_{$i}";
    243255            $result = wp_insert_term( $term, $this->taxonomy );
    244256            $this->assertInternalType( 'array', $result );
    245             $terms_1[$i] = $result['term_id'];
     257            $terms_1[ $i ] = $result['term_id'];
    246258        }
    247259
    248260        // second set: one of the original terms, plus one new term
    249         $terms_2 = array();
     261        $terms_2    = array();
    250262        $terms_2[0] = $terms_1[1];
    251263
    252         $term = 'term';
    253         $result = wp_insert_term( $term, $this->taxonomy );
     264        $term       = 'term';
     265        $result     = wp_insert_term( $term, $this->taxonomy );
    254266        $terms_2[1] = $result['term_id'];
    255 
    256267
    257268        // set the initial terms
    258269        $tt_1 = wp_set_object_terms( $post_id, $terms_1, $this->taxonomy );
    259         $this->assertEquals( 3, count($tt_1) );
     270        $this->assertEquals( 3, count( $tt_1 ) );
    260271
    261272        // make sure they're correct
    262         $terms = wp_get_object_terms($post_id, $this->taxonomy, array('fields' => 'ids', 'orderby' => 'term_id'));
     273        $terms = wp_get_object_terms(
     274            $post_id, $this->taxonomy, array(
     275                'fields'  => 'ids',
     276                'orderby' => 'term_id',
     277            )
     278        );
    263279        $this->assertEquals( $terms_1, $terms );
    264280
    265281        // change the terms
    266282        $tt_2 = wp_set_object_terms( $post_id, $terms_2, $this->taxonomy );
    267         $this->assertEquals( 2, count($tt_2) );
     283        $this->assertEquals( 2, count( $tt_2 ) );
    268284
    269285        // make sure they're correct
    270         $terms = wp_get_object_terms($post_id, $this->taxonomy, array('fields' => 'ids', 'orderby' => 'term_id'));
     286        $terms = wp_get_object_terms(
     287            $post_id, $this->taxonomy, array(
     288                'fields'  => 'ids',
     289                'orderby' => 'term_id',
     290            )
     291        );
    271292        $this->assertEquals( $terms_2, $terms );
    272293
    … …  
    281302        $post_id = self::$post_ids[0];
    282303
    283         $terms_1 = array('foo', 'bar', 'baz');
    284         $terms_2 = array('bar', 'bing');
     304        $terms_1 = array( 'foo', 'bar', 'baz' );
     305        $terms_2 = array( 'bar', 'bing' );
    285306
    286307        // set the initial terms
    287308        $tt_1 = wp_set_object_terms( $post_id, $terms_1, $this->taxonomy );
    288         $this->assertEquals( 3, count($tt_1) );
     309        $this->assertEquals( 3, count( $tt_1 ) );
    289310
    290311        // make sure they're correct
    291         $terms = wp_get_object_terms($post_id, $this->taxonomy, array('fields' => 'names', 'orderby' => 'term_id'));
     312        $terms = wp_get_object_terms(
     313            $post_id, $this->taxonomy, array(
     314                'fields'  => 'names',
     315                'orderby' => 'term_id',
     316            )
     317        );
    292318        $this->assertEquals( $terms_1, $terms );
    293319
    294320        // change the terms
    295321        $tt_2 = wp_set_object_terms( $post_id, $terms_2, $this->taxonomy );
    296         $this->assertEquals( 2, count($tt_2) );
     322        $this->assertEquals( 2, count( $tt_2 ) );
    297323
    298324        // make sure they're correct
    299         $terms = wp_get_object_terms($post_id, $this->taxonomy, array('fields' => 'names', 'orderby' => 'term_id'));
     325        $terms = wp_get_object_terms(
     326            $post_id, $this->taxonomy, array(
     327                'fields'  => 'names',
     328                'orderby' => 'term_id',
     329            )
     330        );
    300331        $this->assertEquals( $terms_2, $terms );
    301332
    … …  
    321352        register_taxonomy( 'wptests_tax', 'post' );
    322353
    323         $t = self::factory()->term->create( array(
    324             'taxonomy' => 'wptests_tax',
    325             'slug' => 'foo',
    326             'name' => 'Bar',
    327         ) );
     354        $t = self::factory()->term->create(
     355            array(
     356                'taxonomy' => 'wptests_tax',
     357                'slug'     => 'foo',
     358                'name'     => 'Bar',
     359            )
     360        );
    328361
    329362        $tt_ids = wp_set_object_terms( self::$post_ids[0], 'foo', 'wptests_tax' );
    … …  
    338371        register_taxonomy( 'wptests_tax', 'post' );
    339372
    340         $t = self::factory()->term->create( array(
    341             'taxonomy' => 'wptests_tax',
    342             'slug' => 'foo',
    343             'name' => 'Bar',
    344         ) );
     373        $t = self::factory()->term->create(
     374            array(
     375                'taxonomy' => 'wptests_tax',
     376                'slug'     => 'foo',
     377                'name'     => 'Bar',
     378            )
     379        );
    345380
    346381        $tt_ids = wp_set_object_terms( self::$post_ids[0], 'Bar', 'wptests_tax' );
    … …  
    355390        register_taxonomy( 'wptests_tax', 'post' );
    356391
    357         $t1 = self::factory()->term->create( array(
    358             'taxonomy' => 'wptests_tax',
    359             'slug' => 'foo',
    360             'name' => 'Bar',
    361         ) );
    362 
    363         $t2 = self::factory()->term->create( array(
    364             'taxonomy' => 'wptests_tax',
    365             'slug' => 'bar',
    366             'name' => 'Foo',
    367         ) );
     392        $t1 = self::factory()->term->create(
     393            array(
     394                'taxonomy' => 'wptests_tax',
     395                'slug'     => 'foo',
     396                'name'     => 'Bar',
     397            )
     398        );
     399
     400        $t2 = self::factory()->term->create(
     401            array(
     402                'taxonomy' => 'wptests_tax',
     403                'slug'     => 'bar',
     404                'name'     => 'Foo',
     405            )
     406        );
    368407
    369408        $tt_ids = wp_set_object_terms( self::$post_ids[0], 'Bar', 'wptests_tax' );
Note: See TracChangeset for help on using the changeset viewer.