NPM Resource Changed bug PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar adarshdubey03
adarshdubey03
10 Mar 2026

Pull Request resolves #44407 .

  • I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.

Summary of Changes

Switching the Client filter (Site / Administrator) in the Menu Items view should reset the Menu filter so a valid menu can be selected for the chosen client. This currently fails because the reset logic in SearchTools.clear() only checks fields inside .js-stools-container-filters, while the client_id and menutype selectors are placed inside .js-stools-container-selector. As a result, the dependent filter is not cleared, an invalid combination may be submitted, and Joomla falls back to the Site client. This change includes .js-stools-container-selector in the reset check so the Menu filter is properly cleared when the Client filter changes.

Testing Instructions

Go to Menus and then All Menu Items in the administrator.
Ensure the filters show something like:
Client: Site
Menu: Main Menu
Change the Client filter from Site to Administrator
Repeat the same test in the opposite direction by changing Client from Administrator back to Site.

Actual result BEFORE applying this Pull Request

When changing the Client filter from Site to Administrator, the page reloads but the Client filter reverts back to Site.

Expected result AFTER applying this Pull Request

When switching the Client filter between Site and Administrator, the Menu filter is automatically reset to “Select Menu” before the form is submitted. The page reloads with the correct client selected, and the Menu dropdown then displays only the menus that belong to that client, allowing the user to switch between Site and Administrator menu items correctly.

Link to documentations

Please select:

  • Documentation link for guide.joomla.org:

  • No documentation changes for guide.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar adarshdubey03 adarshdubey03 - open - 10 Mar 2026
avatar adarshdubey03 adarshdubey03 - change - 10 Mar 2026
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 10 Mar 2026
Category JavaScript Repository NPM Change
avatar brianteeman brianteeman - test_item - 10 Mar 2026 - Tested successfully
avatar brianteeman
brianteeman - comment - 10 Mar 2026

I have tested this item ✅ successfully on 5daa258


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47352.

avatar exlemor exlemor - test_item - 10 Mar 2026 - Tested successfully
avatar exlemor
exlemor - comment - 10 Mar 2026

I have tested this item ✅ successfully on 5daa258

I have tested this successfully. Thanks @adarshdubey03 - nice to see multiple contributions from you!


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47352.

avatar richard67 richard67 - change - 10 Mar 2026
Status Pending Ready to Commit
Labels Added: NPM Resource Changed bug PR-5.4-dev
avatar richard67
richard67 - comment - 10 Mar 2026

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47352.

Add a Comment

Login with GitHub to post a comment