WordPress.org
Get WordPress

Make WordPress Core

Ticket #45934: 45934.diff

File 45934.diff, 38.9 KB (added by pento, 6 years ago)

  • phpcs.xml.dist

     
    1111                </properties>
    1212        </rule>
    1313
     14        <rule ref="WordPress.NamingConventions.ValidVariableName">
     15                <properties>
     16                        <property name="customPropertiesWhitelist" type="array">
     17                                <!-- From database structure queries -->
     18                                <element value="Collation"/>
     19                                <element value="Column_name"/>
     20                                <element value="Default"/>
     21                                <element value="Extra"/>
     22                                <element value="Field"/>
     23                                <element value="Index_type"/>
     24                                <element value="Key"/>
     25                                <element value="Key_name"/>
     26                                <element value="Msg_text"/>
     27                                <element value="Non_unique"/>
     28                                <element value="Null"/>
     29                                <element value="Sub_part"/>
     30                                <element value="Type"/>
     31                                <!-- From plugin/theme data -->
     32                                <element value="authorAndUri"/>
     33                                <element value="Name"/>
     34                                <element value="Version"/>
     35                                <!-- From the result of wp_xmlrpc_server::wp_getPageList() -->
     36                                <element value="dateCreated"/>
     37
     38                                <!-- From DOMDocument -->
     39                                <element value="childNodes"/>
     40                                <element value="formatOutput"/>
     41                                <element value="nodeName"/>
     42                                <element value="nodeType"/>
     43                                <element value="parentNode"/>
     44                                <element value="preserveWhiteSpace"/>
     45                                <element value="textContent"/>
     46                                <!-- From PHPMailer -->
     47                                <element value="AltBody"/>
     48                                <element value="Body"/>
     49                                <element value="CharSet"/>
     50                                <element value="ContentType"/>
     51                                <element value="Encoding"/>
     52                                <element value="Hostname"/>
     53                                <element value="mailHeader"/>
     54                                <element value="MIMEBody"/>
     55                                <element value="MIMEHeader"/>
     56                                <element value="Sender"/>
     57                                <element value="Subject"/>
     58                                <!-- From PHPUnit_Util_Getopt -->
     59                                <element value="longOptions"/>
     60                                <!-- From POP3 -->
     61                                <element value="ERROR"/>
     62                                <!-- From ZipArchive -->
     63                                <element value="numFiles"/>
     64                        </property>
     65                </properties>
     66        </rule>
     67
    1468        <arg name="extensions" value="php"/>
    1569
    1670        <!-- Strip the filepaths down to the relevant bit. -->

  • src/wp-includes/class-wp-comment-query.php

     
    555555                                preg_split( '/[,\s]/', $this->query_vars['orderby'] );
    556556
    557557                        $orderby_array            = array();
    558                         $found_orderby_comment_ID = false;
     558                        $found_orderby_comment_id = false;
    559559                        foreach ( $ordersby as $_key => $_value ) {
    560560                                if ( ! $_value ) {
    561561                                        continue;
    … …  
    569569                                        $_order   = $_value;
    570570                                }
    571571
    572                                 if ( ! $found_orderby_comment_ID && in_array( $_orderby, array( 'comment_ID', 'comment__in' ) ) ) {
    573                                         $found_orderby_comment_ID = true;
     572                                if ( ! $found_orderby_comment_id && in_array( $_orderby, array( 'comment_ID', 'comment__in' ) ) ) {
     573                                        $found_orderby_comment_id = true;
    574574                                }
    575575
    576576                                $parsed = $this->parse_orderby( $_orderby );
    … …  
    593593                        }
    594594
    595595                        // To ensure determinate sorting, always include a comment_ID clause.
    596                         if ( ! $found_orderby_comment_ID ) {
    597                                 $comment_ID_order = '';
     596                        if ( ! $found_orderby_comment_id ) {
     597                                $comment_id_order = '';
    598598
    599599                                // Inherit order from comment_date or comment_date_gmt, if available.
    600600                                foreach ( $orderby_array as $orderby_clause ) {
    601601                                        if ( preg_match( '/comment_date(?:_gmt)*\ (ASC|DESC)/', $orderby_clause, $match ) ) {
    602                                                 $comment_ID_order = $match[1];
     602                                                $comment_id_order = $match[1];
    603603                                                break;
    604604                                        }
    605605                                }
    606606
    607607                                // If no date-related order is available, use the date from the first available clause.
    608                                 if ( ! $comment_ID_order ) {
     608                                if ( ! $comment_id_order ) {
    609609                                        foreach ( $orderby_array as $orderby_clause ) {
    610610                                                if ( false !== strpos( 'ASC', $orderby_clause ) ) {
    611                                                         $comment_ID_order = 'ASC';
     611                                                        $comment_id_order = 'ASC';
    612612                                                } else {
    613                                                         $comment_ID_order = 'DESC';
     613                                                        $comment_id_order = 'DESC';
    614614                                                }
    615615
    616616                                                break;
    … …  
    618618                                }
    619619
    620620                                // Default to DESC.
    621                                 if ( ! $comment_ID_order ) {
    622                                         $comment_ID_order = 'DESC';
     621                                if ( ! $comment_id_order ) {
     622                                        $comment_id_order = 'DESC';
    623623                                }
    624624
    625                                 $orderby_array[] = "$wpdb->comments.comment_ID $comment_ID_order";
     625                                $orderby_array[] = "$wpdb->comments.comment_ID $comment_id_order";
    626626                        }
    627627
    628628                        $orderby = implode( ', ', $orderby_array );

  • tests/phpunit/tests/avatar.php

     
    103103                $this->assertEquals( $url, $url2 );
    104104        }
    105105
    106         protected $fakeURL;
     106        protected $fake_url;
    107107        /**
    108108         * @ticket 21195
    109109         */
    110110        public function test_pre_get_avatar_url_filter() {
    111                 $this->fakeURL = 'haha wat';
     111                $this->fake_url = 'haha wat';
    112112
    113113                add_filter( 'pre_get_avatar_data', array( $this, 'pre_get_avatar_url_filter' ), 10, 1 );
    114114                $url = get_avatar_url( 1 );
    115115                remove_filter( 'pre_get_avatar_data', array( $this, 'pre_get_avatar_url_filter' ), 10 );
    116116
    117                 $this->assertEquals( $url, $this->fakeURL );
     117                $this->assertEquals( $url, $this->fake_url );
    118118        }
    119119        public function pre_get_avatar_url_filter( $args ) {
    120                 $args['url'] = $this->fakeURL;
     120                $args['url'] = $this->fake_url;
    121121                return $args;
    122122        }
    123123
    … …  
    125125         * @ticket 21195
    126126         */
    127127        public function test_get_avatar_url_filter() {
    128                 $this->fakeURL = 'omg lol';
     128                $this->fake_url = 'omg lol';
    129129
    130130                add_filter( 'get_avatar_url', array( $this, 'get_avatar_url_filter' ), 10, 1 );
    131131                $url = get_avatar_url( 1 );
    132132                remove_filter( 'get_avatar_url', array( $this, 'get_avatar_url_filter' ), 10 );
    133133
    134                 $this->assertEquals( $url, $this->fakeURL );
     134                $this->assertEquals( $url, $this->fake_url );
    135135        }
    136136        public function get_avatar_url_filter( $url ) {
    137                 return $this->fakeURL;
     137                return $this->fake_url;
    138138        }
    139139
    140140        /**
    … …  
    207207        }
    208208
    209209
    210         protected $fakeIMG;
     210        protected $fake_img;
    211211        /**
    212212         * @ticket 21195
    213213         */
    214214        public function test_pre_get_avatar_filter() {
    215                 $this->fakeIMG = 'YOU TOO?!';
     215                $this->fake_img = 'YOU TOO?!';
    216216
    217217                add_filter( 'pre_get_avatar', array( $this, 'pre_get_avatar_filter' ), 10, 1 );
    218218                $img = get_avatar( 1 );
    219219                remove_filter( 'pre_get_avatar', array( $this, 'pre_get_avatar_filter' ), 10 );
    220220
    221                 $this->assertEquals( $img, $this->fakeIMG );
     221                $this->assertEquals( $img, $this->fake_img );
    222222        }
    223223        public function pre_get_avatar_filter( $img ) {
    224                 return $this->fakeIMG;
     224                return $this->fake_img;
    225225        }
    226226
    227227        /**
    228228         * @ticket 21195
    229229         */
    230230        public function test_get_avatar_filter() {
    231                 $this->fakeURL = 'YA RLY';
     231                $this->fake_url = 'YA RLY';
    232232
    233233                add_filter( 'get_avatar', array( $this, 'get_avatar_filter' ), 10, 1 );
    234234                $img = get_avatar( 1 );
    235235                remove_filter( 'get_avatar', array( $this, 'get_avatar_filter' ), 10 );
    236236
    237                 $this->assertEquals( $img, $this->fakeURL );
     237                $this->assertEquals( $img, $this->fake_url );
    238238        }
    239239        public function get_avatar_filter( $img ) {
    240                 return $this->fakeURL;
     240                return $this->fake_url;
    241241        }
    242242
    243243        /**

  • tests/phpunit/tests/functions.php

     
    77        function test_wp_parse_args_object() {
    88                $x        = new MockClass;
    99                $x->_baba = 5;
    10                 $x->yZ    = 'baba';
     10                $x->yZ    = 'baba'; // phpcs:ignore WordPress.NamingConventions.ValidVariableName.NotSnakeCaseMemberVar
    1111                $x->a     = array( 5, 111, 'x' );
    1212                $this->assertEquals(
    1313                        array(
    … …  
    4343        function test_wp_parse_args_defaults() {
    4444                $x        = new MockClass;
    4545                $x->_baba = 5;
    46                 $x->yZ    = 'baba';
     46                $x->yZ    = 'baba'; // phpcs:ignore WordPress.NamingConventions.ValidVariableName.NotSnakeCaseMemberVar
    4747                $x->a     = array( 5, 111, 'x' );
    4848                $d        = array( 'pu' => 'bu' );
    4949                $this->assertEquals(

  • tests/phpunit/tests/general/wpError.php

     
    1414         *
    1515         * @var WP_Error
    1616         */
    17         public $WP_Error;
     17        public $wp_error;
    1818
    1919        /**
    2020         * Set up.
    … …  
    2222        public function setUp() {
    2323                parent::setUp();
    2424
    25                 $this->WP_Error = new WP_Error();
     25                $this->wp_error = new WP_Error();
    2626        }
    2727
    2828        public function test_WP_Error_should_be_of_type_WP_Error() {
    29                 $this->assertWPError( $this->WP_Error );
     29                $this->assertWPError( $this->wp_error );
    3030        }
    3131
    3232        public function test_WP_Error_with_default_empty_parameters_should_add_no_errors() {
    33                 $this->assertEmpty( $this->WP_Error->errors );
     33                $this->assertEmpty( $this->wp_error->errors );
    3434        }
    3535
    3636        public function test_WP_Error_with_empty_code_should_add_no_code() {
    37                 $this->assertSame( '', $this->WP_Error->get_error_code() );
     37                $this->assertSame( '', $this->wp_error->get_error_code() );
    3838        }
    3939
    4040        public function test_WP_Error_with_empty_code_should_add_no_message() {
    41                 $this->assertSame( '', $this->WP_Error->get_error_message() );
     41                $this->assertSame( '', $this->wp_error->get_error_message() );
    4242        }
    4343
    4444        public function test_WP_Error_with_empty_code_should_add_no_error_data() {
    45                 $this->assertEmpty( $this->WP_Error->error_data );
     45                $this->assertEmpty( $this->wp_error->error_data );
    4646        }
    4747
    4848        public function test_WP_Error_with_code_and_empty_message_should_add_error_with_that_code() {
    … …  
    103103         * @covers ::get_error_codes()
    104104         */
    105105        public function test_get_error_codes_with_no_errors_should_return_empty_array() {
    106                 $this->assertEmpty( $this->WP_Error->get_error_codes() );
     106                $this->assertEmpty( $this->wp_error->get_error_codes() );
    107107        }
    108108
    109109        /**
    110110         * @covers ::get_error_codes()
    111111         */
    112112        public function test_get_error_codes_with_one_error_should_return_an_array_with_only_that_code() {
    113                 $this->WP_Error->add( 'code', 'message' );
     113                $this->wp_error->add( 'code', 'message' );
    114114
    115                 $this->assertEqualSets( array( 'code' ), $this->WP_Error->get_error_codes() );
     115                $this->assertEqualSets( array( 'code' ), $this->wp_error->get_error_codes() );
    116116        }
    117117
    118118        /**
    119119         * @covers ::get_error_codes()
    120120         */
    121121        public function test_get_error_codes_with_multiple_errors_should_return_an_array_of_those_codes() {
    122                 $this->WP_Error->add( 'code', 'message' );
    123                 $this->WP_Error->add( 'code2', 'message2' );
     122                $this->wp_error->add( 'code', 'message' );
     123                $this->wp_error->add( 'code2', 'message2' );
    124124
    125125                $expected = array( 'code', 'code2' );
    126126
    127                 $this->assertEqualSets( $expected, $this->WP_Error->get_error_codes() );
     127                $this->assertEqualSets( $expected, $this->wp_error->get_error_codes() );
    128128        }
    129129
    130130        /**
    131131         * @covers ::get_error_code()
    132132         */
    133133        public function test_get_error_code_with_no_errors_should_return_an_empty_string() {
    134                 $this->assertSame( '', $this->WP_Error->get_error_code() );
     134                $this->assertSame( '', $this->wp_error->get_error_code() );
    135135        }
    136136
    137137        /**
    138138         * @covers ::get_error_code()
    139139         */
    140140        public function test_get_error_code_with_one_error_should_return_that_error_code() {
    141                 $this->WP_Error->add( 'code', 'message' );
     141                $this->wp_error->add( 'code', 'message' );
    142142
    143                 $this->assertSame( 'code', $this->WP_Error->get_error_code() );
     143                $this->assertSame( 'code', $this->wp_error->get_error_code() );
    144144        }
    145145
    146146        /**
    147147         * @covers ::get_error_code()
    148148         */
    149149        public function test_get_error_code_with_multiple_errors_should_return_only_the_first_error_code() {
    150                 $this->WP_Error->add( 'code', 'message' );
    151                 $this->WP_Error->add( 'code2', 'message2' );
     150                $this->wp_error->add( 'code', 'message' );
     151                $this->wp_error->add( 'code2', 'message2' );
    152152
    153                 $this->assertSame( 'code', $this->WP_Error->get_error_code() );
     153                $this->assertSame( 'code', $this->wp_error->get_error_code() );
    154154        }
    155155
    156156        /**
    157157         * @covers ::get_error_messages()
    158158         */
    159159        public function test_get_error_messages_with_empty_code_and_no_errors_should_return_an_empty_array() {
    160                 $this->assertEmpty( $this->WP_Error->get_error_messages() );
     160                $this->assertEmpty( $this->wp_error->get_error_messages() );
    161161        }
    162162
    163163        /**
    164164         * @covers ::get_error_messages()
    165165         */
    166166        public function test_get_error_messages_with_empty_code_one_error_should_return_an_array_with_that_message() {
    167                 $this->WP_Error->add( 'code', 'message' );
     167                $this->wp_error->add( 'code', 'message' );
    168168
    169                 $this->assertEqualSets( array( 'message' ), $this->WP_Error->get_error_messages() );
     169                $this->assertEqualSets( array( 'message' ), $this->wp_error->get_error_messages() );
    170170        }
    171171
    172172        /**
    173173         * @covers ::get_error_messages()
    174174         */
    175175        public function test_get_error_messages_with_empty_code_multiple_errors_should_return_an_array_of_messages() {
    176                 $this->WP_Error->add( 'code', 'message' );
    177                 $this->WP_Error->add( 'code2', 'message2' );
     176                $this->wp_error->add( 'code', 'message' );
     177                $this->wp_error->add( 'code2', 'message2' );
    178178
    179                 $this->assertEqualSets( array( 'message', 'message2' ), $this->WP_Error->get_error_messages() );
     179                $this->assertEqualSets( array( 'message', 'message2' ), $this->wp_error->get_error_messages() );
    180180        }
    181181
    182182        /**
    183183         * @covers ::get_error_messages()
    184184         */
    185185        public function test_get_error_messages_with_an_invalid_code_should_return_an_empty_array() {
    186                 $this->assertEmpty( $this->WP_Error->get_error_messages( 'code' ) );
     186                $this->assertEmpty( $this->wp_error->get_error_messages( 'code' ) );
    187187        }
    188188
    189189        /**
    190190         * @covers ::get_error_messages()
    191191         */
    192192        public function test_get_error_messages_with_one_error_should_return_an_array_with_that_message() {
    193                 $this->WP_Error->add( 'code', 'message' );
     193                $this->wp_error->add( 'code', 'message' );
    194194
    195                 $this->assertEqualSets( array( 'message' ), $this->WP_Error->get_error_messages( 'code' ) );
     195                $this->assertEqualSets( array( 'message' ), $this->wp_error->get_error_messages( 'code' ) );
    196196        }
    197197
    198198        /**
    199199         * @covers ::get_error_messages()
    200200         */
    201201        public function test_get_error_messages_with_multiple_errors_same_code_should_return_an_array_with_all_messages() {
    202                 $this->WP_Error->add( 'code', 'message' );
    203                 $this->WP_Error->add( 'code', 'message2' );
     202                $this->wp_error->add( 'code', 'message' );
     203                $this->wp_error->add( 'code', 'message2' );
    204204
    205                 $this->assertequalSets( array( 'message', 'message2' ), $this->WP_Error->get_error_messages( 'code' ) );
     205                $this->assertequalSets( array( 'message', 'message2' ), $this->wp_error->get_error_messages( 'code' ) );
    206206        }
    207207
    208208        /**
    209209         * @covers ::get_error_message()
    210210         */
    211211        public function test_get_error_message_with_empty_code_and_no_errors_should_return_an_empty_string() {
    212                 $this->assertSame( '', $this->WP_Error->get_error_message() );
     212                $this->assertSame( '', $this->wp_error->get_error_message() );
    213213        }
    214214
    215215        /**
    216216         * @covers ::get_error_message()
    217217         */
    218218        public function test_get_error_message_with_empty_code_and_one_error_should_return_that_message() {
    219                 $this->WP_Error->add( 'code', 'message' );
     219                $this->wp_error->add( 'code', 'message' );
    220220
    221                 $this->assertSame( 'message', $this->WP_Error->get_error_message() );
     221                $this->assertSame( 'message', $this->wp_error->get_error_message() );
    222222        }
    223223
    224224        /**
    225225         * @covers ::get_error_message()
    226226         */
    227227        public function test_get_error_message_with_empty_code_and_multiple_errors_should_return_the_first_message() {
    228                 $this->WP_Error->add( 'code', 'message' );
    229                 $this->WP_Error->add( 'code2', 'message2' );
     228                $this->wp_error->add( 'code', 'message' );
     229                $this->wp_error->add( 'code2', 'message2' );
    230230
    231                 $this->assertSame( 'message', $this->WP_Error->get_error_message() );
     231                $this->assertSame( 'message', $this->wp_error->get_error_message() );
    232232        }
    233233
    234234        /**
    235235         * @covers ::get_error_message()
    236236         */
    237237        public function test_get_error_message_with_empty_code_and_multiple_errors_multiple_codes_should_return_the_first_message() {
    238                 $this->WP_Error->add( 'code', 'message' );
    239                 $this->WP_Error->add( 'code2', 'message2' );
    240                 $this->WP_Error->add( 'code', 'message2' );
     238                $this->wp_error->add( 'code', 'message' );
     239                $this->wp_error->add( 'code2', 'message2' );
     240                $this->wp_error->add( 'code', 'message2' );
    241241
    242                 $this->assertSame( 'message', $this->WP_Error->get_error_message() );
     242                $this->assertSame( 'message', $this->wp_error->get_error_message() );
    243243        }
    244244
    245245        /**
    246246         * @covers ::get_error_message()
    247247         */
    248248        public function test_get_error_message_with_invalid_code_and_no_errors_should_return_empty_string() {
    249                 $this->assertSame( '', $this->WP_Error->get_error_message( 'invalid' ) );
     249                $this->assertSame( '', $this->wp_error->get_error_message( 'invalid' ) );
    250250        }
    251251
    252252        /**
    253253         * @covers ::get_error_message()
    254254         */
    255255        public function test_get_error_message_with_invalid_code_and_one_error_should_return_an_empty_string() {
    256                 $this->WP_Error->add( 'code', 'message' );
     256                $this->wp_error->add( 'code', 'message' );
    257257
    258                 $this->assertSame( '', $this->WP_Error->get_error_message( 'invalid' ) );
     258                $this->assertSame( '', $this->wp_error->get_error_message( 'invalid' ) );
    259259        }
    260260
    261261        /**
    262262         * @covers ::get_error_message()
    263263         */
    264264        public function test_get_error_message_with_invalid_code_and_multiple_errors_should_return_an_empty_string() {
    265                 $this->WP_Error->add( 'code', 'message' );
    266                 $this->WP_Error->add( 'code2', 'message2' );
     265                $this->wp_error->add( 'code', 'message' );
     266                $this->wp_error->add( 'code2', 'message2' );
    267267
    268                 $this->assertSame( '', $this->WP_Error->get_error_message( 'invalid' ) );
     268                $this->assertSame( '', $this->wp_error->get_error_message( 'invalid' ) );
    269269        }
    270270
    271271        /**
    272272         * @covers ::get_error_data()
    273273         */
    274274        public function test_get_error_data_with_empty_code_and_no_errors_should_evaluate_as_null() {
    275                 $this->assertSame( null, $this->WP_Error->get_error_data() );
     275                $this->assertSame( null, $this->wp_error->get_error_data() );
    276276        }
    277277
    278278        /**
    279279         * @covers ::get_error_data()
    280280         */
    281281        public function test_get_error_data_with_empty_code_one_error_no_data_should_evaluate_as_null() {
    282                 $this->WP_Error->add( 'code', 'message' );
     282                $this->wp_error->add( 'code', 'message' );
    283283
    284                 $this->assertSame( null, $this->WP_Error->get_error_data() );
     284                $this->assertSame( null, $this->wp_error->get_error_data() );
    285285        }
    286286
    287287        /**
    288288         * @covers ::get_error_data()
    289289         */
    290290        public function test_get_error_data_with_empty_code_multiple_errors_no_data_should_evaluate_as_null() {
    291                 $this->WP_Error->add( 'code', 'message' );
    292                 $this->WP_Error->add( 'code2', 'message2' );
     291                $this->wp_error->add( 'code', 'message' );
     292                $this->wp_error->add( 'code2', 'message2' );
    293293
    294                 $this->assertSame( null, $this->WP_Error->get_error_data() );
     294                $this->assertSame( null, $this->wp_error->get_error_data() );
    295295        }
    296296
    297297        /**
    … …  
    299299         */
    300300        public function test_get_error_data_with_empty_code_and_one_error_with_data_should_return_that_data() {
    301301                $expected = array( 'data-key' => 'data-value' );
    302                 $this->WP_Error->add( 'code', 'message', $expected );
     302                $this->wp_error->add( 'code', 'message', $expected );
    303303
    304                 $this->assertEqualSetsWithIndex( $expected, $this->WP_Error->get_error_data() );
     304                $this->assertEqualSetsWithIndex( $expected, $this->wp_error->get_error_data() );
    305305        }
    306306
    307307        /**
    … …  
    309309         */
    310310        public function test_get_error_data_with_empty_code_and_multiple_errors_different_codes_should_return_the_last_data_of_the_first_code() {
    311311                $expected = array( 'data-key' => 'data-value' );
    312                 $this->WP_Error->add( 'code', 'message', $expected );
    313                 $this->WP_Error->add( 'code2', 'message2', 'data2' );
     312                $this->wp_error->add( 'code', 'message', $expected );
     313                $this->wp_error->add( 'code2', 'message2', 'data2' );
    314314
    315                 $this->assertEqualSetsWithIndex( $expected, $this->WP_Error->get_error_data() );
     315                $this->assertEqualSetsWithIndex( $expected, $this->wp_error->get_error_data() );
    316316        }
    317317
    318318        /**
    319319         * @covers ::get_error_data()
    320320         */
    321321        public function test_get_error_data_with_empty_code_and_multiple_errors_same_code_should_return_the_last_data_of_the_first_code() {
    322                 $this->WP_Error->add( 'code', 'message', 'data' );
    323                 $this->WP_Error->add( 'code', 'message2', 'data2' );
    324                 $this->WP_Error->add( 'code2', 'message2', 'data3' );
     322                $this->wp_error->add( 'code', 'message', 'data' );
     323                $this->wp_error->add( 'code', 'message2', 'data2' );
     324                $this->wp_error->add( 'code2', 'message2', 'data3' );
    325325
    326                 $this->assertSame( 'data2', $this->WP_Error->get_error_data() );
     326                $this->assertSame( 'data2', $this->wp_error->get_error_data() );
    327327        }
    328328
    329329        /**
    330330         * @covers ::get_error_data()
    331331         */
    332332        public function test_get_error_data_with_code_and_no_errors_should_evaluate_as_null() {
    333                 $this->assertSame( null, $this->WP_Error->get_error_data( 'code' ) );
     333                $this->assertSame( null, $this->wp_error->get_error_data( 'code' ) );
    334334        }
    335335
    336336        /**
    337337         * @covers ::get_error_data()
    338338         */
    339339        public function test_get_error_data_with_code_and_one_error_with_no_data_should_evaluate_as_null() {
    340                 $this->WP_Error->add( 'code', 'message' );
     340                $this->wp_error->add( 'code', 'message' );
    341341
    342                 $this->assertSame( null, $this->WP_Error->get_error_data( 'code' ) );
     342                $this->assertSame( null, $this->wp_error->get_error_data( 'code' ) );
    343343        }
    344344
    345345        /**
    … …  
    347347         */
    348348        public function test_get_error_data_with_code_and_one_error_with_data_should_return_that_data() {
    349349                $expected = array( 'data-key' => 'data-value' );
    350                 $this->WP_Error->add( 'code', 'message', $expected );
     350                $this->wp_error->add( 'code', 'message', $expected );
    351351
    352                 $this->assertEqualSetsWithIndex( $expected, $this->WP_Error->get_error_data( 'code' ) );
     352                $this->assertEqualSetsWithIndex( $expected, $this->wp_error->get_error_data( 'code' ) );
    353353        }
    354354
    355355        /**
    … …  
    357357         */
    358358        public function test_get_error_data_with_code_and_multiple_errors_different_codes_should_return_the_last_stored_data_of_the_code() {
    359359                $expected = array( 'data3' );
    360                 $this->WP_Error->add( 'code', 'message', 'data' );
    361                 $this->WP_Error->add( 'code2', 'message2', 'data2' );
    362                 $this->WP_Error->add( 'code', 'message3', $expected );
     360                $this->wp_error->add( 'code', 'message', 'data' );
     361                $this->wp_error->add( 'code2', 'message2', 'data2' );
     362                $this->wp_error->add( 'code', 'message3', $expected );
    363363
    364                 $this->assertEqualSetsWithIndex( $expected, $this->WP_Error->get_error_data( 'code' ) );
     364                $this->assertEqualSetsWithIndex( $expected, $this->wp_error->get_error_data( 'code' ) );
    365365        }
    366366
    367367        /**
    368368         * @covers ::get_error_data()
    369369         */
    370370        public function test_get_error_data_with_code_and_multiple_errors_same_code_should_return_the_last_stored_data() {
    371                 $this->WP_Error->add( 'code', 'message', 'data' );
    372                 $this->WP_Error->add( 'code', 'message2', 'data2' );
    373                 $this->WP_Error->add( 'code2', 'message3', 'data3' );
     371                $this->wp_error->add( 'code', 'message', 'data' );
     372                $this->wp_error->add( 'code', 'message2', 'data2' );
     373                $this->wp_error->add( 'code2', 'message3', 'data3' );
    374374
    375                 $this->assertSame( 'data2', $this->WP_Error->get_error_data( 'code' ) );
     375                $this->assertSame( 'data2', $this->wp_error->get_error_data( 'code' ) );
    376376        }
    377377
    378378        /**
    379379         * @covers ::has_errors()
    380380         */
    381381        public function test_has_errors_with_no_errors_returns_false() {
    382                 $this->assertFalse( $this->WP_Error->has_errors() );
     382                $this->assertFalse( $this->wp_error->has_errors() );
    383383        }
    384384
    385385        /**
    386386         * @covers ::has_errors()
    387387         */
    388388        public function test_has_errors_with_errors_returns_true() {
    389                 $this->WP_Error->add( 'code', 'message', 'data' );
    390                 $this->assertTrue( $this->WP_Error->has_errors() );
     389                $this->wp_error->add( 'code', 'message', 'data' );
     390                $this->assertTrue( $this->wp_error->has_errors() );
    391391        }
    392392
    393393        /**
    394394         * @covers ::add()
    395395         */
    396396        public function test_add_with_empty_code_empty_message_empty_data_should_add_empty_key_to_errors_array() {
    397                 $this->WP_Error->add( '', '', 'data' );
     397                $this->wp_error->add( '', '', 'data' );
    398398
    399                 $this->assertArrayHasKey( '', $this->WP_Error->errors );
     399                $this->assertArrayHasKey( '', $this->wp_error->errors );
    400400        }
    401401
    402402        /**
    403403         * @covers ::add()
    404404         */
    405405        public function test_add_with_empty_code_empty_message_empty_data_should_add_empty_message_to_errors_array_under_empty_key() {
    406                 $this->WP_Error->add( '', '', 'data' );
     406                $this->wp_error->add( '', '', 'data' );
    407407
    408                 $this->assertEqualSetsWithIndex( array( '' => array( '' ) ), $this->WP_Error->errors );
     408                $this->assertEqualSetsWithIndex( array( '' => array( '' ) ), $this->wp_error->errors );
    409409        }
    410410
    411411        /**
    412412         * @covers ::add()
    413413         */
    414414        public function test_add_with_empty_code_empty_message_empty_data_should_not_alter_data() {
    415                 $this->WP_Error->add( '', '', '' );
     415                $this->wp_error->add( '', '', '' );
    416416
    417                 $this->assertEmpty( $this->WP_Error->error_data );
     417                $this->assertEmpty( $this->wp_error->error_data );
    418418        }
    419419
    420420        /**
    421421         * @covers ::add()
    422422         */
    423423        public function test_add_with_empty_code_empty_message_non_empty_data_should_store_data_under_an_empty_code_key() {
    424                 $this->WP_Error->add( '', '', 'data' );
     424                $this->wp_error->add( '', '', 'data' );
    425425
    426                 $this->assertEqualSetsWithIndex( array( '' => 'data' ), $this->WP_Error->error_data );
     426                $this->assertEqualSetsWithIndex( array( '' => 'data' ), $this->wp_error->error_data );
    427427        }
    428428
    429429        /**
    430430         * @covers ::add()
    431431         */
    432432        public function test_add_with_code_empty_message_empty_data_should_add_error_with_code() {
    433                 $this->WP_Error->add( 'code', '' );
     433                $this->wp_error->add( 'code', '' );
    434434
    435                 $this->assertSame( 'code', $this->WP_Error->get_error_code() );
     435                $this->assertSame( 'code', $this->wp_error->get_error_code() );
    436436        }
    437437
    438438        /**
    439439         * @covers ::add()
    440440         */
    441441        public function test_add_with_code_empty_message_empty_data_should_add_error_with_empty_message() {
    442                 $this->WP_Error->add( 'code', '' );
     442                $this->wp_error->add( 'code', '' );
    443443
    444                 $this->assertSame( '', $this->WP_Error->get_error_message( 'code' ) );
     444                $this->assertSame( '', $this->wp_error->get_error_message( 'code' ) );
    445445        }
    446446
    447447        /**
    448448         * @covers ::add()
    449449         */
    450450        public function test_add_with_code_empty_message_empty_data_should_not_add_error_data() {
    451                 $this->WP_Error->add( 'code', '' );
     451                $this->wp_error->add( 'code', '' );
    452452
    453                 $this->assertSame( null, $this->WP_Error->get_error_data( 'code' ) );
     453                $this->assertSame( null, $this->wp_error->get_error_data( 'code' ) );
    454454        }
    455455
    456456        /**
    457457         * @covers ::add()
    458458         */
    459459        public function test_add_with_code_and_message_and_empty_data_should_should_add_error_with_that_message() {
    460                 $this->WP_Error->add( 'code', 'message' );
     460                $this->wp_error->add( 'code', 'message' );
    461461
    462                 $this->assertSame( 'message', $this->WP_Error->get_error_message( 'code' ) );
     462                $this->assertSame( 'message', $this->wp_error->get_error_message( 'code' ) );
    463463        }
    464464
    465465        /**
    466466         * @covers ::add()
    467467         */
    468468        public function test_add_with_code_and_message_and_empty_data_should_not_alter_stored_data() {
    469                 $this->WP_Error->add( 'code', 'message' );
     469                $this->wp_error->add( 'code', 'message' );
    470470
    471                 $this->assertSame( null, $this->WP_Error->get_error_data( 'code' ) );
     471                $this->assertSame( null, $this->wp_error->get_error_data( 'code' ) );
    472472        }
    473473
    474474        /**
    475475         * @covers ::add()
    476476         */
    477477        public function test_add_with_code_and_empty_message_and_data_should_add_error_with_that_code() {
    478                 $this->WP_Error->add( 'code', '', 'data' );
     478                $this->wp_error->add( 'code', '', 'data' );
    479479
    480                 $this->assertSame( 'code', $this->WP_Error->get_error_code() );
     480                $this->assertSame( 'code', $this->wp_error->get_error_code() );
    481481        }
    482482
    483483        /**
    484484         * @covers ::add()
    485485         */
    486486        public function test_add_with_code_and_empty_message_and_data_should_store_that_data() {
    487                 $this->WP_Error->add( 'code', '', 'data' );
     487                $this->wp_error->add( 'code', '', 'data' );
    488488
    489                 $this->assertSame( 'data', $this->WP_Error->get_error_data( 'code' ) );
     489                $this->assertSame( 'data', $this->wp_error->get_error_data( 'code' ) );
    490490        }
    491491
    492492        /**
    493493         * @covers ::add()
    494494         */
    495495        public function test_add_with_code_and_message_and_data_should_add_an_error_with_that_code() {
    496                 $this->WP_Error->add( 'code', 'message', 'data' );
     496                $this->wp_error->add( 'code', 'message', 'data' );
    497497
    498                 $this->assertSame( 'code', $this->WP_Error->get_error_code() );
     498                $this->assertSame( 'code', $this->wp_error->get_error_code() );
    499499        }
    500500
    501501        /**
    502502         * @covers ::add()
    503503         */
    504504        public function test_add_with_code_and_message_and_data_should_add_an_error_with_that_message() {
    505                 $this->WP_Error->add( 'code', 'message', 'data' );
     505                $this->wp_error->add( 'code', 'message', 'data' );
    506506
    507                 $this->assertSame( 'message', $this->WP_Error->get_error_message( 'code' ) );
     507                $this->assertSame( 'message', $this->wp_error->get_error_message( 'code' ) );
    508508        }
    509509
    510510        /**
    511511         * @covers ::add()
    512512         */
    513513        public function test_add_with_code_and_message_and_data_should_store_that_data() {
    514                 $this->WP_Error->add( 'code', 'message', 'data' );
     514                $this->wp_error->add( 'code', 'message', 'data' );
    515515
    516                 $this->assertSame( 'data', $this->WP_Error->get_error_data( 'code' ) );
     516                $this->assertSame( 'data', $this->wp_error->get_error_data( 'code' ) );
    517517        }
    518518
    519519        /**
    520520         * @covers ::add()
    521521         */
    522522        public function test_add_multiple_times_with_the_same_code_should_add_additional_messages_for_that_code() {
    523                 $this->WP_Error->add( 'code', 'message' );
    524                 $this->WP_Error->add( 'code', 'message2' );
     523                $this->wp_error->add( 'code', 'message' );
     524                $this->wp_error->add( 'code', 'message2' );
    525525
    526526                $expected = array( 'message', 'message2' );
    527527
    528                 $this->assertEqualSets( $expected, $this->WP_Error->get_error_messages( 'code' ) );
     528                $this->assertEqualSets( $expected, $this->wp_error->get_error_messages( 'code' ) );
    529529        }
    530530
    531531        /**
    532532         * @covers ::add()
    533533         */
    534534        public function test_add_multiple_times_with_the_same_code_and_different_data_should_store_only_the_last_added_data() {
    535                 $this->WP_Error->add( 'code', 'message', 'data-bar' );
    536                 $this->WP_Error->add( 'code', 'message2', 'data-baz' );
     535                $this->wp_error->add( 'code', 'message', 'data-bar' );
     536                $this->wp_error->add( 'code', 'message2', 'data-baz' );
    537537
    538                 $this->assertSame( 'data-baz', $this->WP_Error->get_error_data( 'code' ) );
     538                $this->assertSame( 'data-baz', $this->wp_error->get_error_data( 'code' ) );
    539539        }
    540540
    541541        /**
    542542         * @covers ::add_data()
    543543         */
    544544        public function test_add_data_with_empty_data_empty_code_should_create_orphaned_data_with_no_error() {
    545                 $this->WP_Error->add_data( '' );
     545                $this->wp_error->add_data( '' );
    546546
    547                 $this->assertEmpty( $this->WP_Error->errors );
     547                $this->assertEmpty( $this->wp_error->errors );
    548548        }
    549549
    550550        /**
    551551         * @covers ::add_data()
    552552         */
    553553        public function test_add_data_with_empty_data_empty_code_no_errors_should_create_data_under_an_empty_code_key() {
    554                 $this->WP_Error->add_data( '' );
     554                $this->wp_error->add_data( '' );
    555555
    556                 $this->assertEqualSets( array( '' => '' ), $this->WP_Error->error_data );
     556                $this->assertEqualSets( array( '' => '' ), $this->wp_error->error_data );
    557557        }
    558558
    559559        /**
    560560         * @covers ::add_data()
    561561         */
    562562        public function test_add_data_with_data_empty_code_and_one_error_should_store_the_data_under_that_code() {
    563                 $this->WP_Error->add( 'code', 'message' );
    564                 $this->WP_Error->add_data( 'data' );
     563                $this->wp_error->add( 'code', 'message' );
     564                $this->wp_error->add_data( 'data' );
    565565
    566                 $this->assertSame( 'data', $this->WP_Error->get_error_data( 'code' ) );
     566                $this->assertSame( 'data', $this->wp_error->get_error_data( 'code' ) );
    567567        }
    568568
    569569        /**
    570570         * @covers ::add_data()
    571571         */
    572572        public function test_add_data_with_data_empty_code_and_multiple_errors_with_different_codes_should_store_it_under_the_first_code() {
    573                 $this->WP_Error->add( 'code', 'message' );
    574                 $this->WP_Error->add( 'code2', 'message2' );
     573                $this->wp_error->add( 'code', 'message' );
     574                $this->wp_error->add( 'code2', 'message2' );
    575575
    576                 $this->WP_Error->add_data( 'data' );
     576                $this->wp_error->add_data( 'data' );
    577577
    578                 $this->assertSame( 'data', $this->WP_Error->get_error_data( 'code' ) );
     578                $this->assertSame( 'data', $this->wp_error->get_error_data( 'code' ) );
    579579        }
    580580
    581581        /**
    582582         * @covers ::add_data()
    583583         */
    584584        public function test_add_data_with_data_empty_code_and_multiple_errors_with_same_code_should_store_it_under_the_first_code() {
    585                 $this->WP_Error->add( 'code', 'message' );
    586                 $this->WP_Error->add( 'code2', 'message2' );
    587                 $this->WP_Error->add( 'code', 'message3' );
     585                $this->wp_error->add( 'code', 'message' );
     586                $this->wp_error->add( 'code2', 'message2' );
     587                $this->wp_error->add( 'code', 'message3' );
    588588
    589                 $this->WP_Error->add_data( 'data' );
     589                $this->wp_error->add_data( 'data' );
    590590
    591                 $this->assertSame( 'data', $this->WP_Error->get_error_data( 'code' ) );
     591                $this->assertSame( 'data', $this->wp_error->get_error_data( 'code' ) );
    592592        }
    593593
    594594        /**
    595595         * @covers ::add_data()
    596596         */
    597597        public function test_add_data_with_data_and_code_and_no_errors_should_create_orphaned_data_with_no_error() {
    598                 $this->WP_Error->add_data( 'data', 'code' );
     598                $this->wp_error->add_data( 'data', 'code' );
    599599
    600                 $this->assertEmpty( $this->WP_Error->errors );
     600                $this->assertEmpty( $this->wp_error->errors );
    601601        }
    602602
    603603        /**
    604604         * @covers ::add_data()
    605605         */
    606606        public function test_add_data_with_data_and_code_no_errors_should_create_data_under_that_code_key() {
    607                 $this->WP_Error->add_data( 'data', 'code' );
     607                $this->wp_error->add_data( 'data', 'code' );
    608608
    609                 $this->assertEqualSets( array( 'code' => 'data' ), $this->WP_Error->error_data );
     609                $this->assertEqualSets( array( 'code' => 'data' ), $this->wp_error->error_data );
    610610        }
    611611
    612612        /**
    613613         * @covers ::add_data()
    614614         */
    615615        public function test_add_data_with_data_and_code_one_error_different_code_should_create_orphaned_data_with_no_error() {
    616                 $this->WP_Error->add( 'code', 'message' );
     616                $this->wp_error->add( 'code', 'message' );
    617617
    618                 $this->WP_Error->add_data( 'data', 'code2' );
     618                $this->wp_error->add_data( 'data', 'code2' );
    619619
    620                 $this->assertEqualSetsWithIndex( array( 'code' => array( 'message' ) ), $this->WP_Error->errors );
     620                $this->assertEqualSetsWithIndex( array( 'code' => array( 'message' ) ), $this->wp_error->errors );
    621621        }
    622622
    623623        /**
    624624         * @covers ::add_data()
    625625         */
    626626        public function test_add_data_with_data_and_code_one_error_different_code_should_create_data_under_that_code_key() {
    627                 $this->WP_Error->add( 'code', 'message' );
     627                $this->wp_error->add( 'code', 'message' );
    628628
    629                 $this->WP_Error->add_data( 'data', 'code2' );
     629                $this->wp_error->add_data( 'data', 'code2' );
    630630
    631                 $this->assertEqualSetsWithIndex( array( 'code2' => 'data' ), $this->WP_Error->error_data );
     631                $this->assertEqualSetsWithIndex( array( 'code2' => 'data' ), $this->wp_error->error_data );
    632632        }
    633633
    634634        /**
    635635         * @covers ::add_data()
    636636         */
    637637        public function test_add_data_with_data_and_code_should_add_data() {
    638                 $this->WP_Error->add( 'code', 'message' );
     638                $this->wp_error->add( 'code', 'message' );
    639639
    640                 $this->WP_Error->add_data( 'data', 'code' );
     640                $this->wp_error->add_data( 'data', 'code' );
    641641
    642                 $this->assertSame( 'data', $this->WP_Error->get_error_data( 'code' ) );
     642                $this->assertSame( 'data', $this->wp_error->get_error_data( 'code' ) );
    643643        }
    644644
    645645        /**
    646646         * @covers ::remove()
    647647         */
    648648        public function test_remove_with_no_errors_should_affect_nothing() {
    649                 $before = $this->WP_Error->errors;
     649                $before = $this->wp_error->errors;
    650650
    651                 $this->WP_Error->remove( 'code' );
     651                $this->wp_error->remove( 'code' );
    652652
    653                 $after = $this->WP_Error->errors;
     653                $after = $this->wp_error->errors;
    654654
    655655                $this->assertEqualSetsWithIndex( $before, $after );
    656656        }
    … …  
    659659         * @covers ::remove()
    660660         */
    661661        public function test_remove_empty_code_no_errors_should_affect_nothing() {
    662                 $before = $this->WP_Error->errors;
     662                $before = $this->wp_error->errors;
    663663
    664                 $this->WP_Error->remove( '' );
     664                $this->wp_error->remove( '' );
    665665
    666                 $after = $this->WP_Error->errors;
     666                $after = $this->wp_error->errors;
    667667
    668668                $this->assertEqualSetsWithIndex( $before, $after );
    669669        }
    … …  
    672672         * @covers ::remove()
    673673         */
    674674        public function test_remove_empty_code_and_one_error_with_empty_string_code_should_remove_error() {
    675                 $before = $this->WP_Error->errors;
     675                $before = $this->wp_error->errors;
    676676
    677                 $this->WP_Error->add( '', 'message' );
     677                $this->wp_error->add( '', 'message' );
    678678
    679                 $this->WP_Error->remove( '' );
     679                $this->wp_error->remove( '' );
    680680
    681                 $after = $this->WP_Error->errors;
     681                $after = $this->wp_error->errors;
    682682
    683683                $this->assertEqualSetsWithIndex( $before, $after );
    684684        }
    … …  
    687687         * @covers ::remove()
    688688         */
    689689        public function test_remove_empty_code_and_one_error_with_empty_string_code_should_remove_error_data() {
    690                 $this->WP_Error->add( '', 'message', 'data' );
     690                $this->wp_error->add( '', 'message', 'data' );
    691691
    692                 $this->WP_Error->remove( '' );
     692                $this->wp_error->remove( '' );
    693693
    694                 $after = $this->WP_Error->error_data;
     694                $after = $this->wp_error->error_data;
    695695
    696                 $this->assertEmpty( $this->WP_Error->error_data );
     696                $this->assertEmpty( $this->wp_error->error_data );
    697697        }
    698698
    699699        /**
    700700         * @covers ::remove()
    701701         */
    702702        public function test_remove_should_remove_the_error_with_the_given_code() {
    703                 $this->WP_Error->add( 'code', 'message' );
     703                $this->wp_error->add( 'code', 'message' );
    704704
    705                 $this->WP_Error->remove( 'code' );
     705                $this->wp_error->remove( 'code' );
    706706
    707                 $this->assertEmpty( $this->WP_Error->errors );
     707                $this->assertEmpty( $this->wp_error->errors );
    708708        }
    709709
    710710        /**
    711711         * @covers ::remove()
    712712         */
    713713        public function test_remove_should_remove_the_error_data_associated_with_the_given_code() {
    714                 $this->WP_Error->add( 'code', 'message', 'data' );
     714                $this->wp_error->add( 'code', 'message', 'data' );
    715715
    716                 $this->WP_Error->remove( 'code' );
     716                $this->wp_error->remove( 'code' );
    717717
    718                 $this->assertEmpty( $this->WP_Error->error_data );
     718                $this->assertEmpty( $this->wp_error->error_data );
    719719        }
    720720
    721721}

  • tests/phpunit/tests/http/base.php

     
    1313abstract class WP_HTTP_UnitTestCase extends WP_UnitTestCase {
    1414        // You can use your own version of data/WPHTTP-testcase-redirection-script.php here.
    1515        var $redirection_script = 'http://api.wordpress.org/core/tests/1.0/redirection.php';
    16         var $fileStreamUrl      = 'http://s.w.org/screenshots/3.9/dashboard.png';
     16        var $file_stream_url    = 'http://s.w.org/screenshots/3.9/dashboard.png';
    1717
    1818        protected $http_request_args;
    1919
    … …  
    254254        }
    255255
    256256        function test_file_stream() {
    257                 $url  = $this->fileStreamUrl;
     257                $url  = $this->file_stream_url;
    258258                $size = 153204;
    259259                $res  = wp_remote_request(
    260260                        $url,
    … …  
    282282         * @ticket 26726
    283283         */
    284284        function test_file_stream_limited_size() {
    285                 $url  = $this->fileStreamUrl;
     285                $url  = $this->file_stream_url;
    286286                $size = 10000;
    287287                $res  = wp_remote_request(
    288288                        $url,
    … …  
    311311         * @ticket 31172
    312312         */
    313313        function test_request_limited_size() {
    314                 $url  = $this->fileStreamUrl;
     314                $url  = $this->file_stream_url;
    315315                $size = 10000;
    316316
    317317                $res = wp_remote_request(

  • tests/phpunit/tests/http/curl.php

     
    1515        public function test_http_api_curl_stream_parameter_is_a_reference() {
    1616                add_action( 'http_api_curl', array( $this, '_action_test_http_api_curl_stream_parameter_is_a_reference' ), 10, 3 );
    1717                wp_remote_request(
    18                         $this->fileStreamUrl,
     18                        $this->file_stream_url,
    1919                        array(
    2020                                'stream'  => true,
    2121                                'timeout' => 30,

  • tests/phpunit/tests/user.php

     
    199199         * @ticket 20043
    200200         */
    201201        public function test_user_unset() {
     202                // phpcs:disable WordPress.NamingConventions.ValidVariableName.NotSnakeCaseMemberVar
    202203                $user = new WP_User( self::$author_id );
    203204
    204205                // Test custom fields
    … …  
    207208                unset( $user->customField );
    208209                $this->assertFalse( isset( $user->customField ) );
    209210                return $user;
     211                // phpcs:enable
    210212        }
    211213
    212214        /**