WordPress.org
Get WordPress

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/user/wpDropdownUsers.php

    r38651 r42343  
    1414
    1515        // create a user with a different display_name
    16         $u = $this->factory->user->create( array(
    17             'user_login'   => 'foo',
    18             'display_name' => 'Foo Person'
    19         ) );
     16        $u = $this->factory->user->create(
     17            array(
     18                'user_login'   => 'foo',
     19                'display_name' => 'Foo Person',
     20            )
     21        );
    2022
    21         $found = wp_dropdown_users( array(
    22             'echo' => false
    23         ) );
     23        $found = wp_dropdown_users(
     24            array(
     25                'echo' => false,
     26            )
     27        );
    2428
    2529        $expected = "<option value='$u'>Foo Person</option>";
    … …  
    3438
    3539        // create a user with a different display_name
    36         $u = $this->factory->user->create( array(
    37             'user_login'   => 'foo',
    38             'display_name' => 'Foo Person'
    39         ) );
     40        $u = $this->factory->user->create(
     41            array(
     42                'user_login'   => 'foo',
     43                'display_name' => 'Foo Person',
     44            )
     45        );
    4046
    4147        // Get the result of a non-default, but acceptable input for 'show' parameter to wp_dropdown_users().
    42         $found = wp_dropdown_users( array(
    43             'echo' => false,
    44             'show' => ''
    45         ) );
     48        $found = wp_dropdown_users(
     49            array(
     50                'echo' => false,
     51                'show' => '',
     52            )
     53        );
    4654
    4755        $expected = "<option value='$u'>Foo Person</option>";
    … …  
    5664
    5765        // create a user with a different display_name
    58         $u = $this->factory->user->create( array(
    59             'user_login'   => 'foo',
    60             'display_name' => 'Foo Person'
    61         ) );
     66        $u = $this->factory->user->create(
     67            array(
     68                'user_login'   => 'foo',
     69                'display_name' => 'Foo Person',
     70            )
     71        );
    6272
    6373        // Get the result of a non-default, but acceptable input for 'show' parameter to wp_dropdown_users().
    64         $found = wp_dropdown_users( array(
    65             'echo' => false,
    66             'show' => 'user_login'
    67         ) );
     74        $found = wp_dropdown_users(
     75            array(
     76                'echo' => false,
     77                'show' => 'user_login',
     78            )
     79        );
    6880
    6981        $expected = "<option value='$u'>foo</option>";
    … …  
    7890
    7991        // create a user with a different display_name
    80         $u = $this->factory->user->create( array(
    81             'user_login'   => 'foo',
    82             'display_name' => 'Foo Person'
    83         ) );
     92        $u = $this->factory->user->create(
     93            array(
     94                'user_login'   => 'foo',
     95                'display_name' => 'Foo Person',
     96            )
     97        );
    8498
    8599        // Get the result of a non-default, but acceptable input for 'show' parameter to wp_dropdown_users().
    86         $found = wp_dropdown_users( array(
    87             'echo' => false,
    88             'show' => 'display_name_with_login'
    89         ) );
     100        $found = wp_dropdown_users(
     101            array(
     102                'echo' => false,
     103                'show' => 'display_name_with_login',
     104            )
     105        );
    90106
    91107        $expected = "<option value='$u'>Foo Person (foo)</option>";
    … …  
    100116        $users = self::factory()->user->create_many( 2 );
    101117
    102         $found = wp_dropdown_users( array(
    103             'echo' => false,
    104             'include' => $users[0],
    105             'selected' => $users[1],
    106             'include_selected' => true,
    107             'show' => 'user_login',
    108         ) );
     118        $found = wp_dropdown_users(
     119            array(
     120                'echo'             => false,
     121                'include'          => $users[0],
     122                'selected'         => $users[1],
     123                'include_selected' => true,
     124                'show'             => 'user_login',
     125            )
     126        );
    109127
    110128        $user1 = get_userdata( $users[1] );
    … …  
    119137        $u2 = self::factory()->user->create_and_get( array( 'role' => 'author' ) );
    120138
    121         $found = wp_dropdown_users( array(
    122             'echo' => false,
    123             'role' => 'author',
    124             'show' => 'user_login',
    125         ) );
     139        $found = wp_dropdown_users(
     140            array(
     141                'echo' => false,
     142                'role' => 'author',
     143                'show' => 'user_login',
     144            )
     145        );
    126146
    127147        $this->assertNotContains( $u1->user_login, $found );
    … …  
    136156        $u2 = self::factory()->user->create_and_get( array( 'role' => 'author' ) );
    137157
    138         $found = wp_dropdown_users( array(
    139             'echo' => false,
    140             'role__in' => array( 'author', 'editor' ),
    141             'show' => 'user_login',
    142         ) );
     158        $found = wp_dropdown_users(
     159            array(
     160                'echo'     => false,
     161                'role__in' => array( 'author', 'editor' ),
     162                'show'     => 'user_login',
     163            )
     164        );
    143165
    144166        $this->assertNotContains( $u1->user_login, $found );
    … …  
    153175        $u2 = self::factory()->user->create_and_get( array( 'role' => 'author' ) );
    154176
    155         $found = wp_dropdown_users( array(
    156             'echo' => false,
    157             'role__not_in' => array( 'subscriber', 'editor' ),
    158             'show' => 'user_login',
    159         ) );
     177        $found = wp_dropdown_users(
     178            array(
     179                'echo'         => false,
     180                'role__not_in' => array( 'subscriber', 'editor' ),
     181                'show'         => 'user_login',
     182            )
     183        );
    160184
    161185        $this->assertNotContains( $u1->user_login, $found );
Note: See TracChangeset for help on using the changeset viewer.