WordPress.org
Get WordPress

Make WordPress Core


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

    r39363 r42343  
    3131        $wpdb->query( 'TRUNCATE ' . $wpdb->prefix . 'posts' );
    3232        $this->time = time();
    33         $post_date = date( 'Y-m-d H:i:s', $this->time );
    34         $pages = array();
     33        $post_date  = date( 'Y-m-d H:i:s', $this->time );
     34        $pages      = array();
    3535        self::factory()->user->create();
    36         $pages[] = self::factory()->post->create( array( 'post_type' => 'page', 'post_title' => 'Parent 1', 'post_date' => $post_date ) );
    37         $pages[] = self::factory()->post->create( array( 'post_type' => 'page', 'post_title' => 'Parent 2', 'post_date' => $post_date ) );
    38         $pages[] = self::factory()->post->create( array( 'post_type' => 'page', 'post_title' => 'Parent 3', 'post_author' => '2', 'post_date' => $post_date ) );
     36        $pages[] = self::factory()->post->create(
     37            array(
     38                'post_type'  => 'page',
     39                'post_title' => 'Parent 1',
     40                'post_date'  => $post_date,
     41            )
     42        );
     43        $pages[] = self::factory()->post->create(
     44            array(
     45                'post_type'  => 'page',
     46                'post_title' => 'Parent 2',
     47                'post_date'  => $post_date,
     48            )
     49        );
     50        $pages[] = self::factory()->post->create(
     51            array(
     52                'post_type'   => 'page',
     53                'post_title'  => 'Parent 3',
     54                'post_author' => '2',
     55                'post_date'   => $post_date,
     56            )
     57        );
    3958
    4059        foreach ( $pages as $page ) {
    41             $this->pages[$page] = self::factory()->post->create( array( 'post_parent' => $page, 'post_type' => 'page', 'post_title' => 'Child 1', 'post_date' => $post_date ) );
    42             $this->pages[$page] = self::factory()->post->create( array( 'post_parent' => $page, 'post_type' => 'page', 'post_title' => 'Child 2', 'post_date' => $post_date ) );
    43             $this->pages[$page] = self::factory()->post->create( array( 'post_parent' => $page, 'post_type' => 'page', 'post_title' => 'Child 3', 'post_date' => $post_date ) );
     60            $this->pages[ $page ] = self::factory()->post->create(
     61                array(
     62                    'post_parent' => $page,
     63                    'post_type'   => 'page',
     64                    'post_title'  => 'Child 1',
     65                    'post_date'   => $post_date,
     66                )
     67            );
     68            $this->pages[ $page ] = self::factory()->post->create(
     69                array(
     70                    'post_parent' => $page,
     71                    'post_type'   => 'page',
     72                    'post_title'  => 'Child 2',
     73                    'post_date'   => $post_date,
     74                )
     75            );
     76            $this->pages[ $page ] = self::factory()->post->create(
     77                array(
     78                    'post_parent' => $page,
     79                    'post_type'   => 'page',
     80                    'post_title'  => 'Child 3',
     81                    'post_date'   => $post_date,
     82                )
     83            );
    4484        }
    4585    }
    4686
    4787    function test_wp_list_pages_default() {
    48         $args = array(
    49             'echo' => false
     88        $args                = array(
     89            'echo' => false,
    5090        );
    5191        $expected['default'] = '<li class="pagenav">Pages<ul><li class="page_item page-item-1 page_item_has_children"><a href="' . get_permalink( 1 ) . '">Parent 1</a>
    … …  
    71111</li>
    72112</ul></li>';
    73         $actual = wp_list_pages( $args );
     113        $actual              = wp_list_pages( $args );
    74114        $this->AssertEquals( $expected['default'], $actual );
    75115    }
    76116
    77117    function test_wp_list_pages_depth() {
    78         $args = array(
    79             'echo'  => false,
    80             'depth' => 1
     118        $args              = array(
     119            'echo'  => false,
     120            'depth' => 1,
    81121        );
    82122        $expected['depth'] = '<li class="pagenav">Pages<ul><li class="page_item page-item-1 page_item_has_children"><a href="' . get_permalink( 1 ) . '">Parent 1</a></li>
    … …  
    84124<li class="page_item page-item-3 page_item_has_children"><a href="' . get_permalink( 3 ) . '">Parent 3</a></li>
    85125</ul></li>';
    86         $actual = wp_list_pages( $args );
     126        $actual            = wp_list_pages( $args );
    87127        $this->AssertEquals( $expected['depth'], $actual );
    88128    }
    89129
    90130    function test_wp_list_pages_show_date() {
    91         $args = array(
    92             'echo' => false,
    93             'depth' => 1,
    94             'show_date' => true
    95         );
    96         $date = date( get_option( 'date_format' ), $this->time );
     131        $args                  = array(
     132            'echo'      => false,
     133            'depth'     => 1,
     134            'show_date' => true,
     135        );
     136        $date                  = date( get_option( 'date_format' ), $this->time );
    97137        $expected['show_date'] = '<li class="pagenav">Pages<ul><li class="page_item page-item-1 page_item_has_children"><a href="' . get_permalink( 1 ) . '">Parent 1</a> ' . $date . '</li>
    98138<li class="page_item page-item-2 page_item_has_children"><a href="' . get_permalink( 2 ) . '">Parent 2</a> ' . $date . '</li>
    99139<li class="page_item page-item-3 page_item_has_children"><a href="' . get_permalink( 3 ) . '">Parent 3</a> ' . $date . '</li>
    100140</ul></li>';
    101         $actual = wp_list_pages( $args );
     141        $actual                = wp_list_pages( $args );
    102142        $this->AssertEquals( $expected['show_date'], $actual );
    103143    }
    104144
    105145    function test_wp_list_pages_date_format() {
    106         $args = array(
    107             'echo' => false,
    108             'show_date' => true,
    109             'date_format' => 'l, F j, Y'
    110         );
    111         $date = date( $args['date_format'], $this->time );
     146        $args                    = array(
     147            'echo'        => false,
     148            'show_date'   => true,
     149            'date_format' => 'l, F j, Y',
     150        );
     151        $date                    = date( $args['date_format'], $this->time );
    112152        $expected['date_format'] = '<li class="pagenav">Pages<ul><li class="page_item page-item-1 page_item_has_children"><a href="' . get_permalink( 1 ) . '">Parent 1</a> ' . $date . '
    113153<ul class=\'children\'>
    … …  
    132172</li>
    133173</ul></li>';
    134         $actual = wp_list_pages( $args );
     174        $actual                  = wp_list_pages( $args );
    135175        $this->AssertEquals( $expected['date_format'], $actual );
    136176    }
    137177
    138178    function test_wp_list_pages_child_of() {
    139         $args = array(
    140             'echo' => false,
    141             'child_of' => 2
     179        $args                 = array(
     180            'echo'     => false,
     181            'child_of' => 2,
    142182        );
    143183        $expected['child_of'] = '<li class="pagenav">Pages<ul><li class="page_item page-item-7"><a href="' . get_permalink( 7 ) . '">Child 1</a></li>
    … …  
    145185<li class="page_item page-item-9"><a href="' . get_permalink( 9 ) . '">Child 3</a></li>
    146186</ul></li>';
    147         $actual = wp_list_pages( $args );
     187        $actual               = wp_list_pages( $args );
    148188        $this->AssertEquals( $expected['child_of'], $actual );
    149189    }
    150190
    151191    function test_wp_list_pages_exclude() {
    152         $args = array(
    153             'echo' => false,
    154             'exclude' => '2, 2'
     192        $args                = array(
     193            'echo'    => false,
     194            'exclude' => '2, 2',
    155195        );
    156196        $expected['exclude'] = '<li class="pagenav">Pages<ul><li class="page_item page-item-1 page_item_has_children"><a href="' . get_permalink( 1 ) . '">Parent 1</a>
    … …  
    172212<li class="page_item page-item-9"><a href="' . get_permalink( 9 ) . '">Child 3</a></li>
    173213</ul></li>';
    174         $actual = wp_list_pages( $args );
     214        $actual              = wp_list_pages( $args );
    175215        $this->AssertEquals( $expected['exclude'], $actual );
    176216    }
    177217
    178218    function test_wp_list_pages_title_li() {
    179         $args = array(
    180             'echo' => false,
    181             'depth' => 1,
    182             'title_li' => 'PageTitle'
     219        $args                 = array(
     220            'echo'     => false,
     221            'depth'    => 1,
     222            'title_li' => 'PageTitle',
    183223        );
    184224        $expected['title_li'] = '<li class="pagenav">PageTitle<ul><li class="page_item page-item-1 page_item_has_children"><a href="' . get_permalink( 1 ) . '">Parent 1</a></li>
    … …  
    186226<li class="page_item page-item-3 page_item_has_children"><a href="' . get_permalink( 3 ) . '">Parent 3</a></li>
    187227</ul></li>';
    188         $actual = wp_list_pages( $args );
     228        $actual               = wp_list_pages( $args );
    189229        $this->AssertEquals( $expected['title_li'], $actual );
    190230    }
    191231
    192232    function test_wp_list_pages_echo() {
    193         $args = array(
    194             'echo' => true,
    195             'depth' => 1
     233        $args             = array(
     234            'echo'  => true,
     235            'depth' => 1,
    196236        );
    197237        $expected['echo'] = '<li class="pagenav">Pages<ul><li class="page_item page-item-1 page_item_has_children"><a href="' . get_permalink( 1 ) . '">Parent 1</a></li>
    … …  
    206246
    207247    function test_wp_list_pages_authors() {
    208         $args = array(
    209             'echo' => false,
     248        $args                = array(
     249            'echo'    => false,
    210250            'authors' => '2',
    211251        );
    212         $expected['authors'] = '<li class="pagenav">Pages<ul><li class="page_item page-item-3"><a href="' . get_permalink( 3) . '">Parent 3</a></li>
    213 </ul></li>';
    214         $actual = wp_list_pages( $args );
     252        $expected['authors'] = '<li class="pagenav">Pages<ul><li class="page_item page-item-3"><a href="' . get_permalink( 3 ) . '">Parent 3</a></li>
     253</ul></li>';
     254        $actual              = wp_list_pages( $args );
    215255        $this->AssertEquals( $expected['authors'], $actual );
    216256    }
    217257
    218258    function test_wp_list_pages_number() {
    219         $args = array(
    220             'echo' => false,
     259        $args               = array(
     260            'echo'   => false,
    221261            'number' => 1,
    222262        );
    223263        $expected['number'] = '<li class="pagenav">Pages<ul><li class="page_item page-item-4"><a href="' . get_permalink( 4 ) . '">Child 1</a></li>
    224264</ul></li>';
    225         $actual = wp_list_pages( $args );
     265        $actual             = wp_list_pages( $args );
    226266        $this->AssertEquals( $expected['number'], $actual );
    227267    }
    228268
    229269    function test_wp_list_pages_sort_column() {
    230         $args = array(
    231             'echo' => false,
     270        $args                    = array(
     271            'echo'        => false,
    232272            'sort_column' => 'post_author',
    233             'sort_order' => 'DESC'
     273            'sort_order'  => 'DESC',
    234274        );
    235275        $expected['sort_column'] = '<li class="pagenav">Pages<ul><li class="page_item page-item-3 page_item_has_children"><a href="' . get_permalink( 3 ) . '">Parent 3</a>
    … …  
    255295</li>
    256296</ul></li>';
    257         $actual = wp_list_pages( $args );
     297        $actual                  = wp_list_pages( $args );
    258298        $this->AssertEquals( $expected['sort_column'], $actual );
    259299    }
    260300
    261301    function test_wp_list_pages_link_before() {
    262         $args = array(
    263             'echo' => false,
    264             'link_before' => 'BEFORE'
     302        $args                    = array(
     303            'echo'        => false,
     304            'link_before' => 'BEFORE',
    265305        );
    266306        $expected['link_before'] = '<li class="pagenav">Pages<ul><li class="page_item page-item-1 page_item_has_children"><a href="' . get_permalink( 1 ) . '">BEFOREParent 1</a>
    … …  
    286326</li>
    287327</ul></li>';
    288         $actual = wp_list_pages( $args );
     328        $actual                  = wp_list_pages( $args );
    289329        $this->AssertEquals( $expected['link_before'], $actual );
    290330    }
    291331
    292332    function test_wp_list_pages_link_after() {
    293         $args = array(
    294             'echo' => false,
    295             'link_after' => 'AFTER'
     333        $args                   = array(
     334            'echo'       => false,
     335            'link_after' => 'AFTER',
    296336        );
    297337        $expected['link_after'] = '<li class="pagenav">Pages<ul><li class="page_item page-item-1 page_item_has_children"><a href="' . get_permalink( 1 ) . '">Parent 1AFTER</a>
    … …  
    317357</li>
    318358</ul></li>';
    319         $actual = wp_list_pages( $args );
     359        $actual                 = wp_list_pages( $args );
    320360        $this->AssertEquals( $expected['link_after'], $actual );
    321361    }
    … …  
    323363
    324364    function test_wp_list_pages_include() {
    325         $args = array(
    326             'echo' => false,
    327             'include' => '1,3'
     365        $args                = array(
     366            'echo'    => false,
     367            'include' => '1,3',
    328368        );
    329369        $expected['include'] = '<li class="pagenav">Pages<ul><li class="page_item page-item-1"><a href="' . get_permalink( 1 ) . '">Parent 1</a></li>
    330370<li class="page_item page-item-3"><a href="' . get_permalink( 3 ) . '">Parent 3</a></li>
    331371</ul></li>';
    332         $actual = wp_list_pages( $args );
     372        $actual              = wp_list_pages( $args );
    333373        $this->AssertEquals( $expected['include'], $actual );
    334374    }
    335375
    336376    function test_wp_list_pages_exclude_tree() {
    337         $args = array(
    338             'echo' => false,
    339             'exclude_tree' => '2, 3'
     377        $args                = array(
     378            'echo'         => false,
     379            'exclude_tree' => '2, 3',
    340380        );
    341381        $expected['exclude'] = '<li class="pagenav">Pages<ul><li class="page_item page-item-1 page_item_has_children"><a href="' . get_permalink( 1 ) . '">Parent 1</a>
    … …  
    347387</li>
    348388</ul></li>';
    349         $actual = wp_list_pages( $args );
     389        $actual              = wp_list_pages( $args );
    350390        $this->AssertEquals( $expected['exclude'], $actual );
    351391    }
    352392
    353393    function test_wp_list_pages_discarded_whitespace() {
    354         $args = array(
    355             'echo' => false,
     394        $args                = array(
     395            'echo'         => false,
    356396            'item_spacing' => 'discard',
    357397        );
    358398        $expected['default'] = '<li class="pagenav">Pages<ul><li class="page_item page-item-1 page_item_has_children"><a href="' . get_permalink( 1 ) . '">Parent 1</a><ul class=\'children\'><li class="page_item page-item-4"><a href="' . get_permalink( 4 ) . '">Child 1</a></li><li class="page_item page-item-5"><a href="' . get_permalink( 5 ) . '">Child 2</a></li><li class="page_item page-item-6"><a href="' . get_permalink( 6 ) . '">Child 3</a></li></ul></li><li class="page_item page-item-2 page_item_has_children"><a href="' . get_permalink( 2 ) . '">Parent 2</a><ul class=\'children\'><li class="page_item page-item-7"><a href="' . get_permalink( 7 ) . '">Child 1</a></li><li class="page_item page-item-8"><a href="' . get_permalink( 8 ) . '">Child 2</a></li><li class="page_item page-item-9"><a href="' . get_permalink( 9 ) . '">Child 3</a></li></ul></li><li class="page_item page-item-3 page_item_has_children"><a href="' . get_permalink( 3 ) . '">Parent 3</a><ul class=\'children\'><li class="page_item page-item-10"><a href="' . get_permalink( 10 ) . '">Child 1</a></li><li class="page_item page-item-11"><a href="' . get_permalink( 11 ) . '">Child 2</a></li><li class="page_item page-item-12"><a href="' . get_permalink( 12 ) . '">Child 3</a></li></ul></li></ul></li>';
    359         $actual = wp_list_pages( $args );
     399        $actual              = wp_list_pages( $args );
    360400        $this->AssertEquals( $expected['default'], $actual );
    361401    }
Note: See TracChangeset for help on using the changeset viewer.