? ? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
18 Oct 2019

Pull Request for Issue # .

Summary of Changes

Fixes query error preventing alias menu item types from showing in admin menus.

Testing Instructions

Create admin menu.
Create a menu item.
Create an alias for that menu item.
Publish a menu module.

Expected result

Menu item and its alias are showing.

Actual result

Only first menu item is shown.

Documentation Changes Required

No.

avatar SharkyKZ SharkyKZ - open - 18 Oct 2019
avatar SharkyKZ SharkyKZ - change - 18 Oct 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Oct 2019
Category Administration com_menus
avatar infograf768
infograf768 - comment - 18 Oct 2019

I am a bit confused here.
Could you add screenshots for before and after patch?

avatar infograf768
infograf768 - comment - 18 Oct 2019

OK, I understood. You meant create a "Menu Item Alias" type.
Tested and indeed your PR solves the issue.

avatar infograf768 infograf768 - test_item - 18 Oct 2019 - Tested successfully
avatar infograf768
infograf768 - comment - 18 Oct 2019

I have tested this item successfully on 7314ca2


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

avatar waader waader - test_item - 18 Oct 2019 - Tested successfully
avatar waader
waader - comment - 18 Oct 2019

I have tested this item successfully on 7314ca2


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

avatar SharkyKZ SharkyKZ - change - 18 Oct 2019
Status Pending Ready to Commit
Labels Added: ?
avatar SharkyKZ
SharkyKZ - comment - 18 Oct 2019

RTC


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

avatar wilsonge wilsonge - change - 18 Oct 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-10-18 22:47:16
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 18 Oct 2019
avatar wilsonge wilsonge - merge - 18 Oct 2019
avatar wilsonge
wilsonge - comment - 18 Oct 2019

Thanks!

Add a Comment

Login with GitHub to post a comment