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

    r41624 r42343  
    1919
    2020    public static function wpSetUpBeforeClass( $factory ) {
    21         self::$user_ids[] = self::$contrib_id = $factory->user->create( array(
    22             'user_login' => 'user1',
    23             'user_nicename' => 'userone',
    24             'user_pass'  => 'password',
    25             'first_name' => 'John',
    26             'last_name'  => 'Doe',
    27             'display_name' => 'John Doe',
    28             'user_email' => 'blackburn@battlefield3.com',
    29             'user_url' => 'http://tacos.com',
    30             'role' => 'contributor'
    31         ) );
    32 
    33         self::$user_ids[] = self::$author_id = $factory->user->create( array(
    34             'user_login' => 'author_login',
    35             'user_email' => 'author@email.com',
    36             'role' => 'author'
    37         ) );
     21        self::$user_ids[] = self::$contrib_id = $factory->user->create(
     22            array(
     23                'user_login'    => 'user1',
     24                'user_nicename' => 'userone',
     25                'user_pass'     => 'password',
     26                'first_name'    => 'John',
     27                'last_name'     => 'Doe',
     28                'display_name'  => 'John Doe',
     29                'user_email'    => 'blackburn@battlefield3.com',
     30                'user_url'      => 'http://tacos.com',
     31                'role'          => 'contributor',
     32            )
     33        );
     34
     35        self::$user_ids[] = self::$author_id = $factory->user->create(
     36            array(
     37                'user_login' => 'author_login',
     38                'user_email' => 'author@email.com',
     39                'role'       => 'author',
     40            )
     41        );
    3842
    3943        self::$user_ids[] = self::$admin_id = $factory->user->create( array( 'role' => 'administrator' ) );
    40         self::$user_ids[] = self::$editor_id = $factory->user->create( array(
    41             'role' => 'editor',
    42             'user_email' => 'test@test.com',
    43         ) );
     44        self::$user_ids[] = self::$editor_id = $factory->user->create(
     45            array(
     46                'role'       => 'editor',
     47                'user_email' => 'test@test.com',
     48            )
     49        );
    4450        self::$user_ids[] = self::$sub_id = $factory->user->create( array( 'role' => 'subscriber' ) );
    4551
    … …  
    132138        // some values to set
    133139        $vals = array(
    134             rand_str() => 'val-'.rand_str(),
    135             rand_str() => 'val-'.rand_str(),
    136             rand_str() => 'val-'.rand_str(),
     140            rand_str() => 'val-' . rand_str(),
     141            rand_str() => 'val-' . rand_str(),
     142            rand_str() => 'val-' . rand_str(),
    137143        );
    138144
    … …  
    149155        // so we'll just check to make sure our values are included somewhere.
    150156        foreach ( $vals as $k => $v ) {
    151             $this->assertTrue( isset( $out[$k] ) && $out[$k][0] == $v );
     157            $this->assertTrue( isset( $out[ $k ] ) && $out[ $k ][0] == $v );
    152158        }
    153159        // delete one key and check again
    154         $keys = array_keys( $vals );
     160        $keys          = array_keys( $vals );
    155161        $key_to_delete = array_pop( $keys );
    156162        delete_user_meta( self::$author_id, $key_to_delete );
    157163        $out = get_user_meta( self::$author_id );
    158164        // make sure that key is excluded from the results
    159         foreach ($vals as $k=>$v) {
    160             if ($k == $key_to_delete) {
    161                 $this->assertFalse( isset( $out[$k] ) );
     165        foreach ( $vals as $k => $v ) {
     166            if ( $k == $key_to_delete ) {
     167                $this->assertFalse( isset( $out[ $k ] ) );
    162168            } else {
    163                 $this->assertTrue( isset( $out[$k] ) && $out[$k][0] == $v );
     169                $this->assertTrue( isset( $out[ $k ] ) && $out[ $k ][0] == $v );
    164170            }
    165171        }
    … …  
    254260    function test_user_level_property_back_compat() {
    255261        $roles = array(
    256             self::$admin_id => 10,
    257             self::$editor_id => 7,
    258             self::$author_id => 2,
     262            self::$admin_id   => 10,
     263            self::$editor_id  => 7,
     264            self::$author_id  => 2,
    259265            self::$contrib_id => 1,
    260             self::$sub_id => 0,
     266            self::$sub_id     => 0,
    261267        );
    262268
    … …  
    274280        $this->assertEquals( self::$author_id, $user->ID );
    275281
    276         $user2 = new WP_User( 0,  $user->user_login );
     282        $user2 = new WP_User( 0, $user->user_login );
    277283        $this->assertInstanceOf( 'WP_User', $user2 );
    278284        $this->assertEquals( self::$author_id, $user2->ID );
    … …  
    321327    function test_has_prop() {
    322328        $user = new WP_User( self::$author_id );
    323         $this->assertTrue( $user->has_prop( 'user_email') );
     329        $this->assertTrue( $user->has_prop( 'user_email' ) );
    324330        $this->assertTrue( $user->has_prop( 'use_ssl' ) );
    325331        $this->assertFalse( $user->has_prop( 'field_that_does_not_exist' ) );
    … …  
    335341        $this->assertEquals( 'about me', $user->get( 'description' ) );
    336342
    337         $user_data = array( 'ID' => self::$author_id, 'display_name' => 'test user' );
     343        $user_data = array(
     344            'ID'           => self::$author_id,
     345            'display_name' => 'test user',
     346        );
    338347        wp_update_user( $user_data );
    339348
    … …  
    345354
    346355        // Pass as stdClass
    347         $user_data = array( 'ID' => self::$author_id, 'display_name' => 'a test user' );
     356        $user_data = array(
     357            'ID'           => self::$author_id,
     358            'display_name' => 'a test user',
     359        );
    348360        wp_update_user( (object) $user_data );
    349361
    … …  
    358370        // Test update of fields in _get_additional_user_keys()
    359371        $user_data = array(
    360             'ID' => self::$author_id, 'use_ssl' => 1, 'show_admin_bar_front' => 1,
    361             'rich_editing' => 1, 'syntax_highlighting' => 1, 'first_name' => 'first', 'last_name' => 'last',
    362             'nickname' => 'nick', 'comment_shortcuts' => 'true', 'admin_color' => 'classic',
    363             'description' => 'describe'
     372            'ID'                   => self::$author_id,
     373            'use_ssl'              => 1,
     374            'show_admin_bar_front' => 1,
     375            'rich_editing'         => 1,
     376            'syntax_highlighting'  => 1,
     377            'first_name'           => 'first',
     378            'last_name'            => 'last',
     379            'nickname'             => 'nick',
     380            'comment_shortcuts'    => 'true',
     381            'admin_color'          => 'classic',
     382            'description'          => 'describe',
    364383        );
    365384        wp_update_user( $user_data );
    … …  
    382401        $this->assertInstanceOf( 'WP_User', $userdata );
    383402        $this->assertEquals( $userdata->ID, self::$sub_id );
    384         $prefix = $wpdb->get_blog_prefix();
     403        $prefix  = $wpdb->get_blog_prefix();
    385404        $cap_key = $prefix . 'capabilities';
    386405        $this->assertTrue( isset( $userdata->$cap_key ) );
    … …  
    418437
    419438        $post = array(
    420             'post_author' => self::$author_id,
    421             'post_status' => 'publish',
     439            'post_author'  => self::$author_id,
     440            'post_status'  => 'publish',
    422441            'post_content' => rand_str(),
    423             'post_title' => rand_str(),
    424             'post_type' => 'post'
     442            'post_title'   => rand_str(),
     443            'post_type'    => 'post',
    425444        );
    426445
    … …  
    493512        $post_id_a = self::factory()->post->create( array( 'post_author' => self::$author_id ) );
    494513        $post_id_b = self::factory()->post->create( array( 'post_author' => $user_id_b ) );
    495         $post_id_c = self::factory()->post->create( array( 'post_author' => $user_id_b, 'post_status' => 'private' ) );
     514        $post_id_c = self::factory()->post->create(
     515            array(
     516                'post_author' => $user_id_b,
     517                'post_status' => 'private',
     518            )
     519        );
    496520
    497521        wp_set_current_user( self::$author_id );
    498522        $counts = count_many_users_posts( array( self::$author_id, $user_id_b ), 'post', false );
    499         $this->assertEquals( 1, $counts[self::$author_id] );
    500         $this->assertEquals( 1, $counts[$user_id_b] );
     523        $this->assertEquals( 1, $counts[ self::$author_id ] );
     524        $this->assertEquals( 1, $counts[ $user_id_b ] );
    501525
    502526        $counts = count_many_users_posts( array( self::$author_id, $user_id_b ), 'post', true );
    503         $this->assertEquals( 1, $counts[self::$author_id] );
    504         $this->assertEquals( 1, $counts[$user_id_b] );
     527        $this->assertEquals( 1, $counts[ self::$author_id ] );
     528        $this->assertEquals( 1, $counts[ $user_id_b ] );
    505529
    506530        wp_set_current_user( $user_id_b );
    507531        $counts = count_many_users_posts( array( self::$author_id, $user_id_b ), 'post', false );
    508         $this->assertEquals( 1, $counts[self::$author_id] );
    509         $this->assertEquals( 2, $counts[$user_id_b] );
     532        $this->assertEquals( 1, $counts[ self::$author_id ] );
     533        $this->assertEquals( 2, $counts[ $user_id_b ] );
    510534
    511535        $counts = count_many_users_posts( array( self::$author_id, $user_id_b ), 'post', true );
    512         $this->assertEquals( 1, $counts[self::$author_id] );
    513         $this->assertEquals( 1, $counts[$user_id_b] );
     536        $this->assertEquals( 1, $counts[ self::$author_id ] );
     537        $this->assertEquals( 1, $counts[ $user_id_b ] );
    514538    }
    515539
    … …  
    520544        $user_id = 1;
    521545        // Find me a non-existent user ID.
    522         while ( get_userdata( $user_id ) )
     546        while ( get_userdata( $user_id ) ) {
    523547            ++$user_id;
     548        }
    524549
    525550        // If this test fails, it will error out for calling the to_array() method on a non-object.
    … …  
    532557    function test_wp_update_user_should_not_change_password_when_passed_WP_User_instance() {
    533558        $testuserid = 1;
    534         $user = get_userdata( $testuserid );
     559        $user       = get_userdata( $testuserid );
    535560        $pwd_before = $user->user_pass;
    536561        wp_update_user( $user );
    … …  
    545570     */
    546571    function test_user_meta_error() {
    547         $id1 = wp_insert_user( array(
    548             'user_login' => rand_str(),
    549             'user_pass' => 'password',
    550             'user_email' => 'taco@burrito.com',
    551         ) );
     572        $id1 = wp_insert_user(
     573            array(
     574                'user_login' => rand_str(),
     575                'user_pass'  => 'password',
     576                'user_email' => 'taco@burrito.com',
     577            )
     578        );
    552579        $this->assertEquals( $id1, email_exists( 'taco@burrito.com' ) );
    553580
    554         $id2 = wp_insert_user( array(
    555             'user_login' => rand_str(),
    556             'user_pass' => 'password',
    557             'user_email' => 'taco@burrito.com',
    558         ) );
     581        $id2 = wp_insert_user(
     582            array(
     583                'user_login' => rand_str(),
     584                'user_pass'  => 'password',
     585                'user_email' => 'taco@burrito.com',
     586            )
     587        );
    559588
    560589        if ( ! defined( 'WP_IMPORTING' ) ) {
    … …  
    572601     */
    573602    function test_user_update_email_error() {
    574         $id1 = wp_insert_user( array(
    575             'user_login' => 'blackburn',
    576             'user_pass'  => 'password',
    577             'user_email' => 'blackburn@battlefield4.com',
    578         ) );
     603        $id1 = wp_insert_user(
     604            array(
     605                'user_login' => 'blackburn',
     606                'user_pass'  => 'password',
     607                'user_email' => 'blackburn@battlefield4.com',
     608            )
     609        );
    579610        $this->assertEquals( $id1, email_exists( 'blackburn@battlefield4.com' ) );
    580611
    581         $id2 = wp_insert_user( array(
    582             'user_login' => 'miller',
    583             'user_pass'  => 'password',
    584             'user_email' => 'miller@battlefield4.com',
    585         ) );
     612        $id2 = wp_insert_user(
     613            array(
     614                'user_login' => 'miller',
     615                'user_pass'  => 'password',
     616                'user_email' => 'miller@battlefield4.com',
     617            )
     618        );
    586619        $this->assertEquals( $id2, email_exists( 'miller@battlefield4.com' ) );
    587620
    588         if ( ! is_wp_error( $id2 ) ){
    589             wp_update_user( array(
    590                 'ID'         => $id2,
    591                 'user_email' => 'david@battlefield4.com',
    592             ) );
     621        if ( ! is_wp_error( $id2 ) ) {
     622            wp_update_user(
     623                array(
     624                    'ID'         => $id2,
     625                    'user_email' => 'david@battlefield4.com',
     626                )
     627            );
    593628            $this->assertEquals( $id2, email_exists( 'david@battlefield4.com' ) );
    594629
    595             $return = wp_update_user( array(
    596                 'ID'         => $id2,
    597                 'user_email' => 'blackburn@battlefield4.com',
    598             ) );
     630            $return = wp_update_user(
     631                array(
     632                    'ID'         => $id2,
     633                    'user_email' => 'blackburn@battlefield4.com',
     634                )
     635            );
    599636
    600637            if ( ! defined( 'WP_IMPORTING' ) ) {
    … …  
    624661
    625662        $user_id = wp_insert_user( $user_data );
    626         $user = get_user_by( 'id', $user_id );
     663        $user    = get_user_by( 'id', $user_id );
    627664        $this->assertInstanceOf( 'WP_User', $user );
    628665    }
    … …  
    644681
    645682        $response = register_new_user( $user_login, $user_email );
    646         $user = get_user_by( 'id', $response );
     683        $user     = get_user_by( 'id', $response );
    647684        $this->assertInstanceOf( 'WP_User', $user );
    648685    }
    … …  
    673710    function _illegal_user_logins_data() {
    674711        $data = array(
    675             array( 'testuser' )
     712            array( 'testuser' ),
    676713        );
    677714
    … …  
    720757    }
    721758
    722     /**
     759    /**
    723760     * @ticket 29880
    724761     */
    … …  
    726763        $user_details = array(
    727764            'user_login' => rand_str(),
    728             'user_pass' => 'password',
     765            'user_pass'  => 'password',
    729766            'user_email' => rand_str() . '@example.com',
    730767        );
    … …  
    734771
    735772        // Check that providing an empty password doesn't remove a user's password.
    736         $user_details['ID'] = $user_id;
     773        $user_details['ID']        = $user_id;
    737774        $user_details['user_pass'] = '';
    738775
    739776        $user_id = wp_insert_user( $user_details );
    740         $user = WP_User::get_data_by( 'id', $user_id );
     777        $user    = WP_User::get_data_by( 'id', $user_id );
    741778        $this->assertNotEmpty( $user->user_pass );
    742779    }
    … …  
    748785        $user = $this->author;
    749786
    750         $userdata = $user->to_array();
     787        $userdata                  = $user->to_array();
    751788        $userdata['user_nicename'] = str_replace( '-', '.', $user->user_nicename );
    752789        wp_insert_user( $userdata );
    … …  
    762799    public function test_wp_insert_user_should_accept_user_login_with_60_characters() {
    763800        $user_login = str_repeat( 'a', 60 );
    764         $u = wp_insert_user( array(
    765             'user_login' => $user_login,
    766             'user_email' => $user_login . '@example.com',
    767             'user_pass' => 'password',
    768             'user_nicename' => 'something-short',
    769         ) );
     801        $u          = wp_insert_user(
     802            array(
     803                'user_login'    => $user_login,
     804                'user_email'    => $user_login . '@example.com',
     805                'user_pass'     => 'password',
     806                'user_nicename' => 'something-short',
     807            )
     808        );
    770809
    771810        $this->assertInternalType( 'int', $u );
    … …  
    781820    public function test_wp_insert_user_should_reject_user_login_over_60_characters() {
    782821        $user_login = str_repeat( 'a', 61 );
    783         $u = wp_insert_user( array(
    784             'user_login' => $user_login,
    785             'user_email' => $user_login . '@example.com',
    786             'user_pass' => 'password',
    787             'user_nicename' => 'something-short',
    788         ) );
     822        $u          = wp_insert_user(
     823            array(
     824                'user_login'    => $user_login,
     825                'user_email'    => $user_login . '@example.com',
     826                'user_pass'     => 'password',
     827                'user_nicename' => 'something-short',
     828            )
     829        );
    789830
    790831        $this->assertWPError( $u );
    … …  
    797838    public function test_wp_insert_user_should_reject_user_nicename_over_50_characters() {
    798839        $user_nicename = str_repeat( 'a', 51 );
    799         $u = wp_insert_user( array(
    800             'user_login' => 'mynicenamehas50chars',
    801             'user_email' => $user_nicename . '@example.com',
    802             'user_pass' => 'password',
    803             'user_nicename' => $user_nicename,
    804         ) );
     840        $u             = wp_insert_user(
     841            array(
     842                'user_login'    => 'mynicenamehas50chars',
     843                'user_email'    => $user_nicename . '@example.com',
     844                'user_pass'     => 'password',
     845                'user_nicename' => $user_nicename,
     846            )
     847        );
    805848
    806849        $this->assertWPError( $u );
    … …  
    813856    public function test_wp_insert_user_should_not_generate_user_nicename_longer_than_50_chars() {
    814857        $user_login = str_repeat( 'a', 55 );
    815         $u = wp_insert_user( array(
    816             'user_login' => $user_login,
    817             'user_email' => $user_login . '@example.com',
    818             'user_pass' => 'password',
    819         ) );
     858        $u          = wp_insert_user(
     859            array(
     860                'user_login' => $user_login,
     861                'user_email' => $user_login . '@example.com',
     862                'user_pass'  => 'password',
     863            )
     864        );
    820865
    821866        $this->assertNotEmpty( $u );
    822         $user = new WP_User( $u );
     867        $user     = new WP_User( $u );
    823868        $expected = str_repeat( 'a', 50 );
    824869        $this->assertSame( $expected, $user->user_nicename );
    … …  
    829874     */
    830875    public function test_wp_insert_user_should_not_truncate_to_a_duplicate_user_nicename() {
    831         $u1 = self::factory()->user->create( array(
    832             'user_nicename' => str_repeat( 'a', 50 ),
    833         ) );
     876        $u1 = self::factory()->user->create(
     877            array(
     878                'user_nicename' => str_repeat( 'a', 50 ),
     879            )
     880        );
    834881
    835882        $user1 = new WP_User( $u1 );
    … …  
    839886
    840887        $user_login = str_repeat( 'a', 55 );
    841         $u = wp_insert_user( array(
    842             'user_login' => $user_login,
    843             'user_email' => $user_login . '@example.com',
    844             'user_pass' => 'password',
    845         ) );
     888        $u          = wp_insert_user(
     889            array(
     890                'user_login' => $user_login,
     891                'user_email' => $user_login . '@example.com',
     892                'user_pass'  => 'password',
     893            )
     894        );
    846895
    847896        $this->assertNotEmpty( $u );
    848         $user2 = new WP_User( $u );
     897        $user2    = new WP_User( $u );
    849898        $expected = str_repeat( 'a', 48 ) . '-2';
    850899        $this->assertSame( $expected, $user2->user_nicename );
    … …  
    855904     */
    856905    public function test_wp_insert_user_should_not_truncate_to_a_duplicate_user_nicename_when_suffix_has_more_than_one_character() {
    857         $user_ids = self::factory()->user->create_many( 4, array(
    858             'user_nicename' => str_repeat( 'a', 50 ),
    859         ) );
     906        $user_ids = self::factory()->user->create_many(
     907            4, array(
     908                'user_nicename' => str_repeat( 'a', 50 ),
     909            )
     910        );
    860911
    861912        foreach ( $user_ids as $i => $user_id ) {
    … …  
    870921
    871922        $user_login = str_repeat( 'a', 55 );
    872         $u = wp_insert_user( array(
    873             'user_login' => $user_login,
    874             'user_email' => $user_login . '@example.com',
    875             'user_pass' => 'password',
    876         ) );
     923        $u          = wp_insert_user(
     924            array(
     925                'user_login' => $user_login,
     926                'user_email' => $user_login . '@example.com',
     927                'user_pass'  => 'password',
     928            )
     929        );
    877930
    878931        $this->assertNotEmpty( $u );
    879         $user = new WP_User( $u );
     932        $user     = new WP_User( $u );
    880933        $expected = str_repeat( 'a', 48 ) . '-5';
    881934        $this->assertSame( $expected, $user->user_nicename );
    … …  
    889942        $max_user = $wpdb->get_var( "SELECT MAX(ID) FROM $wpdb->users" );
    890943
    891         $u = wp_insert_user( array(
    892             'ID' => $max_user + 1,
    893             'user_login' => 'whatever',
    894             'user_email' => 'whatever@example.com',
    895             'user_pass' => 'password',
    896         ) );
     944        $u = wp_insert_user(
     945            array(
     946                'ID'         => $max_user + 1,
     947                'user_login' => 'whatever',
     948                'user_email' => 'whatever@example.com',
     949                'user_pass'  => 'password',
     950            )
     951        );
    897952
    898953        $this->assertWPError( $u );
    … …  
    903958     */
    904959    public function test_wp_update_user_should_delete_userslugs_cache() {
    905         $u = self::factory()->user->create();
     960        $u    = self::factory()->user->create();
    906961        $user = get_userdata( $u );
    907962
    908         wp_update_user( array(
    909             'ID' => $u,
    910             'user_nicename' => 'newusernicename',
    911         ) );
     963        wp_update_user(
     964            array(
     965                'ID'            => $u,
     966                'user_nicename' => 'newusernicename',
     967            )
     968        );
    912969        $updated_user = get_userdata( $u );
    913970
    … …  
    9491006
    9501007    public function test_search_users_login() {
    951         $users = get_users( array( 'search' => 'user1', 'fields' => 'ID' ) );
     1008        $users = get_users(
     1009            array(
     1010                'search' => 'user1',
     1011                'fields' => 'ID',
     1012            )
     1013        );
    9521014
    9531015        $this->assertTrue( in_array( self::$contrib_id, $users ) );
    … …  
    9551017
    9561018    public function test_search_users_url() {
    957         $users = get_users( array( 'search' => '*tacos*', 'fields' => 'ID' ) );
     1019        $users = get_users(
     1020            array(
     1021                'search' => '*tacos*',
     1022                'fields' => 'ID',
     1023            )
     1024        );
    9581025
    9591026        $this->assertTrue( in_array( self::$contrib_id, $users ) );
    … …  
    9611028
    9621029    public function test_search_users_email() {
    963         $users = get_users( array( 'search' => '*battle*', 'fields' => 'ID' ) );
     1030        $users = get_users(
     1031            array(
     1032                'search' => '*battle*',
     1033                'fields' => 'ID',
     1034            )
     1035        );
    9641036
    9651037        $this->assertTrue( in_array( self::$contrib_id, $users ) );
    … …  
    9671039
    9681040    public function test_search_users_nicename() {
    969         $users = get_users( array( 'search' => '*one*', 'fields' => 'ID' ) );
     1041        $users = get_users(
     1042            array(
     1043                'search' => '*one*',
     1044                'fields' => 'ID',
     1045            )
     1046        );
    9701047
    9711048        $this->assertTrue( in_array( self::$contrib_id, $users ) );
    … …  
    9731050
    9741051    public function test_search_users_display_name() {
    975         $users = get_users( array( 'search' => '*Doe*', 'fields' => 'ID' ) );
     1052        $users = get_users(
     1053            array(
     1054                'search' => '*Doe*',
     1055                'fields' => 'ID',
     1056            )
     1057        );
    9761058
    9771059        $this->assertTrue( in_array( self::$contrib_id, $users ) );
    … …  
    9841066        // Alter the case of the email address (which stays the same).
    9851067        $userdata = array(
    986             'ID' => self::$editor_id,
     1068            'ID'         => self::$editor_id,
    9871069            'user_email' => 'test@TEST.com',
    9881070        );
    989         $update = wp_update_user( $userdata );
     1071        $update   = wp_update_user( $userdata );
    9901072
    9911073        $this->assertEquals( self::$editor_id, $update );
    … …  
    9981080        // Change the email address.
    9991081        $userdata = array(
    1000             'ID' => self::$editor_id,
     1082            'ID'         => self::$editor_id,
    10011083            'user_email' => 'test2@test.com',
    10021084        );
    1003         $update = wp_update_user( $userdata );
     1085        $update   = wp_update_user( $userdata );
    10041086
    10051087        // Was this successful?
    … …  
    10221104
    10231105        $was_admin_email_sent = false;
    1024         $was_user_email_sent = false;
     1106        $was_user_email_sent  = false;
    10251107
    10261108        wp_new_user_notification( self::$contrib_id, null, $notify );
    … …  
    10351117        if ( $first_recipient ) {
    10361118            $was_admin_email_sent = WP_TESTS_EMAIL === $first_recipient->address;
    1037             $was_user_email_sent = 'blackburn@battlefield3.com' === $first_recipient->address;
     1119            $was_user_email_sent  = 'blackburn@battlefield3.com' === $first_recipient->address;
    10381120        }
    10391121
    … …  
    10431125        }
    10441126
    1045 
    10461127        $this->assertSame( $admin_email_sent_expected, $was_admin_email_sent, 'Admin email result was not as expected in test_wp_new_user_notification' );
    1047         $this->assertSame( $user_email_sent_expected , $was_user_email_sent, 'User email result was not as expected in test_wp_new_user_notification' );
     1128        $this->assertSame( $user_email_sent_expected, $was_user_email_sent, 'User email result was not as expected in test_wp_new_user_notification' );
    10481129    }
    10491130
    … …  
    10981179
    10991180        $was_admin_email_sent = false;
    1100         $was_user_email_sent = false;
     1181        $was_user_email_sent  = false;
    11011182        wp_new_user_notification( self::$contrib_id, 'this_is_a_test_password' );
    11021183
    … …  
    11071188        if ( ! empty( $GLOBALS['phpmailer']->mock_sent ) ) {
    11081189            $was_admin_email_sent = ( isset( $GLOBALS['phpmailer']->mock_sent[0] ) && WP_TESTS_EMAIL == $GLOBALS['phpmailer']->mock_sent[0]['to'][0][0] );
    1109             $was_user_email_sent = ( isset( $GLOBALS['phpmailer']->mock_sent[1] ) && 'blackburn@battlefield3.com' == $GLOBALS['phpmailer']->mock_sent[1]['to'][0][0] );
     1190            $was_user_email_sent  = ( isset( $GLOBALS['phpmailer']->mock_sent[1] ) && 'blackburn@battlefield3.com' == $GLOBALS['phpmailer']->mock_sent[1]['to'][0][0] );
    11101191        }
    11111192
    … …  
    11231204
    11241205        $was_admin_email_sent = false;
    1125         $was_user_email_sent = false;
     1206        $was_user_email_sent  = false;
    11261207        wp_new_user_notification( self::$contrib_id );
    11271208
    … …  
    11321213        if ( ! empty( $GLOBALS['phpmailer']->mock_sent ) ) {
    11331214            $was_admin_email_sent = ( isset( $GLOBALS['phpmailer']->mock_sent[0] ) && WP_TESTS_EMAIL == $GLOBALS['phpmailer']->mock_sent[0]['to'][0][0] );
    1134             $was_user_email_sent = ( isset( $GLOBALS['phpmailer']->mock_sent[1] ) && 'blackburn@battlefield3.com' == $GLOBALS['phpmailer']->mock_sent[1]['to'][0][0] );
     1215            $was_user_email_sent  = ( isset( $GLOBALS['phpmailer']->mock_sent[1] ) && 'blackburn@battlefield3.com' == $GLOBALS['phpmailer']->mock_sent[1]['to'][0][0] );
    11351216        }
    11361217
    … …  
    11411222    /**
    11421223     * Ensure blog's admin email change notification emails do not contain encoded HTML entities
     1224     *
    11431225     * @ticket 40015
    11441226     */
    … …  
    11491231
    11501232        $existing_email = get_option( 'admin_email' );
    1151         $new_email = 'new-admin-email@test.dev';
     1233        $new_email      = 'new-admin-email@test.dev';
    11521234
    11531235        // Give the site a name containing HTML entities
    … …  
    11591241
    11601242        $recipient = $mailer->get_recipient( 'to' );
    1161         $email = $mailer->get_sent();
     1243        $email     = $mailer->get_sent();
    11621244
    11631245        // Assert reciepient is correct
    … …  
    12051287                'not an email',
    12061288                'A confirmation email should not be sent if it is not a valid email',
    1207             )
     1289            ),
    12081290        );
    12091291    }
    … …  
    12211303        $old_current = get_current_user_id();
    12221304
    1223         $user_id = self::factory()->user->create( array(
    1224             'role'       => 'subscriber',
    1225             'user_email' => 'email@test.dev',
    1226         ) );
     1305        $user_id = self::factory()->user->create(
     1306            array(
     1307                'role'       => 'subscriber',
     1308                'user_email' => 'email@test.dev',
     1309            )
     1310        );
    12271311        wp_set_current_user( $user_id );
    12281312
    1229         self::factory()->user->create( array(
    1230             'role'       => 'subscriber',
    1231             'user_email' => 'another-user@test.dev',
    1232         ) );
     1313        self::factory()->user->create(
     1314            array(
     1315                'role'       => 'subscriber',
     1316                'user_email' => 'another-user@test.dev',
     1317            )
     1318        );
    12331319
    12341320        reset_phpmailer_instance();
    … …  
    12361322        // Set $_POST['email'] with new email and $_POST['id'] with user's ID.
    12371323        $_POST['user_id'] = $user_id;
    1238         $_POST['email'] = $email;
     1324        $_POST['email']   = $email;
    12391325        send_confirmation_on_profile_email();
    12401326
    … …  
    12791365     */
    12801366    function test_edit_user_blank_pw() {
    1281         $_POST = $_GET = $_REQUEST = array();
    1282         $_POST['role'] = 'subscriber';
    1283         $_POST['email'] = 'user1@example.com';
    1284         $_POST['user_login'] = 'user_login1';
    1285         $_POST['first_name'] = 'first_name1';
    1286         $_POST['last_name'] = 'last_name1';
    1287         $_POST['nickname'] = 'nickname1';
     1367        $_POST                 = $_GET = $_REQUEST = array();
     1368        $_POST['role']         = 'subscriber';
     1369        $_POST['email']        = 'user1@example.com';
     1370        $_POST['user_login']   = 'user_login1';
     1371        $_POST['first_name']   = 'first_name1';
     1372        $_POST['last_name']    = 'last_name1';
     1373        $_POST['nickname']     = 'nickname1';
    12881374        $_POST['display_name'] = 'display_name1';
    12891375
    … …  
    12981384
    12991385        $user_id = edit_user();
    1300         $user = get_user_by( 'ID', $user_id );
     1386        $user    = get_user_by( 'ID', $user_id );
    13011387
    13021388        $this->assertInternalType( 'int', $user_id );
    … …  
    13061392        // Check updating user with empty password.
    13071393        $_POST['nickname'] = 'nickname_updated';
    1308         $_POST['pass1'] = $_POST['pass2'] = '';
     1394        $_POST['pass1']    = $_POST['pass2'] = '';
    13091395
    13101396        $user_id = edit_user( $user_id );
    … …  
    13151401        // Check updating user with missing second password.
    13161402        $_POST['nickname'] = 'nickname_updated2';
    1317         $_POST['pass1'] = 'blank_pass2';
    1318         $_POST['pass2'] = '';
     1403        $_POST['pass1']    = 'blank_pass2';
     1404        $_POST['pass2']    = '';
    13191405
    13201406        $response = edit_user( $user_id );
    … …  
    13471433        $was_confirmation_email_sent = false;
    13481434
    1349         $user = $this->factory()->user->create_and_get( array(
    1350             'user_email' => 'before@example.com',
    1351         ) );
     1435        $user = $this->factory()->user->create_and_get(
     1436            array(
     1437                'user_email' => 'before@example.com',
     1438            )
     1439        );
    13521440
    13531441        $_POST['email']   = 'after@example.com';
    … …  
    13821470        $was_confirmation_email_sent = false;
    13831471
    1384         $user = $this->factory()->user->create_and_get( array(
    1385             'user_email' => 'before@example.com',
    1386         ) );
     1472        $user = $this->factory()->user->create_and_get(
     1473            array(
     1474                'user_email' => 'before@example.com',
     1475            )
     1476        );
    13871477
    13881478        $_POST['email']   = 'after@example.com';
    … …  
    14151505     */
    14161506    function test_send_confirmation_on_profile_email_html_entities_decoded() {
    1417         $user_id = self::factory()->user->create( array(
    1418             'role'       => 'subscriber',
    1419             'user_email' => 'old-email@test.dev',
    1420         ) );
     1507        $user_id = self::factory()->user->create(
     1508            array(
     1509                'role'       => 'subscriber',
     1510                'user_email' => 'old-email@test.dev',
     1511            )
     1512        );
    14211513        wp_set_current_user( $user_id );
    14221514
    … …  
    14311523        $_POST['email']   = 'new-email@test.dev';
    14321524
    1433         send_confirmation_on_profile_email( );
     1525        send_confirmation_on_profile_email();
    14341526
    14351527        $mailer = tests_retrieve_phpmailer_instance();
Note: See TracChangeset for help on using the changeset viewer.