WordPress.org

Make WordPress Core

Ticket #13174: handle_menu_items_for_trashed_items.13174.diff

File handle_menu_items_for_trashed_items.13174.diff, 12.5 KB (added by filosofo, 5 years ago)
  • wp-includes/default-filters.php

     
    224224add_action( 'save_post',                  '_save_post_hook',          5, 2 ); 
    225225add_action( 'transition_post_status',     '_transition_post_status',  5, 3 ); 
    226226add_action( 'comment_form', 'wp_comment_form_unfiltered_html_nonce'        ); 
    227 add_action( 'wp_scheduled_delete',        'wp_scheduled_delete' ); 
     227add_action( 'wp_scheduled_delete',        'wp_scheduled_delete'            ); 
     228add_action( 'trash_post',                 '_wp_trash_menu_item'            ); 
     229add_action( 'untrash_post',               '_wp_untrash_menu_item'          ); 
     230add_action( 'delete_post',                '_wp_delete_post_menu_item'      ); 
     231add_action( 'delete_term',                '_wp_delete_tax_menu_item'       ); 
    228232 
    229233// Post Thumbnail CSS class filtering 
    230234add_action( 'begin_fetch_post_thumbnail_html', '_wp_post_thumbnail_class_filter_add'    ); 
  • wp-includes/post.php

     
    21672167                } elseif ( !empty($post_parent) ) { 
    21682168                        $parent_post = get_post($post_parent); 
    21692169                        // Check for circular dependency 
    2170                         if ( $parent_post->post_parent == $post_ID ) 
     2170                        if ( isset( $parent_post->post_parent ) && $parent_post->post_parent == $post_ID ) 
    21712171                                $post_parent = 0; 
    21722172                } 
    21732173        } 
  • wp-includes/nav-menu.php

     
    268268                if ( 'taxonomy' == $args['menu-item-type'] ) { 
    269269                        $original_title = get_term_field( 'name', $args['menu-item-object-id'], $args['menu-item-object'], 'raw' ); 
    270270                } elseif ( 'post_type' == $args['menu-item-type'] ) { 
     271 
    271272                        $original_object = get_post( $args['menu-item-object-id'] ); 
    272273                        $original_title = $original_object->post_title; 
     274 
     275                        if ( 'trash' == get_post_status( $args['menu-item-object-id'] ) ) { 
     276                                $post_type_object = get_post_type_object( $args['menu-item-object'] ); 
     277                                if ( isset( $post_type_object->singular_label ) ) 
     278                                        return new WP_Error('update_nav_menu_item_failed', sprintf(__('The menu item "%1$s" belongs to a %2$s that is in the trash, so it cannot be updated.'), $args['menu-item-title'], $post_type_object->singular_label ) ); 
     279                                else 
     280                                        return new WP_Error('update_nav_menu_item_failed', sprintf(__('The menu item "%1$s" belongs to something that is in the trash, so it cannot be updated.'), $args['menu-item-title'] ) ); 
     281                        } 
    273282                } 
    274283 
    275284                if ( empty( $args['menu-item-title'] ) || $args['menu-item-title'] == $original_title ) { 
     
    528537 
    529538        return apply_filters( 'wp_setup_nav_menu_item', $menu_item ); 
    530539} 
     540 
     541/** 
     542 * Get the menu items associated with a particular object. 
     543 * 
     544 * @since 3.0.0 
     545 *  
     546 * @param int $object_id The ID of the original object. 
     547 * @param string $object_type The type of object, such as "taxonomy" or "post_type." 
     548 * @return array The array of menu item IDs; empty array if none; 
     549 */ 
     550function wp_get_associated_nav_menu_items( $object_id = 0, $object_type = 'post_type' ) { 
     551        $object_id = (int) $object_id; 
     552        $menu_item_ids = array(); 
     553 
     554        $query = new WP_Query; 
     555        $menu_items = $query->query( 
     556                array( 
     557                        'meta_key' => '_menu_item_object_id', 
     558                        'meta_value' => $object_id, 
     559                        'post_status' => 'any', 
     560                        'post_type' => 'nav_menu_item', 
     561                        'showposts' => -1, 
     562                ) 
     563        ); 
     564        foreach( (array) $menu_items as $menu_item ) { 
     565                if ( isset( $menu_item->ID ) && is_nav_menu_item( $menu_item->ID ) ) { 
     566                        if ( get_post_meta($menu_item->ID, '_menu_item_type', true) != $object_type ) 
     567                                continue; 
     568 
     569                        $menu_item_ids[] = (int) $menu_item->ID; 
     570                } 
     571        } 
     572 
     573        return array_unique( $menu_item_ids ); 
     574} 
     575 
     576/** 
     577 * Callback for handling a menu item when its original object is trashed. 
     578 * 
     579 * @since 3.0.0 
     580 * @access private 
     581 * 
     582 * @param int $object_id The ID of the original object being trashed. 
     583 * 
     584 */ 
     585function _wp_trash_menu_item( $object_id = 0 ) { 
     586        $object_id = (int) $object_id; 
     587 
     588        $menu_item_ids = wp_get_associated_nav_menu_items( $object_id ); 
     589 
     590        foreach( (array) $menu_item_ids as $menu_item_id ) { 
     591                $menu_item = get_post( $menu_item_id, ARRAY_A ); 
     592                $menu_item['post_status'] = 'draft'; 
     593                wp_insert_post($menu_item); 
     594        } 
     595} 
     596 
     597/** 
     598 * Callback for handling a menu item when its original object is un-trashed. 
     599 * 
     600 * @since 3.0.0 
     601 * @access private 
     602 * 
     603 * @param int $object_id The ID of the original object being untrashed. 
     604 * 
     605 */ 
     606function _wp_untrash_menu_item( $object_id = 0 ) { 
     607        $object_id = (int) $object_id; 
     608 
     609        $menu_item_ids = wp_get_associated_nav_menu_items( $object_id ); 
     610 
     611        foreach( (array) $menu_item_ids as $menu_item_id ) { 
     612                $menu_item = get_post( $menu_item_id, ARRAY_A ); 
     613                $menu_item['post_status'] = 'publish'; 
     614                wp_insert_post($menu_item); 
     615        } 
     616} 
     617 
     618/** 
     619 * Callback for handling a menu item when its original object is deleted. 
     620 * 
     621 * @since 3.0.0 
     622 * @access private 
     623 * 
     624 * @param int $object_id The ID of the original object being trashed. 
     625 * 
     626 */ 
     627function _wp_delete_post_menu_item( $object_id = 0 ) { 
     628        $object_id = (int) $object_id; 
     629 
     630        $menu_item_ids = wp_get_associated_nav_menu_items( $object_id, 'post_type' ); 
     631 
     632        foreach( (array) $menu_item_ids as $menu_item_id ) { 
     633                wp_delete_post( $menu_item_id, true ); 
     634        } 
     635} 
     636 
     637/** 
     638 * Callback for handling a menu item when its original object is deleted. 
     639 * 
     640 * @since 3.0.0 
     641 * @access private 
     642 * 
     643 * @param int $object_id The ID of the original object being trashed. 
     644 * 
     645 */ 
     646function _wp_delete_tax_menu_item( $object_id = 0 ) { 
     647        $object_id = (int) $object_id; 
     648 
     649        $menu_item_ids = wp_get_associated_nav_menu_items( $object_id, 'taxonomy' ); 
     650 
     651        foreach( (array) $menu_item_ids as $menu_item_id ) { 
     652                wp_delete_post( $menu_item_id, true ); 
     653        } 
     654} 
     655 
    531656?> 
  • wp-admin/admin-ajax.php

     
    827827        $menu_id = (int) $_POST['menu']; 
    828828        if ( isset( $_POST['menu-item'] ) ) { 
    829829                $item_ids = wp_save_nav_menu_item( $menu_id, $_POST['menu-item'] ); 
     830                if ( is_wp_error( $item_ids ) )  
     831                        die('-1'); 
    830832        } else { 
    831833                $item_ids = array(); 
    832834        } 
  • wp-admin/includes/nav-menu.php

     
    805805 * 
    806806 * @param int $menu_id The menu ID for which to save this item. 
    807807 * @param array $menu_data The unsanitized posted menu item data. 
    808  * @return array The database IDs of the items saved. 
     808 * @return array The database IDs of the items saved 
    809809 */ 
    810810function wp_save_nav_menu_item( $menu_id = 0, $menu_data = array() ) { 
    811811        $menu_id = (int) $menu_id; 
  • wp-admin/nav-menus.php

     
    3838wp_enqueue_script( 'postbox' ); 
    3939 
    4040// Container for any messages displayed to the user 
    41 $messages_div = ''; 
     41$messages = array(); 
    4242 
    4343// Container that stores the name of the active menu 
    4444$nav_menu_selected_title = ''; 
     
    169169                if ( is_nav_menu_item( $menu_item_id ) ) { 
    170170                        if ( wp_delete_post( $menu_item_id, true ) ) { 
    171171                                 
    172                                 $messages_div = '<div id="message" class="updated"><p>' . __('The menu item has been successfully deleted.') . '</p></div>'; 
     172                                $messages[] = '<div id="message" class="updated"><p>' . __('The menu item has been successfully deleted.') . '</p></div>'; 
    173173                        } 
    174174                } 
    175175                break; 
     
    180180                        $delete_nav_menu = wp_delete_nav_menu( $nav_menu_selected_id ); 
    181181 
    182182                        if ( is_wp_error($delete_nav_menu) ) { 
    183                                 $messages_div = '<div id="message" class="error"><p>' . $delete_nav_menu->get_error_message() . '</p></div>'; 
     183                                $messages[] = '<div id="message" class="error"><p>' . $delete_nav_menu->get_error_message() . '</p></div>'; 
    184184                        } else { 
    185                                 $messages_div = '<div id="message" class="updated"><p>' . __('The menu has been successfully deleted.') . '</p></div>'; 
     185                                $messages[] = '<div id="message" class="updated"><p>' . __('The menu has been successfully deleted.') . '</p></div>'; 
    186186                                $nav_menu_selected_id = 0; // Reset the selected menu 
    187187                        } 
    188188                        unset( $delete_nav_menu ); 
     
    201201                                        $_nav_menu_selected_id = wp_update_nav_menu_object( 0, array('menu-name' => $new_menu_title) ); 
    202202 
    203203                                        if ( is_wp_error( $_nav_menu_selected_id ) ) { 
    204                                                 $messages_div = '<div id="message" class="error"><p>' . $_nav_menu_selected_id->get_error_message() . '</p></div>'; 
     204                                                $messages[] = '<div id="message" class="error"><p>' . $_nav_menu_selected_id->get_error_message() . '</p></div>'; 
    205205                                        } else { 
    206206                                                $_menu_object = wp_get_nav_menu_object( $_nav_menu_selected_id ); 
    207207                                                $nav_menu_selected_id = $_nav_menu_selected_id; 
    208208                                                $nav_menu_selected_title = $_menu_object->name; 
    209                                                 $messages_div = '<div id="message" class="updated"><p>' . sprintf( __('The <strong>%s</strong> menu has been successfully created.'), $nav_menu_selected_title ) . '</p></div>'; 
     209                                                $messages[] = '<div id="message" class="updated"><p>' . sprintf( __('The <strong>%s</strong> menu has been successfully created.'), $nav_menu_selected_title ) . '</p></div>'; 
    210210                                        } 
    211211                                } else { 
    212                                         $messages_div = '<div id="message" class="error"><p>' . __('Please enter a valid menu name.') . '</p></div>'; 
     212                                        $messages[] = '<div id="message" class="error"><p>' . __('Please enter a valid menu name.') . '</p></div>'; 
    213213                                } 
    214214                        } 
    215215 
     
    222222                                $_nav_menu_selected_id = wp_update_nav_menu_object( $nav_menu_selected_id, array( 'menu-name' => $_POST['menu-name'] ) ); 
    223223                                if ( is_wp_error( $_nav_menu_selected_id ) ) { 
    224224                                        $_menu_object = $_nav_menu_selected_id; 
    225                                         $messages_div = '<div id="message" class="error"><p>' . $_nav_menu_selected_id->get_error_message() . '</p></div>'; 
     225                                        $messages[] = '<div id="message" class="error"><p>' . $_nav_menu_selected_id->get_error_message() . '</p></div>'; 
    226226                                } else { 
    227227                                        $_menu_object = wp_get_nav_menu_object( $_nav_menu_selected_id ); 
    228228                                        $nav_menu_selected_title = $_menu_object->name; 
     
    248248 
    249249                                                $menu_item_db_id = wp_update_nav_menu_item( $nav_menu_selected_id, ( $_POST['menu-item-db-id'][$_key] != $_key ? 0 : $_key ), $args ); 
    250250 
    251                                                 if ( ! is_wp_error( $menu_item_db_id ) && isset( $menu_items[$menu_item_db_id] ) ) { 
     251                                                if ( is_wp_error( $menu_item_db_id ) )  
     252                                                        $messages[] = '<div id="message" class="error"><p>' . $menu_item_db_id->get_error_message() . '</p></div>'; 
     253                                                elseif ( isset( $menu_items[$menu_item_db_id] ) ) 
    252254                                                        unset( $menu_items[$menu_item_db_id] ); 
    253                                                 } 
    254255                                        } 
    255256                                } 
    256257 
     
    265266 
    266267                                do_action( 'wp_update_nav_menu', $nav_menu_selected_id ); 
    267268 
    268                                 $messages_div = '<div id="message" class="updated"><p>' . sprintf( __('The <strong>%s</strong> menu has been updated.'), $nav_menu_selected_title ) . '</p></div>'; 
     269                                $messages[] = '<div id="message" class="updated"><p>' . sprintf( __('The <strong>%s</strong> menu has been updated.'), $nav_menu_selected_title ) . '</p></div>'; 
    269270                                unset( $menu_items ); 
    270271                        } 
    271272                } 
     
    310311} elseif ( current_theme_supports('widgets') ) { 
    311312        // Set up nav menu 
    312313        wp_nav_menu_setup(); 
    313         $messages_div = '<div id="message" class="error"><p>' . __('The current theme does not natively support menus, but you can use the &#8220;Navigation Menu&#8221; widget to add any menus you create here to the theme&#8217;s sidebar.') . '</p></div>'; 
     314        $messages[] = '<div id="message" class="error"><p>' . __('The current theme does not natively support menus, but you can use the &#8220;Navigation Menu&#8221; widget to add any menus you create here to the theme&#8217;s sidebar.') . '</p></div>'; 
    314315 
    315316// The theme supports neither menus nor widgets. 
    316317} else { 
    317318        remove_meta_box( 'create-menu', 'nav-menus', 'side' ); 
    318         $messages_div = '<div id="message" class="error"><p>' . __('The current theme does not support menus.') . '</p></div>'; 
     319        $messages[] = '<div id="message" class="error"><p>' . __('The current theme does not support menus.') . '</p></div>'; 
    319320} 
    320321 
    321322// Get the admin header 
     
    324325<div class="wrap"> 
    325326        <?php screen_icon(); ?> 
    326327        <h2><?php esc_html_e('Menus'); ?></h2> 
    327         <?php echo $messages_div; ?> 
     328        <?php  
     329        foreach( $messages as $message ) : 
     330                echo $message . "\n";  
     331        endforeach; 
     332        ?> 
    328333         
    329334        <?php if ( current_theme_supports('nav-menus') || current_theme_supports('widgets') ) : ?> 
    330335        <div id="nav-menus-frame">