[#7905] - [UX] Form Field Menuitem language
- Closed
- 6 Apr 2016
- Medium
- Build: staging
- Easy Test
- # 7905
- Diff
- Devportobello:UXFieldMenuitem
- Success Human Test Results: 3 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
Summary
Form field menuitem doesnt indicate language of an item.
Important thing
This PR could conflict with #7148
Test
- Create many menuitem with different language
- Enable plugin "System - Language Filter "
- Create menu module and use " Base Item"
- Unable to know language of menuitem
- Apply patch
- Now you can know language of menuitem
Other
I wanted to add flag picture but choosen dont allow that :/
Preview
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
zero-24
- | Category | ⇒ | Administration UI/UX |
| Easy | No | ⇒ | Yes |
You right, not noticed multilang already cached.
I have tested this item 
successfully on 0783009
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7905.
I have tested this item successfully on 0783009
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7905.
I have tested this item successfully on 0783009
Test is sucsessfull. But why do you get a.published you don´t use?
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7905.
I have tested this item successfully on 0783009
Test is successful - code could be improved a bit (you added a.published, which you don't use)
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7905.
| Status | Pending | ⇒ | Needs Review |
@rdeutz please make a decision if this can be merged regardless of the unnecessary select
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7905.
brianteeman
- | Category | Administration UI/UX | ⇒ | Administration Multilanguage UI/UX |
| Status | Needs Review | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-04-06 10:04:18 |
| Closed_By | ⇒ | Devportobello |
No use to define a new variable
$multilangIsEnabled = JLanguageMultilang::isEnabled();Please use the existing
JLanguageMultilang::isEnabled()directly.