WordPress.org
Get WordPress

Make WordPress Core

Changeset 38523


Ignore:
Timestamp:
09/06/2016 09:05:45 AM (10 years ago)
Author:
peterwilsoncc
Message:

Menus: Add white space option to wp_nav_menu() and wp_list_pages().

Adds an item_spacing option to the arguments array for the functions wp_nav_menu(), wp_list_pages(), and wp_page_menu(). item_spacing is a boolean accepting either preserve or discard.

Previously, certain CSS choices could result in a site's layout changing if wp_nav_menu() fell back to the default wp_list_pages() due to differences in the whitespace within the HTML. The new argument ensures a function outputs consistant HTML while maintaining backward compatibility.

Fixes #35206.

Location:
trunk
Files:
7 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-includes/class-walker-nav-menu.php

    r37640 r38523  
    5151         */
    5252        public function start_lvl( &$output, $depth = 0, $args = array() ) {
    53                 $indent = str_repeat("\t", $depth);
    54                 $output .= "\n$indent<ul class=\"sub-menu\">\n";
     53                if ( 'preserve' === $args->item_spacing ) {
     54                        $t = "\t";
     55                        $n = "\n";
     56                } else {
     57                        $t = '';
     58                        $n = '';
     59                }
     60                $indent = str_repeat( $t, $depth );
     61                $output .= "{$n}{$indent}<ul class=\"sub-menu\">{$n}";
    5562        }
    5663
    … …  
    6774         */
    6875        public function end_lvl( &$output, $depth = 0, $args = array() ) {
    69                 $indent = str_repeat("\t", $depth);
    70                 $output .= "$indent</ul>\n";
     76                if ( 'preserve' === $args->item_spacing ) {
     77                        $t = "\t";
     78                        $n = "\n";
     79                } else {
     80                        $t = '';
     81                        $n = '';
     82                }
     83                $indent = str_repeat( $t, $depth );
     84                $output .= "$indent</ul>{$n}";
    7185        }
    7286
    … …  
    86100         */
    87101        public function start_el( &$output, $item, $depth = 0, $args = array(), $id = 0 ) {
    88                 $indent = ( $depth ) ? str_repeat( "\t", $depth ) : '';
     102                if ( 'preserve' === $args->item_spacing ) {
     103                        $t = "\t";
     104                        $n = "\n";
     105                } else {
     106                        $t = '';
     107                        $n = '';
     108                }
     109                $indent = ( $depth ) ? str_repeat( $t, $depth ) : '';
    89110
    90111                $classes = empty( $item->classes ) ? array() : (array) $item->classes;
    … …  
    217238         */
    218239        public function end_el( &$output, $item, $depth = 0, $args = array() ) {
    219                 $output .= "</li>\n";
     240                if ( 'preserve' === $args->item_spacing ) {
     241                        $t = "\t";
     242                        $n = "\n";
     243                } else {
     244                        $t = '';
     245                        $n = '';
     246                }
     247                $output .= "</li>{$n}";
    220248        }
    221249

  • trunk/src/wp-includes/class-walker-page.php

    r37518 r38523  
    5454         */
    5555        public function start_lvl( &$output, $depth = 0, $args = array() ) {
    56                 $indent = str_repeat("\t", $depth);
    57                 $output .= "\n$indent<ul class='children'>\n";
     56                if ( 'preserve' === $args['item_spacing'] ) {
     57                        $t = "\t";
     58                        $n = "\n";
     59                } else {
     60                        $t = '';
     61                        $n = '';
     62                }
     63                $indent = str_repeat( $t, $depth );
     64                $output .= "{$n}{$indent}<ul class='children'>{$n}";
    5865        }
    5966
    … …  
    7279         */
    7380        public function end_lvl( &$output, $depth = 0, $args = array() ) {
    74                 $indent = str_repeat("\t", $depth);
    75                 $output .= "$indent</ul>\n";
     81                if ( 'preserve' === $args['item_spacing'] ) {
     82                        $t = "\t";
     83                        $n = "\n";
     84                } else {
     85                        $t = '';
     86                        $n = '';
     87                }
     88                $indent = str_repeat( $t, $depth );
     89                $output .= "{$indent}</ul>{$n}";
    7690        }
    7791
    … …  
    90104         */
    91105        public function start_el( &$output, $page, $depth = 0, $args = array(), $current_page = 0 ) {
     106                if ( 'preserve' === $args['item_spacing'] ) {
     107                        $t = "\t";
     108                        $n = "\n";
     109                } else {
     110                        $t = '';
     111                        $n = '';
     112                }
    92113                if ( $depth ) {
    93                         $indent = str_repeat( "\t", $depth );
     114                        $indent = str_repeat( $t, $depth );
    94115                } else {
    95116                        $indent = '';
    … …  
    176197         */
    177198        public function end_el( &$output, $page, $depth = 0, $args = array() ) {
    178                 $output .= "</li>\n";
     199                if ( 'preserve' === $args['item_spacing'] ) {
     200                        $t = "\t";
     201                        $n = "\n";
     202                } else {
     203                        $t = '';
     204                        $n = '';
     205                }
     206                $output .= "</li>{$n}";
    179207        }
    180208

  • trunk/src/wp-includes/nav-menu-template.php

    r38470 r38523  
    4141 *     @type string             $items_wrap      How the list items should be wrapped. Default is a ul with an id and class.
    4242 *                                               Uses printf() format with numbered placeholders.
     43 *     @type string             $item_spacing    Whether whitespace format the menu's HTML: 'discard' or 'preserve' (default).
    4344 * }
    4445 * @return object|false|void Menu output if $echo is false, false if there are no items or no menu was found.
    … …  
    4849
    4950        $defaults = array( 'menu' => '', 'container' => 'div', 'container_class' => '', 'container_id' => '', 'menu_class' => 'menu', 'menu_id' => '',
    50         'echo' => true, 'fallback_cb' => 'wp_page_menu', 'before' => '', 'after' => '', 'link_before' => '', 'link_after' => '', 'items_wrap' => '<ul id="%1$s" class="%2$s">%3$s</ul>',
     51        'echo' => true, 'fallback_cb' => 'wp_page_menu', 'before' => '', 'after' => '', 'link_before' => '', 'link_after' => '', 'items_wrap' => '<ul id="%1$s" class="%2$s">%3$s</ul>', 'item_spacing' => 'preserve',
    5152        'depth' => 0, 'walker' => '', 'theme_location' => '' );
    5253
    5354        $args = wp_parse_args( $args, $defaults );
     55
     56        if ( ! in_array( $args['item_spacing'], array( 'preserve', 'discard' ), true ) ) {
     57                // invalid value, fall back to default.
     58                $args['item_spacing'] = $defaults['item_spacing'];
     59        }
     60
    5461        /**
    5562         * Filters the arguments used to display a navigation menu.

  • trunk/src/wp-includes/post-template.php

    r38371 r38523  
    11391139 *     @type string $title_li     List heading. Passing a null or empty value will result in no heading, and the list
    11401140 *                                will not be wrapped with unordered list `<ul>` tags. Default 'Pages'.
     1141 *     @type string $item_spacing Whether whitespace format the menu's HTML: 'discard' or 'preserve' (default).
    11411142 *     @type Walker $walker       Walker instance to use for listing pages. Default empty (Walker_Page).
    11421143 * }
    … …  
    11501151                'title_li' => __( 'Pages' ), 'echo' => 1,
    11511152                'authors' => '', 'sort_column' => 'menu_order, post_title',
    1152                 'link_before' => '', 'link_after' => '', 'walker' => '',
     1153                'link_before' => '', 'link_after' => '', 'item_spacing' => 'preserve', 'walker' => '',
    11531154        );
    11541155
    11551156        $r = wp_parse_args( $args, $defaults );
     1157
     1158        if ( ! in_array( $r['item_spacing'], array( 'preserve', 'discard' ), true ) ) {
     1159                // invalid value, fall back to default.
     1160                $r['item_spacing'] = $defaults['item_spacing'];
     1161        }
    11561162
    11571163        $output = '';
    … …  
    12311237 *     Optional. Arguments to generate a page menu. See wp_list_pages() for additional arguments.
    12321238 *
    1233  *     @type string          $sort_column How to short the list of pages. Accepts post column names.
    1234  *                                        Default 'menu_order, post_title'.
    1235  *     @type string          $menu_id     ID for the div containing the page list. Default is empty string.
    1236  *     @type string          $menu_class  Class to use for the element containing the page list. Default 'menu'.
    1237  *     @type string          $container   Element to use for the element containing the page list. Default 'div'.
    1238  *     @type bool            $echo        Whether to echo the list or return it. Accepts true (echo) or false (return).
    1239  *                                        Default true.
    1240  *     @type int|bool|string $show_home   Whether to display the link to the home page. Can just enter the text
    1241  *                                        you'd like shown for the home link. 1|true defaults to 'Home'.
    1242  *     @type string          $link_before The HTML or text to prepend to $show_home text. Default empty.
    1243  *     @type string          $link_after  The HTML or text to append to $show_home text. Default empty.
    1244  *     @type string          $before      The HTML or text to prepend to the menu. Default is '<ul>'.
    1245  *     @type string          $after       The HTML or text to append to the menu. Default is '</ul>'.
    1246  *     @type Walker          $walker      Walker instance to use for listing pages. Default empty (Walker_Page).
     1239 *     @type string          $sort_column  How to short the list of pages. Accepts post column names.
     1240 *                                         Default 'menu_order, post_title'.
     1241 *     @type string          $menu_id      ID for the div containing the page list. Default is empty string.
     1242 *     @type string          $menu_class   Class to use for the element containing the page list. Default 'menu'.
     1243 *     @type string          $container    Element to use for the element containing the page list. Default 'div'.
     1244 *     @type bool            $echo         Whether to echo the list or return it. Accepts true (echo) or false (return).
     1245 *                                         Default true.
     1246 *     @type int|bool|string $show_home    Whether to display the link to the home page. Can just enter the text
     1247 *                                         you'd like shown for the home link. 1|true defaults to 'Home'.
     1248 *     @type string          $link_before  The HTML or text to prepend to $show_home text. Default empty.
     1249 *     @type string          $link_after   The HTML or text to append to $show_home text. Default empty.
     1250 *     @type string          $before       The HTML or text to prepend to the menu. Default is '<ul>'.
     1251 *     @type string          $after        The HTML or text to append to the menu. Default is '</ul>'.
     1252 *     @type string          $item_spacing Whether whitespace format the menu's HTML: 'discard' or 'preserve' (default).
     1253 *     @type Walker          $walker       Walker instance to use for listing pages. Default empty (Walker_Page).
    12471254 * }
    12481255 * @return string|void HTML menu
    … …  
    12501257function wp_page_menu( $args = array() ) {
    12511258        $defaults = array(
    1252                 'sort_column' => 'menu_order, post_title',
    1253                 'menu_id'     => '',
    1254                 'menu_class'  => 'menu',
    1255                 'container'   => 'div',
    1256                 'echo'        => true,
    1257                 'link_before' => '',
    1258                 'link_after'  => '',
    1259                 'before'      => '<ul>',
    1260                 'after'       => '</ul>',
    1261                 'walker'      => '',
     1259                'sort_column'  => 'menu_order, post_title',
     1260                'menu_id'      => '',
     1261                'menu_class'   => 'menu',
     1262                'container'    => 'div',
     1263                'echo'         => true,
     1264                'link_before'  => '',
     1265                'link_after'   => '',
     1266                'before'       => '<ul>',
     1267                'after'        => '</ul>',
     1268                'item_spacing' => 'discard',
     1269                'walker'       => '',
    12621270        );
    12631271        $args = wp_parse_args( $args, $defaults );
     1272
     1273        if ( ! in_array( $args['item_spacing'], array( 'preserve', 'discard' ) ) ) {
     1274                // invalid value, fall back to default.
     1275                $args['item_spacing'] = $defaults['item_spacing'];
     1276        }
     1277
     1278        if ( 'preserve' === $args['item_spacing'] ) {
     1279                $t = "\t";
     1280                $n = "\n";
     1281        } else {
     1282                $t = '';
     1283                $n = '';
     1284        }
    12641285
    12651286        /**
    … …  
    13011322        $list_args['echo'] = false;
    13021323        $list_args['title_li'] = '';
    1303         $menu .= str_replace( array( "\r", "\n", "\t" ), '', wp_list_pages($list_args) );
     1324        $menu .= wp_list_pages( $list_args );
    13041325
    13051326        $container = sanitize_text_field( $args['container'] );
    … …  
    13161337                        'wp_page_menu' === $args['fallback_cb'] &&
    13171338                        'ul' !== $container ) {
    1318                         $args['before'] = '<ul>';
     1339                        $args['before'] = "<ul>{$n}";
    13191340                        $args['after'] = '</ul>';
    13201341                }
    … …  
    13321353        }
    13331354
    1334         $menu = "<{$container}{$attrs}>" . $menu . "</{$container}>\n";
     1355        $menu = "<{$container}{$attrs}>" . $menu . "</{$container}>{$n}";
    13351356
    13361357        /**

  • trunk/tests/phpunit/tests/post/listPages.php

    r35242 r38523  
    1818                'link_after' => '',
    1919                'walker' => '',
     20                'item_spacing' => 'preserve',
    2021                'include'      => '',
    2122                'post_type'    => 'page',
    … …  
    343344                $this->AssertEquals( $expected['exclude'], $actual );
    344345        }
     346
     347        function test_wp_list_pages_discarded_whitespace() {
     348                $args = array(
     349                        'echo' => false,
     350                        'item_spacing' => 'discard',
     351                );
     352                $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>';
     353                $actual = wp_list_pages( $args );
     354                $this->AssertEquals( $expected['default'], $actual );
     355        }
    345356}

  • trunk/tests/phpunit/tests/post/nav-menu.php

    r38400 r38523  
    278278
    279279        /**
     280         * @ticket 35206
     281         */
     282        function test_wp_nav_menu_whitespace_options() {
     283                $post_id1 = self::factory()->post->create();
     284                $post_id2 = self::factory()->post->create();
     285                $post_id3 = self::factory()->post->create();
     286                $post_id4 = self::factory()->post->create();
     287
     288                $post_insert = wp_update_nav_menu_item( $this->menu_id, 0, array(
     289                        'menu-item-type' => 'post_type',
     290                        'menu-item-object' => 'post',
     291                        'menu-item-object-id' => $post_id1,
     292                        'menu-item-status' => 'publish'
     293                ) );
     294
     295                $post_inser2 = wp_update_nav_menu_item( $this->menu_id, 0, array(
     296                        'menu-item-type' => 'post_type',
     297                        'menu-item-object' => 'post',
     298                        'menu-item-object-id' => $post_id2,
     299                        'menu-item-status' => 'publish'
     300                ) );
     301
     302                $post_insert3 = wp_update_nav_menu_item( $this->menu_id, 0, array(
     303                        'menu-item-type' => 'post_type',
     304                        'menu-item-object' => 'post',
     305                        'menu-item-parent-id' => $post_insert,
     306                        'menu-item-object-id' => $post_id3,
     307                        'menu-item-status' => 'publish'
     308                ) );
     309
     310                $post_insert4 = wp_update_nav_menu_item( $this->menu_id, 0, array(
     311                        'menu-item-type' => 'post_type',
     312                        'menu-item-object' => 'post',
     313                        'menu-item-parent-id' => $post_insert,
     314                        'menu-item-object-id' => $post_id4,
     315                        'menu-item-status' => 'publish'
     316                ) );
     317
     318                // No whitespace suppression.
     319                $menu = wp_nav_menu( array(
     320                        'echo' => false,
     321                        'menu' => $this->menu_id,
     322                ) );
     323
     324                // The markup should include whitespace between <li>s
     325                $this->assertRegExp( '/\s<li.*>|<\/li>\s/U', $menu );
     326                $this->assertNotRegExp( '/<\/li><li.*>/U', $menu );
     327
     328
     329                // Whitepsace suppressed.
     330                $menu = wp_nav_menu( array(
     331                        'echo'         => false,
     332                        'item_spacing' => 'discard',
     333                        'menu'         => $this->menu_id,
     334                ) );
     335
     336                // The markup should not include whitespace around <li>s
     337                $this->assertNotRegExp( '/\s<li.*>|<\/li>\s/U', $menu );
     338                $this->assertRegExp( '/><li.*>|<\/li></U', $menu );
     339        }
     340
     341        /*
    280342         * Confirm `wp_nav_menu()` and `Walker_Nav_Menu` passes an $args object to filters.
    281343         *

  • trunk/tests/phpunit/tests/post/template.php

    r35242 r38523  
    300300                $this->assertRegExp( '/<\/ul><\/div>/', $menu );
    301301
     302                // After falling back, the markup should include whitespace around <li>s
     303                $this->assertRegExp( '/\s<li.*>|<\/li>\s/U', $menu );
     304                $this->assertNotRegExp( '/><li.*>|<\/li></U', $menu );
     305
    302306                // No menus + wp_nav_menu() falls back to wp_page_menu(), this time without a container.
    303307                $menu = wp_nav_menu( array(
    … …  
    308312                // After falling back, the empty 'container' argument should still return a container element.
    309313                $this->assertRegExp( '/<div class="menu">/', $menu );
     314
     315                // No menus + wp_nav_menu() falls back to wp_page_menu(), this time without white-space.
     316                $menu = wp_nav_menu( array(
     317                        'echo'         => false,
     318                        'item_spacing' => 'discard',
     319                ) );
     320
     321                // After falling back, the markup should not include whitespace around <li>s
     322                $this->assertNotRegExp( '/\s<li.*>|<\/li>\s/U', $menu );
     323                $this->assertRegExp( '/><li.*>|<\/li></U', $menu );
     324
    310325        }
    311326}
Note: See TracChangeset for help on using the changeset viewer.