WordPress.org
Get WordPress

Make WordPress Core


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

    r38398 r42343  
    2222
    2323                if ( false !== strpos( $test_url, '%d' ) ) {
    24                         if ( false !== strpos( $test_url, '/?author=%d' ) )
     24                        if ( false !== strpos( $test_url, '/?author=%d' ) ) {
    2525                                $test_url = sprintf( $test_url, self::$author_id );
    26                         if ( false !== strpos( $test_url, '?cat=%d' ) )
     26                        }
     27                        if ( false !== strpos( $test_url, '?cat=%d' ) ) {
    2728                                $test_url = sprintf( $test_url, self::$terms[ $expected['url'] ] );
     29                        }
    2830                }
    2931
    … …  
    4951                        array( '?cat=%d', array( 'url' => '/category/parent/child-1/' ), 15256 ),
    5052                        array( '?cat=%d', array( 'url' => '/category/parent/child-1/child-2/' ) ), // no children
    51                         array( '/category/uncategorized/', array( 'url' => '/category/uncategorized/', 'qv' => array( 'category_name' => 'uncategorized' ) ) ),
    52                         array( '/category/uncategorized/page/2/', array( 'url' => '/category/uncategorized/page/2/', 'qv' => array( 'category_name' => 'uncategorized', 'paged' => 2) ) ),
    53                         array( '/category/uncategorized/?paged=2', array( 'url' => '/category/uncategorized/page/2/', 'qv' => array( 'category_name' => 'uncategorized', 'paged' => 2) ) ),
    54                         array( '/category/uncategorized/?paged=2&category_name=uncategorized', array( 'url' => '/category/uncategorized/page/2/', 'qv' => array( 'category_name' => 'uncategorized', 'paged' => 2) ), 17174 ),
     53                        array(
     54                                '/category/uncategorized/',
     55                                array(
     56                                        'url' => '/category/uncategorized/',
     57                                        'qv'  => array( 'category_name' => 'uncategorized' ),
     58                                ),
     59                        ),
     60                        array(
     61                                '/category/uncategorized/page/2/',
     62                                array(
     63                                        'url' => '/category/uncategorized/page/2/',
     64                                        'qv'  => array(
     65                                                'category_name' => 'uncategorized',
     66                                                'paged'         => 2,
     67                                        ),
     68                                ),
     69                        ),
     70                        array(
     71                                '/category/uncategorized/?paged=2',
     72                                array(
     73                                        'url' => '/category/uncategorized/page/2/',
     74                                        'qv'  => array(
     75                                                'category_name' => 'uncategorized',
     76                                                'paged'         => 2,
     77                                        ),
     78                                ),
     79                        ),
     80                        array(
     81                                '/category/uncategorized/?paged=2&category_name=uncategorized',
     82                                array(
     83                                        'url' => '/category/uncategorized/page/2/',
     84                                        'qv'  => array(
     85                                                'category_name' => 'uncategorized',
     86                                                'paged'         => 2,
     87                                        ),
     88                                ),
     89                                17174,
     90                        ),
    5591
    5692                        // Categories & Intersections with other vars
    57                         array( '/category/uncategorized/?tag=post-formats', array( 'url' => '/category/uncategorized/?tag=post-formats', 'qv' => array('category_name' => 'uncategorized', 'tag' => 'post-formats') ) ),
    58                         array( '/?category_name=cat-a,cat-b', array( 'url' => '/?category_name=cat-a,cat-b', 'qv' => array( 'category_name' => 'cat-a,cat-b' ) ) ),
     93                        array(
     94                                '/category/uncategorized/?tag=post-formats',
     95                                array(
     96                                        'url' => '/category/uncategorized/?tag=post-formats',
     97                                        'qv'  => array(
     98                                                'category_name' => 'uncategorized',
     99                                                'tag'           => 'post-formats',
     100                                        ),
     101                                ),
     102                        ),
     103                        array(
     104                                '/?category_name=cat-a,cat-b',
     105                                array(
     106                                        'url' => '/?category_name=cat-a,cat-b',
     107                                        'qv'  => array( 'category_name' => 'cat-a,cat-b' ),
     108                                ),
     109                        ),
    59110
    60111                        // Taxonomies with extra Query Vars
    61                         array( '/category/cat-a/page/1/?test=one%20two', '/category/cat-a/?test=one%20two', 18086), // Extra query vars should stay encoded
     112                        array( '/category/cat-a/page/1/?test=one%20two', '/category/cat-a/?test=one%20two', 18086 ), // Extra query vars should stay encoded
    62113
    63114                        // Categories with Dates
    64                         array( '/2008/04/?cat=1', array( 'url' => '/2008/04/?cat=1', 'qv' => array('cat' => '1', 'year' => '2008', 'monthnum' => '04' ) ), 17661 ),
    65 //                      array( '/2008/?category_name=cat-a', array( 'url' => '/2008/?category_name=cat-a', 'qv' => array('category_name' => 'cat-a', 'year' => '2008' ) ) ),
    66 
    67                         // Pages
    68                         array( '/child-page-1/', '/parent-page/child-page-1/'),
    69                         array( '/?page_id=144', '/parent-page/child-page-1/'),
     115                        array(
     116                                '/2008/04/?cat=1',
     117                                array(
     118                                        'url' => '/2008/04/?cat=1',
     119                                        'qv'  => array(
     120                                                'cat'      => '1',
     121                                                'year'     => '2008',
     122                                                'monthnum' => '04',
     123                                        ),
     124                                ),
     125                                17661,
     126                        ),
     127                        //          array( '/2008/?category_name=cat-a', array( 'url' => '/2008/?category_name=cat-a', 'qv' => array('category_name' => 'cat-a', 'year' => '2008' ) ) ),
     128
     129                                                        // Pages
     130                                                        array( '/child-page-1/', '/parent-page/child-page-1/' ),
     131                        array( '/?page_id=144', '/parent-page/child-page-1/' ),
    70132                        array( '/abo', '/about/' ),
    71133                        array( '/parent/child1/grandchild/', '/parent/child1/grandchild/' ),
    … …  
    73135
    74136                        // Posts
    75                         array( '?p=587', '/2008/06/02/post-format-test-audio/'),
    76                         array( '/?name=images-test', '/2008/09/03/images-test/'),
     137                        array( '?p=587', '/2008/06/02/post-format-test-audio/' ),
     138                        array( '/?name=images-test', '/2008/09/03/images-test/' ),
    77139                        // Incomplete slug should resolve and remove the ?name= parameter
    78                         array( '/?name=images-te', '/2008/09/03/images-test/', 20374),
     140                        array( '/?name=images-te', '/2008/09/03/images-test/', 20374 ),
    79141                        // Page slug should resolve to post slug and remove the ?pagename= parameter
    80                         array( '/?pagename=images-test', '/2008/09/03/images-test/', 20374),
    81 
    82                         array( '/2008/06/02/post-format-test-au/', '/2008/06/02/post-format-test-audio/'),
    83                         array( '/2008/06/post-format-test-au/', '/2008/06/02/post-format-test-audio/'),
    84                         array( '/2008/post-format-test-au/', '/2008/06/02/post-format-test-audio/'),
    85                         array( '/2010/post-format-test-au/', '/2008/06/02/post-format-test-audio/'), // A Year the post is not in
    86                         array( '/post-format-test-au/', '/2008/06/02/post-format-test-audio/'),
    87 
    88                         array( '/2008/09/03/images-test/3/', array( 'url' => '/2008/09/03/images-test/3/', 'qv' => array( 'name' => 'images-test', 'year' => '2008', 'monthnum' => '09', 'day' => '03', 'page' => '3' ) ) ),
     142                        array( '/?pagename=images-test', '/2008/09/03/images-test/', 20374 ),
     143
     144                        array( '/2008/06/02/post-format-test-au/', '/2008/06/02/post-format-test-audio/' ),
     145                        array( '/2008/06/post-format-test-au/', '/2008/06/02/post-format-test-audio/' ),
     146                        array( '/2008/post-format-test-au/', '/2008/06/02/post-format-test-audio/' ),
     147                        array( '/2010/post-format-test-au/', '/2008/06/02/post-format-test-audio/' ), // A Year the post is not in
     148                        array( '/post-format-test-au/', '/2008/06/02/post-format-test-audio/' ),
     149
     150                        array(
     151                                '/2008/09/03/images-test/3/',
     152                                array(
     153                                        'url' => '/2008/09/03/images-test/3/',
     154                                        'qv'  => array(
     155                                                'name'     => 'images-test',
     156                                                'year'     => '2008',
     157                                                'monthnum' => '09',
     158                                                'day'      => '03',
     159                                                'page'     => '3',
     160                                        ),
     161                                ),
     162                        ),
    89163                        array( '/2008/09/03/images-test/?page=3', '/2008/09/03/images-test/3/' ),
    90164                        array( '/2008/09/03/images-te?page=3', '/2008/09/03/images-test/3/' ),
    … …  
    99173                        // Dates
    100174                        array( '/?m=2008', '/2008/' ),
    101                         array( '/?m=200809', '/2008/09/'),
    102                         array( '/?m=20080905', '/2008/09/05/'),
    103 
    104                         array( '/2008/?day=05', '/2008/?day=05'), // no redirect
    105                         array( '/2008/09/?day=05', '/2008/09/05/'),
    106                         array( '/2008/?monthnum=9', '/2008/09/'),
    107 
    108                         array( '/?year=2008', '/2008/'),
    109 
    110                         array( '/2012/13/', '/2012/'),
     175                        array( '/?m=200809', '/2008/09/' ),
     176                        array( '/?m=20080905', '/2008/09/05/' ),
     177
     178                        array( '/2008/?day=05', '/2008/?day=05' ), // no redirect
     179                        array( '/2008/09/?day=05', '/2008/09/05/' ),
     180                        array( '/2008/?monthnum=9', '/2008/09/' ),
     181
     182                        array( '/?year=2008', '/2008/' ),
     183
     184                        array( '/2012/13/', '/2012/' ),
    111185                        array( '/2012/11/51/', '/2012/11/', 0, array( 'WP_Date_Query' ) ),
    112186
    113187                        // Authors
    114188                        array( '/?author=%d', '/author/canonical-author/' ),
    115 //                      array( '/?author=%d&year=2008', '/2008/?author=3'),
    116 //                      array( '/author/canonical-author/?year=2008', '/2008/?author=3'), //Either or, see previous testcase.
    117 
    118                         // Feeds
    119                         array( '/?feed=atom', '/feed/atom/' ),
     189                        //          array( '/?author=%d&year=2008', '/2008/?author=3'),
     190                        //          array( '/author/canonical-author/?year=2008', '/2008/?author=3'), //Either or, see previous testcase.
     191
     192                                // Feeds
     193                                array( '/?feed=atom', '/feed/atom/' ),
    120194                        array( '/?feed=rss2', '/feed/' ),
    121                         array( '/?feed=comments-rss2', '/comments/feed/'),
    122                         array( '/?feed=comments-atom', '/comments/feed/atom/'),
     195                        array( '/?feed=comments-rss2', '/comments/feed/' ),
     196                        array( '/?feed=comments-atom', '/comments/feed/atom/' ),
    123197
    124198                        // Feeds (per-post)
    125                         array( '/2008/03/03/comment-test/?feed=comments-atom', '/2008/03/03/comment-test/feed/atom/'),
    126                         array( '/?p=149&feed=comments-atom', '/2008/03/03/comment-test/feed/atom/'),
     199                        array( '/2008/03/03/comment-test/?feed=comments-atom', '/2008/03/03/comment-test/feed/atom/' ),
     200                        array( '/?p=149&feed=comments-atom', '/2008/03/03/comment-test/feed/atom/' ),
    127201
    128202                        // Index
Note: See TracChangeset for help on using the changeset viewer.