WordPress.org

Make WordPress Core

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

File 11817-added-comma-removal.diff, 7.5 KB (added by markmcwilliams, 11 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 {