WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (9 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.