Prevent error in Menu location checkbox settings by sabernhardt · Pull Request #10859 · WordPress/wordpress-develop

sabernhardt · 2026-02-03T23:34:00Z

Creates a text string to display for invalid menu ID.
Assigns id and aria-describedby to the connect the span with the checkboxes.

This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.

…bedby` for checkboxes

github-actions · 2026-02-03T23:34:12Z

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Core Committers: Use this line as a base for the props when committing in SVN:

Props sabernhardt, westonruter, audrasjb, mukesh27.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

github-actions · 2026-02-03T23:40:00Z

Test using WordPress Playground

The changes in this pull request can previewed and tested using a WordPress Playground instance.

WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser.

Some things to be aware of

The Plugin and Theme Directories cannot be accessed within Playground.
All changes will be lost when closing a tab with a Playground instance.
All changes will be lost when refreshing the page.
A fresh instance is created each time the link below is clicked.
Every time this pull request is updated, a new ZIP file containing all changes is created. If changes are not reflected in the Playground instance,
it's possible that the most recent build failed, or has not completed. Check the list of workflow runs to be sure.

For more details about these limitations and more, check out the Limitations page in the WordPress Playground documentation.

Test this pull request with WordPress Playground.

sabernhardt · 2026-02-03T23:52:27Z

Before the patch, an invalid menu ID can result in a PHP error:

With the patch, the text says that the ID is invalid (but it is not an error message):

Note: To assign an invalid ID, with Twenty Twenty-One as the active theme, I needed to edit the theme_mods_twentytwentyone value in the database, in the *_options table.

src/wp-admin/nav-menus.php

mukeshpanchal27 · 2026-02-04T04:55:58Z

@sabernhardt Request some changes on PR.

sabernhardt · 2026-02-06T18:11:13Z

src/wp-admin/nav-menus.php

 										foreach ( $locations as $location => $description ) :
-											$checked = false;
+											$checked          = false;
+											$aria_describedby = '';


The $aria_describedby variable is defined at the top of the foreach loop. When I tried setting the variable conditionally, checking isset() pulled values from other display locations (the variable was set earlier in the loop).

src/wp-admin/nav-menus.php

Copilot

Pull request overview

Updates the nav menus admin screen to avoid errors when a theme location is assigned to an invalid menu ID, and improves accessibility by associating the “currently set to” text with the location checkbox via aria-describedby.

Changes:

Add fallback display text when a theme location points to an invalid menu ID (avoids fatal when accessing ->name).
Add id to the “currently set to” <span> and conditionally add aria-describedby on the corresponding checkbox.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-06T19:38:38Z

src/wp-admin/nav-menus.php

 															_x( '(Currently set to: %s)', 'menu location' ),
-															wp_get_nav_menu_object( $menu_locations[ $location ] )->name
+															is_nav_menu( $menu_locations[ $location ] ) ? wp_get_nav_menu_object( $menu_locations[ $location ] )->name : __( 'an invalid menu ID' )


This uses is_nav_menu() and then calls wp_get_nav_menu_object() again, which results in duplicate lookups (since is_nav_menu() already calls wp_get_nav_menu_object()). Consider fetching the menu object once, validating it, and then using its name to avoid redundant work and simplify the expression.

Copilot · 2026-02-06T19:38:38Z

src/wp-admin/nav-menus.php

+													<span class="theme-location-set" id="theme-location-set-<?php echo esc_attr( $location ); ?>">
 													<?php
 														printf(
 															/* translators: %s: Menu name. */


The translator comment says %s is a menu name, but in the invalid-ID case %s becomes a fallback message instead. Please update the translator comment (or refactor) so it accurately describes all possible substitution values.

Suggested change

/* translators: %s: Menu name. */

/* translators: %s: Menu name or a message indicating an invalid menu ID. */

The text would depend on whether the message changes, but possibly:
Menu name or a message indicating that the menu was not found.

src/wp-admin/nav-menus.php

- edit fallback message and translator comment - add `esc_html()` (for `$description` as well as for the menu name) - change indentation

Copilot

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

src/wp-admin/nav-menus.php

westonruter

LGTM

Co-authored-by: Weston Ruter <[email protected]>

src/wp-admin/nav-menus.php

mukeshpanchal27

Thanks. Looks good to go. Left one non blocking feedback.

src/wp-admin/nav-menus.php

mukeshpanchal27 · 2026-02-10T04:24:27Z

@westonruter Let's commit this one. It's 10 year old 👴 ticket

westonruter · 2026-02-13T19:00:06Z

@audrasjb Are you intending to commit?

audrasjb · 2026-02-14T15:07:06Z

@westonruter yep

audrasjb · 2026-02-17T07:50:57Z

fixed

Create text string for invalid menu ID, and add id and `aria-descri…

2429319

…bedby` for checkboxes

mukeshpanchal27 reviewed Feb 4, 2026

View reviewed changes

src/wp-admin/nav-menus.php Outdated Show resolved Hide resolved

mukeshpanchal27 reviewed Feb 4, 2026

View reviewed changes

src/wp-admin/nav-menus.php Outdated Show resolved Hide resolved

Move aria-describedby escaping later

d07b0b3

sabernhardt commented Feb 6, 2026

View reviewed changes

src/wp-admin/nav-menus.php Show resolved Hide resolved

westonruter requested a review from Copilot February 6, 2026 19:34

Copilot started reviewing on behalf of westonruter February 6, 2026 19:35 View session

Copilot AI reviewed Feb 6, 2026

View reviewed changes

apply some Copilot recommendations

ff46ec7

- edit fallback message and translator comment - add `esc_html()` (for `$description` as well as for the menu name) - change indentation

westonruter requested a review from Copilot February 7, 2026 23:12

Copilot started reviewing on behalf of westonruter February 7, 2026 23:13 View session

Copilot AI reviewed Feb 7, 2026

View reviewed changes

westonruter reviewed Feb 7, 2026

View reviewed changes

src/wp-admin/nav-menus.php Outdated Show resolved Hide resolved

Rename variable

0eb2770

westonruter reviewed Feb 9, 2026

View reviewed changes

src/wp-admin/nav-menus.php Outdated Show resolved Hide resolved

westonruter approved these changes Feb 9, 2026

View reviewed changes

Apply readability suggestion from @westonruter

7031dd6

Co-authored-by: Weston Ruter <[email protected]>

westonruter reviewed Feb 9, 2026

View reviewed changes

src/wp-admin/nav-menus.php Outdated Show resolved Hide resolved

Fix trailing whitespace

720a6c2

mukeshpanchal27 approved these changes Feb 10, 2026

View reviewed changes

src/wp-admin/nav-menus.php Show resolved Hide resolved

Merge branch 'trunk' into nav-menu-set-to

9ca05bc

Merge branch 'trunk' into nav-menu-set-to

7d14c2c

audrasjb closed this Feb 17, 2026

	/* translators: %s: Menu name. */
	/* translators: %s: Menu name or a message indicating an invalid menu ID. */

Conversation

sabernhardt commented Feb 3, 2026

Uh oh!

github-actions bot commented Feb 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Feb 3, 2026

Test using WordPress Playground

Some things to be aware of

Uh oh!

sabernhardt commented Feb 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

mukeshpanchal27 commented Feb 4, 2026

Uh oh!

sabernhardt Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

sabernhardt Feb 7, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

westonruter left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

mukeshpanchal27 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

mukeshpanchal27 commented Feb 10, 2026

Uh oh!

westonruter commented Feb 13, 2026

Uh oh!

audrasjb commented Feb 14, 2026

Uh oh!

audrasjb commented Feb 17, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

github-actions bot commented Feb 3, 2026 •

edited

Loading

sabernhardt commented Feb 3, 2026 •

edited

Loading