Skip to content

Conversation

@laoneo
Copy link
Member

@laoneo laoneo commented Apr 23, 2022

Summary of Changes

Make sure the menu class has always a database instance.

Testing Instructions

  • Open the back end
  • Create any menu item
  • Open the front end
  • Navigate to the menu item

Actual result BEFORE applying this Pull Request

All works.

Expected result AFTER applying this Pull Request

All works.

@toivo
Copy link
Contributor

toivo commented Apr 30, 2022

I have tested this item ✅ successfully on 238c32e

Tested successfully in Joomla 4.2.0-alpha3-dev of 30 April in Wampserver 3.2.8 using PHP 8.0.15.


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

@pabloarias
Copy link

I have tested this item ✅ successfully on 238c32e

Tested with Joomla 4.2-alpha3-dev and PHP 8.1.5.


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

@laoneo
Copy link
Member Author

laoneo commented May 14, 2022

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label May 14, 2022
@roland-d roland-d merged commit cfe2869 into joomla:4.2-dev May 14, 2022
@roland-d roland-d deleted the j4/menu/ensure-db branch May 14, 2022 18:53
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label May 14, 2022
@roland-d
Copy link
Contributor

Thanks everybody

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants