WordPress.org

Make WordPress Core

Ticket #44286: 44286.2.diff

File 44286.2.diff, 4.2 KB (added by garrett-eclipse, 15 months ago)

Refreshed patch and addressed setting 0 to nav_menu_locations theme mod which caused a default location to be checked when creating a menu.

  • src/wp-admin/css/nav-menus.css

     
    6565        position: relative;
    6666}
    6767
    68 .blank-slate .menu-settings {
    69         border: none;
    70         margin-top: 0;
    71         padding-top: 0;
    72         overflow: hidden;
    73 }
    74 
    7568.is-submenu {
    7669        color: #555d66; /* #fafafa background */
    7770        font-style: italic;
     
    749742        float: right;
    750743}
    751744
    752 .nav-menus-php .blank-slate .menu-settings {
    753         display: none;
    754 }
    755 
    756745/* Same as the Publish Meta Box #delete-action */
    757746.nav-menus-php .delete-action {
    758747        float: left;
  • src/wp-admin/nav-menus.php

     
    300300        case 'update':
    301301                check_admin_referer( 'update-nav_menu', 'update-nav-menu-nonce' );
    302302
    303                 // Remove menu locations that have been unchecked.
    304                 foreach ( $locations as $location => $description ) {
    305                         if ( ( empty( $_POST['menu-locations'] ) || empty( $_POST['menu-locations'][ $location ] ) ) && isset( $menu_locations[ $location ] ) && $menu_locations[ $location ] == $nav_menu_selected_id ) {
    306                                 unset( $menu_locations[ $location ] );
    307                         }
    308                 }
    309 
    310303                // Merge new and existing menu locations if any new ones are set.
    311304                if ( isset( $_POST['menu-locations'] ) ) {
    312305                        $new_menu_locations = array_map( 'absint', $_POST['menu-locations'] );
     
    313306                        $menu_locations     = array_merge( $menu_locations, $new_menu_locations );
    314307                }
    315308
    316                 // Set menu locations.
    317                 set_theme_mod( 'nav_menu_locations', $menu_locations );
    318 
    319309                // Add Menu.
    320310                if ( 0 == $nav_menu_selected_id ) {
    321311                        $new_menu_title = trim( esc_html( $_POST['menu-name'] ) );
     
    332322                                        if ( isset( $_REQUEST['menu-item'] ) ) {
    333323                                                wp_save_nav_menu_items( $nav_menu_selected_id, absint( $_REQUEST['menu-item'] ) );
    334324                                        }
    335                                         if ( isset( $_REQUEST['zero-menu-state'] ) ) {
     325                    // Set the menu_location value correctly for the newly created menu.
     326                    foreach ( $menu_locations as $location => $id ) {
     327                            if ( 0 === $id ) {
     328                                    $menu_locations[ $location ] = $nav_menu_selected_id;
     329                            }
     330                    }
     331                    set_theme_mod( 'nav_menu_locations', $menu_locations );
     332                    if ( isset( $_REQUEST['zero-menu-state'] ) || ! empty( $_POST['auto-add-pages'] ) ) {
    336333                                                // If there are menu items, add them.
    337334                                                wp_nav_menu_update_menu_items( $nav_menu_selected_id, $nav_menu_selected_title );
     335                                        }
     336                                        if ( isset( $_REQUEST['zero-menu-state'] ) ) {
    338337                                                // Auto-save nav_menu_locations.
    339338                                                $locations = get_nav_menu_locations();
    340339                                                foreach ( $locations as $location => $menu_id ) {
     
    360359                                $messages[] = '<div id="message" class="error notice is-dismissible"><p>' . __( 'Please enter a valid menu name.' ) . '</p></div>';
    361360                        }
    362361
    363                         // Update existing menu.
     362                // Update existing menu.
    364363                } else {
     364                        // Remove menu locations that have been unchecked.
     365                        foreach ( $locations as $location => $description ) {
     366                                if ( ( empty( $_POST['menu-locations'] ) || empty( $_POST['menu-locations'][ $location ] ) ) && isset( $menu_locations[ $location ] ) && $menu_locations[ $location ] == $nav_menu_selected_id ) {
     367                                        unset( $menu_locations[ $location ] );
     368                                }
     369                        }
    365370
     371                        // Set menu locations.
     372                        set_theme_mod( 'nav_menu_locations', $menu_locations );
     373
    366374                        $_menu_object = wp_get_nav_menu_object( $nav_menu_selected_id );
    367375
    368376                        $menu_title = trim( esc_html( $_POST['menu-name'] ) );
     
    887895        <div id="menu-management-liquid">
    888896                <div id="menu-management">
    889897                        <form id="update-nav-menu" method="post" enctype="multipart/form-data">
    890                         <?php
    891                                 $new_screen_class = '';
    892                         if ( $add_new_screen ) {
    893                                 $new_screen_class = 'blank-slate';
    894                         }
    895                         ?>
    896898                                <h2><?php _e( 'Menu structure' ); ?></h2>
    897                                 <div class="menu-edit <?php echo $new_screen_class; ?>">
     899                                <div class="menu-edit">
    898900                                        <input type="hidden" name="nav-menu-data">
    899901                                        <?php
    900902                                        wp_nonce_field( 'closedpostboxes', 'closedpostboxesnonce', false );