Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (6 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/rest-api/rest-post-meta-fields.php

    r42228 r42343  
    1616    public static function wpSetUpBeforeClass( $factory ) {
    1717        self::$wp_meta_keys_saved = $GLOBALS['wp_meta_keys'];
    18         self::$post_id = $factory->post->create();
     18        self::$post_id            = $factory->post->create();
    1919    }
    2020
     
    2727        parent::setUp();
    2828
    29         register_meta( 'post', 'test_single', array(
    30             'show_in_rest' => true,
    31             'single' => true,
    32             'type' => 'string',
    33         ));
    34         register_meta( 'post', 'test_multi', array(
    35             'show_in_rest' => true,
    36             'single' => false,
    37             'type' => 'string',
    38         ));
    39         register_meta( 'post', 'test_bad_auth', array(
    40             'show_in_rest' => true,
    41             'single' => true,
    42             'auth_callback' => '__return_false',
    43             'type' => 'string',
    44         ));
    45         register_meta( 'post', 'test_bad_auth_multi', array(
    46             'show_in_rest' => true,
    47             'single' => false,
    48             'auth_callback' => '__return_false',
    49             'type' => 'string',
    50         ));
     29        register_meta(
     30            'post', 'test_single', array(
     31                'show_in_rest' => true,
     32                'single'       => true,
     33                'type'         => 'string',
     34            )
     35        );
     36        register_meta(
     37            'post', 'test_multi', array(
     38                'show_in_rest' => true,
     39                'single'       => false,
     40                'type'         => 'string',
     41            )
     42        );
     43        register_meta(
     44            'post', 'test_bad_auth', array(
     45                'show_in_rest'  => true,
     46                'single'        => true,
     47                'auth_callback' => '__return_false',
     48                'type'          => 'string',
     49            )
     50        );
     51        register_meta(
     52            'post', 'test_bad_auth_multi', array(
     53                'show_in_rest'  => true,
     54                'single'        => false,
     55                'auth_callback' => '__return_false',
     56                'type'          => 'string',
     57            )
     58        );
    5159        register_meta( 'post', 'test_no_rest', array() );
    52         register_meta( 'post', 'test_rest_disabled', array(
    53             'show_in_rest' => false,
    54             'type' => 'string',
    55         ));
    56         register_meta( 'post', 'test_custom_schema', array(
    57             'single' => true,
    58             'type' => 'integer',
    59             'show_in_rest' => array(
    60                 'schema' => array(
    61                     'type' => 'number',
     60        register_meta(
     61            'post', 'test_rest_disabled', array(
     62                'show_in_rest' => false,
     63                'type'         => 'string',
     64            )
     65        );
     66        register_meta(
     67            'post', 'test_custom_schema', array(
     68                'single'       => true,
     69                'type'         => 'integer',
     70                'show_in_rest' => array(
     71                    'schema' => array(
     72                        'type' => 'number',
     73                    ),
    6274                ),
    63             ),
    64         ));
    65         register_meta( 'post', 'test_custom_schema_multi', array(
    66             'single' => false,
    67             'type' => 'integer',
    68             'show_in_rest' => array(
    69                 'schema' => array(
    70                     'type' => 'number',
     75            )
     76        );
     77        register_meta(
     78            'post', 'test_custom_schema_multi', array(
     79                'single'       => false,
     80                'type'         => 'integer',
     81                'show_in_rest' => array(
     82                    'schema' => array(
     83                        'type' => 'number',
     84                    ),
    7185                ),
    72             ),
    73         ));
    74         register_meta( 'post', 'test_invalid_type', array(
    75             'single' => true,
    76             'type' => 'lalala',
    77             'show_in_rest' => true,
    78         ));
    79         register_meta( 'post', 'test_no_type', array(
    80             'single' => true,
    81             'type' => null,
    82             'show_in_rest' => true,
    83         ));
    84 
    85         register_meta( 'post', 'test_custom_name', array(
    86             'single' => true,
    87             'type' => 'string',
    88             'show_in_rest' => array(
    89                 'name'  => 'new_name',
    90             ),
    91         ));
    92 
    93         register_meta( 'post', 'test_custom_name_multi', array(
    94             'single' => false,
    95             'type' => 'string',
    96             'show_in_rest' => array(
    97                 'name'  => 'new_name_multi',
    98             ),
    99         ));
     86            )
     87        );
     88        register_meta(
     89            'post', 'test_invalid_type', array(
     90                'single'       => true,
     91                'type'         => 'lalala',
     92                'show_in_rest' => true,
     93            )
     94        );
     95        register_meta(
     96            'post', 'test_no_type', array(
     97                'single'       => true,
     98                'type'         => null,
     99                'show_in_rest' => true,
     100            )
     101        );
     102
     103        register_meta(
     104            'post', 'test_custom_name', array(
     105                'single'       => true,
     106                'type'         => 'string',
     107                'show_in_rest' => array(
     108                    'name' => 'new_name',
     109                ),
     110            )
     111        );
     112
     113        register_meta(
     114            'post', 'test_custom_name_multi', array(
     115                'single'       => false,
     116                'type'         => 'string',
     117                'show_in_rest' => array(
     118                    'name' => 'new_name_multi',
     119                ),
     120            )
     121        );
    100122
    101123        /** @var WP_REST_Server $wp_rest_server */
     
    107129    protected function grant_write_permission() {
    108130        // Ensure we have write permission.
    109         $user = $this->factory->user->create( array(
    110             'role' => 'editor',
    111         ));
     131        $user = $this->factory->user->create(
     132            array(
     133                'role' => 'editor',
     134            )
     135        );
    112136        wp_set_current_user( $user );
    113137    }
     
    116140        add_post_meta( self::$post_id, 'test_single', 'testvalue' );
    117141
    118         $request = new WP_REST_Request( 'GET', sprintf( '/wp/v2/posts/%d', self::$post_id ) );
     142        $request  = new WP_REST_Request( 'GET', sprintf( '/wp/v2/posts/%d', self::$post_id ) );
    119143        $response = $this->server->dispatch( $request );
    120144
     
    202226
    203227    public function test_get_value_types() {
    204         register_meta( 'post', 'test_string', array(
    205             'show_in_rest' => true,
    206             'single' => true,
    207             'type' => 'string',
    208         ));
    209         register_meta( 'post', 'test_number', array(
    210             'show_in_rest' => true,
    211             'single' => true,
    212             'type' => 'number',
    213         ));
    214         register_meta( 'post', 'test_bool', array(
    215             'show_in_rest' => true,
    216             'single' => true,
    217             'type' => 'boolean',
    218         ));
     228        register_meta(
     229            'post', 'test_string', array(
     230                'show_in_rest' => true,
     231                'single'       => true,
     232                'type'         => 'string',
     233            )
     234        );
     235        register_meta(
     236            'post', 'test_number', array(
     237                'show_in_rest' => true,
     238                'single'       => true,
     239                'type'         => 'number',
     240            )
     241        );
     242        register_meta(
     243            'post', 'test_bool', array(
     244                'show_in_rest' => true,
     245                'single'       => true,
     246                'type'         => 'boolean',
     247            )
     248        );
    219249
    220250        /** @var WP_REST_Server $wp_rest_server */
     
    227257        add_post_meta( self::$post_id, 'test_bool', 1 );
    228258
    229         $request = new WP_REST_Request( 'GET', sprintf( '/wp/v2/posts/%d', self::$post_id ) );
     259        $request  = new WP_REST_Request( 'GET', sprintf( '/wp/v2/posts/%d', self::$post_id ) );
    230260        $response = $this->server->dispatch( $request );
    231261        $this->assertEquals( 200, $response->get_status() );
     
    250280        add_post_meta( self::$post_id, 'test_custom_name', 'janet' );
    251281
    252         $request = new WP_REST_Request( 'GET', sprintf( '/wp/v2/posts/%d', self::$post_id ) );
     282        $request  = new WP_REST_Request( 'GET', sprintf( '/wp/v2/posts/%d', self::$post_id ) );
    253283        $response = $this->server->dispatch( $request );
    254284
     
    273303        $this->grant_write_permission();
    274304
    275         $data = array(
     305        $data    = array(
    276306            'meta' => array(
    277307                'test_single' => 'test_value',
     
    305335        $this->grant_write_permission();
    306336
    307         $data = array(
     337        $data    = array(
    308338            'meta' => array(
    309339                'test_single' => 'test_value',
     
    404434        $this->grant_write_permission();
    405435
    406         $data = array(
     436        $data    = array(
    407437            'meta' => array(
    408438                'test_invalid_type' => 'test_value',
     
    423453        $this->grant_write_permission();
    424454
    425         $data = array(
     455        $data    = array(
    426456            'meta' => array(
    427457                'test_multi' => array( 'val1' ),
     
    467497        $this->grant_write_permission();
    468498
    469         $data = array(
     499        $data    = array(
    470500            'meta' => array(
    471501                'test_multi' => array( 'c', 'n' ),
     
    495525        wp_set_current_user( 0 );
    496526
    497         $data = array(
     527        $data    = array(
    498528            'meta' => array(
    499529                'test_multi' => array( 'val1' ),
     
    511541
    512542    public function test_set_value_invalid_value() {
    513         register_meta( 'post', 'my_meta_key', array(
    514             'show_in_rest' => true,
    515             'single' => true,
    516             'type' => 'string',
    517         ));
    518 
    519         $this->grant_write_permission();
    520 
    521         $data = array(
     543        register_meta(
     544            'post', 'my_meta_key', array(
     545                'show_in_rest' => true,
     546                'single'       => true,
     547                'type'         => 'string',
     548            )
     549        );
     550
     551        $this->grant_write_permission();
     552
     553        $data    = array(
    522554            'meta' => array(
    523555                'my_meta_key' => array( 'c', 'n' ),
     
    532564
    533565    public function test_set_value_invalid_value_multiple() {
    534         register_meta( 'post', 'my_meta_key', array(
    535             'show_in_rest' => true,
    536             'single' => false,
    537             'type' => 'string',
    538         ));
    539 
    540         $this->grant_write_permission();
    541 
    542         $data = array(
     566        register_meta(
     567            'post', 'my_meta_key', array(
     568                'show_in_rest' => true,
     569                'single'       => false,
     570                'type'         => 'string',
     571            )
     572        );
     573
     574        $this->grant_write_permission();
     575
     576        $data    = array(
    543577            'meta' => array(
    544578                'my_meta_key' => array( array( 'a' ) ),
     
    553587
    554588    public function test_set_value_sanitized() {
    555         register_meta( 'post', 'my_meta_key', array(
    556             'show_in_rest' => true,
    557             'single' => true,
    558             'type' => 'integer',
    559         ));
    560 
    561         $this->grant_write_permission();
    562 
    563         $data = array(
     589        register_meta(
     590            'post', 'my_meta_key', array(
     591                'show_in_rest' => true,
     592                'single'       => true,
     593                'type'         => 'integer',
     594            )
     595        );
     596
     597        $this->grant_write_permission();
     598
     599        $data    = array(
    564600            'meta' => array(
    565601                'my_meta_key' => '1', // Set to a string.
     
    570606
    571607        $response = $this->server->dispatch( $request );
    572         $data = $response->get_data();
     608        $data     = $response->get_data();
    573609        $this->assertEquals( 1, $data['meta']['my_meta_key'] );
    574610    }
    575611
    576612    public function test_set_value_csv() {
    577         register_meta( 'post', 'my_meta_key', array(
    578             'show_in_rest' => true,
    579             'single' => false,
    580             'type' => 'integer',
    581         ));
    582 
    583         $this->grant_write_permission();
    584 
    585         $data = array(
     613        register_meta(
     614            'post', 'my_meta_key', array(
     615                'show_in_rest' => true,
     616                'single'       => false,
     617                'type'         => 'integer',
     618            )
     619        );
     620
     621        $this->grant_write_permission();
     622
     623        $data    = array(
    586624            'meta' => array(
    587625                'my_meta_key' => '1,2,3', // Set to a string.
     
    592630
    593631        $response = $this->server->dispatch( $request );
    594         $data = $response->get_data();
     632        $data     = $response->get_data();
    595633        $this->assertEquals( array( 1, 2, 3 ), $data['meta']['my_meta_key'] );
    596634    }
     
    623661        $this->grant_write_permission();
    624662
    625         $data = array(
     663        $data    = array(
    626664            'meta' => array(
    627665                'test_multi' => array( 'val1' ),
     
    656694        $this->grant_write_permission();
    657695
    658         $data = array(
     696        $data    = array(
    659697            'meta' => array(
    660698                'test_custom_schema' => 3,
     
    685723        $this->grant_write_permission();
    686724
    687         $data = array(
     725        $data    = array(
    688726            'meta' => array(
    689727                'test_custom_schema_multi' => array( 2 ),
     
    729767        $this->grant_write_permission();
    730768
    731         $data = array(
     769        $data    = array(
    732770            'meta' => array(
    733771                'new_name' => 'janet',
     
    758796        $this->grant_write_permission();
    759797
    760         $data = array(
     798        $data    = array(
    761799            'meta' => array(
    762800                'new_name_multi' => array( 'janet' ),
     
    858896        $this->grant_write_permission();
    859897
    860         $data = array(
     898        $data    = array(
    861899            'meta' => array(
    862900                'test_multi' => array(),
     
    889927        $this->grant_write_permission();
    890928
    891         $data = array(
     929        $data    = array(
    892930            'meta' => array(
    893931                'test_single' => null,
     
    914952        $this->grant_write_permission();
    915953
    916         $data = array(
     954        $data    = array(
    917955            'meta' => array(
    918956                'test_bad_auth' => null,
     
    939977        $this->grant_write_permission();
    940978
    941         $data = array(
     979        $data    = array(
    942980            'meta' => array(
    943981                'test_single' => null,
     
    9681006        $this->grant_write_permission();
    9691007
    970         $data = array(
     1008        $data    = array(
    9711009            'meta' => array(
    9721010                'new_name' => null,
     
    9841022
    9851023    public function test_get_schema() {
    986         $request = new WP_REST_Request( 'OPTIONS', sprintf( '/wp/v2/posts/%d', self::$post_id ) );
    987         $response = $this->server->dispatch( $request );
    988 
    989         $data = $response->get_data();
     1024        $request  = new WP_REST_Request( 'OPTIONS', sprintf( '/wp/v2/posts/%d', self::$post_id ) );
     1025        $response = $this->server->dispatch( $request );
     1026
     1027        $data   = $response->get_data();
    9901028        $schema = $data['schema'];
    9911029
Note: See TracChangeset for help on using the changeset viewer.