WordPress.org

Make WordPress Core

Ticket #13551: make-classes-property-consistent.13551.diff

File make-classes-property-consistent.13551.diff, 8.2 KB (added by filosofo, 8 years ago)
  • wp-includes/nav-menu-template.php

     
    7272
    7373                $classes = array( 'menu-item', 'menu-item-type-' . $item->type );
    7474                if ( !empty($item->classes) )
    75                         $classes = array_merge($classes, explode(' ', $item->classes) );
     75                        $classes = array_merge( $classes, $item->classes );
    7676
    7777                if ( 'custom' != $item->type ) {
    7878                        $classes[] = 'menu-item-object-' . $item->object;
     
    170170                $output .= '<input type="hidden" class="menu-item-target" name="menu-item[' . $possible_object_id . '][menu-item-target]" value="'. esc_attr( $item->target ) .'" />';
    171171                $output .= '<input type="hidden" class="menu-item-attr_title" name="menu-item[' . $possible_object_id . '][menu-item-attr_title]" value="'. esc_attr( $item->attr_title ) .'" />';
    172172                $output .= '<input type="hidden" class="menu-item-description" name="menu-item[' . $possible_object_id . '][menu-item-description]" value="'. esc_attr( $item->description ) .'" />';
    173                 $output .= '<input type="hidden" class="menu-item-classes" name="menu-item[' . $possible_object_id . '][menu-item-classes]" value="'. esc_attr( $item->classes ) .'" />';
     173                $output .= '<input type="hidden" class="menu-item-classes" name="menu-item[' . $possible_object_id . '][menu-item-classes]" value="'. esc_attr( implode( ' ', $item->classes ) ) .'" />';
    174174                $output .= '<input type="hidden" class="menu-item-xfn" name="menu-item[' . $possible_object_id . '][menu-item-xfn]" value="'. esc_attr( $item->xfn ) .'" />';
    175175        }
    176176}
     
    338338                                ( 'taxonomy' == $menu_item->type && ( $wp_query->is_category || $wp_query->is_tag || $wp_query->is_tax ) )
    339339                        )
    340340                ) {
    341                         $menu_items[$key]->classes = trim( $menu_items[$key]->classes . ' ' . 'current-menu-item' );
     341                        $menu_items[$key]->classes[] = 'current-menu-item';
    342342                        $active_parent_item_ids[] = (int) $menu_item->menu_item_parent;
    343343                        $active_parent_object_ids[] = (int) $menu_item->post_parent;
    344344                        $active_object = $menu_item->object;
     
    348348                        $current_url = ( is_ssl() ? 'https://' : 'http://' ) . $_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI'];
    349349                        $item_url = strpos( $menu_item->url, '#' ) ? substr( $menu_item->url, 0, strpos( $menu_item->url, '#' ) ) : $menu_item->url;
    350350                        if ( $item_url == $current_url ) {
    351                                 $menu_items[$key]->classes = trim( $menu_item->classes . ' ' . 'current-menu-item' );
     351                                $menu_items[$key]->classes[] = 'current-menu-item';
    352352                                if ( untrailingslashit($current_url) == home_url() )
    353                                         $menu_items[$key]->classes .= ' menu-item-home';
     353                                        $menu_items[$key]->classes[] = 'menu-item-home';
    354354                                $active_parent_item_ids[] = (int) $menu_item->menu_item_parent;
    355355                                $active_parent_object_ids[] = (int) $menu_item->post_parent;
    356356                                $active_object = $menu_item->object;
     
    359359               
    360360                // back-compat with wp_page_menu: add "current_page_parent" to static home page link for any non-page query
    361361                if ( ! empty( $home_page_id ) && 'post_type' == $menu_item->type && empty( $wp_query->is_page ) && $home_page_id == $menu_item->object_id )
    362                         $menu_items[$key]->classes = trim( $menu_items[$key]->classes . ' ' . 'current_page_parent' );
     362                        $menu_items[$key]->classes[] = 'current_page_parent';
    363363        }
    364364
    365365        $active_parent_item_ids = array_filter( array_unique( $active_parent_item_ids ) );
     
    374374                        is_post_type_hierarchical( $queried_object->post_type ) &&
    375375                        in_array( $parent_item->object_id, $queried_object->ancestors )
    376376                )
    377                         $menu_items[$key]->classes = trim( $parent_item->classes . ' ' . 'current-' . $queried_object->post_type . '-ancestor current-menu-ancestor' );
     377                        $menu_items[$key]->classes[] = 'current-' . $queried_object->post_type . '-ancestor current-menu-ancestor';
    378378                if ( in_array( $parent_item->db_id, $active_parent_item_ids ) )
    379                         $menu_items[$key]->classes = trim( $parent_item->classes . ' ' . 'current-menu-parent' );
     379                        $menu_items[$key]->classes[] = 'current-menu-parent';
    380380                if ( in_array( $parent_item->object_id, $active_parent_object_ids ) )
    381                         $menu_items[$key]->classes = trim( $parent_item->classes . ' ' . 'current-' . $active_object . '-parent' );
     381                        $menu_items[$key]->classes[] = 'current-' . $active_object . '-parent';
    382382        }
    383383}
    384384
  • wp-includes/nav-menu.php

     
    253253 * @return int The menu item's database ID or WP_Error object on failure.
    254254 */
    255255function wp_update_nav_menu_item( $menu_id = 0, $menu_item_db_id = 0, $menu_item_data = array() ) {
    256 
    257256        $menu_id = (int) $menu_id;
    258257        $menu_item_db_id = (int) $menu_item_db_id;
    259258
     
    370369                update_post_meta( $menu_item_db_id, '_menu_item_object_id', (int) $args['menu-item-object-id'] );
    371370                update_post_meta( $menu_item_db_id, '_menu_item_object', sanitize_key($args['menu-item-object']) );
    372371                update_post_meta( $menu_item_db_id, '_menu_item_target', sanitize_key($args['menu-item-target']) );
    373                 // @todo sanitize_html_classes()
    374                 foreach( array( 'menu-item-classes', 'menu-item-xfn') as $arg )
    375                         $args[$arg] = implode( ' ', array_map( 'sanitize_html_class', explode( ' ', $args[$arg] ) ) );
     372               
     373                $args['menu-item-classes'] = array_map( 'sanitize_html_class', explode( ' ', $args['menu-item-classes'] ) );
     374                $args['menu-item-xfn'] = implode( ' ', array_map( 'sanitize_html_class', explode( ' ', $args['menu-item-xfn'] ) ) );
    376375                update_post_meta( $menu_item_db_id, '_menu_item_classes', $args['menu-item-classes'] );
    377376                update_post_meta( $menu_item_db_id, '_menu_item_xfn', $args['menu-item-xfn'] );
    378377
     
    533532 * - title:             The title of this menu item.
    534533 * - target:            The target attribute of the link element for this menu item.
    535534 * - attr_title:        The title attribute of the link element for this menu item.
    536  * - classes:           The class attribute value for the link element of this menu item.
     535 * - classes:           The array of class attribute values for the link element of this menu item.
    537536 * - xfn:               The XFN relationship expressed in the link of this menu item.
    538537 * - description:       The description of this menu item.
    539538 *
     
    580579                        $menu_item->attr_title = apply_filters( 'nav_menu_attr_title', $menu_item->post_excerpt );
    581580                        $menu_item->description = apply_filters( 'nav_menu_description', $menu_item->post_content );
    582581
    583                         $menu_item->classes = get_post_meta( $menu_item->ID, '_menu_item_classes', true );
     582                        $menu_item->classes = (array) get_post_meta( $menu_item->ID, '_menu_item_classes', true );
    584583                        $menu_item->xfn = get_post_meta( $menu_item->ID, '_menu_item_xfn', true );
    585584                } else {
    586585                        $menu_item->db_id = 0;
     
    598597
    599598                        $menu_item->attr_title = apply_filters( 'nav_menu_attr_title', $menu_item->post_excerpt );
    600599                        $menu_item->description = apply_filters( 'nav_menu_description', $menu_item->post_content );
    601                         $menu_item->classes = '';
     600                        $menu_item->classes = array();
    602601                        $menu_item->xfn = '';
    603602                }
    604603        } elseif ( isset( $menu_item->taxonomy ) ) {
     
    618617                $menu_item->target = '';
    619618                $menu_item->attr_title = '';
    620619                $menu_item->description = get_term_field( 'description', $menu_item->term_id, $menu_item->taxonomy );
    621                 $menu_item->classes = '';
     620                $menu_item->classes = array();
    622621                $menu_item->xfn = '';
    623622
    624623        }
  • wp-admin/includes/nav-menu.php

     
    132132                                <p class="field-css-classes description description-thin">
    133133                                        <label for="edit-menu-item-classes-<?php echo $item_id; ?>">
    134134                                                <?php _e( 'CSS Classes (optional)' ); ?><br />
    135                                                 <input type="text" id="edit-menu-item-classes-<?php echo $item_id; ?>" class="widefat code edit-menu-item-classes" name="menu-item-classes[<?php echo $item_id; ?>]" value="<?php echo esc_attr( $item->classes ); ?>" />
     135                                                <input type="text" id="edit-menu-item-classes-<?php echo $item_id; ?>" class="widefat code edit-menu-item-classes" name="menu-item-classes[<?php echo $item_id; ?>]" value="<?php echo esc_attr( implode(' ', $item->classes ) ); ?>" />
    136136                                        </label>
    137137                                </p>
    138138                                <p class="field-xfn description description-thin">