Make WordPress Core


Ignore:
Timestamp:
09/19/2020 10:53:15 AM (4 years ago)
Author:
SergeyBiryukov
Message:

Coding Standards: Give the $id variable in slashed data tests a more descriptive name.

See #50767, #51344.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/tests/phpunit/tests/user/slashes.php

    r49003 r49004  
    4949        $_POST = add_magic_quotes( $_POST ); // The add_user() function will strip slashes.
    5050
    51         $id  = add_user();
    52         $user = get_user_to_edit( $id );
     51        $user_id = add_user();
     52        $user    = get_user_to_edit( $user_id );
    5353
    5454        $this->assertSame( $this->slash_1, $user->first_name );
     
    7474        $_POST = add_magic_quotes( $_POST ); // The add_user() function will strip slashes.
    7575
    76         $id  = add_user();
    77         $user = get_user_to_edit( $id );
     76        $user_id = add_user();
     77        $user    = get_user_to_edit( $user_id );
    7878
    7979        $this->assertSame( $this->slash_2, $user->first_name );
     
    8888     */
    8989    function test_edit_user() {
    90         $id = self::factory()->user->create();
     90        $user_id = self::factory()->user->create();
    9191
    9292        $_POST                 = array();
     
    103103        $_POST = add_magic_quotes( $_POST ); // The edit_user() function will strip slashes.
    104104
    105         $id   = edit_user( $id );
    106         $user = get_user_to_edit( $id );
     105        $user_id = edit_user( $user_id );
     106        $user    = get_user_to_edit( $user_id );
    107107
    108108        $this->assertSame( $this->slash_1, $user->first_name );
     
    125125        $_POST = add_magic_quotes( $_POST ); // The edit_user() function will strip slashes.
    126126
    127         $id   = edit_user( $id );
    128         $user = get_user_to_edit( $id );
     127        $user_id = edit_user( $user_id );
     128        $user    = get_user_to_edit( $user_id );
    129129
    130130        $this->assertSame( $this->slash_2, $user->first_name );
     
    139139     */
    140140    function test_wp_insert_user() {
    141         $id  = wp_insert_user(
     141        $user_id = wp_insert_user(
    142142            array(
    143143                'user_login'   => 'slash_example_user_3',
     
    152152            )
    153153        );
    154         $user = get_user_to_edit( $id );
     154        $user    = get_user_to_edit( $user_id );
    155155
    156156        $this->assertSame( wp_unslash( $this->slash_1 ), $user->first_name );
     
    160160        $this->assertSame( wp_unslash( $this->slash_3 ), $user->description );
    161161
    162         $id  = wp_insert_user(
     162        $user_id = wp_insert_user(
    163163            array(
    164164                'user_login'   => 'slash_example_user_4',
     
    173173            )
    174174        );
    175         $user = get_user_to_edit( $id );
     175        $user    = get_user_to_edit( $user_id );
    176176
    177177        $this->assertSame( wp_unslash( $this->slash_2 ), $user->first_name );
     
    186186     */
    187187    function test_wp_update_user() {
    188         $id  = self::factory()->user->create();
    189         $id  = wp_update_user(
    190             array(
    191                 'ID'           => $id,
     188        $user_id = self::factory()->user->create();
     189        $user_id = wp_update_user(
     190            array(
     191                'ID'           => $user_id,
    192192                'role'         => 'subscriber',
    193193                'first_name'   => $this->slash_1,
     
    198198            )
    199199        );
    200         $user = get_user_to_edit( $id );
     200        $user    = get_user_to_edit( $user_id );
    201201
    202202        $this->assertSame( wp_unslash( $this->slash_1 ), $user->first_name );
     
    206206        $this->assertSame( wp_unslash( $this->slash_3 ), $user->description );
    207207
    208         $id  = wp_update_user(
    209             array(
    210                 'ID'           => $id,
     208        $user_id = wp_update_user(
     209            array(
     210                'ID'           => $user_id,
    211211                'role'         => 'subscriber',
    212212                'first_name'   => $this->slash_2,
     
    217217            )
    218218        );
    219         $user = get_user_to_edit( $id );
     219        $user    = get_user_to_edit( $user_id );
    220220
    221221        $this->assertSame( wp_unslash( $this->slash_2 ), $user->first_name );
Note: See TracChangeset for help on using the changeset viewer.