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-term-meta-fields.php

    r43378 r43571  
    1616
    1717    public static function wpSetUpBeforeClass( $factory ) {
    18         register_taxonomy( 'customtax', 'post', array(
    19             'show_in_rest' => true,
    20         ) );
     18        register_taxonomy(
     19            'customtax',
     20            'post',
     21            array(
     22                'show_in_rest' => true,
     23            )
     24        );
    2125
    2226        self::$wp_meta_keys_saved = isset( $GLOBALS['wp_meta_keys'] ) ? $GLOBALS['wp_meta_keys'] : array();
     
    3741
    3842        register_meta(
    39             'term', 'test_single', array(
     43            'term',
     44            'test_single',
     45            array(
    4046                'show_in_rest' => true,
    4147                'single'       => true,
     
    4450        );
    4551        register_meta(
    46             'term', 'test_multi', array(
     52            'term',
     53            'test_multi',
     54            array(
    4755                'show_in_rest' => true,
    4856                'single'       => false,
     
    5159        );
    5260        register_meta(
    53             'term', 'test_bad_auth', array(
     61            'term',
     62            'test_bad_auth',
     63            array(
    5464                'show_in_rest'  => true,
    5565                'single'        => true,
     
    5969        );
    6070        register_meta(
    61             'term', 'test_bad_auth_multi', array(
     71            'term',
     72            'test_bad_auth_multi',
     73            array(
    6274                'show_in_rest'  => true,
    6375                'single'        => false,
     
    6880        register_meta( 'term', 'test_no_rest', array() );
    6981        register_meta(
    70             'term', 'test_rest_disabled', array(
     82            'term',
     83            'test_rest_disabled',
     84            array(
    7185                'show_in_rest' => false,
    7286                'type'         => 'string',
     
    7488        );
    7589        register_meta(
    76             'term', 'test_custom_schema', array(
     90            'term',
     91            'test_custom_schema',
     92            array(
    7793                'single'       => true,
    7894                'type'         => 'integer',
     
    85101        );
    86102        register_meta(
    87             'term', 'test_custom_schema_multi', array(
     103            'term',
     104            'test_custom_schema_multi',
     105            array(
    88106                'single'       => false,
    89107                'type'         => 'integer',
     
    96114        );
    97115        register_meta(
    98             'term', 'test_invalid_type', array(
     116            'term',
     117            'test_invalid_type',
     118            array(
    99119                'single'       => true,
    100120                'type'         => 'lalala',
     
    103123        );
    104124        register_meta(
    105             'term', 'test_no_type', array(
     125            'term',
     126            'test_no_type',
     127            array(
    106128                'single'       => true,
    107129                'type'         => null,
     
    111133
    112134        register_meta(
    113             'term', 'test_custom_name', array(
     135            'term',
     136            'test_custom_name',
     137            array(
    114138                'single'       => true,
    115139                'type'         => 'string',
     
    121145
    122146        register_meta(
    123             'term', 'test_custom_name_multi', array(
     147            'term',
     148            'test_custom_name_multi',
     149            array(
    124150                'single'       => false,
    125151                'type'         => 'string',
     
    130156        );
    131157
    132         register_taxonomy( 'customtax', 'post', array(
    133             'show_in_rest' => true,
    134         ) );
    135 
    136         register_term_meta( 'customtax', 'test_customtax_single', array(
    137             'show_in_rest'   => true,
    138             'single'         => true,
    139         ) );
    140 
    141         register_term_meta( 'customtax', 'test_customtax_multi', array(
    142             'show_in_rest'   => true,
    143             'single'         => false,
    144         ) );
     158        register_taxonomy(
     159            'customtax',
     160            'post',
     161            array(
     162                'show_in_rest' => true,
     163            )
     164        );
     165
     166        register_term_meta(
     167            'customtax',
     168            'test_customtax_single',
     169            array(
     170                'show_in_rest' => true,
     171                'single'       => true,
     172            )
     173        );
     174
     175        register_term_meta(
     176            'customtax',
     177            'test_customtax_multi',
     178            array(
     179                'show_in_rest' => true,
     180                'single'       => false,
     181            )
     182        );
    145183
    146184        // Register 'test_single' on subtype to override for bad auth.
    147         register_term_meta( 'customtax', 'test_single', array(
    148             'show_in_rest'   => true,
    149             'single'         => true,
    150             'auth_callback'  => '__return_false',
    151         ) );
     185        register_term_meta(
     186            'customtax',
     187            'test_single',
     188            array(
     189                'show_in_rest'  => true,
     190                'single'        => true,
     191                'auth_callback' => '__return_false',
     192            )
     193        );
    152194
    153195        /** @var WP_REST_Server $wp_rest_server */
     
    257299    public function test_get_value_types() {
    258300        register_meta(
    259             'term', 'test_string', array(
     301            'term',
     302            'test_string',
     303            array(
    260304                'show_in_rest' => true,
    261305                'single'       => true,
     
    264308        );
    265309        register_meta(
    266             'term', 'test_number', array(
     310            'term',
     311            'test_number',
     312            array(
    267313                'show_in_rest' => true,
    268314                'single'       => true,
     
    271317        );
    272318        register_meta(
    273             'term', 'test_bool', array(
     319            'term',
     320            'test_bool',
     321            array(
    274322                'show_in_rest' => true,
    275323                'single'       => true,
     
    572620    public function test_set_value_invalid_value() {
    573621        register_meta(
    574             'term', 'my_meta_key', array(
     622            'term',
     623            'my_meta_key',
     624            array(
    575625                'show_in_rest' => true,
    576626                'single'       => true,
     
    595645    public function test_set_value_invalid_value_multiple() {
    596646        register_meta(
    597             'term', 'my_meta_key', array(
     647            'term',
     648            'my_meta_key',
     649            array(
    598650                'show_in_rest' => true,
    599651                'single'       => false,
     
    618670    public function test_set_value_sanitized() {
    619671        register_meta(
    620             'term', 'my_meta_key', array(
     672            'term',
     673            'my_meta_key',
     674            array(
    621675                'show_in_rest' => true,
    622676                'single'       => true,
     
    642696    public function test_set_value_csv() {
    643697        register_meta(
    644             'term', 'my_meta_key', array(
     698            'term',
     699            'my_meta_key',
     700            array(
    645701                'show_in_rest' => true,
    646702                'single'       => false,
     
    10941150        add_term_meta( $term_id, $meta_key, $meta_value );
    10951151
    1096         $request = new WP_REST_Request( 'GET', sprintf( '/wp/v2/%s/%d', $endpoint, $term_id ) );
     1152        $request  = new WP_REST_Request( 'GET', sprintf( '/wp/v2/%s/%d', $endpoint, $term_id ) );
    10971153        $response = rest_get_server()->dispatch( $request );
    10981154
     
    11471203
    11481204        $request = new WP_REST_Request( 'POST', sprintf( '/wp/v2/%s/%d', $endpoint, $term_id ) );
    1149         $request->set_body_params( array(
    1150             'meta' => array(
    1151                 $meta_key => $meta_value,
    1152             ),
    1153         ) );
     1205        $request->set_body_params(
     1206            array(
     1207                'meta' => array(
     1208                    $meta_key => $meta_value,
     1209                ),
     1210            )
     1211        );
    11541212
    11551213        $response = rest_get_server()->dispatch( $request );
Note: See TracChangeset for help on using the changeset viewer.