WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
08/17/2018 01:50:26 AM (8 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/customize/nav-menu-item-setting.php

    r42343 r43571  
    155155                $item_title = 'Greetings';
    156156                $item_id    = wp_update_nav_menu_item(
    157                         $menu_id, 0, array(
     157                        $menu_id,
     158                        0,
     159                        array(
    158160                                'menu-item-type'      => 'post_type',
    159161                                'menu-item-object'    => 'post',
    … …  
    180182                $other_menu_id = wp_create_nav_menu( 'Menu2' );
    181183                wp_update_nav_menu_item(
    182                         $other_menu_id, $item_id, array(
     184                        $other_menu_id,
     185                        $item_id,
     186                        array(
    183187                                'menu-item-title' => 'Hola',
    184188                        )
    … …  
    202206                $menu_id = wp_create_nav_menu( 'Menu' );
    203207                $item_id = wp_update_nav_menu_item(
    204                         $menu_id, 0, array(
     208                        $menu_id,
     209                        0,
     210                        array(
    205211                                'menu-item-type'      => 'post_type',
    206212                                'menu-item-object'    => 'post',
    … …  
    232238                $item_title = 'Greetings';
    233239                $item_id    = wp_update_nav_menu_item(
    234                         $menu_id, 0, array(
     240                        $menu_id,
     241                        0,
     242                        array(
    235243                                'menu-item-type'      => 'taxonomy',
    236244                                'menu-item-object'    => 'category',
    … …  
    267275                $menu_id = wp_create_nav_menu( 'Menu' );
    268276                $item_id = wp_update_nav_menu_item(
    269                         $menu_id, 0, array(
     277                        $menu_id,
     278                        0,
     279                        array(
    270280                                'menu-item-type'   => 'custom_type',
    271281                                'menu-item-object' => 'custom_object',
    … …  
    311321                $post_id = self::factory()->post->create( array( 'post_title' => 'Hello World' ) );
    312322                $item_id = wp_update_nav_menu_item(
    313                         $menu_id, 0, array(
     323                        $menu_id,
     324                        0,
     325                        array(
    314326                                'menu-item-type'      => 'post_type',
    315327                                'menu-item-object'    => 'post',
    … …  
    344356                $item_title        = 'Greetings';
    345357                $item_id           = wp_update_nav_menu_item(
    346                         $primary_menu_id, 0, array(
     358                        $primary_menu_id,
     359                        0,
     360                        array(
    347361                                'menu-item-type'      => 'post_type',
    348362                                'menu-item-object'    => 'post',
    … …  
    395409                for ( $i = 0; $i < 5; $i += 1 ) {
    396410                        $item_id    = wp_update_nav_menu_item(
    397                                 $menu_id, 0, array(
     411                                $menu_id,
     412                                0,
     413                                array(
    398414                                        'menu-item-type'      => 'post_type',
    399415                                        'menu-item-object'    => 'post',
    … …  
    452468                for ( $i = 0; $i < 5; $i += 1 ) {
    453469                        $item_id    = wp_update_nav_menu_item(
    454                                 $menu_id, 0, array(
     470                                $menu_id,
     471                                0,
     472                                array(
    455473                                        'menu-item-type'      => 'post_type',
    456474                                        'menu-item-object'    => 'post',
    … …  
    574592
    575593                $nav_menu_item_id = wp_update_nav_menu_item(
    576                         $menu_id, 0, wp_slash(
     594                        $menu_id,
     595                        0,
     596                        wp_slash(
    577597                                array(
    578598                                        'menu-item-object-id'   => $unsanitized['object_id'],
    … …  
    626646                $item_title        = 'Greetings';
    627647                $item_id           = wp_update_nav_menu_item(
    628                         $primary_menu_id, 0, array(
     648                        $primary_menu_id,
     649                        0,
     650                        array(
    629651                                'menu-item-type'      => 'post_type',
    630652                                'menu-item-object'    => 'post',
    … …  
    691713                for ( $i = 0; $i < 5; $i += 1 ) {
    692714                        $item_id    = wp_update_nav_menu_item(
    693                                 $menu_id, 0, array(
     715                                $menu_id,
     716                                0,
     717                                array(
    694718                                        'menu-item-type'      => 'post_type',
    695719                                        'menu-item-object'    => 'post',
    … …  
    762786                for ( $i = 0; $i < 5; $i += 1 ) {
    763787                        $item_id    = wp_update_nav_menu_item(
    764                                 $menu_id, 0, array(
     788                                $menu_id,
     789                                0,
     790                                array(
    765791                                        'menu-item-type'      => 'post_type',
    766792                                        'menu-item-object'    => 'post',
    … …  
    807833                $menu_id = wp_create_nav_menu( 'Primary' );
    808834                register_post_type(
    809                         'poem', array(
     835                        'poem',
     836                        array(
    810837                                'public' => true,
    811838                        )
    … …  
    820847                $post       = get_post( $post_id );
    821848                $item_id    = wp_update_nav_menu_item(
    822                         $menu_id, 0, array(
     849                        $menu_id,
     850                        0,
     851                        array(
    823852                                'menu-item-type'      => 'post_type',
    824853                                'menu-item-object'    => 'poem',
    … …  
    921950                $term_id = self::factory()->term->create( array( 'taxonomy' => 'category' ) );
    922951                register_post_type(
    923                         'press_release', array(
     952                        'press_release',
     953                        array(
    924954                                'has_archive' => true,
    925955                        )
    … …  
    933963                );
    934964                $this->wp_customize->set_post_value(
    935                         $setting->id, array(
     965                        $setting->id,
     966                        array(
    936967                                'type'      => 'taxonomy',
    937968                                'object'    => 'category',
    … …  
    951982                );
    952983                $this->wp_customize->set_post_value(
    953                         $setting->id, array(
     984                        $setting->id,
     985                        array(
    954986                                'type'      => 'post_type',
    955987                                'object'    => 'press_release',
    … …  
    9691001                );
    9701002                $this->wp_customize->set_post_value(
    971                         $setting->id, array(
     1003                        $setting->id,
     1004                        array(
    9721005                                'type'   => 'post_type_archive',
    9731006                                'object' => 'press_release',
    … …  
    9901023                $menu_id = wp_create_nav_menu( 'Menu' );
    9911024                register_post_type(
    992                         'press_release', array(
     1025                        'press_release',
     1026                        array(
    9931027                                'has_archive' => true,
    9941028                                'labels'      => array(
    … …  
    10161050                // Post: existing nav menu item.
    10171051                $nav_menu_item_id = wp_update_nav_menu_item(
    1018                         $menu_id, 0, array(
     1052                        $menu_id,
     1053                        0,
     1054                        array(
    10191055                                'menu-item-object-id' => $post_id,
    10201056                                'menu-item-type'      => 'post_type',
    … …  
    10451081                );
    10461082                $this->wp_customize->set_post_value(
    1047                         $setting->id, array(
     1083                        $setting->id,
     1084                        array(
    10481085                                'object_id' => $post_id,
    10491086                                'type'      => 'post_type',
    … …  
    10671104                // Term: existing nav menu item.
    10681105                $nav_menu_item_id = wp_update_nav_menu_item(
    1069                         $menu_id, 0, array(
     1106                        $menu_id,
     1107                        0,
     1108                        array(
    10701109                                'menu-item-object-id' => $term_id,
    10711110                                'menu-item-type'      => 'taxonomy',
    … …  
    10961135                );
    10971136                $this->wp_customize->set_post_value(
    1098                         $setting->id, array(
     1137                        $setting->id,
     1138                        array(
    10991139                                'object_id' => $term_id,
    11001140                                'type'      => 'taxonomy',
    … …  
    11181158                // Post Type Archive: existing nav menu item.
    11191159                $nav_menu_item_id = wp_update_nav_menu_item(
    1120                         $menu_id, 0, array(
     1160                        $menu_id,
     1161                        0,
     1162                        array(
    11211163                                'menu-item-type'   => 'post_type_archive',
    11221164                                'menu-item-object' => 'press_release',
    … …  
    11461188                );
    11471189                $this->wp_customize->set_post_value(
    1148                         $setting->id, array(
     1190                        $setting->id,
     1191                        array(
    11491192                                'type'   => 'post_type_archive',
    11501193                                'object' => 'press_release',
Note: See TracChangeset for help on using the changeset viewer.