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/slashes.php

    r35244 r42343  
    2626    /**
    2727     * Tests the controller function that expects slashed data
    28      *
    2928     */
    3029    function test_add_user() {
    31         $_POST = $_GET = $_REQUEST = array();
    32         $_POST['user_login'] = 'slash_example_user_1';
    33         $_POST['pass1'] = 'password';
    34         $_POST['pass2'] = 'password';
    35         $_POST['role'] = 'subscriber';
    36         $_POST['email'] = 'user1@example.com';
    37         $_POST['first_name'] = $this->slash_1;
    38         $_POST['last_name'] = $this->slash_3;
    39         $_POST['nickname'] = $this->slash_5;
     30        $_POST                 = $_GET = $_REQUEST = array();
     31        $_POST['user_login']   = 'slash_example_user_1';
     32        $_POST['pass1']        = 'password';
     33        $_POST['pass2']        = 'password';
     34        $_POST['role']         = 'subscriber';
     35        $_POST['email']        = 'user1@example.com';
     36        $_POST['first_name']   = $this->slash_1;
     37        $_POST['last_name']    = $this->slash_3;
     38        $_POST['nickname']     = $this->slash_5;
    4039        $_POST['display_name'] = $this->slash_7;
    41         $_POST['description'] = $this->slash_3;
    42         $_POST = add_magic_quotes( $_POST ); // the edit_post() function will strip slashes
    43 
    44         $id = add_user();
     40        $_POST['description']  = $this->slash_3;
     41        $_POST                 = add_magic_quotes( $_POST ); // the edit_post() function will strip slashes
     42
     43        $id   = add_user();
    4544        $user = get_user_to_edit( $id );
    4645
    … …  
    5150        $this->assertEquals( $this->slash_3, $user->description );
    5251
    53         $_POST = $_GET = $_REQUEST = array();
    54         $_POST['user_login'] = 'slash_example_user_2';
    55         $_POST['pass1'] = 'password';
    56         $_POST['pass2'] = 'password';
    57         $_POST['role'] = 'subscriber';
    58         $_POST['email'] = 'user2@example.com';
    59         $_POST['first_name'] = $this->slash_2;
    60         $_POST['last_name'] = $this->slash_4;
    61         $_POST['nickname'] = $this->slash_6;
     52        $_POST                 = $_GET = $_REQUEST = array();
     53        $_POST['user_login']   = 'slash_example_user_2';
     54        $_POST['pass1']        = 'password';
     55        $_POST['pass2']        = 'password';
     56        $_POST['role']         = 'subscriber';
     57        $_POST['email']        = 'user2@example.com';
     58        $_POST['first_name']   = $this->slash_2;
     59        $_POST['last_name']    = $this->slash_4;
     60        $_POST['nickname']     = $this->slash_6;
    6261        $_POST['display_name'] = $this->slash_2;
    63         $_POST['description'] = $this->slash_4;
    64         $_POST = add_magic_quotes( $_POST ); // the edit_post() function will strip slashes
    65 
    66         $id = add_user();
     62        $_POST['description']  = $this->slash_4;
     63        $_POST                 = add_magic_quotes( $_POST ); // the edit_post() function will strip slashes
     64
     65        $id   = add_user();
    6766        $user = get_user_to_edit( $id );
    6867
    … …  
    7675    /**
    7776     * Tests the controller function that expects slashed data
    78      *
    7977     */
    8078    function test_edit_user() {
    8179        $id = self::factory()->user->create();
    8280
    83         $_POST = $_GET = $_REQUEST = array();
    84         $_POST['role'] = 'subscriber';
    85         $_POST['email'] = 'user1@example.com';
    86         $_POST['first_name'] = $this->slash_1;
    87         $_POST['last_name'] = $this->slash_3;
    88         $_POST['nickname'] = $this->slash_5;
     81        $_POST                 = $_GET = $_REQUEST = array();
     82        $_POST['role']         = 'subscriber';
     83        $_POST['email']        = 'user1@example.com';
     84        $_POST['first_name']   = $this->slash_1;
     85        $_POST['last_name']    = $this->slash_3;
     86        $_POST['nickname']     = $this->slash_5;
    8987        $_POST['display_name'] = $this->slash_7;
    90         $_POST['description'] = $this->slash_3;
    91         $_POST = add_magic_quotes( $_POST ); // the edit_post() function will strip slashes
    92 
    93         $id = edit_user( $id );
     88        $_POST['description']  = $this->slash_3;
     89        $_POST                 = add_magic_quotes( $_POST ); // the edit_post() function will strip slashes
     90
     91        $id   = edit_user( $id );
    9492        $user = get_user_to_edit( $id );
    9593
    … …  
    10098        $this->assertEquals( $this->slash_3, $user->description );
    10199
    102         $_POST = $_GET = $_REQUEST = array();
    103         $_POST['role'] = 'subscriber';
    104         $_POST['email'] = 'user2@example.com';
    105         $_POST['first_name'] = $this->slash_2;
    106         $_POST['last_name'] = $this->slash_4;
    107         $_POST['nickname'] = $this->slash_6;
     100        $_POST                 = $_GET = $_REQUEST = array();
     101        $_POST['role']         = 'subscriber';
     102        $_POST['email']        = 'user2@example.com';
     103        $_POST['first_name']   = $this->slash_2;
     104        $_POST['last_name']    = $this->slash_4;
     105        $_POST['nickname']     = $this->slash_6;
    108106        $_POST['display_name'] = $this->slash_2;
    109         $_POST['description'] = $this->slash_4;
    110         $_POST = add_magic_quotes( $_POST ); // the edit_post() function will strip slashes
    111 
    112         $id = edit_user( $id );
     107        $_POST['description']  = $this->slash_4;
     108        $_POST                 = add_magic_quotes( $_POST ); // the edit_post() function will strip slashes
     109
     110        $id   = edit_user( $id );
    113111        $user = get_user_to_edit( $id );
    114112
    … …  
    122120    /**
    123121     * Tests the model function that expects slashed data
    124      *
    125122     */
    126123    function test_wp_insert_user() {
    127         $id = wp_insert_user( array(
    128             'user_login' => 'slash_example_user_3',
    129             'role' => 'subscriber',
    130             'email' => 'user3@example.com',
    131             'first_name' => $this->slash_1,
    132             'last_name' => $this->slash_3,
    133             'nickname' => $this->slash_5,
    134             'display_name' => $this->slash_7,
    135             'description' => $this->slash_3,
    136             'user_pass' => ''
    137         ) );
     124        $id   = wp_insert_user(
     125            array(
     126                'user_login'   => 'slash_example_user_3',
     127                'role'         => 'subscriber',
     128                'email'        => 'user3@example.com',
     129                'first_name'   => $this->slash_1,
     130                'last_name'    => $this->slash_3,
     131                'nickname'     => $this->slash_5,
     132                'display_name' => $this->slash_7,
     133                'description'  => $this->slash_3,
     134                'user_pass'    => '',
     135            )
     136        );
    138137        $user = get_user_to_edit( $id );
    139138
    … …  
    144143        $this->assertEquals( wp_unslash( $this->slash_3 ), $user->description );
    145144
    146         $id = wp_insert_user( array(
    147             'user_login' => 'slash_example_user_4',
    148             'role' => 'subscriber',
    149             'email' => 'user3@example.com',
    150             'first_name' => $this->slash_2,
    151             'last_name' => $this->slash_4,
    152             'nickname' => $this->slash_6,
    153             'display_name' => $this->slash_2,
    154             'description' => $this->slash_4,
    155             'user_pass' => ''
    156         ) );
     145        $id   = wp_insert_user(
     146            array(
     147                'user_login'   => 'slash_example_user_4',
     148                'role'         => 'subscriber',
     149                'email'        => 'user3@example.com',
     150                'first_name'   => $this->slash_2,
     151                'last_name'    => $this->slash_4,
     152                'nickname'     => $this->slash_6,
     153                'display_name' => $this->slash_2,
     154                'description'  => $this->slash_4,
     155                'user_pass'    => '',
     156            )
     157        );
    157158        $user = get_user_to_edit( $id );
    158159
    … …  
    166167    /**
    167168     * Tests the model function that expects slashed data
    168      *
    169169     */
    170170    function test_wp_update_user() {
    171         $id = self::factory()->user->create();
    172         $id = wp_update_user(array(
    173             'ID' => $id,
    174             'role' => 'subscriber',
    175             'first_name' => $this->slash_1,
    176             'last_name' => $this->slash_3,
    177             'nickname' => $this->slash_5,
    178             'display_name' => $this->slash_7,
    179             'description' => $this->slash_3,
    180         ));
     171        $id   = self::factory()->user->create();
     172        $id   = wp_update_user(
     173            array(
     174                'ID'           => $id,
     175                'role'         => 'subscriber',
     176                'first_name'   => $this->slash_1,
     177                'last_name'    => $this->slash_3,
     178                'nickname'     => $this->slash_5,
     179                'display_name' => $this->slash_7,
     180                'description'  => $this->slash_3,
     181            )
     182        );
    181183        $user = get_user_to_edit( $id );
    182184
    … …  
    187189        $this->assertEquals( wp_unslash( $this->slash_3 ), $user->description );
    188190
    189         $id = wp_update_user(array(
    190             'ID' => $id,
    191             'role' => 'subscriber',
    192             'first_name' => $this->slash_2,
    193             'last_name' => $this->slash_4,
    194             'nickname' => $this->slash_6,
    195             'display_name' => $this->slash_2,
    196             'description' => $this->slash_4,
    197         ));
     191        $id   = wp_update_user(
     192            array(
     193                'ID'           => $id,
     194                'role'         => 'subscriber',
     195                'first_name'   => $this->slash_2,
     196                'last_name'    => $this->slash_4,
     197                'nickname'     => $this->slash_6,
     198                'display_name' => $this->slash_2,
     199                'description'  => $this->slash_4,
     200            )
     201        );
    198202        $user = get_user_to_edit( $id );
    199203
Note: See TracChangeset for help on using the changeset viewer.