Problem/Motivation

Required for multiple registration role does not match core's required field behavior.

When a boolean field is marked required and the form display widget chosen is "Check boxes/radio buttons", then the "N/A" option is not shown. Required for multiple registration role still shows the "N/A" option.

Screenshot showing the difference in behavior between core and multiple registration.

Note: The required behavior does work as expected, therefore the "N/A" option is not accepted as valid.

CommentFileSizeAuthor
#5 multiple_registration_this_field_is_required_for_n_a_option-3059370-5.patch1.06 KBysamoylenko
#4 multiple_registration-field_access-3021682-4.patch1.41 KBysamoylenko
#3 multiple_registration_this_field_is_required_for_n_a_option-3059370-3.patch1.02 KBysamoylenko
Screen Shot 2019-06-04 at 10.18.49 AM.png61.54 KBjcandan

Comments

jcandan created an issue. See original summary.

jcandan’s picture

jcandan commented
Issue summary: View changes
ysamoylenko’s picture

ysamoylenko commented
Status: Active » Needs review
StatusFileSize
new multiple_registration_this_field_is_required_for_n_a_option-3059370-3.patch1.02 KB

Hi @jcandan,

Thank you for your report.
I've created a patch which should fix the problem you described.

Please follow the next steps:
1. Apply the patch from this post.
2. Clear your site cache.
3. Check widgets again.

ysamoylenko’s picture

ysamoylenko commented
StatusFileSize
new multiple_registration-field_access-3021682-4.patch1.41 KB

I've updated the patch. Please use the latest version.

ysamoylenko’s picture

ysamoylenko commented
StatusFileSize
new multiple_registration_this_field_is_required_for_n_a_option-3059370-5.patch1.06 KB

I've updated the patch. Please ignore the previous one.

IreneV’s picture

IreneV commented

Looks great! Works
Please create another task to refactor

IreneV’s picture

IreneV commented
Status: Needs review » Reviewed & tested by the community

  • ysamoylenko committed bd63a55 on 8.x-2.x 
    Issue #3059370 by ysamoylenko, jcandan, IreneV: N/A option should not...
ysamoylenko’s picture

ysamoylenko commented
Status: Reviewed & tested by the community » Fixed

Thank you for the report and review.

The patch was committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.