? ? Pending

User tests: Successful: Unsuccessful:

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

avatar joeforjoomla joeforjoomla - open - 13 Mar 2017
avatar joeforjoomla joeforjoomla - change - 13 Mar 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 13 Mar 2017
Category Administration com_menus
avatar joeforjoomla joeforjoomla - change - 13 Mar 2017
The description was changed
avatar joeforjoomla joeforjoomla - edited - 13 Mar 2017
avatar joeforjoomla joeforjoomla - change - 13 Mar 2017
Labels Added: ?
avatar infograf768
infograf768 - comment - 14 Mar 2017

@izharaazmi
Can you look at this?

avatar infograf768
infograf768 - comment - 14 Mar 2017

Looks like Virtuemart is not using a normal xml

<name>VIRTUEMART</name>
instead of
<name>COM_VIRTUEMART</name>
or
<name>com_virtuemart</name>

which would explain that issue.

avatar izharaazmi
izharaazmi - comment - 14 Mar 2017

I am looking at this now. However I believe that since the language key string comes from the component manifest itself, the translation should still work if written in the extension.

avatar zero-24
zero-24 - comment - 14 Mar 2017

Looks like Virtuemart is not using a normal xml

The name don't need to be a translation string it can also be just text. So I think the element is the better choice here 😄

avatar infograf768
infograf768 - comment - 14 Mar 2017

Tested patch and, yes, this PR makes sense and will work in any case.

avatar infograf768 infograf768 - test_item - 14 Mar 2017 - Tested successfully
avatar infograf768
infograf768 - comment - 14 Mar 2017

I have tested this item successfully on 0d978ea


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

avatar izharaazmi
izharaazmi - comment - 14 Mar 2017

Yeah right. I'd too choose element against name. Not sure why I did that in the first place.
Patch is good to me. 👍

avatar infograf768 infograf768 - alter_testresult - 14 Mar 2017 - izharaazmi: Tested successfully
avatar infograf768 infograf768 - change - 14 Mar 2017
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 14 Mar 2017

RTC. Good find!


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

avatar wilsonge wilsonge - change - 14 Mar 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-03-14 11:11:29
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 14 Mar 2017
avatar wilsonge wilsonge - merge - 14 Mar 2017

Add a Comment

Login with GitHub to post a comment