User tests: Successful: Unsuccessful:
Pull Request for Issue #16421
Adding conditionals to display the default placeholder with null value defined in the field Option
Edit an item which contains a choice for a menu item field using the modal_menu type.
Menu Item Alias. Choosing the menu item to alias to.Create Article menu item. Choosing the menu item Submission Redirect in Options.Login Form menu item. Choosing the Menu Item Login Redirect and Menu Item Logout Redirect in the Options.Logout menu item. Choosing the Logout Redirection Page in the Options.Login Form module. Choosing the Login Redirection Page and the Logout Redirection Page.Menu module. Choosing the Base Item.The xml code for the Menu module is for example
<field
name="base"
type="modal_menu"
label="MOD_MENU_FIELD_ACTIVE_LABEL"
description="MOD_MENU_FIELD_ACTIVE_DESC"
select="true"
new="true"
edit="true"
clear="true"
>
<option value="">JCURRENT</option>
</field>
In each case, select a menu item in the modal. Then Clear that menu item.
Select a Menu ItemDefaultDefault for bothDefaultDefault for bothCurrentAll have Select a Menu Item
Patch and test again. Do not forget to choose a menu item and Clear to confirm that the correct value is displayed.
Here is an example —after patch— concerning the original issue with Menu module.
| Status | New | ⇒ | Pending |
| Category | ⇒ | Administration com_menus |
Looks a bit wrong to me to use the <option> tag for this. The tag could be present more than once but your code only supports one. So imho it would make more sense to use an attribute (eg placeholder or whatever you like) of the field instead a child element.
this pr deals with existing xmls. its purpose was not to refactor them.
if you think we should, then let's do this, but it means modifying each of them
Ah sorry, yes then of course we need to take what is there already.
I have tested this item
| Status | Pending | ⇒ | Ready to Commit |
RTC after two successful tests.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-06-06 18:24:12 |
| Closed_By | ⇒ | rdeutz | |
| Labels |
Added:
?
|
||
I have tested this item✅ successfully on 113c374
Patch ok for me
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16459.