?
avatar PhilETaylor
PhilETaylor
2 Feb 2021

Steps to reproduce the issue

was #18725 closed with no action

Create a menu item of type iframe wrapper - set url to be something thats certainly not a url

Expected result

Validation error message

Actual result

Menu Item Saved

Screenshot 2021-02-02 at 20 28 44

avatar PhilETaylor PhilETaylor - open - 2 Feb 2021
avatar joomla-cms-bot joomla-cms-bot - change - 2 Feb 2021
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 2 Feb 2021
avatar PhilETaylor
PhilETaylor - comment - 2 Feb 2021

same with Menu Item Type = URL

Screenshot 2021-02-02 at 21 12 56

avatar brianteeman
brianteeman - comment - 2 Feb 2021

Someone might have fixed it for you if you hadnt closed the original issue

image

avatar PhilETaylor
PhilETaylor - comment - 2 Feb 2021

Someone might have fixed it for you if you hadnt closed the original issue

Its not about me. No one is fixing anything "for me". Im not asking, or demanding, anyone to do anything. Im simply reporting the issues. Like or not. I don't care.

avatar joomdonation
joomdonation - comment - 7 Feb 2021

PR #32342 should fix the first part of this issue. For the URL of Menu Item Type URL, I'm afraid of it is hard to fix/validate because look like we also accept relative/none-sef URL there, so maybe we will have to rely on Administrators to enter valid url.

avatar sandramay0905
sandramay0905 - comment - 11 Apr 2021

@joomdonation is right, the first part about IFrame Wrapper URL Not validated is solved.

Should the issue be closed and a new one created about Menu Item Type URL Not validated?

avatar PhilETaylor PhilETaylor - change - 18 Apr 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-04-18 18:23:48
Closed_By PhilETaylor
avatar PhilETaylor PhilETaylor - close - 18 Apr 2021

Add a Comment

Login with GitHub to post a comment