Ticket #48409: 48409.2.diff

src/wp-admin/includes/class-walker-nav-menu-checklist.php

@@ -62 +62 @@
          *
          * @since 3.0.0
          *
-         * @global int $_nav_menu_placeholder
+         * @global int        $_nav_menu_placeholder
+         * @global int|string $nav_menu_selected_id
          *
          * @param string $output Used to append additional content (passed by reference).
          * @param object $item   Menu item data object.
@@ -71 +72 @@
          * @param int    $id     Not used.
          */
         public function start_el( &$output, $item, $depth = 0, $args = array(), $id = 0 ) {
-                global $_nav_menu_placeholder;
+                global $_nav_menu_placeholder, $nav_menu_selected_id;
 
                 $_nav_menu_placeholder = ( 0 > $_nav_menu_placeholder ) ? intval( $_nav_menu_placeholder ) - 1 : -1;
                 $possible_object_id    = isset( $item->post_type ) && 'nav_menu_item' == $item->post_type ? $item->object_id : $_nav_menu_placeholder;
@@ -81 +82 @@
 
                 $output .= $indent . '<li>';
                 $output .= '<label class="menu-item-title">';
-                $output .= '<input type="checkbox" class="menu-item-checkbox';
+                $output .= '<input type="checkbox"' . wp_nav_menu_disabled_check( $nav_menu_selected_id, false ) . ' class="menu-item-checkbox';
 
                 if ( ! empty( $item->front_or_home ) ) {
                         $output .= ' add-to-top';

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

@@ -255 +255 @@
 }
 
 /**
- * Check whether to disable the Menu Locations meta box submit button
+ * Check whether to disable the Menu Locations meta box submit button and inputs.
  *
  * @since 3.6.0
+ * @since 5.3.1 The `$echo` parameter was added.
  *
  * @global bool $one_theme_location_no_menus to determine if no menus exist
  *
- * @param int|string $nav_menu_selected_id (id, name or slug) of the currently-selected menu
- * @return string Disabled attribute if at least one menu exists, false if not
+ * @param int|string $nav_menu_selected_id ID, name, or slug of the currently selected menu.
+ * @param bool       $echo                 Whether to echo or just return the string.
+ * @return string|false Disabled attribute if at least one menu exists, false if not.
  */
-function wp_nav_menu_disabled_check( $nav_menu_selected_id ) {
+function wp_nav_menu_disabled_check( $nav_menu_selected_id, $echo = true ) {
         global $one_theme_location_no_menus;
 
         if ( $one_theme_location_no_menus ) {
@@ -271 +273 @@
                 return false;
         }
 
-        return disabled( $nav_menu_selected_id, 0 );
+        return disabled( $nav_menu_selected_id, 0, $echo );
 }
 
 /**
@@ -292 +294 @@
                 <input type="hidden" value="custom" name="menu-item[<?php echo $_nav_menu_placeholder; ?>][menu-item-type]" />
                 <p id="menu-item-url-wrap" class="wp-clearfix">
                         <label class="howto" for="custom-menu-item-url"><?php _e( 'URL' ); ?></label>
-                        <input id="custom-menu-item-url" name="menu-item[<?php echo $_nav_menu_placeholder; ?>][menu-item-url]" type="text" class="code menu-item-textbox" placeholder="https://" />
+                        <input id="custom-menu-item-url" name="menu-item[<?php echo $_nav_menu_placeholder; ?>][menu-item-url]" type="text"<?php wp_nav_menu_disabled_check( $nav_menu_selected_id ); ?> class="code menu-item-textbox" placeholder="https://" />
                 </p>
 
                 <p id="menu-item-name-wrap" class="wp-clearfix">
                         <label class="howto" for="custom-menu-item-name"><?php _e( 'Link Text' ); ?></label>
-                        <input id="custom-menu-item-name" name="menu-item[<?php echo $_nav_menu_placeholder; ?>][menu-item-title]" type="text" class="regular-text menu-item-textbox" />
+                        <input id="custom-menu-item-name" name="menu-item[<?php echo $_nav_menu_placeholder; ?>][menu-item-title]" type="text"<?php wp_nav_menu_disabled_check( $nav_menu_selected_id ); ?> class="regular-text menu-item-textbox" />
                 </p>
 
                 <p class="button-controls wp-clearfix">
@@ -569 +571 @@
                         ?>
                         <p class="quick-search-wrap">
                                 <label for="quick-search-posttype-<?php echo $post_type_name; ?>" class="screen-reader-text"><?php _e( 'Search' ); ?></label>
-                                <input type="search" class="quick-search" value="<?php echo $searched; ?>" name="quick-search-posttype-<?php echo $post_type_name; ?>" id="quick-search-posttype-<?php echo $post_type_name; ?>" />
+                                <input type="search"<?php wp_nav_menu_disabled_check( $nav_menu_selected_id ); ?> class="quick-search" value="<?php echo $searched; ?>" name="quick-search-posttype-<?php echo $post_type_name; ?>" id="quick-search-posttype-<?php echo $post_type_name; ?>" />
                                 <span class="spinner"></span>
                                 <?php submit_button( __( 'Search' ), 'small quick-search-submit hide-if-js', 'submit', false, array( 'id' => 'submit-quick-search-posttype-' . $post_type_name ) ); ?>
                         </p>
@@ -651 +653 @@
 
                 <p class="button-controls wp-clearfix" data-items-type="posttype-<?php echo esc_attr( $post_type_name ); ?>">
                         <span class="list-controls hide-if-no-js">
-                                <input type="checkbox" id="<?php echo esc_attr( $post_type_name . '-tab' ); ?>" class="select-all" />
+                                <input type="checkbox"<?php wp_nav_menu_disabled_check( $nav_menu_selected_id ); ?> id="<?php echo esc_attr( $post_type_name . '-tab' ); ?>" class="select-all" />
                                 <label for="<?php echo esc_attr( $post_type_name . '-tab' ); ?>"><?php _e( 'Select All' ); ?></label>
                         </span>
 
@@ -880 +882 @@
 
                 <p class="button-controls wp-clearfix" data-items-type="taxonomy-<?php echo esc_attr( $taxonomy_name ); ?>">
                         <span class="list-controls hide-if-no-js">
-                                <input type="checkbox" id="<?php echo esc_attr( $taxonomy_name . '-tab' ); ?>" class="select-all" />
+                                <input type="checkbox"<?php wp_nav_menu_disabled_check( $nav_menu_selected_id ); ?> id="<?php echo esc_attr( $taxonomy_name . '-tab' ); ?>" class="select-all" />
                                 <label for="<?php echo esc_attr( $taxonomy_name . '-tab' ); ?>"><?php _e( 'Select All' ); ?></label>
                         </span>