? Pending

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
3 Jun 2017

Pull Request for Issue #16421

Summary of Changes

Adding conditionals to display the default placeholder with null value defined in the field Option

Testing Instructions

Edit an item which contains a choice for a menu item field using the modal_menu type.

  1. Creating a Menu Item Alias. Choosing the menu item to alias to.
  2. Creating a Create Article menu item. Choosing the menu item Submission Redirect in Options.
  3. Creating a Login Form menu item. Choosing the Menu Item Login Redirect and Menu Item Logout Redirect in the Options.
  4. Creating a Logout menu item. Choosing the Logout Redirection Page in the Options.
  5. Editing the Site Login Form module. Choosing the Login Redirection Page and the Logout Redirection Page.
  6. Editing the Site 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.

Expected result for the placeholder value

  1. Select a Menu Item
  2. Default
  3. Default for both
  4. Default
  5. Default for both
  6. Current

Actual result

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.

menumodalfield

@epidote

avatar infograf768 infograf768 - open - 3 Jun 2017
avatar infograf768 infograf768 - change - 3 Jun 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 3 Jun 2017
Category Administration com_menus
avatar AlexRed
AlexRed - comment - 3 Jun 2017

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.

avatar AlexRed AlexRed - test_item - 3 Jun 2017 - Tested successfully
avatar Bakual
Bakual - comment - 3 Jun 2017

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.

avatar infograf768
infograf768 - comment - 3 Jun 2017

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

avatar Bakual
Bakual - comment - 3 Jun 2017

Ah sorry, yes then of course we need to take what is there already.

avatar carcam
carcam - comment - 3 Jun 2017

I have tested this item successfully on 113c374


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

avatar carcam carcam - test_item - 3 Jun 2017 - Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - change - 3 Jun 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 3 Jun 2017

RTC after two successful tests.

avatar rdeutz rdeutz - change - 6 Jun 2017
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: ?
avatar rdeutz rdeutz - close - 6 Jun 2017
avatar rdeutz rdeutz - merge - 6 Jun 2017

Add a Comment

Login with GitHub to post a comment