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/general/paginateLinks.php

    r41371 r42343  
    1212
    1313        function test_defaults() {
    14                 $page2 = get_pagenum_link( 2 );
    15                 $page3 = get_pagenum_link( 3 );
     14                $page2  = get_pagenum_link( 2 );
     15                $page3  = get_pagenum_link( 3 );
    1616                $page50 = get_pagenum_link( 50 );
    1717
    18                 $expected =<<<EXPECTED
     18                $expected = <<<EXPECTED
    1919<span aria-current='page' class='page-numbers current'>1</span>
    2020<a class='page-numbers' href='$page2'>2</a>
    … …  
    3030
    3131        function test_format() {
    32                 $page2 = home_url( '/page/2/' );
    33                 $page3 = home_url( '/page/3/' );
     32                $page2  = home_url( '/page/2/' );
     33                $page3  = home_url( '/page/3/' );
    3434                $page50 = home_url( '/page/50/' );
    3535
    36                 $expected =<<<EXPECTED
     36                $expected = <<<EXPECTED
    3737<span aria-current='page' class='page-numbers current'>1</span>
    3838<a class='page-numbers' href='$page2'>2</a>
    … …  
    4343EXPECTED;
    4444
    45                 $links = paginate_links( array( 'total' => 50, 'format' => 'page/%#%/' ) );
     45                $links = paginate_links(
     46                        array(
     47                                'total'  => 50,
     48                                'format' => 'page/%#%/',
     49                        )
     50                );
    4651                $this->assertEquals( $expected, $links );
    4752        }
    4853
    4954        function test_prev_next_false() {
    50                 $home = home_url( '/' );
    51                 $page3 = get_pagenum_link( 3 );
    52                 $page4 = get_pagenum_link( 4 );
     55                $home   = home_url( '/' );
     56                $page3  = get_pagenum_link( 3 );
     57                $page4  = get_pagenum_link( 4 );
    5358                $page50 = get_pagenum_link( 50 );
    5459
    55                 $expected =<<<EXPECTED
     60                $expected = <<<EXPECTED
    5661<a class='page-numbers' href='$home'>1</a>
    5762<span aria-current='page' class='page-numbers current'>2</span>
    … …  
    6267EXPECTED;
    6368
    64                 $links = paginate_links( array( 'total' => 50, 'prev_next' => false, 'current' => 2 ) );
     69                $links = paginate_links(
     70                        array(
     71                                'total'     => 50,
     72                                'prev_next' => false,
     73                                'current'   => 2,
     74                        )
     75                );
    6576                $this->assertEquals( $expected, $links );
    6677        }
    6778
    6879        function test_prev_next_true() {
    69                 $home = home_url( '/' );
    70                 $page3 = get_pagenum_link( 3 );
    71                 $page4 = get_pagenum_link( 4 );
     80                $home   = home_url( '/' );
     81                $page3  = get_pagenum_link( 3 );
     82                $page4  = get_pagenum_link( 4 );
    7283                $page50 = get_pagenum_link( 50 );
    7384
    74                 $expected =<<<EXPECTED
     85                $expected = <<<EXPECTED
    7586<a class="prev page-numbers" href="$home">&laquo; Previous</a>
    7687<a class='page-numbers' href='$home'>1</a>
    … …  
    8394EXPECTED;
    8495
    85                 $links = paginate_links( array( 'total' => 50, 'prev_next' => true, 'current' => 2 ) );
     96                $links = paginate_links(
     97                        array(
     98                                'total'     => 50,
     99                                'prev_next' => true,
     100                                'current'   => 2,
     101                        )
     102                );
    86103                $this->assertEquals( $expected, $links );
    87104        }
    … …  
    97114                $this->i18n_count = 0;
    98115                add_filter( 'number_format_i18n', array( $this, 'increment_i18n_count' ) );
    99                 paginate_links( array(
    100                         'total'     => 100,
    101                         'current'   => 50,
    102                         'show_all'  => false,
    103                         'prev_next' => true,
    104                         'end_size'  => 1,
    105                         'mid_size'  => 1,
    106                 ) );
     116                paginate_links(
     117                        array(
     118                                'total'     => 100,
     119                                'current'   => 50,
     120                                'show_all'  => false,
     121                                'prev_next' => true,
     122                                'end_size'  => 1,
     123                                'mid_size'  => 1,
     124                        )
     125                );
    107126                // The links should be:
    108127                // < Previous 1 ... 49 50 51 ... 100 Next >
    … …  
    117136
    118137                // Current page: 2
    119                 $links = paginate_links( array(
    120                         'current'  => 2,
    121                         'total'    => 5,
    122                         'end_size' => 1,
    123                         'mid_size' => 1,
    124                         'type'     => 'array',
    125                 ) );
     138                $links = paginate_links(
     139                        array(
     140                                'current'  => 2,
     141                                'total'    => 5,
     142                                'end_size' => 1,
     143                                'mid_size' => 1,
     144                                'type'     => 'array',
     145                        )
     146                );
    126147
    127148                $expected_attributes = array(
    128149                        array(
    129150                                'href'  => home_url( '/' ),
    130                                 'class' => 'prev page-numbers'
     151                                'class' => 'prev page-numbers',
    131152                        ),
    132153                        array(
    133154                                'href'  => home_url( '/' ),
    134                                 'class' => 'page-numbers'
    135                         )
    136                 );
    137 
    138                 $document = new DOMDocument();
     155                                'class' => 'page-numbers',
     156                        ),
     157                );
     158
     159                $document                     = new DOMDocument();
    139160                $document->preserveWhiteSpace = false;
    140161
    … …  
    142163                foreach ( $expected_attributes as $link_idx => $attributes ) {
    143164
    144                         $document->loadHTML( $links[$link_idx] );
     165                        $document->loadHTML( $links[ $link_idx ] );
    145166                        $tag = $document->getElementsByTagName( 'a' )->item( 0 );
    146167
    … …  
    155176
    156177                // Current page: 1
    157                 $links = paginate_links( array(
    158                         'current'  => 1,
    159                         'total'    => 5,
    160                         'end_size' => 1,
    161                         'mid_size' => 1,
    162                         'type'     => 'array',
    163                 ) );
     178                $links = paginate_links(
     179                        array(
     180                                'current'  => 1,
     181                                'total'    => 5,
     182                                'end_size' => 1,
     183                                'mid_size' => 1,
     184                                'type'     => 'array',
     185                        )
     186                );
    164187
    165188                $document->loadHTML( $links[0] );
    … …  
    179202
    180203        function add_query_arg( $url ) {
    181                 return add_query_arg( array(
    182                         'foo' => 'bar',
    183                         's'   => 'search+term',
    184                 ), $url );
     204                return add_query_arg(
     205                        array(
     206                                'foo' => 'bar',
     207                                's'   => 'search+term',
     208                        ), $url
     209                );
    185210        }
    186211
    … …  
    190215        function test_paginate_links_query_args() {
    191216                add_filter( 'get_pagenum_link', array( $this, 'add_query_arg' ) );
    192                 $links = paginate_links( array(
    193                         'current'  => 2,
    194                         'total'    => 5,
    195                         'end_size' => 1,
    196                         'mid_size' => 1,
    197                         'type'     => 'array',
    198                 ) );
     217                $links = paginate_links(
     218                        array(
     219                                'current'  => 2,
     220                                'total'    => 5,
     221                                'end_size' => 1,
     222                                'mid_size' => 1,
     223                                'type'     => 'array',
     224                        )
     225                );
    199226                remove_filter( 'get_pagenum_link', array( $this, 'add_query_arg' ) );
    200227
    201                 $document = new DOMDocument();
     228                $document                     = new DOMDocument();
    202229                $document->preserveWhiteSpace = false;
    203230
    … …  
    226253        function test_paginate_links_with_custom_query_args() {
    227254                add_filter( 'get_pagenum_link', array( $this, 'add_query_arg' ) );
    228                 $links = paginate_links( array(
    229                         'current'  => 2,
    230                         'total'    => 5,
    231                         'end_size' => 1,
    232                         'mid_size' => 1,
    233                         'type'     => 'array',
    234                         'add_args' => array(
    235                                 'baz' => 'qux',
    236                         ),
    237                 ) );
     255                $links = paginate_links(
     256                        array(
     257                                'current'  => 2,
     258                                'total'    => 5,
     259                                'end_size' => 1,
     260                                'mid_size' => 1,
     261                                'type'     => 'array',
     262                                'add_args' => array(
     263                                        'baz' => 'qux',
     264                                ),
     265                        )
     266                );
    238267                remove_filter( 'get_pagenum_link', array( $this, 'add_query_arg' ) );
    239268
    240                 $document = new DOMDocument();
     269                $document                     = new DOMDocument();
    241270                $document->preserveWhiteSpace = false;
    242271
    … …  
    264293        public function test_paginate_links_should_allow_non_default_format_without_add_args() {
    265294                // Fake the query params.
    266                 $request_uri = $_SERVER['REQUEST_URI'];
     295                $request_uri            = $_SERVER['REQUEST_URI'];
    267296                $_SERVER['REQUEST_URI'] = add_query_arg( 'foo', 3, home_url() );
    268297
    269                 $links = paginate_links( array(
    270                         'base'    => add_query_arg( 'foo', '%#%' ),
    271                         'format'  => '',
    272                         'total'   => 5,
    273                         'current' => 3,
    274                         'type'    => 'array',
    275                 ) );
     298                $links = paginate_links(
     299                        array(
     300                                'base'    => add_query_arg( 'foo', '%#%' ),
     301                                'format'  => '',
     302                                'total'   => 5,
     303                                'current' => 3,
     304                                'type'    => 'array',
     305                        )
     306                );
    276307
    277308                $this->assertContains( '?foo=1', $links[1] );
    … …  
    288319        public function test_paginate_links_should_allow_add_args_to_be_bool_false() {
    289320                // Fake the query params.
    290                 $request_uri = $_SERVER['REQUEST_URI'];
     321                $request_uri            = $_SERVER['REQUEST_URI'];
    291322                $_SERVER['REQUEST_URI'] = add_query_arg( 'foo', 3, home_url() );
    292323
    293                 $links = paginate_links( array(
    294                         'add_args' => false,
    295                         'base'    => add_query_arg( 'foo', '%#%' ),
    296                         'format'  => '',
    297                         'total'   => 5,
    298                         'current' => 3,
    299                         'type'    => 'array',
    300                 ) );
     324                $links = paginate_links(
     325                        array(
     326                                'add_args' => false,
     327                                'base'     => add_query_arg( 'foo', '%#%' ),
     328                                'format'   => '',
     329                                'total'    => 5,
     330                                'current'  => 3,
     331                                'type'     => 'array',
     332                        )
     333                );
    301334
    302335                $this->assertContains( "<span aria-current='page' class='page-numbers current'>3</span>", $links );
    … …  
    308341        public function test_custom_base_query_arg_should_be_stripped_from_current_url_before_generating_pag_links() {
    309342                // Fake the current URL: example.com?foo
    310                 $request_uri = $_SERVER['REQUEST_URI'];
     343                $request_uri            = $_SERVER['REQUEST_URI'];
    311344                $_SERVER['REQUEST_URI'] = add_query_arg( 'foo', '', $request_uri );
    312345
    313                 $links = paginate_links( array(
    314                         'base'    => add_query_arg( 'foo', '%_%', home_url() ),
    315                         'format'  => '%#%',
    316                         'total'   => 5,
    317                         'current' => 1,
    318                         'type'    => 'array',
    319                 ) );
     346                $links = paginate_links(
     347                        array(
     348                                'base'    => add_query_arg( 'foo', '%_%', home_url() ),
     349                                'format'  => '%#%',
     350                                'total'   => 5,
     351                                'current' => 1,
     352                                'type'    => 'array',
     353                        )
     354                );
    320355
    321356                $page_2_url = home_url() . '?foo=2';
Note: See TracChangeset for help on using the changeset viewer.