RTC NPM Resource Changed bug PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar hiteshm0
hiteshm0
29 Dec 2025

Pull Request for Issue #46472

Summary of Changes

When creating a menu item via "save to menu" from articles or categories, the menu type is preselected on page load. But, the parent item dropdown does not have any items. This is because the parent item dropdown is populated only when a change event is fired on a menu type field. And no change event occurs when the page is loaded with a preselected menu item.

Fix: Added initialization logic to check if a menu type is already selected on page load. If yes, then the onChange function is trigerred manually.

Testing Instructions

  1. Go to content → articles/categories
  2. Open/create any article/category
  3. Click save to menu button
  4. Check the parent item dropdown

Actual result BEFORE applying this Pull Request

Parent Item dropdown is empty

Expected result AFTER applying this Pull Request

Parent item dropdown has the correct items

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar hiteshm0 hiteshm0 - open - 29 Dec 2025
avatar hiteshm0 hiteshm0 - change - 29 Dec 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 29 Dec 2025
Category JavaScript Repository NPM Change
avatar ceford ceford - test_item - 5 Jan 2026 - Tested successfully
avatar ceford
ceford - comment - 5 Jan 2026

I have tested this item ✅ successfully on 9e64196


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

avatar tecpromotion tecpromotion - change - 9 Jan 2026
Labels Added: NPM Resource Changed bug PR-5.4-dev
avatar tecpromotion tecpromotion - alter_testresult - 9 Jan 2026 - ceford: Tested successfully
avatar coolcat-creations coolcat-creations - test_item - 9 Jan 2026 - Tested successfully
avatar coolcat-creations
coolcat-creations - comment - 9 Jan 2026

I have tested this item ✅ successfully on 8110997

Works as described and is very useful, thank you!


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

avatar richard67 richard67 - change - 9 Jan 2026
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 9 Jan 2026

RTC


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

avatar richard67 richard67 - change - 9 Jan 2026
Labels Added: RTC
avatar muhme
muhme - comment - 12 Jan 2026

✅ Final test before merge with JBT

  • It was difficult for me to reproduce the issue, but finally (perhaps after setting ‘Error Reporting’ to system default and disabling ‘Debug System’ and ‘Log Almost Everything’) I was able to reproduce the issue and saw that the list of parent menu items was empty and the suggested workaround of selecting a different menu was also reproduced.
  • PR installed with patch tester and npm run build:js
  • Successfully verified that the list of menu entries is loaded (in 'Save to Menu' for articles and in both 'Save to Menu as List' and 'Save to Menu as Blog' for categories). The list of parent menu items is always loaded with the correct entries.
avatar muhme muhme - change - 12 Jan 2026
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2026-01-12 13:28:38
Closed_By muhme
avatar muhme muhme - close - 12 Jan 2026
avatar muhme muhme - merge - 12 Jan 2026
avatar muhme
muhme - comment - 12 Jan 2026

Thank you @hiteshm0 for your contribution. Thank you @ceford and @coolcat-creations for testing.

avatar heelc29
heelc29 - comment - 12 Jan 2026

@richard67 I think this PR is responsible for the recent failed system tests. I haven't investigated this further yet.
image

avatar richard67
richard67 - comment - 12 Jan 2026

@richard67 I think this PR is responsible for the recent failed system tests. I haven't investigated this further yet.

@heelc29 I think the same, but I also have not had time to investigate yet.

avatar richard67
richard67 - comment - 13 Jan 2026

It seems this PR was not the right fix. Meanwhile there is a new one, see #46681 .

avatar hiteshm0
hiteshm0 - comment - 15 Jan 2026

sorry y'all 😅

Add a Comment

Login with GitHub to post a comment