No Code Attached Yet PHP 8.x bug
avatar brianteeman
brianteeman
19 Oct 2024

Steps to reproduce the issue

Create a new admin menu. Either use a preset or add a menu item to it.
Publish this new admin menu in a module in position menu
After the message to turn on the menu recovery mode is displayed make sure you click on it to enable recovery mode

Expected result

no errors

Actual result

you might need to navigate to another page but a series of errors are displayed such as

Warning: Undefined property: Joomla\CMS\Menu\AdministratorMenuItem::$element in /home/public_html/administrator/modules/mod_menu/src/Menu/CssMenu.php on line 319

Warning: Undefined property: Joomla\CMS\Menu\AdministratorMenuItem::$class in /home/public_html/administrator/modules/mod_menu/src/Menu/CssMenu.php on line 495

System information (as much as possible)

Additional comments

avatar brianteeman brianteeman - open - 19 Oct 2024
avatar joomla-cms-bot joomla-cms-bot - change - 19 Oct 2024
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 19 Oct 2024
avatar chmst chmst - change - 20 Oct 2024
Status New Closed
Closed_Date 0000-00-00 00:00:00 2024-10-20 08:32:30
Closed_By chmst
avatar chmst chmst - close - 20 Oct 2024
avatar chmst
chmst - comment - 20 Oct 2024

Closing as there is a PR. Please test #44318

avatar chmst chmst - change - 20 Oct 2024
Labels Added: PHP 8.x
avatar chmst chmst - labeled - 20 Oct 2024
avatar chmst chmst - change - 23 Oct 2024
Status Closed New
Closed_Date 2024-10-20 08:32:30
Closed_By chmst
avatar chmst chmst - reopen - 23 Oct 2024
avatar chmst
chmst - comment - 23 Oct 2024

Re-opening. The PR #44318 fixes all PHP warnings, but not the issue with recovery mode.

avatar chmst chmst - change - 26 Oct 2024
Labels Added: bug
avatar chmst chmst - labeled - 26 Oct 2024
avatar chmst chmst - change - 26 Oct 2024
Status New Closed
Closed_Date 0000-00-00 00:00:00 2024-10-26 20:27:05
Closed_By chmst
avatar chmst chmst - close - 26 Oct 2024
avatar chmst
chmst - comment - 26 Oct 2024

Please test #44318 again with recovery mode.

Add a Comment

Login with GitHub to post a comment