WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (8 years ago)
Author:
pento
Message:

Code is Poetry.
WordPress' code just... wasn't.
This is now dealt with.

Props jrf, pento, netweb, GaryJ, jdgrimes, westonruter, Greg Sherwood from PHPCS, and everyone who's ever contributed to WPCS and PHPCS.
Fixes #41057.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/tests/phpunit/tests/rewrite.php

    r41786 r42343  
    5353        $redirect = 'index.php?test_var1=$matches[1]&test_var2=1';
    5454
    55         $wp_rewrite->add_rule( $pattern, array(
    56             'test_var1' => '$matches[1]',
    57             'test_var2' => '1'
    58         ) );
     55        $wp_rewrite->add_rule(
     56            $pattern, array(
     57                'test_var1' => '$matches[1]',
     58                'test_var2' => '1',
     59            )
     60        );
    5961
    6062        $wp_rewrite->flush_rules();
    … …  
    9395
    9496    function test_url_to_postid_set_url_scheme_https_to_http() {
    95         $post_id = self::factory()->post->create();
     97        $post_id   = self::factory()->post->create();
    9698        $permalink = get_permalink( $post_id );
    9799        $this->assertEquals( $post_id, url_to_postid( set_url_scheme( $permalink, 'https' ) ) );
    98100
    99         $post_id = self::factory()->post->create( array( 'post_type' => 'page' ) );
     101        $post_id   = self::factory()->post->create( array( 'post_type' => 'page' ) );
    100102        $permalink = get_permalink( $post_id );
    101103        $this->assertEquals( $post_id, url_to_postid( set_url_scheme( $permalink, 'https' ) ) );
    … …  
    125127        $_SERVER['HTTPS'] = 'on';
    126128
    127         $network_home = home_url();
     129        $network_home        = home_url();
    128130        $this->blog_id_35531 = self::factory()->blog->create();
    129131
    … …  
    182184    function test_url_to_postid_hierarchical() {
    183185
    184         $parent_id = self::factory()->post->create( array( 'post_title' => 'Parent', 'post_type' => 'page' ) );
    185         $child_id = self::factory()->post->create( array( 'post_title' => 'Child', 'post_type' => 'page', 'post_parent' => $parent_id ) );
     186        $parent_id = self::factory()->post->create(
     187            array(
     188                'post_title' => 'Parent',
     189                'post_type'  => 'page',
     190            )
     191        );
     192        $child_id  = self::factory()->post->create(
     193            array(
     194                'post_title'  => 'Child',
     195                'post_type'   => 'page',
     196                'post_parent' => $parent_id,
     197            )
     198        );
    186199
    187200        $this->assertEquals( $parent_id, url_to_postid( get_permalink( $parent_id ) ) );
    … …  
    191204    function test_url_to_postid_hierarchical_with_matching_leaves() {
    192205
    193         $parent_id = self::factory()->post->create( array(
    194             'post_name' => 'parent',
    195             'post_type' => 'page',
    196         ) );
    197         $child_id_1 = self::factory()->post->create( array(
    198             'post_name'   => 'child1',
    199             'post_type'   => 'page',
    200             'post_parent' => $parent_id,
    201         ) );
    202         $child_id_2 = self::factory()->post->create( array(
    203             'post_name'   => 'child2',
    204             'post_type'   => 'page',
    205             'post_parent' => $parent_id,
    206         ) );
    207         $grandchild_id_1 = self::factory()->post->create( array(
    208             'post_name'   => 'grandchild',
    209             'post_type'   => 'page',
    210             'post_parent' => $child_id_1,
    211         ) );
    212         $grandchild_id_2 = self::factory()->post->create( array(
    213             'post_name'   => 'grandchild',
    214             'post_type'   => 'page',
    215             'post_parent' => $child_id_2,
    216         ) );
     206        $parent_id       = self::factory()->post->create(
     207            array(
     208                'post_name' => 'parent',
     209                'post_type' => 'page',
     210            )
     211        );
     212        $child_id_1      = self::factory()->post->create(
     213            array(
     214                'post_name'   => 'child1',
     215                'post_type'   => 'page',
     216                'post_parent' => $parent_id,
     217            )
     218        );
     219        $child_id_2      = self::factory()->post->create(
     220            array(
     221                'post_name'   => 'child2',
     222                'post_type'   => 'page',
     223                'post_parent' => $parent_id,
     224            )
     225        );
     226        $grandchild_id_1 = self::factory()->post->create(
     227            array(
     228                'post_name'   => 'grandchild',
     229                'post_type'   => 'page',
     230                'post_parent' => $child_id_1,
     231            )
     232        );
     233        $grandchild_id_2 = self::factory()->post->create(
     234            array(
     235                'post_name'   => 'grandchild',
     236                'post_type'   => 'page',
     237                'post_parent' => $child_id_2,
     238            )
     239        );
    217240
    218241        $this->assertEquals( home_url( 'parent/child1/grandchild/' ), get_permalink( $grandchild_id_1 ) );
    … …  
    226249        update_option( 'home', home_url( '/example/' ) );
    227250
    228         $id = self::factory()->post->create( array( 'post_title' => 'Hi', 'post_type' => 'page', 'post_name' => 'examp' ) );
     251        $id = self::factory()->post->create(
     252            array(
     253                'post_title' => 'Hi',
     254                'post_type'  => 'page',
     255                'post_name'  => 'examp',
     256            )
     257        );
    229258        $this->assertEquals( $id, url_to_postid( get_permalink( $id ) ) );
    230         $this->assertEquals( $id, url_to_postid( site_url('/example/examp' ) ) );
     259        $this->assertEquals( $id, url_to_postid( site_url( '/example/examp' ) ) );
    231260        $this->assertEquals( $id, url_to_postid( '/example/examp/' ) );
    232261        $this->assertEquals( $id, url_to_postid( '/example/examp' ) );
    … …  
    250279
    251280        $this->go_to( $home_url . 'page' );
    252         $this->assertEquals( array( 'page' => '', 'pagename' => 'page' ), $GLOBALS['wp']->query_vars );
     281        $this->assertEquals(
     282            array(
     283                'page'     => '',
     284                'pagename' => 'page',
     285            ), $GLOBALS['wp']->query_vars
     286        );
    253287    }
    254288
    … …  
    257291     */
    258292    function test_parse_request_home_path_with_regex_character() {
    259         $home_url = home_url( '/ma.ch/' );
     293        $home_url       = home_url( '/ma.ch/' );
    260294        $not_a_home_url = home_url( '/match/' );
    261295        update_option( 'home', $home_url );
    … …  
    265299
    266300        $this->go_to( $home_url . 'page' );
    267         $this->assertEquals( array( 'page' => '', 'pagename' => 'page' ), $GLOBALS['wp']->query_vars );
     301        $this->assertEquals(
     302            array(
     303                'page'     => '',
     304                'pagename' => 'page',
     305            ), $GLOBALS['wp']->query_vars
     306        );
    268307
    269308        $this->go_to( $not_a_home_url . 'page' );
    270         $this->assertNotEquals( array( 'page' => '', 'pagename' => 'page' ), $GLOBALS['wp']->query_vars );
    271         $this->assertEquals( array( 'page' => '', 'pagename' => 'match/page' ), $GLOBALS['wp']->query_vars );
     309        $this->assertNotEquals(
     310            array(
     311                'page'     => '',
     312                'pagename' => 'page',
     313            ), $GLOBALS['wp']->query_vars
     314        );
     315        $this->assertEquals(
     316            array(
     317                'page'     => '',
     318                'pagename' => 'match/page',
     319            ), $GLOBALS['wp']->query_vars
     320        );
    272321    }
    273322
    … …  
    288337
    289338    function test_url_to_postid_dupe_path() {
    290         update_option( 'home', home_url('/example/') );
    291 
    292         $id = self::factory()->post->create( array( 'post_title' => 'Hi', 'post_type' => 'page', 'post_name' => 'example' ) );
     339        update_option( 'home', home_url( '/example/' ) );
     340
     341        $id = self::factory()->post->create(
     342            array(
     343                'post_title' => 'Hi',
     344                'post_type'  => 'page',
     345                'post_name'  => 'example',
     346            )
     347        );
    293348
    294349        $this->assertEquals( $id, url_to_postid( get_permalink( $id ) ) );
    … …  
    304359        update_option( 'home', home_url( '/example' ) );
    305360
    306         self::factory()->post->create( array( 'post_title' => 'Collision', 'post_type' => 'page', 'post_name' => 'collision' ) );
     361        self::factory()->post->create(
     362            array(
     363                'post_title' => 'Collision',
     364                'post_type'  => 'page',
     365                'post_name'  => 'collision',
     366            )
     367        );
    307368
    308369        // This url should NOT return a post ID
    … …  
    313374    /**
    314375     * Reveals bug introduced in WP 3.0
     376     *
    315377     * @group ms-required
    316378     */
    … …  
    319381        switch_to_blog( $blog_id );
    320382
    321         self::factory()->post->create( array( 'post_title' => 'Collision ', 'post_type' => 'page' ) );
     383        self::factory()->post->create(
     384            array(
     385                'post_title' => 'Collision ',
     386                'post_type'  => 'page',
     387            )
     388        );
    322389
    323390        // This url should NOT return a post ID
    … …  
    334401        $this->set_permalink_structure( '/%postname%/' );
    335402
    336         $page_id = self::factory()->post->create( array( 'post_type' => 'page', 'post_status' => 'trash' ) );
     403        $page_id = self::factory()->post->create(
     404            array(
     405                'post_type'   => 'page',
     406                'post_status' => 'trash',
     407            )
     408        );
    337409        $post_id = self::factory()->post->create( array( 'post_title' => get_post( $page_id )->post_title ) );
    338410
    … …  
    366438        $this->set_permalink_structure( '/%postname%/' );
    367439
    368         $post_id = self::factory()->post->create( array( 'post_name' => 'foo-bar-baz' ) );
     440        $post_id   = self::factory()->post->create( array( 'post_name' => 'foo-bar-baz' ) );
    369441        $permalink = get_permalink( $post_id );
    370         $url = str_replace( home_url(), 'http://some-other-domain.com', get_permalink( $post_id ) );
     442        $url       = str_replace( home_url(), 'http://some-other-domain.com', get_permalink( $post_id ) );
    371443
    372444        $this->assertSame( $post_id, url_to_postid( $permalink ) );
    … …  
    380452        $this->set_permalink_structure( '/%postname%/' );
    381453
    382         $page_id = self::factory()->post->create( array( 'post_type' => 'page', 'post_status' => 'trash' ) );
     454        $page_id = self::factory()->post->create(
     455            array(
     456                'post_type'   => 'page',
     457                'post_status' => 'trash',
     458            )
     459        );
    383460        $post_id = self::factory()->post->create( array( 'post_title' => get_post( $page_id )->post_title ) );
    384461
Note: See TracChangeset for help on using the changeset viewer.