Hi,

what i try to to is this:

open a field if checkboxes A+B are checked. But it doesnt work via conditionals. Field appears when only one of the checkboxes is checked. But i set the condition that this should happens only when both are checked (see screenshots).

best,

CommentFileSizeAuthor
#11 conditionals_with-2855258-11.patch2.52 KBjrockowitz
#9 conditionals_with-2855258-9.patch1.93 KBjrockowitz
#6 webform.JPG33.38 KBvgutekunst
#2 conditionals_with-2855258-2.patch1.26 KBjrockowitz
webform2.JPG17.89 KBvgutekunst
webform-conditionals.JPG89.62 KBvgutekunst

Comments

lolhonk created an issue. See original summary.

jrockowitz’s picture

jrockowitz commented
Status: Active » Needs review
StatusFileSize
new conditionals_with-2855258-2.patch1.26 KB

The attached patch adds the below example of multiple checkboxes being used as trigger to 'Example: Elements: Condition Logic' (form/example-elements-states)

checkboxes_example:
  '#type': details
  '#title': 'Checkboxes example'
  '#open': true
  checkboxes:
    '#type': checkboxes
    '#title': 'Please check all'
    '#options':
      one: One
      two: Two
  checkboxes_explain:
    '#type': textarea
    '#title': 'Please explain why you checked all the above box'
    '#states':
      visible:
        ':input[name="checkboxes[one]"]':
          checked: true
        ':input[name="checkboxes[two]"]':
          checked: true
      required:
        ':input[name="checkboxes[one]"]':
          checked: true
        ':input[name="checkboxes[two]"]':
          checked: true

  • jrockowitz committed cbe29a8 on 8.x-5.x 
    Issue #2855258 by jrockowitz, lolhonk: conditionals with multiple...
jrockowitz’s picture

jrockowitz commented
Status: Needs review » Closed (works as designed)

I committed the working example. Please checkout the latest dev release and review.

vgutekunst’s picture

vgutekunst commented

youre crazy fast and good!

i checked it and i can confirm it works with element "textarea" but not with element "message".

here the yaml

checkboxes_example:
  '#type': details
  '#title': 'Checkboxes example'
  '#open': true
  checkboxes:
    '#type': checkboxes
    '#title': 'Please check all'
    '#options':
      one: One
      two: Two
  checkboxes_explain:
    '#type': textarea
    '#title': 'Please explain why you checked all the above box'
    '#states':
      visible:
        ':input[name="checkboxes[one]"]':
          checked: true
        ':input[name="checkboxes[two]"]':
          checked: true
      required:
        ':input[name="checkboxes[one]"]':
          checked: true
        ':input[name="checkboxes[two]"]':
          checked: true
  455:
    '#type': message
    '#states':
      visible:
        - ':input[name="checkboxes[one]"]':
            checked: true
        - ':input[name="checkboxes[two]"]':
            checked: true
      required:
        - ':input[name="checkboxes[one]"]':
            checked: true
        - ':input[name="checkboxes[two]"]':
            checked: true
    '#message_message': |
      <pre>
      <code>Message body

vgutekunst’s picture

vgutekunst commented
StatusFileSize
new webform.JPG33.38 KB
vgutekunst’s picture

vgutekunst commented
Status: Closed (works as designed) » Needs review
jrockowitz’s picture

jrockowitz commented

I can duplicate the issue. The below code...

455:
    '#type': message
    '#states':
      visible:
        ':input[name="checkboxes[one]"]':
          checked: true
        ':input[name="checkboxes[two]"]':
          checked: true
    '#message_message': 'Message body'

...is being converted via the UI to...

455:
    '#type': message
    '#states':
      visible:
        - ':input[name="checkboxes[one]"]':
            checked: true
        - ':input[name="checkboxes[two]"]':
            checked: true
    '#message_message': 'Message body'

which is being interpreted as an 'or' instead of an 'and' operator.

jrockowitz’s picture

jrockowitz commented
StatusFileSize
new conditionals_with-2855258-9.patch1.93 KB

Status: Needs review » Needs work

The last submitted patch, 9: conditionals_with-2855258-9.patch, failed testing.

jrockowitz’s picture

jrockowitz commented
Status: Needs work » Needs review
StatusFileSize
new conditionals_with-2855258-11.patch2.52 KB

  • jrockowitz committed b3c237e on 8.x-5.x 
    Issue #2855258 by jrockowitz, lolhonk: conditionals with multiple...
jrockowitz’s picture

jrockowitz commented
Status: Needs review » Fixed

Fixed. Pleas checkout or download the latest dev release.

vgutekunst’s picture

vgutekunst commented

It works! Thank you very much jrockowitz!

Status: Fixed » Closed (fixed)

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