? ? Success

User tests: Successful: Unsuccessful:

avatar Denitz
Denitz
22 Jun 2017

Steps to reproduce the issue

Create custom field extended from JFormFieldCheckboxes and add multiple options with disabled option in the middle of options.

Or test with XML:

<field name="test" type="checkboxes">
	<option value="1">1</option>
	<option value="2" disabled="true">2</option>
	<option value="3">3</option>
</field>

Expected result

Only 2nd checkbox is disabled.

Actual result

2nd and 3rd checkboxes are disabled.

System information (as much as possible)

As usual.

Additional comments

/layout/joomla/form/field/checkboxes.php:

The idea is to not use previous/field-specific 'disabled' attribute if there are more than 1 option.

avatar Denitz Denitz - open - 22 Jun 2017
avatar Denitz Denitz - change - 22 Jun 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 22 Jun 2017
Category Layout
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 22 Jun 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 22 Jun 2017

I have tested this item successfully on 2ef1888

Test on templateDetails.xml of Protostar.


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

avatar Denitz Denitz - change - 22 Jun 2017
The description was changed
avatar Denitz Denitz - edited - 22 Jun 2017
avatar Quy Quy - test_item - 22 Jun 2017 - Tested successfully
avatar Quy
Quy - comment - 22 Jun 2017

I have tested this item successfully on 2ef1888


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 22 Jun 2017
Status Pending Ready to Commit
avatar joomla-cms-bot joomla-cms-bot - edited - 22 Jun 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 22 Jun 2017

RTC after two successful tests.

avatar zero-24 zero-24 - change - 22 Jun 2017
Milestone Added:
avatar rdeutz rdeutz - change - 5 Jul 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-07-05 11:40:29
Closed_By rdeutz
avatar rdeutz rdeutz - close - 5 Jul 2017
avatar rdeutz rdeutz - merge - 5 Jul 2017

Add a Comment

Login with GitHub to post a comment