Changeset 44415

Timestamp:

01/07/2019 12:53:22 PM (6 years ago)

Author:

SergeyBiryukov

Message:

Pages, Post Types: Escape CSS classes in Walker_Page::start_el() after the page_css_class filter runs.

Don't add an empty class attribute if there are no classes, for consistency with Walker_Nav_Menu::start_el().

Props abhijitrakas, mukesh27.
Fixes #44880.

File:

• trunk/src/wp-includes/class-walker-page.php (modified) (2 diffs)

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

@@ -149 +149 @@
          */
         $css_classes = implode( ' ', apply_filters( 'page_css_class', $css_class, $page, $depth, $args, $current_page ) );
+        $css_classes = $css_classes ? ' class="' . esc_attr( $css_classes ) . '"' : '';
 
         if ( '' === $page->post_title ) {
@@ -187 +188 @@
 
         $output .= $indent . sprintf(
-            '<li class="%s"><a%s>%s%s%s</a>',
+            '<li%s><a%s>%s%s%s</a>',
             $css_classes,
             $attributes,