Make WordPress Core


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

    r38654 r42343  
    1212    /**
    1313     * @ticket 24785
    14      *
    1514     */
    1615    function test_nested_loop_reset_postdata() {
    17         $post_id = self::factory()->post->create();
     16        $post_id        = self::factory()->post->create();
    1817        $nested_post_id = self::factory()->post->create();
    1918
    2019        $first_query = new WP_Query( array( 'post__in' => array( $post_id ) ) );
    21         while ( $first_query->have_posts() ) { $first_query->the_post();
     20        while ( $first_query->have_posts() ) {
     21            $first_query->the_post();
    2222            $second_query = new WP_Query( array( 'post__in' => array( $nested_post_id ) ) );
    2323            while ( $second_query->have_posts() ) {
     
    9595        remove_action( 'pre_get_posts', array( $this, 'pre_get_posts_tax_category_tax_query' ) );
    9696        register_taxonomy( 'wptests_tax', 'post' );
    97         $terms = self::factory()->term->create_many( 2, array(
    98             'taxonomy' => 'wptests_tax',
    99         ) );
     97        $terms = self::factory()->term->create_many(
     98            2, array(
     99                'taxonomy' => 'wptests_tax',
     100            )
     101        );
    100102
    101103        $posts = self::factory()->post->create_many( 2 );
     
    105107
    106108        add_action( 'parse_query', array( $this, 'filter_parse_query_to_remove_tax' ) );
    107         $q = new WP_Query( array(
    108             'fields' => 'ids',
    109             'wptests_tax' => $terms[1],
    110         ) );
     109        $q = new WP_Query(
     110            array(
     111                'fields'      => 'ids',
     112                'wptests_tax' => $terms[1],
     113            )
     114        );
    111115
    112116        remove_action( 'parse_query', array( $this, 'filter_parse_query_to_remove_tax' ) );
     
    125129        register_taxonomy( 'wptests_tax', 'post' );
    126130
    127         $q = new WP_Query( array(
    128             'tax_query' => array(
    129                 array(
    130                     'taxonomy' => 'wptests_tax',
    131                     'operator' => 'NOT EXISTS',
     131        $q = new WP_Query(
     132            array(
     133                'tax_query' => array(
     134                    array(
     135                        'taxonomy' => 'wptests_tax',
     136                        'operator' => 'NOT EXISTS',
     137                    ),
    132138                ),
    133             ),
    134         ) );
     139            )
     140        );
    135141
    136142        $queried_object = $q->get_queried_object();
     
    141147        global $wpdb;
    142148
    143         $q = new WP_Query( array(
    144             'orderby' => 'title date',
    145             'order' => 'DESC',
    146         ) );
     149        $q = new WP_Query(
     150            array(
     151                'orderby' => 'title date',
     152                'order'   => 'DESC',
     153            )
     154        );
    147155
    148156        $this->assertContains( "ORDER BY $wpdb->posts.post_title DESC, $wpdb->posts.post_date DESC", $q->request );
     
    150158
    151159    public function test_cat_querystring_single_term() {
    152         $c1 = self::factory()->category->create( array(
    153             'name' => 'Test Category 1',
    154             'slug' => 'test1',
    155         ) );
    156         $c2 = self::factory()->category->create( array(
    157             'name' => 'Test Category 2',
    158             'slug' => 'test2',
    159         ) );
     160        $c1 = self::factory()->category->create(
     161            array(
     162                'name' => 'Test Category 1',
     163                'slug' => 'test1',
     164            )
     165        );
     166        $c2 = self::factory()->category->create(
     167            array(
     168                'name' => 'Test Category 2',
     169                'slug' => 'test2',
     170            )
     171        );
    160172
    161173        $p1 = self::factory()->post->create();
     
    167179        wp_set_object_terms( $p3, $c2, 'category' );
    168180
    169         $url = add_query_arg( array(
    170             'cat' => $c1,
    171         ), '/' );
     181        $url = add_query_arg(
     182            array(
     183                'cat' => $c1,
     184            ), '/'
     185        );
    172186
    173187        $this->go_to( $url );
     
    179193
    180194    public function test_category_querystring_multiple_terms_comma_separated() {
    181         $c1 = self::factory()->category->create( array(
    182             'name' => 'Test Category 1',
    183             'slug' => 'test1',
    184         ) );
    185         $c2 = self::factory()->category->create( array(
    186             'name' => 'Test Category 2',
    187             'slug' => 'test2',
    188         ) );
    189         $c3 = self::factory()->category->create( array(
    190             'name' => 'Test Category 3',
    191             'slug' => 'test3',
    192         ) );
     195        $c1 = self::factory()->category->create(
     196            array(
     197                'name' => 'Test Category 1',
     198                'slug' => 'test1',
     199            )
     200        );
     201        $c2 = self::factory()->category->create(
     202            array(
     203                'name' => 'Test Category 2',
     204                'slug' => 'test2',
     205            )
     206        );
     207        $c3 = self::factory()->category->create(
     208            array(
     209                'name' => 'Test Category 3',
     210                'slug' => 'test3',
     211            )
     212        );
    193213
    194214        $p1 = self::factory()->post->create();
     
    202222        wp_set_object_terms( $p4, $c3, 'category' );
    203223
    204         $url = add_query_arg( array(
    205             'cat' => implode( ',',array( $c1,$c2 ) ),
    206         ), '/' );
     224        $url = add_query_arg(
     225            array(
     226                'cat' => implode( ',', array( $c1, $c2 ) ),
     227            ), '/'
     228        );
    207229
    208230        $this->go_to( $url );
     
    217239     */
    218240    public function test_category_querystring_multiple_terms_formatted_as_array() {
    219         $c1 = self::factory()->category->create( array(
    220             'name' => 'Test Category 1',
    221             'slug' => 'test1',
    222         ) );
    223         $c2 = self::factory()->category->create( array(
    224             'name' => 'Test Category 2',
    225             'slug' => 'test2',
    226         ) );
    227         $c3 = self::factory()->category->create( array(
    228             'name' => 'Test Category 3',
    229             'slug' => 'test3',
    230         ) );
     241        $c1 = self::factory()->category->create(
     242            array(
     243                'name' => 'Test Category 1',
     244                'slug' => 'test1',
     245            )
     246        );
     247        $c2 = self::factory()->category->create(
     248            array(
     249                'name' => 'Test Category 2',
     250                'slug' => 'test2',
     251            )
     252        );
     253        $c3 = self::factory()->category->create(
     254            array(
     255                'name' => 'Test Category 3',
     256                'slug' => 'test3',
     257            )
     258        );
    231259
    232260        $p1 = self::factory()->post->create();
     
    240268        wp_set_object_terms( $p4, $c3, 'category' );
    241269
    242         $url = add_query_arg( array(
    243             'cat' => array( $c1, $c2 ),
    244         ), '/' );
     270        $url = add_query_arg(
     271            array(
     272                'cat' => array( $c1, $c2 ),
     273            ), '/'
     274        );
    245275
    246276        $this->go_to( $url );
     
    253283
    254284    public function test_tag_querystring_single_term() {
    255         $t1 = self::factory()->tag->create_and_get( array(
    256             'name' => 'Test Tag 1',
    257             'slug' => 'test1',
    258         ) );
    259         $t2 = self::factory()->tag->create_and_get( array(
    260             'name' => 'Test Tag 2',
    261             'slug' => 'test2',
    262         ) );
     285        $t1 = self::factory()->tag->create_and_get(
     286            array(
     287                'name' => 'Test Tag 1',
     288                'slug' => 'test1',
     289            )
     290        );
     291        $t2 = self::factory()->tag->create_and_get(
     292            array(
     293                'name' => 'Test Tag 2',
     294                'slug' => 'test2',
     295            )
     296        );
    263297
    264298        $p1 = self::factory()->post->create();
     
    270304        wp_set_object_terms( $p3, $t2->slug, 'post_tag' );
    271305
    272         $url = add_query_arg( array(
    273             'tag' => $t1->slug,
    274         ), '/' );
     306        $url = add_query_arg(
     307            array(
     308                'tag' => $t1->slug,
     309            ), '/'
     310        );
    275311
    276312        $this->go_to( $url );
     
    282318
    283319    public function test_tag_querystring_multiple_terms_comma_separated() {
    284         $c1 = self::factory()->tag->create_and_get( array(
    285             'name' => 'Test Tag 1',
    286             'slug' => 'test1',
    287         ) );
    288         $c2 = self::factory()->tag->create_and_get( array(
    289             'name' => 'Test Tag 2',
    290             'slug' => 'test2',
    291         ) );
    292         $c3 = self::factory()->tag->create_and_get( array(
    293             'name' => 'Test Tag 3',
    294             'slug' => 'test3',
    295         ) );
     320        $c1 = self::factory()->tag->create_and_get(
     321            array(
     322                'name' => 'Test Tag 1',
     323                'slug' => 'test1',
     324            )
     325        );
     326        $c2 = self::factory()->tag->create_and_get(
     327            array(
     328                'name' => 'Test Tag 2',
     329                'slug' => 'test2',
     330            )
     331        );
     332        $c3 = self::factory()->tag->create_and_get(
     333            array(
     334                'name' => 'Test Tag 3',
     335                'slug' => 'test3',
     336            )
     337        );
    296338
    297339        $p1 = self::factory()->post->create();
     
    305347        wp_set_object_terms( $p4, $c3->slug, 'post_tag' );
    306348
    307         $url = add_query_arg( array(
    308             'tag' => implode( ',',array( $c1->slug,$c2->slug ) ),
    309         ), '/' );
     349        $url = add_query_arg(
     350            array(
     351                'tag' => implode( ',', array( $c1->slug, $c2->slug ) ),
     352            ), '/'
     353        );
    310354
    311355        $this->go_to( $url );
     
    320364     */
    321365    public function test_tag_querystring_multiple_terms_formatted_as_array() {
    322         $c1 = self::factory()->tag->create_and_get( array(
    323             'name' => 'Test Tag 1',
    324             'slug' => 'test1',
    325         ) );
    326         $c2 = self::factory()->tag->create_and_get( array(
    327             'name' => 'Test Tag 2',
    328             'slug' => 'test2',
    329         ) );
    330         $c3 = self::factory()->tag->create_and_get( array(
    331             'name' => 'Test Tag 3',
    332             'slug' => 'test3',
    333         ) );
     366        $c1 = self::factory()->tag->create_and_get(
     367            array(
     368                'name' => 'Test Tag 1',
     369                'slug' => 'test1',
     370            )
     371        );
     372        $c2 = self::factory()->tag->create_and_get(
     373            array(
     374                'name' => 'Test Tag 2',
     375                'slug' => 'test2',
     376            )
     377        );
     378        $c3 = self::factory()->tag->create_and_get(
     379            array(
     380                'name' => 'Test Tag 3',
     381                'slug' => 'test3',
     382            )
     383        );
    334384
    335385        $p1 = self::factory()->post->create();
     
    343393        wp_set_object_terms( $p4, $c3->slug, 'post_tag' );
    344394
    345         $url = add_query_arg( array(
    346             'tag' => array($c1->slug,$c2->slug),
    347         ), '/' );
     395        $url = add_query_arg(
     396            array(
     397                'tag' => array( $c1->slug, $c2->slug ),
     398            ), '/'
     399        );
    348400
    349401        $this->go_to( $url );
     
    369421        wp_set_object_terms( $p3, 'test2', 'test_tax_cat' );
    370422
    371         $url = add_query_arg( array(
    372             'test_tax_cat' => 'test1',
    373         ), '/' );
     423        $url = add_query_arg(
     424            array(
     425                'test_tax_cat' => 'test1',
     426            ), '/'
     427        );
    374428
    375429        $this->go_to( $url );
     
    393447        wp_set_object_terms( $p2, array( 'test1', 'test2' ), 'test_tax_cat' );
    394448        wp_set_object_terms( $p3, 'test2', 'test_tax_cat' );
    395         wp_set_object_terms( $p4, "test3", 'test_tax_cat' );
    396 
    397         $url = add_query_arg( array(
    398             'test_tax_cat' => 'test1,test2',
    399         ), '/' );
     449        wp_set_object_terms( $p4, 'test3', 'test_tax_cat' );
     450
     451        $url = add_query_arg(
     452            array(
     453                'test_tax_cat' => 'test1,test2',
     454            ), '/'
     455        );
    400456
    401457        $this->go_to( $url );
     
    422478        wp_set_object_terms( $p2, array( 'test1', 'test2' ), 'test_tax_cat' );
    423479        wp_set_object_terms( $p3, 'test2', 'test_tax_cat' );
    424         wp_set_object_terms( $p4, "test3", 'test_tax_cat' );
    425 
    426         $url = add_query_arg( array(
    427             'test_tax_cat' => array( 'test1', 'test2' ),
    428         ), '/' );
     480        wp_set_object_terms( $p4, 'test3', 'test_tax_cat' );
     481
     482        $url = add_query_arg(
     483            array(
     484                'test_tax_cat' => array( 'test1', 'test2' ),
     485            ), '/'
     486        );
    429487
    430488        $this->go_to( $url );
     
    437495     */
    438496    public function test_pages_dont_404_when_queried_post_id_is_modified() {
    439         $post_id = self::factory()->post->create( array( 'post_title' => 'A Test Page', 'post_type' => 'page' ) );
     497        $post_id = self::factory()->post->create(
     498            array(
     499                'post_title' => 'A Test Page',
     500                'post_type'  => 'page',
     501            )
     502        );
    440503
    441504        add_action( 'parse_query', array( $this, 'filter_parse_query_to_modify_queried_post_id' ) );
     
    456519        global $wp_rewrite;
    457520
    458         register_post_type( 'guide', array( 'name' => 'Guide', 'public' => true, 'hierarchical' => true ) );
     521        register_post_type(
     522            'guide', array(
     523                'name'         => 'Guide',
     524                'public'       => true,
     525                'hierarchical' => true,
     526            )
     527        );
    459528        $wp_rewrite->flush_rules();
    460         $post_id = self::factory()->post->create( array( 'post_title' => 'A Test Guide', 'post_type' => 'guide' ) );
     529        $post_id = self::factory()->post->create(
     530            array(
     531                'post_title' => 'A Test Guide',
     532                'post_type'  => 'guide',
     533            )
     534        );
    461535
    462536        add_action( 'parse_query', array( $this, 'filter_parse_query_to_modify_queried_post_id' ) );
     
    479553     */
    480554    public function test_offset_0_should_override_page() {
    481         $q = new WP_Query( array(
    482             'paged' => 2,
    483             'posts_per_page' => 5,
    484             'offset' => 0,
    485         ) );
     555        $q = new WP_Query(
     556            array(
     557                'paged'          => 2,
     558                'posts_per_page' => 5,
     559                'offset'         => 0,
     560            )
     561        );
    486562
    487563        $this->assertContains( 'LIMIT 0, 5', $q->request );
     
    492568     */
    493569    public function test_offset_should_be_ignored_when_not_set() {
    494         $q = new WP_Query( array(
    495             'paged' => 2,
    496             'posts_per_page' => 5,
    497         ) );
     570        $q = new WP_Query(
     571            array(
     572                'paged'          => 2,
     573                'posts_per_page' => 5,
     574            )
     575        );
    498576
    499577        $this->assertContains( 'LIMIT 5, 5', $q->request );
     
    504582     */
    505583    public function test_offset_should_be_ignored_when_passed_a_non_numeric_value() {
    506         $q = new WP_Query( array(
    507             'paged' => 2,
    508             'posts_per_page' => 5,
    509             'offset' => '',
    510         ) );
     584        $q = new WP_Query(
     585            array(
     586                'paged'          => 2,
     587                'posts_per_page' => 5,
     588                'offset'         => '',
     589            )
     590        );
    511591
    512592        $this->assertContains( 'LIMIT 5, 5', $q->request );
     
    520600        $p2 = self::factory()->post->create( array( 'comment_status' => 'closed' ) );
    521601
    522         $q = new WP_Query( array(
    523             'fields' => 'ids',
    524             'comment_status' => 'closed',
    525         ) );
     602        $q = new WP_Query(
     603            array(
     604                'fields'         => 'ids',
     605                'comment_status' => 'closed',
     606            )
     607        );
    526608
    527609        $this->assertSame( array( $p2 ), $q->posts );
     
    535617        $p2 = self::factory()->post->create( array( 'ping_status' => 'closed' ) );
    536618
    537         $q = new WP_Query( array(
    538             'fields' => 'ids',
    539             'ping_status' => 'closed',
    540         ) );
     619        $q = new WP_Query(
     620            array(
     621                'fields'      => 'ids',
     622                'ping_status' => 'closed',
     623            )
     624        );
    541625
    542626        $this->assertSame( array( $p2 ), $q->posts );
     
    550634        register_taxonomy( 'tax2', 'post' );
    551635
    552         $term1 = $this->factory->term->create( array( 'taxonomy' => 'tax1', 'name' => 'term1' ) );
    553         $term2 = $this->factory->term->create( array( 'taxonomy' => 'tax2', 'name' => 'term2' ) );
     636        $term1   = $this->factory->term->create(
     637            array(
     638                'taxonomy' => 'tax1',
     639                'name'     => 'term1',
     640            )
     641        );
     642        $term2   = $this->factory->term->create(
     643            array(
     644                'taxonomy' => 'tax2',
     645                'name'     => 'term2',
     646            )
     647        );
    554648        $post_id = $this->factory->post->create();
    555649        wp_set_object_terms( $post_id, 'term1', 'tax1' );
     
    570664        register_taxonomy( 'tax2', 'post' );
    571665
    572         $term1 = $this->factory->term->create( array( 'taxonomy' => 'tax1', 'name' => 'term1' ) );
    573         $term2 = $this->factory->term->create( array( 'taxonomy' => 'tax2', 'name' => 'term2' ) );
     666        $term1   = $this->factory->term->create(
     667            array(
     668                'taxonomy' => 'tax1',
     669                'name'     => 'term1',
     670            )
     671        );
     672        $term2   = $this->factory->term->create(
     673            array(
     674                'taxonomy' => 'tax2',
     675                'name'     => 'term2',
     676            )
     677        );
    574678        $post_id = $this->factory->post->create();
    575679        wp_set_object_terms( $post_id, 'term1', 'tax1' );
Note: See TracChangeset for help on using the changeset viewer.