Ticket #11817: 11817-added-comma-removal.diff

File 11817-added-comma-removal.diff, 7.5 KB (added by markmcwilliams, 3 years ago)

Removes a stray comma ('after_link' => ,) that made it's way into the end of the default array() for wp_nav_menu(). Plus a little couple of cleanups, as in whitespace, which I think have been done properly and as they should? :) Main thing was to just remove the stray commas!

  • nav-menu-template.php

     
    2424 */ 
    2525function wp_nav_menu( $args = array() ) { 
    2626        $defaults = array( 'menu' => '', 'menu_class' => 'menu', 'format' => 'div', 'echo' => true, 
    27         'fallback_cb' => 'wp_page_menu', 'link_before' => '', 'link_after' => '', 'before_link' => '', 'after_link' => '', ); 
     27        'fallback_cb' => 'wp_page_menu', 'link_before' => '', 'link_after' => '', 'before_link' => '', 'after_link' => '' ); 
    2828         
    2929        $args = wp_parse_args( $args, $defaults ); 
    3030        $args = apply_filters( 'wp_nav_menu_args', $args ); 
    … …  
    3838        if ( !$menu ) { 
    3939                $menus = wp_get_nav_menus(); 
    4040                foreach ( $menus as $menu_maybe ) { 
    41                         if ( wp_get_nav_menu_items($menu_maybe->term_id) ) { 
     41                        if ( wp_get_nav_menu_items( $menu_maybe->term_id ) ) { 
    4242                                $menu = $menu_maybe; 
    4343                                break; 
    4444                        } 
    … …  
    7373 **/ 
    7474function wp_get_nav_menu( $args = array() ) { 
    7575        $defaults = array( 'menu' => '', 'menu_class' => 'menu', 'ul_class' => '', 'format' => 'div', 'type' => 'frontend', 
    76         'fallback_cb' => '', 'link_before' => '', 'link_after' => '', 'before_link' => '', 'after_link' => '', ); 
     76        'fallback_cb' => '', 'link_before' => '', 'link_after' => '', 'before_link' => '', 'after_link' => '' ); 
    7777         
    7878        $args = wp_parse_args( $args, $defaults ); 
    7979        $args = apply_filters( 'wp_get_nav_menu_args', $args ); 
    … …  
    8282        $menu = wp_get_nav_menu_object( $args->menu ); 
    8383         
    8484        // If no menu was found, call the fallback_cb 
    85         if ( !$menu || is_wp_error($menu) ) { 
    86                 if ( function_exists($args->fallback_cb) ) 
     85        if ( !$menu || is_wp_error( $menu ) ) { 
     86                if ( function_exists( $args->fallback_cb ) ) 
    8787                        return call_user_func( $args->fallback_cb, $args ); 
    8888        } 
    8989         
    … …  
    9797        foreach ( $menu_items as $menu_item ) 
    9898                $parent_menu_order[ $menu_item->ID ] = $menu_item->menu_order; 
    9999         
    100         $ul_class = isset($args->ul_class) ? ' class="'. $args->ul_class .'"' : ''; 
     100        $ul_class = isset( $args->ul_class ) ? ' class="'. $args->ul_class .'"' : ''; 
    101101        $nav_menu .= '<ul'. $ul_class .'>'; 
    102102         
    103103        // Display Loop 
    … …  
    116116                if ( $last_item || $current_parent != $menu_items[ $key + 1 ]->post_parent ) { 
    117117                        if ( $last_item || in_array( $menu_items[ $key + 1 ]->post_parent, $parent_stack ) ) { 
    118118                                $nav_menu .= '</li>'; 
    119                                 while ( !empty( $parent_stack ) && ($last_item || $menu_items[ $key + 1 ]->post_parent != $current_parent ) ) { 
     119                                while ( !empty( $parent_stack ) && ( $last_item || $menu_items[ $key + 1 ]->post_parent != $current_parent ) ) { 
    120120                                        $nav_menu .= '</ul></li>'; 
    121121                                        $current_parent = array_pop( $parent_stack ); 
    122122                                } 
    … …  
    149149        $item = ''; 
    150150        switch ( $context ) { 
    151151                case 'frontend': 
    152                         $attr_title = ( isset($menu_item->anchor_title) && '' != $menu_item->anchor_title ) ? ' title="'. esc_attr($menu_item->anchor_title) .'"' : ''; 
     152                        $attr_title = ( isset( $menu_item->anchor_title ) && '' != $menu_item->anchor_title ) ? ' title="'. esc_attr( $menu_item->anchor_title ) .'"' : ''; 
    153153                        $href = isset($menu_item->link) ? ' href="'. esc_url($menu_item->link) .'"' : ''; 
    154154                         
    155155                        $item .= '<a'. $attr_title . $href . $menu_item->target .'>'; 
    … …  
    160160                 
    161161                case 'backend': 
    162162                        $item .= '<dl><dt>'; 
    163                         $item .= '<span class="item-title">'. esc_html($menu_item->title) .'</span>'; 
     163                        $item .= '<span class="item-title">'. esc_html( $menu_item->title ) .'</span>'; 
    164164                        $item .= '<span class="item-controls">'; 
    165                         $item .= '<span class="item-type">'. esc_html($menu_item->type) .'</span>'; 
     165                        $item .= '<span class="item-type">'. esc_html( $menu_item->type ) .'</span>'; 
    166166                         
    167167                        // Actions 
    168168                        $item .= '<a class="item-edit thickbox" id="edit'. esc_attr( $menu_item->menu_order ) .'" value="'. esc_attr( $menu_item->menu_order ) .'" title="'. __('Edit Menu Item') .'" href="#TB_inline?height=380&width=300&inlineId=menu-item-settings">'. __('Edit') .'</a> | '; 
    … …  
    185185                         
    186186                case 'default': 
    187187                        $menu_id = 'menu-item-' . $menu_item->ID; 
    188                         $item .= '<label class="item-title"><input type="checkbox" id="'. esc_attr($menu_id) .'" name="'. esc_attr( $menu_item->title ) .'" value="'. esc_attr( $menu_item->link ) .'" />'. $menu_item->title .'</label>'; 
     188                        $item .= '<label class="item-title"><input type="checkbox" id="'. esc_attr( $menu_id ) .'" name="'. esc_attr( $menu_item->title ) .'" value="'. esc_attr( $menu_item->link ) .'" />'. $menu_item->title .'</label>'; 
    189189                         
    190190                        // Menu Item Settings 
    191191                        $item .= '<input type="hidden" class="item-type" value="'. esc_attr( $menu_item->append ) .'" />'; 

  • nav-menu.php

     
    7777        else 
    7878                $slug = $menu_name; 
    7979 
    80         $menu = wp_insert_term( $menu_name, 'nav_menu', array('slug' => $slug) ); 
     80        $menu = wp_insert_term( $menu_name, 'nav_menu', array( 'slug' => $slug ) ); 
    8181 
    8282        if ( is_wp_error($menu) ) 
    8383                return $menu; 
    … …  
    9595 */ 
    9696function wp_delete_nav_menu( $menu ) { 
    9797        $menu = wp_get_nav_menu_object( $menu ); 
    98         if ( !$menu  ) 
     98        if ( !$menu ) 
    9999                return false; 
    100100 
    101101        $menu_objects = get_objects_in_term( $menu->term_id, 'nav_menu' ); 
    … …  
    116116 * @since 3.0.0 
    117117 */ 
    118118function wp_create_default_nav_menu() { 
    119         $menu = wp_create_nav_menu( __('Menu 1'), array('slug' => 'menu-1') ); 
     119        $menu = wp_create_nav_menu( __('Menu 1'), array( 'slug' => 'menu-1' ) ); 
    120120 
    121121        if ( is_wp_error($menu) ) 
    122122                return; 
    123123 
    124         $pages = get_pages( array('parent' => 0, 'number' => 15) ); 
     124        $pages = get_pages( array( 'parent' => 0, 'number' => 15 ) ); 
    125125        $counter = 1; 
    126126        foreach ( $pages as $page ) { 
    127                 $item = array( 'post_status' => 'publish', 'post_type' => 'nav_menu_item', 'menu_order' => $counter, 'tax_input' => array( 'nav_menu' => $menu->name), 'post_title' => addslashes($page->post_title) ); 
     127                $item = array( 'post_status' => 'publish', 'post_type' => 'nav_menu_item', 'menu_order' => $counter, 'tax_input' => array( 'nav_menu' => $menu->name ), 'post_title' => addslashes( $page->post_title ) ); 
    128128                $item_id = wp_insert_post($item, true); 
    129129                update_post_meta( $item_id, 'menu_type', 'page' ); 
    130130                update_post_meta( $item_id, 'object_id', $page->ID ); 
    … …  
    225225                case 'category' : 
    226226                        $menu_item->link = get_category_link( $menu_item->object_id ); 
    227227 
    228                         if ( empty($menu_item->post_title) ) { 
     228                        if ( empty( $menu_item->post_title ) ) { 
    229229                                $title_raw = get_category( $menu_item->object_id ); 
    230230                                $menu_item->title =  $title_raw->cat_name; 
    231231                        } else { 
    232232                                $menu_item->title = $menu_item->post_title; 
    233233                        } 
    234234 
    235                         if ( empty($menu_item->post_content) ) 
     235                        if ( empty( $menu_item->post_content ) ) 
    236236                                $menu_item->description = strip_tags( category_description( $menu_item->object_id ) ); 
    237237                        else 
    238238                                $menu_item->description = $menu_item->post_content; 
    … …  
    257257/* @todo: update to use tax/post data 
    258258 
    259259                        //SET anchor title 
    260                         if (isset($wp_custom_nav_menu_items->custom_anchor_title)) { 
     260                        if ( isset( $wp_custom_nav_menu_items->custom_anchor_title ) ) { 
    261261                                $anchor_title = $wp_custom_nav_menu_items->custom_anchor_title; 
    262262                        } 
    263263                        else { 
    264264                                $anchor_title = $title; 
    265265                        } 
    266266 
    267                         if ($queried_id == $wp_custom_nav_menu_items->post_id) { 
     267                        if ( $queried_id == $wp_custom_nav_menu_items->post_id ) { 
    268268                                $li_class = 'class="current_page_item"'; 
    269269                        } 
    270270 
    271                         if (isset($wp_custom_nav_menu_items->new_window)) { 
    272                                 if ($wp_custom_nav_menu_items->new_window > 0) { 
     271                        if ( isset( $wp_custom_nav_menu_items->new_window ) ) { 
     272                                if ( $wp_custom_nav_menu_items->new_window > 0 ) { 
    273273                                        $target = 'target="_blank"'; 
    274274                                } 
    275275                                else {