WordPress.org

Make WordPress Core

Ticket #22189: 22189.3.diff

File 22189.3.diff, 4.3 KB (added by nacin, 19 months ago)

Explanation to follow

  • wp-includes/nav-menu.php

     
    279279        if ( ( ! $menu && 0 !== $menu_id ) || is_wp_error( $menu ) ) 
    280280                return $menu; 
    281281 
    282         $menu_items = 0 == $menu_id ? array() : (array) wp_get_nav_menu_items( $menu_id, array( 'post_status' => 'publish,draft' ) ); 
    283  
    284         $count = count( $menu_items ); 
    285  
    286282        $defaults = array( 
    287283                'menu-item-db-id' => $menu_item_db_id, 
    288284                'menu-item-object-id' => 0, 
     
    305301        if ( 0 == $menu_id ) { 
    306302                $args['menu-item-position'] = 1; 
    307303        } elseif ( 0 == (int) $args['menu-item-position'] ) { 
     304                $menu_items = 0 == $menu_id ? array() : (array) wp_get_nav_menu_items( $menu_id, array( 'post_status' => 'publish,draft' ) ); 
    308305                $last_item = array_pop( $menu_items ); 
    309                 $args['menu-item-position'] = ( $last_item && isset( $last_item->menu_order ) ) ? 1 + $last_item->menu_order : $count; 
     306                $args['menu-item-position'] = ( $last_item && isset( $last_item->menu_order ) ) ? 1 + $last_item->menu_order : count( $menu_items ); 
    310307        } 
    311308 
    312309        $original_parent = 0 < $menu_item_db_id ? get_post_field( 'post_parent', $menu_item_db_id ) : 0; 
     
    350347                'post_type' => 'nav_menu_item', 
    351348        ); 
    352349 
    353         if ( 0 != $menu_id ) 
     350        if ( $menu_id && ! is_object_in_term( $menu_item_db_id, 'nav_menu', (int) $menu->term_id ) ) 
    354351                $post['tax_input'] = array( 'nav_menu' => array( intval( $menu->term_id ) ) ); 
    355352 
     353        $update = 0 != $menu_item_db_id; 
     354 
    356355        // New menu item. Default is draft status 
    357         if ( 0 == $menu_item_db_id ) { 
     356        if ( ! $update ) { 
    358357                $post['ID'] = 0; 
    359358                $post['post_status'] = 'publish' == $args['menu-item-status'] ? 'publish' : 'draft'; 
    360359                $menu_item_db_id = wp_insert_post( $post ); 
    361  
    362         // Update existing menu item. Default is publish status 
    363         } else { 
    364                 $post['ID'] = $menu_item_db_id; 
    365                 $post['post_status'] = 'draft' == $args['menu-item-status'] ? 'draft' : 'publish'; 
    366                 wp_update_post( $post ); 
     360                if ( ! $menu_item_db_id || is_wp_error( $menu_item_db_id ) ) 
     361                        return $menu_item_db_id; 
    367362        } 
    368363 
    369364        if ( 'custom' == $args['menu-item-type'] ) { 
     
    371366                $args['menu-item-object'] = 'custom'; 
    372367        } 
    373368 
    374         if ( ! $menu_item_db_id || is_wp_error( $menu_item_db_id ) ) 
    375                 return $menu_item_db_id; 
    376  
    377369        $menu_item_db_id = (int) $menu_item_db_id; 
    378370 
    379371        update_post_meta( $menu_item_db_id, '_menu_item_type', sanitize_key($args['menu-item-type']) ); 
    380         update_post_meta( $menu_item_db_id, '_menu_item_menu_item_parent', (int) $args['menu-item-parent-id'] ); 
    381         update_post_meta( $menu_item_db_id, '_menu_item_object_id', (int) $args['menu-item-object-id'] ); 
     372        update_post_meta( $menu_item_db_id, '_menu_item_menu_item_parent', strval( (int) $args['menu-item-parent-id'] ) ); 
     373        update_post_meta( $menu_item_db_id, '_menu_item_object_id', strval( (int) $args['menu-item-object-id'] ) ); 
    382374        update_post_meta( $menu_item_db_id, '_menu_item_object', sanitize_key($args['menu-item-object']) ); 
    383375        update_post_meta( $menu_item_db_id, '_menu_item_target', sanitize_key($args['menu-item-target']) ); 
    384376 
     
    389381        update_post_meta( $menu_item_db_id, '_menu_item_url', esc_url_raw($args['menu-item-url']) ); 
    390382 
    391383        if ( 0 == $menu_id ) 
    392                 update_post_meta( $menu_item_db_id, '_menu_item_orphaned', time() ); 
    393         else 
     384                update_post_meta( $menu_item_db_id, '_menu_item_orphaned', (string) time() ); 
     385        elseif ( get_post_meta( $menu_item_db_id, '_menu_item_orphaned' ) ) 
    394386                delete_post_meta( $menu_item_db_id, '_menu_item_orphaned' ); 
    395387 
     388        // Update existing menu item. Default is publish status 
     389        if ( $update ) { 
     390                $post['ID'] = $menu_item_db_id; 
     391                $post['post_status'] = 'draft' == $args['menu-item-status'] ? 'draft' : 'publish'; 
     392                wp_update_post( $post ); 
     393        } 
     394 
    396395        do_action('wp_update_nav_menu_item', $menu_id, $menu_item_db_id, $args ); 
    397396 
    398397        return $menu_item_db_id; 
     
    483482                return $items; 
    484483 
    485484        $defaults = array( 'order' => 'ASC', 'orderby' => 'menu_order', 'post_type' => 'nav_menu_item', 
    486                 'post_status' => 'publish', 'output' => ARRAY_A, 'output_key' => 'menu_order', 'nopaging' => true, 
    487                 'update_post_term_cache' => false ); 
     485                'post_status' => 'publish', 'output' => ARRAY_A, 'output_key' => 'menu_order', 'nopaging' => true ); 
    488486        $args = wp_parse_args( $args, $defaults ); 
    489487        if ( count( $items ) > 1 ) 
    490488                $args['include'] = implode( ',', $items );