? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
5 Jul 2020

In the admin there are no SEF urls so there is no need to have a menu item alias for admin menu items. It just clutters the interface and confuses the user.

This PR only removes the display of the alias it does not stop it being created.

Testing Instructions

create an admin menu.
Create a menu item in that menu

Actual result BEFORE applying this Pull Request

image

image

Expected result AFTER applying this Pull Request

image

image

avatar brianteeman brianteeman - open - 5 Jul 2020
avatar brianteeman brianteeman - change - 5 Jul 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 5 Jul 2020
Category Administration com_menus
avatar Fedik
Fedik - comment - 5 Jul 2020

not that easy, if in some reason you need 2 (and more) similar title (in different menus or different type), you will get an error "alias duplication", and no possibility to edit it

avatar brianteeman
brianteeman - comment - 5 Jul 2020

grrh

so I guess I will need to update the alias generation code then so it includes the menuname in the alias. That will make it unique

avatar brianteeman brianteeman - change - 5 Jul 2020
Title
[4.0] Custom admin menu items alias
WIP [4.0] Custom admin menu items alias
avatar brianteeman brianteeman - edited - 5 Jul 2020
avatar Fedik
Fedik - comment - 5 Jul 2020

... so it includes the menuname in the alias

or maybe better/safer use datetime with seconds 'Y-m-d-H-i-s' (eg 2020-07-05-13-52-11) for all admin aliases

avatar brianteeman brianteeman - change - 5 Jul 2020
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2020-07-05 20:57:32
Closed_By brianteeman
Labels Added: ?
avatar brianteeman brianteeman - close - 5 Jul 2020

Add a Comment

Login with GitHub to post a comment