You must specify a non-empty path to clean about joomla-cms HOT 5 CLOSED

A stack trace is never desirable when a simple fix can be made.

You only get the stack trace if you have debug enabled. In normal use you get

But if you have a fix that is more helpful please submit a pull request with that code

from joomla-cms.

Hi Brian. Ah yes! I do have debug on and you are correct. It would still be preferable to enable the user to edit the menu item than to receive the error.

I've never submitted a pull request before, and I don't know if the fix I would suggest is acceptable, however it works for me (tested) by simply replacing the "throw new..." line I quoted above with "return;"

The thing I don't know is if there are ever any circumstances where one would desire the error message instead of silently ignoring the problem because I don't have enough in-depth knowledge of the code that may call it under other circumstances. It certainly fixes this specific scenario.

I shall see if I can submit the pull request. :-)

from joomla-cms.

I think your fix is at the wrong place and would have to many side effects. Then main problem here is that the application can't load the form definition for the menu item. This is something you can't fix in a proper way. I think in such situations deleting the menu item and creating a new one would be the way to go.

from joomla-cms.

Please test #43604 as this should solve the issue at the com_menus level instead of globally on the Path class.

from joomla-cms.

closing as we have a pr #43604

from joomla-cms.