Changeset 58854 for trunk/src/wp-includes/nav-menu.php

Timestamp:

08/05/2024 06:58:49 PM (18 months ago)

Author:

SergeyBiryukov

Message:

Menus: Check if taxonomy term exists in wp_update_nav_menu_item().

When inserting a term from a non-existing taxonomy as a nav item, the post_title property should be empty, and the function should not throw a fatal error for wp_specialchars_decode().

Includes bringing some consistency to similar checks for post types and post type archives in the same code fragment.

Follow-up to [14283], [14450], [35382], [36095].

Props dd32, narenin, mukesh27, SergeyBiryukov.
Fixes #61799.

File:

• trunk/src/wp-includes/nav-menu.php (modified) (1 diff)

trunk/src/wp-includes/nav-menu.php

@@ -492 +492 @@
 
         $original_title = '';
+
         if ( 'taxonomy' === $args['menu-item-type'] ) {
-            $original_parent = get_term_field( 'parent', $args['menu-item-object-id'], $args['menu-item-object'], 'raw' );
-            $original_title  = get_term_field( 'name', $args['menu-item-object-id'], $args['menu-item-object'], 'raw' );
+            $original_object = get_term( $args['menu-item-object-id'], $args['menu-item-object'] );
+
+            if ( $original_object instanceof WP_Term ) {
+                $original_parent = get_term_field( 'parent', $args['menu-item-object-id'], $args['menu-item-object'], 'raw' );
+                $original_title  = get_term_field( 'name', $args['menu-item-object-id'], $args['menu-item-object'], 'raw' );
+            }
         } elseif ( 'post_type' === $args['menu-item-type'] ) {
-
             $original_object = get_post( $args['menu-item-object-id'] );
-            $original_parent = (int) $original_object->post_parent;
-            $original_title  = $original_object->post_title;
+
+            if ( $original_object instanceof WP_Post ) {
+                $original_parent = (int) $original_object->post_parent;
+                $original_title  = $original_object->post_title;
+            }
         } elseif ( 'post_type_archive' === $args['menu-item-type'] ) {
             $original_object = get_post_type_object( $args['menu-item-object'] );
-            if ( $original_object ) {
+
+            if ( $original_object instanceof WP_Post_Type ) {
                 $original_title = $original_object->labels->archives;
             }