User tests: Successful: Unsuccessful:
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_menus |
Labels |
Added:
?
|
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.
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.
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
Tested patch and, yes, this PR makes sense and will work in any case.
I have tested this item
Yeah right. I'd too choose element against name. Not sure why I did that in the first place.
Patch is good to me.
Status | Pending | ⇒ | Ready to Commit |
RTC. Good find!
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:
?
|
@izharaazmi
Can you look at this?