?
avatar joeforjoomla
joeforjoomla
13 Mar 2017

Steps to reproduce the issue

Install an extension such as Virtuemart

Expected result

All values in 'Show or Hide Menu Items' are translated as the standard Joomla admin menu

Actual result

Only item marked as COM_XXX in the XML manifest file are translated, none of the XXX are translated
menu_container_translation

System information (as much as possible)

In the file root/administrator/components/com_menus/views/item/tmpl/edit_container.php
the language file loading is based in the component name, it should be based on the admin menu item value as for the standard admin menu:

if ($extension = $link->componentname): $lang->load("$extension.sys", JPATH_ADMINISTRATOR, null, false, true) || $lang->load("$extension.sys", JPATH_ADMINISTRATOR . '/components/' . $extension, null, false, true); endif;

Additional comments

avatar joeforjoomla joeforjoomla - open - 13 Mar 2017
avatar joomla-cms-bot joomla-cms-bot - change - 13 Mar 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 13 Mar 2017
avatar zero-24
zero-24 - comment - 13 Mar 2017

@joeforjoomla please send the change as PR than ?

avatar joeforjoomla
joeforjoomla - comment - 13 Mar 2017

Mmmm it looks that i'm no more able to open a pull request for 3.7

Please do that if you know how to, based on the following changes:

File root/administrator/components/com_menus/views/item/tmpl/edit_container.php line 95:

Change from:
if ($extension = $link->componentname):``

to:
if ($extension = $link->element):

File root/administrator/components/com_menus/helpers/menus.php line 184:

Change from:
$query->select('e.name as componentname')

to:
$query->select('e.name as componentname, e.element')

avatar zero-24
zero-24 - comment - 13 Mar 2017

Mmmm it looks that i'm no more able to open a pull request for 3.7

Why do you think so? Just send a PR against staging and you are fine ?

avatar joeforjoomla
joeforjoomla - comment - 13 Mar 2017

Done: #14584

avatar zero-24
zero-24 - comment - 13 Mar 2017

Looks like you closed the pr?

avatar joeforjoomla
joeforjoomla - comment - 13 Mar 2017

Sorry just created a new one #14585

avatar joeforjoomla
joeforjoomla - comment - 13 Mar 2017

Closing this issue hence we have a PR #14585

avatar joeforjoomla joeforjoomla - change - 13 Mar 2017
Status New Closed
Closed_Date 0000-00-00 00:00:00 2017-03-13 16:57:46
Closed_By joeforjoomla
avatar joeforjoomla joeforjoomla - close - 13 Mar 2017

Add a Comment

Login with GitHub to post a comment