Changeset 51690

Timestamp:

08/30/2021 01:38:26 PM (3 years ago)

Author:

desrosj

Message:

Menus: Save display locations for new menus.

Fixes a bug preventing the locations saving when creating a new menu. Introduced in [50938].

Props dlh, andraganescu, mukesh27, desrosj, peterwilsoncc.
Merges [51688] to the 5.8 branch.
Fixes #53877.

Location:

branches/5.8

Files:

• . (modified) (1 prop)
• src/wp-admin/nav-menus.php (modified) (2 diffs)

branches/5.8/src/wp-admin/nav-menus.php

@@ -317 +317 @@
 
         // Merge new and existing menu locations if any new ones are set.
+        $new_menu_locations = array();
         if ( isset( $_POST['menu-locations'] ) ) {
             $new_menu_locations = array_map( 'absint', $_POST['menu-locations'] );
@@ -352 +353 @@
                                 $locations[ $location ] = $nav_menu_selected_id;
                                 break; // There should only be 1.
+                        }
+
+                        set_theme_mod( 'nav_menu_locations', $locations );
+                    } elseif ( count( $new_menu_locations ) > 0 ) {
+                        // If locations have been selected for the new menu, save those.
+                        $locations = get_nav_menu_locations();
+
+                        foreach ( array_keys( $new_menu_locations ) as $location ) {
+                            $locations[ $location ] = $nav_menu_selected_id;
                         }