? ? Failure

User tests: Successful: Unsuccessful:

avatar Fedik
Fedik
3 Jun 2017

Pull Request for Issue #16110

Summary of Changes

The patch rerender Chosen.js if it was there before

Testing Instructions

please see #16110 and #14556 for detail

Example xml which can be used for test:

<field name="repeatable" type="repeatable" icon="list" label="Repeatable" select="select">
	<fieldset hidden="true" name="repeatable_modal" repeat="true">

			<field type="list" name="list1" label="List1">
			<option>- Select -</option>
			<option value="val1">val 1</option>
			<option value="val2">val 2</option>
			<option value="val3">val 3</option>
		</field>

		<field type="list" name="list2" label="List2" multiple="true">
			<option>- Select -</option>
			<option value="val4">val 4</option>
			<option value="val5">val 5</option>
			<option value="val6">val 6</option>
			<option value="val7">val 7</option>
			<option value="val8">val 8</option>
		</field>
	</fieldset>
</field>

Expected result

Chosen.js rendered

Actual result

Only default <selec> list

Documentation Changes Required

none

avatar Fedik Fedik - open - 3 Jun 2017
avatar Fedik Fedik - change - 3 Jun 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 3 Jun 2017
Category JavaScript
avatar Fedik
Fedik - comment - 3 Jun 2017

@N6REJ please test also

avatar roland-d
roland-d - comment - 3 Jun 2017

@Fedik I applied the patch but I am still getting a plain select list.

image

avatar Fedik
Fedik - comment - 3 Jun 2017

@roland-d It will apply Chosen.js only if it available on the page, it do not force one.
I have tested in the Isis template params.

avatar roland-d
roland-d - comment - 4 Jun 2017

I have tested this item successfully on f5337c9

The dropdown was a regular select before the patch but after applying the patch it is now a chosen dropdown.


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

avatar roland-d roland-d - test_item - 4 Jun 2017 - Tested successfully
avatar N6REJ
N6REJ - comment - 7 Jun 2017

sorry I missed this, for some reason git is not emailing me with notices.. I'll get this tested in next 72hrs

avatar roland-d
roland-d - comment - 4 Jul 2017

@N6REJ The 72 hours are up, are you able to test this again?

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 20 Jul 2017

@N6REJ can you please test?

avatar N6REJ
N6REJ - comment - 24 Jul 2017

sorry guys, idk why but github refuses to send me notification for joomla. I get them for neard but not from joomla. I keep forgetting to get this done.. I"m going to leave the tab open so I don't forget.. give me 30 min.

avatar N6REJ N6REJ - test_item - 24 Jul 2017 - Tested successfully
avatar N6REJ
N6REJ - comment - 24 Jul 2017

I have tested this item successfully on f5337c9

This worked very nicely. RTC


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 24 Jul 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 24 Jul 2017

RTC after two successful tests.

avatar mbabker mbabker - change - 25 Jul 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-07-25 23:43:49
Closed_By mbabker
Labels Added: ?
avatar mbabker mbabker - close - 25 Jul 2017
avatar mbabker mbabker - merge - 25 Jul 2017

Add a Comment

Login with GitHub to post a comment