WordPress.org

Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (4 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/meta.php

    r38762 r42343  
    99    function setUp() {
    1010        parent::setUp();
    11         $this->author = new WP_User( self::factory()->user->create( array( 'role' => 'author' ) ) );
    12         $this->meta_id = add_metadata( 'user', $this->author->ID, 'meta_key', 'meta_value' );
     11        $this->author         = new WP_User( self::factory()->user->create( array( 'role' => 'author' ) ) );
     12        $this->meta_id        = add_metadata( 'user', $this->author->ID, 'meta_key', 'meta_value' );
    1313        $this->delete_meta_id = add_metadata( 'user', $this->author->ID, 'delete_meta_key', 'delete_meta_value' );
    1414    }
    1515
    16     function _meta_sanitize_cb ( $meta_value, $meta_key, $meta_type ) {
     16    function _meta_sanitize_cb( $meta_value, $meta_key, $meta_type ) {
    1717        return 'sanitized';
    1818    }
     
    6969        // Let's try some invalid meta data
    7070        $this->assertFalse( update_metadata_by_mid( 'user', 0, 'meta_value' ) );
    71         $this->assertFalse( update_metadata_by_mid( 'user', $this->meta_id, 'meta_value', array('invalid', 'key' ) ) );
     71        $this->assertFalse( update_metadata_by_mid( 'user', $this->meta_id, 'meta_value', array( 'invalid', 'key' ) ) );
    7272
    7373        // Let's see if caches get cleared after updates.
    74         $meta = get_metadata_by_mid( 'user', $this->meta_id );
     74        $meta  = get_metadata_by_mid( 'user', $this->meta_id );
    7575        $first = get_user_meta( $meta->user_id, $meta->meta_key );
    7676        $this->assertTrue( update_metadata_by_mid( 'user', $this->meta_id, 'other_meta_value' ) );
     
    9797        remove_action( 'updated_postmeta', array( $this, 'updated_meta' ) );
    9898
    99         $found = $this->updated_mids;
     99        $found              = $this->updated_mids;
    100100        $this->updated_mids = array();
    101101
     
    106106
    107107    function test_metadata_exists() {
    108         $this->assertFalse( metadata_exists( 'user',  $this->author->ID, 'foobarbaz' ) );
    109         $this->assertTrue( metadata_exists( 'user',  $this->author->ID, 'meta_key' ) );
    110         $this->assertFalse( metadata_exists( 'user',  1234567890, 'foobarbaz' ) );
    111         $this->assertFalse( metadata_exists( 'user',  1234567890, 'meta_key' ) );
     108        $this->assertFalse( metadata_exists( 'user', $this->author->ID, 'foobarbaz' ) );
     109        $this->assertTrue( metadata_exists( 'user', $this->author->ID, 'meta_key' ) );
     110        $this->assertFalse( metadata_exists( 'user', 1234567890, 'foobarbaz' ) );
     111        $this->assertFalse( metadata_exists( 'user', 1234567890, 'meta_key' ) );
    112112    }
    113113
     
    127127     */
    128128    function test_user_metadata_not_exists() {
    129         $u = get_users( array(
    130             'meta_query' => array(
    131                 array( 'key' => 'meta_key', 'compare' => 'NOT EXISTS' )
    132             )
    133         ) );
     129        $u = get_users(
     130            array(
     131                'meta_query' => array(
     132                    array(
     133                        'key'     => 'meta_key',
     134                        'compare' => 'NOT EXISTS',
     135                    ),
     136                ),
     137            )
     138        );
    134139
    135140        $this->assertEquals( 1, count( $u ) );
     
    139144
    140145        // Test EXISTS and NOT EXISTS together, no users should be found
    141         $this->assertEquals( 0, count( get_users( array(
    142             'meta_query' => array(
    143                 array( 'key' => 'meta_key', 'compare' => 'NOT EXISTS' ),
    144                 array( 'key' => 'delete_meta_key', 'compare' => 'EXISTS' )
    145             )
    146         ) ) ) );
    147 
    148         $this->assertEquals( 2, count( get_users( array(
    149             'meta_query' => array(
    150                 array( 'key' => 'non_existing_meta', 'compare' => 'NOT EXISTS' )
    151             )
    152         ) ) ) );
     146        $this->assertEquals(
     147            0, count(
     148                get_users(
     149                    array(
     150                        'meta_query' => array(
     151                            array(
     152                                'key'     => 'meta_key',
     153                                'compare' => 'NOT EXISTS',
     154                            ),
     155                            array(
     156                                'key'     => 'delete_meta_key',
     157                                'compare' => 'EXISTS',
     158                            ),
     159                        ),
     160                    )
     161                )
     162            )
     163        );
     164
     165        $this->assertEquals(
     166            2, count(
     167                get_users(
     168                    array(
     169                        'meta_query' => array(
     170                            array(
     171                                'key'     => 'non_existing_meta',
     172                                'compare' => 'NOT EXISTS',
     173                            ),
     174                        ),
     175                    )
     176                )
     177            )
     178        );
    153179
    154180        delete_metadata( 'user', $this->author->ID, 'meta_key' );
    155181
    156         $this->assertEquals( 2, count( get_users( array(
    157             'meta_query' => array(
    158                 array( 'key' => 'meta_key', 'compare' => 'NOT EXISTS' )
    159             )
    160         ) ) ) );
     182        $this->assertEquals(
     183            2, count(
     184                get_users(
     185                    array(
     186                        'meta_query' => array(
     187                            array(
     188                                'key'     => 'meta_key',
     189                                'compare' => 'NOT EXISTS',
     190                            ),
     191                        ),
     192                    )
     193                )
     194            )
     195        );
    161196    }
    162197
    163198    function test_metadata_slashes() {
    164         $key = __FUNCTION__;
    165         $value = 'Test\\singleslash';
    166         $expected = 'Testsingleslash';
    167         $value2 = 'Test\\\\doubleslash';
     199        $key       = __FUNCTION__;
     200        $value     = 'Test\\singleslash';
     201        $expected  = 'Testsingleslash';
     202        $value2    = 'Test\\\\doubleslash';
    168203        $expected2 = 'Test\\doubleslash';
    169204        $this->assertFalse( metadata_exists( 'user', $this->author->ID, $key ) );
     
    202237        add_post_meta( $post_id2, 'num_as_longtext_desc', 100 );
    203238
    204         $posts = new WP_Query( array(
    205             'fields' => 'ids',
    206             'post_type' => 'any',
    207             'meta_key' => 'num_as_longtext',
    208             'meta_value' => '0',
    209             'meta_compare' => '>',
    210             'meta_type' => 'UNSIGNED',
    211             'orderby' => 'meta_value',
    212             'order' => 'ASC'
    213         ) );
     239        $posts = new WP_Query(
     240            array(
     241                'fields'       => 'ids',
     242                'post_type'    => 'any',
     243                'meta_key'     => 'num_as_longtext',
     244                'meta_value'   => '0',
     245                'meta_compare' => '>',
     246                'meta_type'    => 'UNSIGNED',
     247                'orderby'      => 'meta_value',
     248                'order'        => 'ASC',
     249            )
     250        );
    214251
    215252        $this->assertEquals( array( $post_id2, $post_id1 ), $posts->posts );
     
    217254
    218255        // Make sure the newer meta_query syntax behaves in a consistent way
    219         $posts = new WP_Query( array(
    220             'fields' => 'ids',
    221             'post_type' => 'any',
    222             'meta_query' => array(
    223                 array(
    224                     'key' => 'num_as_longtext',
    225                     'value' => '0',
    226                     'compare' => '>',
    227                     'type' => 'UNSIGNED',
     256        $posts = new WP_Query(
     257            array(
     258                'fields'     => 'ids',
     259                'post_type'  => 'any',
     260                'meta_query' => array(
     261                    array(
     262                        'key'     => 'num_as_longtext',
     263                        'value'   => '0',
     264                        'compare' => '>',
     265                        'type'    => 'UNSIGNED',
     266                    ),
    228267                ),
    229             ),
    230             'orderby' => 'meta_value',
    231             'order' => 'ASC'
    232         ) );
     268                'orderby'    => 'meta_value',
     269                'order'      => 'ASC',
     270            )
     271        );
    233272
    234273        $this->assertEquals( array( $post_id2, $post_id1 ), $posts->posts );
     
    236275
    237276        // The legacy `meta_key` value should take precedence.
    238         $posts = new WP_Query( array(
    239             'fields' => 'ids',
    240             'post_type' => 'any',
    241             'meta_key' => 'num_as_longtext',
    242             'meta_compare' => '>',
    243             'meta_type' => 'UNSIGNED',
    244             'meta_query' => array(
    245                 array(
    246                     'key' => 'num_as_longtext_desc',
    247                     'value' => '0',
    248                     'compare' => '>',
    249                     'type' => 'UNSIGNED',
     277        $posts = new WP_Query(
     278            array(
     279                'fields'       => 'ids',
     280                'post_type'    => 'any',
     281                'meta_key'     => 'num_as_longtext',
     282                'meta_compare' => '>',
     283                'meta_type'    => 'UNSIGNED',
     284                'meta_query'   => array(
     285                    array(
     286                        'key'     => 'num_as_longtext_desc',
     287                        'value'   => '0',
     288                        'compare' => '>',
     289                        'type'    => 'UNSIGNED',
     290                    ),
    250291                ),
    251             ),
    252             'orderby' => 'meta_value',
    253             'order' => 'ASC'
    254         ) );
     292                'orderby'      => 'meta_value',
     293                'order'        => 'ASC',
     294            )
     295        );
    255296
    256297        $this->assertEquals( array( $post_id2, $post_id1 ), $posts->posts );
     
    260301    function test_meta_cache_order_asc() {
    261302        $post_id = self::factory()->post->create();
    262         $colors = array( 'red', 'blue', 'yellow', 'green' );
    263         foreach ( $colors as $color )
     303        $colors  = array( 'red', 'blue', 'yellow', 'green' );
     304        foreach ( $colors as $color ) {
    264305            add_post_meta( $post_id, 'color', $color );
     306        }
    265307
    266308        foreach ( range( 1, 10 ) as $i ) {
     
    268310            $this->assertEquals( $meta, $colors );
    269311
    270             if ( 0 === $i % 2 )
     312            if ( 0 === $i % 2 ) {
    271313                wp_cache_delete( $post_id, 'post_meta' );
     314            }
    272315        }
    273316    }
     
    335378     */
    336379    public function test_get_metadata_with_empty_key_array_value() {
    337         $data = array( 1, 2 );
     380        $data  = array( 1, 2 );
    338381        $value = serialize( $data );
    339382        add_metadata( 'user', $this->author->ID, 'foo', $data );
     
    347390     */
    348391    public function test_get_metadata_with_empty_key_object_value() {
    349         $data = new stdClass;
     392        $data      = new stdClass;
    350393        $data->foo = 'bar';
     394        $value     = serialize( $data );
     395        add_metadata( 'user', $this->author->ID, 'foo', $data );
     396        $found = get_metadata( 'user', $this->author->ID );
     397
     398        $this->assertEquals( array( $value ), $found['foo'] );
     399    }
     400
     401    /**
     402     * @ticket 15030
     403     */
     404    public function test_get_metadata_with_empty_key_nested_array_value() {
     405        $data  = array(
     406            array( 1, 2 ),
     407            array( 3, 4 ),
     408        );
    351409        $value = serialize( $data );
    352410        add_metadata( 'user', $this->author->ID, 'foo', $data );
    353411        $found = get_metadata( 'user', $this->author->ID );
    354412
    355         $this->assertEquals( array( $value ), $found['foo'] );
    356     }
    357 
    358     /**
    359      * @ticket 15030
    360      */
    361     public function test_get_metadata_with_empty_key_nested_array_value() {
    362         $data = array(
    363             array( 1, 2 ),
    364             array( 3, 4 ),
    365         );
    366         $value = serialize( $data );
    367         add_metadata( 'user', $this->author->ID, 'foo', $data );
    368         $found = get_metadata( 'user', $this->author->ID );
    369 
    370413        $this->assertSame( array( $value ), $found['foo'] );
    371414    }
    372415
    373     /** Helpers **********************************************************/
     416    /** Helpers */
    374417
    375418    public function updated_meta( $meta_id ) {
Note: See TracChangeset for help on using the changeset viewer.