WordPress.org

Make WordPress Core

Changeset 44573


Ignore:
Timestamp:
01/12/2019 06:05:55 AM (6 months ago)
Author:
pento
Message:

Coding Standards: Fix and whitelist variable names.

From the WordPress.NamingConventions.ValidVariableName sniff, this commit fixes/whitelists all NotSnakeCaseMemberVar, MemberNotSnakeCase, and StringNotSnakeCase violations. It also fixes a handful of the NotSnakeCase violations.

See #45934.

Location:
trunk
Files:
8 edited

Legend:

Unmodified
Added
Removed
  • trunk/phpcs.xml.dist

    r44571 r44573  
    99        <properties>
    1010            <property name="allowMultipleArguments" value="false"/>
     11        </properties>
     12    </rule>
     13
     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>
    1165        </properties>
    1266    </rule>
  • trunk/src/wp-includes/class-wp-comment-query.php

    r44546 r44573  
    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 ) {
     
    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
     
    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                    }
     
    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
     
    619619
    620620                // Default to DESC.
    621                 if ( ! $comment_ID_order ) {
    622                     $comment_ID_order = 'DESC';
    623                 }
    624 
    625                 $orderby_array[] = "$wpdb->comments.comment_ID $comment_ID_order";
     621                if ( ! $comment_id_order ) {
     622                    $comment_id_order = 'DESC';
     623                }
     624
     625                $orderby_array[] = "$wpdb->comments.comment_ID $comment_id_order";
    626626            }
    627627
  • trunk/tests/phpunit/tests/avatar.php

    r44499 r44573  
    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 );
     
    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    }
     
    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 );
     
    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
     
    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 );
     
    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
     
    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 );
     
    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
  • trunk/tests/phpunit/tests/functions.php

    r44546 r44573  
    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(
     
    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' );
  • trunk/tests/phpunit/tests/general/wpError.php

    r42761 r44573  
    1515     * @var WP_Error
    1616     */
    17     public $WP_Error;
     17    public $wp_error;
    1818
    1919    /**
     
    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
     
    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
     
    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' );
    114 
    115         $this->assertEqualSets( array( 'code' ), $this->WP_Error->get_error_codes() );
     113        $this->wp_error->add( 'code', 'message' );
     114
     115        $this->assertEqualSets( array( 'code' ), $this->wp_error->get_error_codes() );
    116116    }
    117117
     
    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
     
    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
     
    139139     */
    140140    public function test_get_error_code_with_one_error_should_return_that_error_code() {
    141         $this->WP_Error->add( 'code', 'message' );
    142 
    143         $this->assertSame( 'code', $this->WP_Error->get_error_code() );
     141        $this->wp_error->add( 'code', 'message' );
     142
     143        $this->assertSame( 'code', $this->wp_error->get_error_code() );
    144144    }
    145145
     
    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' );
    152 
    153         $this->assertSame( 'code', $this->WP_Error->get_error_code() );
     150        $this->wp_error->add( 'code', 'message' );
     151        $this->wp_error->add( 'code2', 'message2' );
     152
     153        $this->assertSame( 'code', $this->wp_error->get_error_code() );
    154154    }
    155155
     
    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
     
    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' );
    168 
    169         $this->assertEqualSets( array( 'message' ), $this->WP_Error->get_error_messages() );
     167        $this->wp_error->add( 'code', 'message' );
     168
     169        $this->assertEqualSets( array( 'message' ), $this->wp_error->get_error_messages() );
    170170    }
    171171
     
    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' );
    178 
    179         $this->assertEqualSets( array( 'message', 'message2' ), $this->WP_Error->get_error_messages() );
     176        $this->wp_error->add( 'code', 'message' );
     177        $this->wp_error->add( 'code2', 'message2' );
     178
     179        $this->assertEqualSets( array( 'message', 'message2' ), $this->wp_error->get_error_messages() );
    180180    }
    181181
     
    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
     
    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' );
    194 
    195         $this->assertEqualSets( array( 'message' ), $this->WP_Error->get_error_messages( 'code' ) );
     193        $this->wp_error->add( 'code', 'message' );
     194
     195        $this->assertEqualSets( array( 'message' ), $this->wp_error->get_error_messages( 'code' ) );
    196196    }
    197197
     
    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' );
    204 
    205         $this->assertequalSets( array( 'message', 'message2' ), $this->WP_Error->get_error_messages( 'code' ) );
     202        $this->wp_error->add( 'code', 'message' );
     203        $this->wp_error->add( 'code', 'message2' );
     204
     205        $this->assertequalSets( array( 'message', 'message2' ), $this->wp_error->get_error_messages( 'code' ) );
    206206    }
    207207
     
    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
     
    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' );
    220 
    221         $this->assertSame( 'message', $this->WP_Error->get_error_message() );
     219        $this->wp_error->add( 'code', 'message' );
     220
     221        $this->assertSame( 'message', $this->wp_error->get_error_message() );
    222222    }
    223223
     
    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' );
    230 
    231         $this->assertSame( 'message', $this->WP_Error->get_error_message() );
     228        $this->wp_error->add( 'code', 'message' );
     229        $this->wp_error->add( 'code2', 'message2' );
     230
     231        $this->assertSame( 'message', $this->wp_error->get_error_message() );
    232232    }
    233233
     
    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' );
    241 
    242         $this->assertSame( 'message', $this->WP_Error->get_error_message() );
     238        $this->wp_error->add( 'code', 'message' );
     239        $this->wp_error->add( 'code2', 'message2' );
     240        $this->wp_error->add( 'code', 'message2' );
     241
     242        $this->assertSame( 'message', $this->wp_error->get_error_message() );
    243243    }
    244244
     
    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
     
    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' );
    257 
    258         $this->assertSame( '', $this->WP_Error->get_error_message( 'invalid' ) );
     256        $this->wp_error->add( 'code', 'message' );
     257
     258        $this->assertSame( '', $this->wp_error->get_error_message( 'invalid' ) );
    259259    }
    260260
     
    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' );
    267 
    268         $this->assertSame( '', $this->WP_Error->get_error_message( 'invalid' ) );
     265        $this->wp_error->add( 'code', 'message' );
     266        $this->wp_error->add( 'code2', 'message2' );
     267
     268        $this->assertSame( '', $this->wp_error->get_error_message( 'invalid' ) );
    269269    }
    270270
     
    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
     
    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' );
    283 
    284         $this->assertSame( null, $this->WP_Error->get_error_data() );
     282        $this->wp_error->add( 'code', 'message' );
     283
     284        $this->assertSame( null, $this->wp_error->get_error_data() );
    285285    }
    286286
     
    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' );
    293 
    294         $this->assertSame( null, $this->WP_Error->get_error_data() );
     291        $this->wp_error->add( 'code', 'message' );
     292        $this->wp_error->add( 'code2', 'message2' );
     293
     294        $this->assertSame( null, $this->wp_error->get_error_data() );
    295295    }
    296296
     
    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 );
    303 
    304         $this->assertEqualSetsWithIndex( $expected, $this->WP_Error->get_error_data() );
     302        $this->wp_error->add( 'code', 'message', $expected );
     303
     304        $this->assertEqualSetsWithIndex( $expected, $this->wp_error->get_error_data() );
    305305    }
    306306
     
    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' );
    314 
    315         $this->assertEqualSetsWithIndex( $expected, $this->WP_Error->get_error_data() );
     312        $this->wp_error->add( 'code', 'message', $expected );
     313        $this->wp_error->add( 'code2', 'message2', 'data2' );
     314
     315        $this->assertEqualSetsWithIndex( $expected, $this->wp_error->get_error_data() );
    316316    }
    317317
     
    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' );
    325 
    326         $this->assertSame( 'data2', $this->WP_Error->get_error_data() );
     322        $this->wp_error->add( 'code', 'message', 'data' );
     323        $this->wp_error->add( 'code', 'message2', 'data2' );
     324        $this->wp_error->add( 'code2', 'message2', 'data3' );
     325
     326        $this->assertSame( 'data2', $this->wp_error->get_error_data() );
    327327    }
    328328
     
    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
     
    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' );
    341 
    342         $this->assertSame( null, $this->WP_Error->get_error_data( 'code' ) );
     340        $this->wp_error->add( 'code', 'message' );
     341
     342        $this->assertSame( null, $this->wp_error->get_error_data( 'code' ) );
    343343    }
    344344
     
    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 );
    351 
    352         $this->assertEqualSetsWithIndex( $expected, $this->WP_Error->get_error_data( 'code' ) );
     350        $this->wp_error->add( 'code', 'message', $expected );
     351
     352        $this->assertEqualSetsWithIndex( $expected, $this->wp_error->get_error_data( 'code' ) );
    353353    }
    354354
     
    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 );
    363 
    364         $this->assertEqualSetsWithIndex( $expected, $this->WP_Error->get_error_data( 'code' ) );
     360        $this->wp_error->add( 'code', 'message', 'data' );
     361        $this->wp_error->add( 'code2', 'message2', 'data2' );
     362        $this->wp_error->add( 'code', 'message3', $expected );
     363
     364        $this->assertEqualSetsWithIndex( $expected, $this->wp_error->get_error_data( 'code' ) );
    365365    }
    366366
     
    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' );
    374 
    375         $this->assertSame( 'data2', $this->WP_Error->get_error_data( 'code' ) );
     371        $this->wp_error->add( 'code', 'message', 'data' );
     372        $this->wp_error->add( 'code', 'message2', 'data2' );
     373        $this->wp_error->add( 'code2', 'message3', 'data3' );
     374
     375        $this->assertSame( 'data2', $this->wp_error->get_error_data( 'code' ) );
    376376    }
    377377
     
    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
     
    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
     
    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' );
    398 
    399         $this->assertArrayHasKey( '', $this->WP_Error->errors );
     397        $this->wp_error->add( '', '', 'data' );
     398
     399        $this->assertArrayHasKey( '', $this->wp_error->errors );
    400400    }
    401401
     
    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' );
    407 
    408         $this->assertEqualSetsWithIndex( array( '' => array( '' ) ), $this->WP_Error->errors );
     406        $this->wp_error->add( '', '', 'data' );
     407
     408        $this->assertEqualSetsWithIndex( array( '' => array( '' ) ), $this->wp_error->errors );
    409409    }
    410410
     
    413413     */
    414414    public function test_add_with_empty_code_empty_message_empty_data_should_not_alter_data() {
    415         $this->WP_Error->add( '', '', '' );
    416 
    417         $this->assertEmpty( $this->WP_Error->error_data );
     415        $this->wp_error->add( '', '', '' );
     416
     417        $this->assertEmpty( $this->wp_error->error_data );
    418418    }
    419419
     
    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' );
    425 
    426         $this->assertEqualSetsWithIndex( array( '' => 'data' ), $this->WP_Error->error_data );
     424        $this->wp_error->add( '', '', 'data' );
     425
     426        $this->assertEqualSetsWithIndex( array( '' => 'data' ), $this->wp_error->error_data );
    427427    }
    428428
     
    431431     */
    432432    public function test_add_with_code_empty_message_empty_data_should_add_error_with_code() {
    433         $this->WP_Error->add( 'code', '' );
    434 
    435         $this->assertSame( 'code', $this->WP_Error->get_error_code() );
     433        $this->wp_error->add( 'code', '' );
     434
     435        $this->assertSame( 'code', $this->wp_error->get_error_code() );
    436436    }
    437437
     
    440440     */
    441441    public function test_add_with_code_empty_message_empty_data_should_add_error_with_empty_message() {
    442         $this->WP_Error->add( 'code', '' );
    443 
    444         $this->assertSame( '', $this->WP_Error->get_error_message( 'code' ) );
     442        $this->wp_error->add( 'code', '' );
     443
     444        $this->assertSame( '', $this->wp_error->get_error_message( 'code' ) );
    445445    }
    446446
     
    449449     */
    450450    public function test_add_with_code_empty_message_empty_data_should_not_add_error_data() {
    451         $this->WP_Error->add( 'code', '' );
    452 
    453         $this->assertSame( null, $this->WP_Error->get_error_data( 'code' ) );
     451        $this->wp_error->add( 'code', '' );
     452
     453        $this->assertSame( null, $this->wp_error->get_error_data( 'code' ) );
    454454    }
    455455
     
    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' );
    461 
    462         $this->assertSame( 'message', $this->WP_Error->get_error_message( 'code' ) );
     460        $this->wp_error->add( 'code', 'message' );
     461
     462        $this->assertSame( 'message', $this->wp_error->get_error_message( 'code' ) );
    463463    }
    464464
     
    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' );
    470 
    471         $this->assertSame( null, $this->WP_Error->get_error_data( 'code' ) );
     469        $this->wp_error->add( 'code', 'message' );
     470
     471        $this->assertSame( null, $this->wp_error->get_error_data( 'code' ) );
    472472    }
    473473
     
    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' );
    479 
    480         $this->assertSame( 'code', $this->WP_Error->get_error_code() );
     478        $this->wp_error->add( 'code', '', 'data' );
     479
     480        $this->assertSame( 'code', $this->wp_error->get_error_code() );
    481481    }
    482482
     
    485485     */
    486486    public function test_add_with_code_and_empty_message_and_data_should_store_that_data() {
    487         $this->WP_Error->add( 'code', '', 'data' );
    488 
    489         $this->assertSame( 'data', $this->WP_Error->get_error_data( 'code' ) );
     487        $this->wp_error->add( 'code', '', 'data' );
     488
     489        $this->assertSame( 'data', $this->wp_error->get_error_data( 'code' ) );
    490490    }
    491491
     
    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' );
    497 
    498         $this->assertSame( 'code', $this->WP_Error->get_error_code() );
     496        $this->wp_error->add( 'code', 'message', 'data' );
     497
     498        $this->assertSame( 'code', $this->wp_error->get_error_code() );
    499499    }
    500500
     
    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' );
    506 
    507         $this->assertSame( 'message', $this->WP_Error->get_error_message( 'code' ) );
     505        $this->wp_error->add( 'code', 'message', 'data' );
     506
     507        $this->assertSame( 'message', $this->wp_error->get_error_message( 'code' ) );
    508508    }
    509509
     
    512512     */
    513513    public function test_add_with_code_and_message_and_data_should_store_that_data() {
    514         $this->WP_Error->add( 'code', 'message', 'data' );
    515 
    516         $this->assertSame( 'data', $this->WP_Error->get_error_data( 'code' ) );
     514        $this->wp_error->add( 'code', 'message', 'data' );
     515
     516        $this->assertSame( 'data', $this->wp_error->get_error_data( 'code' ) );
    517517    }
    518518
     
    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
     
    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' );
    537 
    538         $this->assertSame( 'data-baz', $this->WP_Error->get_error_data( 'code' ) );
     535        $this->wp_error->add( 'code', 'message', 'data-bar' );
     536        $this->wp_error->add( 'code', 'message2', 'data-baz' );
     537
     538        $this->assertSame( 'data-baz', $this->wp_error->get_error_data( 'code' ) );
    539539    }
    540540
     
    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( '' );
    546 
    547         $this->assertEmpty( $this->WP_Error->errors );
     545        $this->wp_error->add_data( '' );
     546
     547        $this->assertEmpty( $this->wp_error->errors );
    548548    }
    549549
     
    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( '' );
    555 
    556         $this->assertEqualSets( array( '' => '' ), $this->WP_Error->error_data );
     554        $this->wp_error->add_data( '' );
     555
     556        $this->assertEqualSets( array( '' => '' ), $this->wp_error->error_data );
    557557    }
    558558
     
    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' );
    565 
    566         $this->assertSame( 'data', $this->WP_Error->get_error_data( 'code' ) );
     563        $this->wp_error->add( 'code', 'message' );
     564        $this->wp_error->add_data( 'data' );
     565
     566        $this->assertSame( 'data', $this->wp_error->get_error_data( 'code' ) );
    567567    }
    568568
     
    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' );
    575 
    576         $this->WP_Error->add_data( 'data' );
    577 
    578         $this->assertSame( 'data', $this->WP_Error->get_error_data( 'code' ) );
     573        $this->wp_error->add( 'code', 'message' );
     574        $this->wp_error->add( 'code2', 'message2' );
     575
     576        $this->wp_error->add_data( 'data' );
     577
     578        $this->assertSame( 'data', $this->wp_error->get_error_data( 'code' ) );
    579579    }
    580580
     
    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' );
    588 
    589         $this->WP_Error->add_data( 'data' );
    590 
    591         $this->assertSame( 'data', $this->WP_Error->get_error_data( 'code' ) );
     585        $this->wp_error->add( 'code', 'message' );
     586        $this->wp_error->add( 'code2', 'message2' );
     587        $this->wp_error->add( 'code', 'message3' );
     588
     589        $this->wp_error->add_data( 'data' );
     590
     591        $this->assertSame( 'data', $this->wp_error->get_error_data( 'code' ) );
    592592    }
    593593
     
    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' );
    599 
    600         $this->assertEmpty( $this->WP_Error->errors );
     598        $this->wp_error->add_data( 'data', 'code' );
     599
     600        $this->assertEmpty( $this->wp_error->errors );
    601601    }
    602602
     
    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' );
    608 
    609         $this->assertEqualSets( array( 'code' => 'data' ), $this->WP_Error->error_data );
     607        $this->wp_error->add_data( 'data', 'code' );
     608
     609        $this->assertEqualSets( array( 'code' => 'data' ), $this->wp_error->error_data );
    610610    }
    611611
     
    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' );
    617 
    618         $this->WP_Error->add_data( 'data', 'code2' );
    619 
    620         $this->assertEqualSetsWithIndex( array( 'code' => array( 'message' ) ), $this->WP_Error->errors );
     616        $this->wp_error->add( 'code', 'message' );
     617
     618        $this->wp_error->add_data( 'data', 'code2' );
     619
     620        $this->assertEqualSetsWithIndex( array( 'code' => array( 'message' ) ), $this->wp_error->errors );
    621621    }
    622622
     
    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' );
    628 
    629         $this->WP_Error->add_data( 'data', 'code2' );
    630 
    631         $this->assertEqualSetsWithIndex( array( 'code2' => 'data' ), $this->WP_Error->error_data );
     627        $this->wp_error->add( 'code', 'message' );
     628
     629        $this->wp_error->add_data( 'data', 'code2' );
     630
     631        $this->assertEqualSetsWithIndex( array( 'code2' => 'data' ), $this->wp_error->error_data );
    632632    }
    633633
     
    636636     */
    637637    public function test_add_data_with_data_and_code_should_add_data() {
    638         $this->WP_Error->add( 'code', 'message' );
    639 
    640         $this->WP_Error->add_data( 'data', 'code' );
    641 
    642         $this->assertSame( 'data', $this->WP_Error->get_error_data( 'code' ) );
     638        $this->wp_error->add( 'code', 'message' );
     639
     640        $this->wp_error->add_data( 'data', 'code' );
     641
     642        $this->assertSame( 'data', $this->wp_error->get_error_data( 'code' ) );
    643643    }
    644644
     
    647647     */
    648648    public function test_remove_with_no_errors_should_affect_nothing() {
    649         $before = $this->WP_Error->errors;
    650 
    651         $this->WP_Error->remove( 'code' );
    652 
    653         $after = $this->WP_Error->errors;
     649        $before = $this->wp_error->errors;
     650
     651        $this->wp_error->remove( 'code' );
     652
     653        $after = $this->wp_error->errors;
    654654
    655655        $this->assertEqualSetsWithIndex( $before, $after );
     
    660660     */
    661661    public function test_remove_empty_code_no_errors_should_affect_nothing() {
    662         $before = $this->WP_Error->errors;
    663 
    664         $this->WP_Error->remove( '' );
    665 
    666         $after = $this->WP_Error->errors;
     662        $before = $this->wp_error->errors;
     663
     664        $this->wp_error->remove( '' );
     665
     666        $after = $this->wp_error->errors;
    667667
    668668        $this->assertEqualSetsWithIndex( $before, $after );
     
    673673     */
    674674    public function test_remove_empty_code_and_one_error_with_empty_string_code_should_remove_error() {
    675         $before = $this->WP_Error->errors;
    676 
    677         $this->WP_Error->add( '', 'message' );
    678 
    679         $this->WP_Error->remove( '' );
    680 
    681         $after = $this->WP_Error->errors;
     675        $before = $this->wp_error->errors;
     676
     677        $this->wp_error->add( '', 'message' );
     678
     679        $this->wp_error->remove( '' );
     680
     681        $after = $this->wp_error->errors;
    682682
    683683        $this->assertEqualSetsWithIndex( $before, $after );
     
    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' );
    691 
    692         $this->WP_Error->remove( '' );
    693 
    694         $after = $this->WP_Error->error_data;
    695 
    696         $this->assertEmpty( $this->WP_Error->error_data );
     690        $this->wp_error->add( '', 'message', 'data' );
     691
     692        $this->wp_error->remove( '' );
     693
     694        $after = $this->wp_error->error_data;
     695
     696        $this->assertEmpty( $this->wp_error->error_data );
    697697    }
    698698
     
    701701     */
    702702    public function test_remove_should_remove_the_error_with_the_given_code() {
    703         $this->WP_Error->add( 'code', 'message' );
    704 
    705         $this->WP_Error->remove( 'code' );
    706 
    707         $this->assertEmpty( $this->WP_Error->errors );
     703        $this->wp_error->add( 'code', 'message' );
     704
     705        $this->wp_error->remove( 'code' );
     706
     707        $this->assertEmpty( $this->wp_error->errors );
    708708    }
    709709
     
    712712     */
    713713    public function test_remove_should_remove_the_error_data_associated_with_the_given_code() {
    714         $this->WP_Error->add( 'code', 'message', 'data' );
    715 
    716         $this->WP_Error->remove( 'code' );
    717 
    718         $this->assertEmpty( $this->WP_Error->error_data );
     714        $this->wp_error->add( 'code', 'message', 'data' );
     715
     716        $this->wp_error->remove( 'code' );
     717
     718        $this->assertEmpty( $this->wp_error->error_data );
    719719    }
    720720
  • trunk/tests/phpunit/tests/http/base.php

    r43571 r44573  
    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;
     
    255255
    256256    function test_file_stream() {
    257         $url  = $this->fileStreamUrl;
     257        $url  = $this->file_stream_url;
    258258        $size = 153204;
    259259        $res  = wp_remote_request(
     
    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(
     
    312312     */
    313313    function test_request_limited_size() {
    314         $url  = $this->fileStreamUrl;
     314        $url  = $this->file_stream_url;
    315315        $size = 10000;
    316316
  • trunk/tests/phpunit/tests/http/curl.php

    r43571 r44573  
    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,
  • trunk/tests/phpunit/tests/user.php

    r43571 r44573  
    200200     */
    201201    public function test_user_unset() {
     202        // phpcs:disable WordPress.NamingConventions.ValidVariableName.NotSnakeCaseMemberVar
    202203        $user = new WP_User( self::$author_id );
    203204
     
    208209        $this->assertFalse( isset( $user->customField ) );
    209210        return $user;
     211        // phpcs:enable
    210212    }
    211213
Note: See TracChangeset for help on using the changeset viewer.