WordPress.org

Make WordPress Core


Ignore:
Timestamp:
08/17/2018 01:50:26 AM (3 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/rest-api/rest-post-meta-fields.php

    r43378 r43571  
    1616
    1717    public static function wpSetUpBeforeClass( $factory ) {
    18         register_post_type( 'cpt', array(
    19             'show_in_rest' => true,
    20             'supports'     => array( 'custom-fields' ),
    21         ) );
     18        register_post_type(
     19            'cpt',
     20            array(
     21                'show_in_rest' => true,
     22                'supports'     => array( 'custom-fields' ),
     23            )
     24        );
    2225
    2326        self::$wp_meta_keys_saved = isset( $GLOBALS['wp_meta_keys'] ) ? $GLOBALS['wp_meta_keys'] : array();
     
    3841
    3942        register_meta(
    40             'post', 'test_single', array(
     43            'post',
     44            'test_single',
     45            array(
    4146                'show_in_rest' => true,
    4247                'single'       => true,
     
    4550        );
    4651        register_meta(
    47             'post', 'test_multi', array(
     52            'post',
     53            'test_multi',
     54            array(
    4855                'show_in_rest' => true,
    4956                'single'       => false,
     
    5259        );
    5360        register_meta(
    54             'post', 'test_bad_auth', array(
     61            'post',
     62            'test_bad_auth',
     63            array(
    5564                'show_in_rest'  => true,
    5665                'single'        => true,
     
    6069        );
    6170        register_meta(
    62             'post', 'test_bad_auth_multi', array(
     71            'post',
     72            'test_bad_auth_multi',
     73            array(
    6374                'show_in_rest'  => true,
    6475                'single'        => false,
     
    6980        register_meta( 'post', 'test_no_rest', array() );
    7081        register_meta(
    71             'post', 'test_rest_disabled', array(
     82            'post',
     83            'test_rest_disabled',
     84            array(
    7285                'show_in_rest' => false,
    7386                'type'         => 'string',
     
    7588        );
    7689        register_meta(
    77             'post', 'test_custom_schema', array(
     90            'post',
     91            'test_custom_schema',
     92            array(
    7893                'single'       => true,
    7994                'type'         => 'integer',
     
    86101        );
    87102        register_meta(
    88             'post', 'test_custom_schema_multi', array(
     103            'post',
     104            'test_custom_schema_multi',
     105            array(
    89106                'single'       => false,
    90107                'type'         => 'integer',
     
    97114        );
    98115        register_meta(
    99             'post', 'test_invalid_type', array(
     116            'post',
     117            'test_invalid_type',
     118            array(
    100119                'single'       => true,
    101120                'type'         => 'lalala',
     
    104123        );
    105124        register_meta(
    106             'post', 'test_no_type', array(
     125            'post',
     126            'test_no_type',
     127            array(
    107128                'single'       => true,
    108129                'type'         => null,
     
    112133
    113134        register_meta(
    114             'post', 'test_custom_name', array(
     135            'post',
     136            'test_custom_name',
     137            array(
    115138                'single'       => true,
    116139                'type'         => 'string',
     
    122145
    123146        register_meta(
    124             'post', 'test_custom_name_multi', array(
     147            'post',
     148            'test_custom_name_multi',
     149            array(
    125150                'single'       => false,
    126151                'type'         => 'string',
     
    131156        );
    132157
    133         register_post_type( 'cpt', array(
    134             'show_in_rest' => true,
    135             'supports'     => array( 'custom-fields' ),
    136         ) );
    137 
    138         register_post_meta( 'cpt', 'test_cpt_single', array(
    139             'show_in_rest'   => true,
    140             'single'         => true,
    141         ) );
    142 
    143         register_post_meta( 'cpt', 'test_cpt_multi', array(
    144             'show_in_rest'   => true,
    145             'single'         => false,
    146         ) );
     158        register_post_type(
     159            'cpt',
     160            array(
     161                'show_in_rest' => true,
     162                'supports'     => array( 'custom-fields' ),
     163            )
     164        );
     165
     166        register_post_meta(
     167            'cpt',
     168            'test_cpt_single',
     169            array(
     170                'show_in_rest' => true,
     171                'single'       => true,
     172            )
     173        );
     174
     175        register_post_meta(
     176            'cpt',
     177            'test_cpt_multi',
     178            array(
     179                'show_in_rest' => true,
     180                'single'       => false,
     181            )
     182        );
    147183
    148184        // Register 'test_single' on subtype to override for bad auth.
    149         register_post_meta( 'cpt', 'test_single', array(
    150             'show_in_rest'   => true,
    151             'single'         => true,
    152             'auth_callback'  => '__return_false',
    153         ) );
     185        register_post_meta(
     186            'cpt',
     187            'test_single',
     188            array(
     189                'show_in_rest'  => true,
     190                'single'        => true,
     191                'auth_callback' => '__return_false',
     192            )
     193        );
    154194
    155195        /** @var WP_REST_Server $wp_rest_server */
     
    259299    public function test_get_value_types() {
    260300        register_meta(
    261             'post', 'test_string', array(
     301            'post',
     302            'test_string',
     303            array(
    262304                'show_in_rest' => true,
    263305                'single'       => true,
     
    266308        );
    267309        register_meta(
    268             'post', 'test_number', array(
     310            'post',
     311            'test_number',
     312            array(
    269313                'show_in_rest' => true,
    270314                'single'       => true,
     
    273317        );
    274318        register_meta(
    275             'post', 'test_bool', array(
     319            'post',
     320            'test_bool',
     321            array(
    276322                'show_in_rest' => true,
    277323                'single'       => true,
     
    574620    public function test_set_value_invalid_value() {
    575621        register_meta(
    576             'post', 'my_meta_key', array(
     622            'post',
     623            'my_meta_key',
     624            array(
    577625                'show_in_rest' => true,
    578626                'single'       => true,
     
    597645    public function test_set_value_invalid_value_multiple() {
    598646        register_meta(
    599             'post', 'my_meta_key', array(
     647            'post',
     648            'my_meta_key',
     649            array(
    600650                'show_in_rest' => true,
    601651                'single'       => false,
     
    620670    public function test_set_value_sanitized() {
    621671        register_meta(
    622             'post', 'my_meta_key', array(
     672            'post',
     673            'my_meta_key',
     674            array(
    623675                'show_in_rest' => true,
    624676                'single'       => true,
     
    644696    public function test_set_value_csv() {
    645697        register_meta(
    646             'post', 'my_meta_key', array(
     698            'post',
     699            'my_meta_key',
     700            array(
    647701                'show_in_rest' => true,
    648702                'single'       => false,
     
    10961150        add_post_meta( $post_id, $meta_key, $meta_value );
    10971151
    1098         $request = new WP_REST_Request( 'GET', sprintf( '/wp/v2/%s/%d', $endpoint, $post_id ) );
     1152        $request  = new WP_REST_Request( 'GET', sprintf( '/wp/v2/%s/%d', $endpoint, $post_id ) );
    10991153        $response = rest_get_server()->dispatch( $request );
    11001154
     
    11491203
    11501204        $request = new WP_REST_Request( 'POST', sprintf( '/wp/v2/%s/%d', $endpoint, $post_id ) );
    1151         $request->set_body_params( array(
    1152             'meta' => array(
    1153                 $meta_key => $meta_value,
    1154             ),
    1155         ) );
     1205        $request->set_body_params(
     1206            array(
     1207                'meta' => array(
     1208                    $meta_key => $meta_value,
     1209                ),
     1210            )
     1211        );
    11561212
    11571213        $response = rest_get_server()->dispatch( $request );
Note: See TracChangeset for help on using the changeset viewer.