Steps to reproduce the issue

Create a menu item that points to something belonging to a component. For example a form in Chronoforms.

Remove that component (perhaps you've installed a newer version of it)

You can see that the menu page shows you "component not found". Try to edit the menu item.

Expected result

Be able to edit the menu item to be able to point it elsewhere (for example a newer version of the component)

Actual result

Stack trace is produced as follows:

Function Location

1 () JROOT/libraries/vendor/joomla/filesystem/src/Path.php:194
2 Joomla\Filesystem\Path::clean() JROOT/administrator/components/com_menus/src/Model/ItemModel.php:1131
3 Joomla\Component\Menus\Administrator\Model\ItemModel->preprocessForm() JROOT/libraries/src/MVC/Model/FormBehaviorTrait.php:115
4 Joomla\CMS\MVC\Model\FormModel->loadForm() JROOT/administrator/components/com_menus/src/Model/ItemModel.php:517
5 Joomla\Component\Menus\Administrator\Model\ItemModel->getForm() JROOT/libraries/src/MVC/View/AbstractView.php:159
6 Joomla\CMS\MVC\View\AbstractView->get() JROOT/administrator/components/com_menus/src/View/Item/HtmlView.php:88
7 Joomla\Component\Menus\Administrator\View\Item\HtmlView->display() JROOT/libraries/src/MVC/Controller/BaseController.php:697
8 Joomla\CMS\MVC\Controller\BaseController->display() JROOT/administrator/components/com_menus/src/Controller/DisplayController.php:74
9 Joomla\Component\Menus\Administrator\Controller\DisplayController->display() JROOT/libraries/src/MVC/Controller/BaseController.php:730
10 Joomla\CMS\MVC\Controller\BaseController->execute() JROOT/libraries/src/Dispatcher/ComponentDispatcher.php:143
11 Joomla\CMS\Dispatcher\ComponentDispatcher->dispatch() JROOT/libraries/src/Component/ComponentHelper.php:361
12 Joomla\CMS\Component\ComponentHelper::renderComponent() JROOT/libraries/src/Application/AdministratorApplication.php:150
13 Joomla\CMS\Application\AdministratorApplication->dispatch() JROOT/libraries/src/Application/AdministratorApplication.php:195
14 Joomla\CMS\Application\AdministratorApplication->doExecute() JROOT/libraries/src/Application/CMSApplication.php:306
15 Joomla\CMS\Application\CMSApplication->execute() JROOT/administrator/includes/app.php:58
16 require_once() JROOT/administrator/index.php:32

System information (as much as possible)

Joomla! Version Joomla! 5.1.0 Stable [ Kudumisha ] 16-April-2024 16:00 GMT

Additional comments

In the file JROOT/libraries/vendor/joomla/filesystem/src/Path.php

In this scenario the $path is a boolean, not a string (because the component doesn't exist)

At this statement:

public static function clean($path, $ds = \DIRECTORY_SEPARATOR)
{
if (!\is_string($path)) {
throw new \InvalidArgumentException('You must specify a non-empty path to clean');
}

Before the if statement, you could check if $path is (presumably) False, and if so, just return. There is no need to crash out. This enables the editing of the menu item to proceed without the user having to resort to editing the Path.php file to fix it, or having to delete their menu item and re-create it. A stack trace is never desirable when a simple fix can be made.