Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (8 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/customize/nav-menu-setting.php

    r38858 r42343  
    2727        global $wp_customize;
    2828        $this->wp_customize = new WP_Customize_Manager();
    29         $wp_customize = $this->wp_customize;
     29        $wp_customize       = $this->wp_customize;
    3030    }
    3131
     
    115115        do_action( 'customize_register', $this->wp_customize );
    116116        $default = array(
    117             'name' => 'Lorem \\o/',
     117            'name'        => 'Lorem \\o/',
    118118            'description' => 'ipsum \\o/',
    119             'parent' => 123,
     119            'parent'      => 123,
    120120        );
    121121        $setting = new WP_Customize_Nav_Menu_Setting( $this->wp_customize, 'nav_menu[-5]', compact( 'default' ) );
     
    132132        do_action( 'customize_register', $this->wp_customize );
    133133
    134         $menu_name = 'Test 123 \\o/';
     134        $menu_name      = 'Test 123 \\o/';
    135135        $parent_menu_id = wp_create_nav_menu( wp_slash( "Parent $menu_name" ) );
    136         $description = 'Hello my world \\o/.';
    137         $menu_id = wp_update_nav_menu_object( 0, wp_slash( array(
    138             'menu-name' => $menu_name,
    139             'parent' => $parent_menu_id,
    140             'description' => $description,
    141         ) ) );
     136        $description    = 'Hello my world \\o/.';
     137        $menu_id        = wp_update_nav_menu_object(
     138            0, wp_slash(
     139                array(
     140                    'menu-name'   => $menu_name,
     141                    'parent'      => $parent_menu_id,
     142                    'description' => $description,
     143                )
     144            )
     145        );
    142146
    143147        $setting_id = "nav_menu[$menu_id]";
    144         $setting = new WP_Customize_Nav_Menu_Setting( $this->wp_customize, $setting_id );
     148        $setting    = new WP_Customize_Nav_Menu_Setting( $this->wp_customize, $setting_id );
    145149
    146150        $value = $setting->value();
     
    167171        do_action( 'customize_register', $this->wp_customize );
    168172
    169         $menu_id = wp_update_nav_menu_object( 0, wp_slash( array(
    170             'menu-name' => 'Name 1 \\o/',
    171             'description' => 'Description 1 \\o/',
    172             'parent' => 0,
    173         ) ) );
     173        $menu_id    = wp_update_nav_menu_object(
     174            0, wp_slash(
     175                array(
     176                    'menu-name'   => 'Name 1 \\o/',
     177                    'description' => 'Description 1 \\o/',
     178                    'parent'      => 0,
     179                )
     180            )
     181        );
    174182        $setting_id = "nav_menu[$menu_id]";
    175         $setting = new WP_Customize_Nav_Menu_Setting( $this->wp_customize, $setting_id );
     183        $setting    = new WP_Customize_Nav_Menu_Setting( $this->wp_customize, $setting_id );
    176184
    177185        $nav_menu_options = $this->get_nav_menu_items_option();
     
    179187
    180188        $post_value = array(
    181             'name' => 'Name 2 \\o/',
     189            'name'        => 'Name 2 \\o/',
    182190            'description' => 'Description 2 \\o/',
    183             'parent' => 1,
    184             'auto_add' => true,
     191            'parent'      => 1,
     192            'auto_add'    => true,
    185193        );
    186194        $this->wp_customize->set_post_value( $setting_id, $post_value );
     
    213221        $this->assertContains( $menu_id, $nav_menu_options['auto_add'] );
    214222
    215         $menus = wp_get_nav_menus();
     223        $menus     = wp_get_nav_menus();
    216224        $menus_ids = wp_list_pluck( $menus, 'term_id' );
    217         $i = array_search( $menu_id, $menus_ids );
     225        $i         = array_search( $menu_id, $menus_ids );
    218226        $this->assertInternalType( 'int', $i, 'Update-previewed menu does not appear in wp_get_nav_menus()' );
    219227        $filtered_menu = $menus[ $i ];
     
    229237        do_action( 'customize_register', $this->wp_customize );
    230238
    231         $menu_id = -123;
     239        $menu_id    = -123;
    232240        $post_value = array(
    233             'name' => 'New Menu Name 1 \\o/',
     241            'name'        => 'New Menu Name 1 \\o/',
    234242            'description' => 'New Menu Description 1 \\o/',
    235             'parent' => 0,
    236             'auto_add' => false,
     243            'parent'      => 0,
     244            'auto_add'    => false,
    237245        );
    238246        $setting_id = "nav_menu[$menu_id]";
    239         $setting = new WP_Customize_Nav_Menu_Setting( $this->wp_customize, $setting_id );
     247        $setting    = new WP_Customize_Nav_Menu_Setting( $this->wp_customize, $setting_id );
    240248
    241249        $this->wp_customize->set_post_value( $setting->id, $post_value );
     
    258266        $this->assertNotContains( $menu_id, $nav_menu_options['auto_add'] );
    259267
    260         $menus = wp_get_nav_menus();
     268        $menus     = wp_get_nav_menus();
    261269        $menus_ids = wp_list_pluck( $menus, 'term_id' );
    262         $i = array_search( $menu_id, $menus_ids );
     270        $i         = array_search( $menu_id, $menus_ids );
    263271        $this->assertInternalType( 'int', $i, 'Insert-previewed menu was not injected into wp_get_nav_menus()' );
    264272        $filtered_menu = $menus[ $i ];
     
    274282        do_action( 'customize_register', $this->wp_customize );
    275283
    276         $menu_id = wp_update_nav_menu_object( 0, wp_slash( array(
    277             'menu-name' => 'Name 1 \\o/',
    278             'description' => 'Description 1 \\o/',
    279             'parent' => 0,
    280         ) ) );
    281         $setting_id = "nav_menu[$menu_id]";
    282         $setting = new WP_Customize_Nav_Menu_Setting( $this->wp_customize, $setting_id );
    283         $nav_menu_options = $this->get_nav_menu_items_option();
     284        $menu_id                        = wp_update_nav_menu_object(
     285            0, wp_slash(
     286                array(
     287                    'menu-name'   => 'Name 1 \\o/',
     288                    'description' => 'Description 1 \\o/',
     289                    'parent'      => 0,
     290                )
     291            )
     292        );
     293        $setting_id                     = "nav_menu[$menu_id]";
     294        $setting                        = new WP_Customize_Nav_Menu_Setting( $this->wp_customize, $setting_id );
     295        $nav_menu_options               = $this->get_nav_menu_items_option();
    284296        $nav_menu_options['auto_add'][] = $menu_id;
    285297        update_option( 'nav_menu_options', $nav_menu_options );
     
    312324        $this->assertNull( $setting->sanitize( 123 ) );
    313325
    314         $value = array(
    315             'name' => ' Hello \\o/ <b>world</b> ',
     326        $value     = array(
     327            'name'        => ' Hello \\o/ <b>world</b> ',
    316328            'description' => "New\nline \\o/",
    317             'parent' => -12,
    318             'auto_add' => true,
    319             'extra' => 'ignored',
     329            'parent'      => -12,
     330            'auto_add'    => true,
     331            'extra'       => 'ignored',
    320332        );
    321333        $sanitized = $setting->sanitize( $value );
     
    327339
    328340        $value['name'] = '    '; // Blank spaces.
    329         $sanitized = $setting->sanitize( $value );
     341        $sanitized     = $setting->sanitize( $value );
    330342        $this->assertEquals( '(unnamed)', $sanitized['name'] );
    331343    }
     
    339351        do_action( 'customize_register', $this->wp_customize );
    340352
    341         $menu_id = wp_update_nav_menu_object( 0, wp_slash( array(
    342             'menu-name' => 'Name 1 \\o/',
    343             'description' => 'Description 1 \\o/',
    344             'parent' => 0,
    345         ) ) );
    346         $nav_menu_options = $this->get_nav_menu_items_option();
     353        $menu_id                        = wp_update_nav_menu_object(
     354            0, wp_slash(
     355                array(
     356                    'menu-name'   => 'Name 1 \\o/',
     357                    'description' => 'Description 1 \\o/',
     358                    'parent'      => 0,
     359                )
     360            )
     361        );
     362        $nav_menu_options               = $this->get_nav_menu_items_option();
    347363        $nav_menu_options['auto_add'][] = $menu_id;
    348364        update_option( 'nav_menu_options', $nav_menu_options );
    349365
    350366        $setting_id = "nav_menu[$menu_id]";
    351         $setting = new WP_Customize_Nav_Menu_Setting( $this->wp_customize, $setting_id );
    352 
    353         $auto_add = false;
     367        $setting    = new WP_Customize_Nav_Menu_Setting( $this->wp_customize, $setting_id );
     368
     369        $auto_add  = false;
    354370        $new_value = array(
    355             'name' => 'Name 2 \\o/',
     371            'name'        => 'Name 2 \\o/',
    356372            'description' => 'Description 2 \\o/',
    357             'parent' => 1,
    358             'auto_add' => $auto_add,
     373            'parent'      => 1,
     374            'auto_add'    => $auto_add,
    359375        );
    360376
     
    399415        do_action( 'customize_register', $this->wp_customize );
    400416
    401         $menu_id = -123;
     417        $menu_id    = -123;
    402418        $post_value = array(
    403             'name' => 'New Menu Name 1 \\o/',
     419            'name'        => 'New Menu Name 1 \\o/',
    404420            'description' => 'New Menu Description 1 \\o/',
    405             'parent' => 0,
    406             'auto_add' => true,
     421            'parent'      => 0,
     422            'auto_add'    => true,
    407423        );
    408424        $setting_id = "nav_menu[$menu_id]";
    409         $setting = new WP_Customize_Nav_Menu_Setting( $this->wp_customize, $setting_id );
     425        $setting    = new WP_Customize_Nav_Menu_Setting( $this->wp_customize, $setting_id );
    410426
    411427        $this->wp_customize->set_post_value( $setting->id, $post_value );
     
    451467        wp_update_nav_menu_object( 0, wp_slash( array( 'menu-name' => $menu_name ) ) );
    452468
    453         $menu_id = -123;
     469        $menu_id    = -123;
    454470        $setting_id = "nav_menu[$menu_id]";
    455         $setting = new WP_Customize_Nav_Menu_Setting( $this->wp_customize, $setting_id );
     471        $setting    = new WP_Customize_Nav_Menu_Setting( $this->wp_customize, $setting_id );
    456472        $this->wp_customize->set_post_value( $setting->id, array( 'name' => $menu_name ) );
    457473        $setting->save();
    458474
    459475        $expected_resolved_menu_name = "$menu_name (2)";
    460         $new_menu = wp_get_nav_menu_object( $setting->term_id );
     476        $new_menu                    = wp_get_nav_menu_object( $setting->term_id );
    461477        $this->assertEquals( $expected_resolved_menu_name, $new_menu->name );
    462478
     
    473489        do_action( 'customize_register', $this->wp_customize );
    474490
    475         $menu_name = 'Lorem Ipsum \\o/';
    476         $menu_id = wp_create_nav_menu( wp_slash( $menu_name ) );
    477         $setting_id = "nav_menu[$menu_id]";
    478         $setting = new WP_Customize_Nav_Menu_Setting( $this->wp_customize, $setting_id );
    479         $nav_menu_options = $this->get_nav_menu_items_option();
     491        $menu_name                      = 'Lorem Ipsum \\o/';
     492        $menu_id                        = wp_create_nav_menu( wp_slash( $menu_name ) );
     493        $setting_id                     = "nav_menu[$menu_id]";
     494        $setting                        = new WP_Customize_Nav_Menu_Setting( $this->wp_customize, $setting_id );
     495        $nav_menu_options               = $this->get_nav_menu_items_option();
    480496        $nav_menu_options['auto_add'][] = $menu_id;
    481497        update_option( 'nav_menu_options', $nav_menu_options );
Note: See TracChangeset for help on using the changeset viewer.