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 Item
Default
Default
for bothDefault
Default
for bothCurrent
All 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.