Feeling Lucky
NPM Resource Changed
?
?
[#28942] - [4.0] Fix required checkbox client-side validation
- Fixed in Code Base
- 18 May 2020
- Medium
- Build: 4.0-dev
- # 28942
- Diff
- SharkyKZ:j4/js/validate/checkbox
- Pending Hound Hound is busy sniffing around... Details
User tests: Successful: Unsuccessful:
Summary of Changes
Fixes JS validation for required checkboxes.
Testing Instructions
Create a form containing required checkbox.
E.g. add required="true" to both field in administrator/components/com_languages/forms/override.xml:
<field
name="both"
type="checkbox"
label="COM_LANGUAGES_OVERRIDE_FIELD_BOTH_LABEL"
value="true"
filter="boolean"
required="true"
/>
Go to Language Override form. Check the checkbox and click somewhere on the page.
Expected result
No validation warning shown.
Actual result
Warning shown:
One of the options must be selected
Documentation Changes Required
No.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | JavaScript Repository NPM Change |
chmst
- comment
- 16 May 2020
The test of js is successfull but the result is not correct.
If a checkbox is required, but the user did not check, the messag is "One of the options must be selected".
While there are not several options.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28942.
chmst
- comment
- 16 May 2020
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28942.
| Status | Pending | ⇒ | Ready to Commit |
richard67
- comment
- 16 May 2020
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28942.
| Labels |
Added:
?
?
NPM Resource Changed
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-05-18 22:09:45 |
| Closed_By | ⇒ | wilsonge | |
| Labels | |||
wilsonge
- comment
- 18 May 2020
Thanks!
I have tested this item✅ successfully on b7d1382
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28942.