?
avatar shawnmcginn
shawnmcginn
14 Oct 2017

Steps to reproduce the issue

Joomla version 3.7.5
Frontend or backend, in article editor select Menu to insert a menu link, search for a menu item in the modal and select any menu item.

Expected result

Should write a link for the menu item at the cursor in the article window

Actual result

Does nothing. If you look at admin-items-modal.min.js this condition evaluates to true because Joomla.getOptions('xtd-menus') is undefined following a search:
if (!Joomla.getOptions('xtd-menus')) {
// Something went wrong!
window.parent.jModalClose();
return false;
}
Note, if you do not search for a menu item and select from the list of available items it works as expected. Therefore you can work around the problem by searching for the desired menu item, select said item, the modal will close, re-open the modal, your search query is retained and you can now choose the desired element.

System information (as much as possible)

Joomla version 3.7.5
running on linux shared hosting

Additional comments

my first defect submission, my apologies if I'm lacking any necessary info

avatar shawnmcginn shawnmcginn - open - 14 Oct 2017
avatar joomla-cms-bot joomla-cms-bot - labeled - 14 Oct 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 15 Oct 2017

Please update on latest Version and test using Core-Template without 3rd-Party-Components and Plug-Ins if this Issues is same.


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 15 Oct 2017
Status New Information Required
avatar infograf768
infograf768 - comment - 15 Oct 2017

Issue confirmed on staging.
As soon as Search is used in the modal, the link to the menu item selected is NOT entered in the text field. One has to reload the modal to be able to enter the link.

  • the Clear button is broken
avatar infograf768
infograf768 - comment - 15 Oct 2017

@dgt41 Can you look into this?

avatar infograf768 infograf768 - change - 15 Oct 2017
Priority Low Medium
Status Information Required Confirmed
avatar infograf768 infograf768 - change - 15 Oct 2017
Build 3.7.5 staging
avatar dgt41
dgt41 - comment - 17 Oct 2017

Please test #18359

avatar brianteeman
brianteeman - comment - 17 Oct 2017

Closed bas we have a pr for testing

avatar brianteeman brianteeman - change - 17 Oct 2017
Status Confirmed Closed
Closed_Date 0000-00-00 00:00:00 2017-10-17 20:35:39
Closed_By brianteeman
avatar brianteeman brianteeman - close - 17 Oct 2017
avatar infograf768
infograf768 - comment - 18 Oct 2017

@shawnmcginn

Please test/confirm #18359

Add a Comment

Login with GitHub to post a comment