No Code Attached Yet J4 Issue PHP 8.x
avatar chmst
chmst
11 Jun 2022

Steps to reproduce the issue

I want to add an own Administrator Menu and use a preset.

Go to Menu - manage
New menu: enter a name and choose "Alternative Menu Preset"
Save and Close
A new menu is on the menus view

Open the list of menu items

click the first item (home dashboard)

Expected result

Actual result

grafik

System information (as much as possible)

win11, xampp, php 8.1.5

Additional comments

avatar chmst chmst - open - 11 Jun 2022
avatar joomla-cms-bot joomla-cms-bot - change - 11 Jun 2022
Labels Added: No Code Attached Yet J4 Issue PHP 8.x
avatar joomla-cms-bot joomla-cms-bot - labeled - 11 Jun 2022
avatar brianteeman
brianteeman - comment - 13 Jun 2022

Confirmed on php 8.1

avatar brianteeman
brianteeman - comment - 13 Jun 2022

There are two different issues here

The first is that there is a bug in the preset and it is not using the correct value for the menu item type #38042)

The second is that in php 8.1 the bug above results in the deprecation errors.

avatar richard67 richard67 - close - 13 Jun 2022
avatar richard67
richard67 - comment - 13 Jun 2022

Closing as having a pull request. Please test #38042 . Thanks in advance.

avatar richard67 richard67 - change - 13 Jun 2022
Status New Closed
Closed_Date 0000-00-00 00:00:00 2022-06-13 08:32:27
Closed_By richard67
avatar brianteeman
brianteeman - comment - 13 Jun 2022

Please re-open. My pr only resolves part of the issue

avatar richard67 richard67 - change - 13 Jun 2022
Status Closed New
Closed_Date 2022-06-13 08:32:27
Closed_By richard67
avatar richard67 richard67 - reopen - 13 Jun 2022
avatar richard67
richard67 - comment - 13 Jun 2022

Please re-open. My pr only resolves part of the issue

@brianteeman Then please note that somehow in your PR, e.g. by changing the Pull Request for Issue #38032 . to Pull Request for Issue #38032 (part). or something like that.

avatar brianteeman
brianteeman - comment - 13 Jun 2022

I did write that above here but have now added the word partial there

avatar chmst chmst - change - 17 Jun 2022
Status New Closed
Closed_Date 0000-00-00 00:00:00 2022-06-17 08:36:02
Closed_By chmst
avatar chmst chmst - close - 17 Jun 2022
avatar chmst
chmst - comment - 17 Jun 2022

Please test #38081. Closing as now both warnings are resolved.

Add a Comment

Login with GitHub to post a comment