This is not the same as #735528: FAPI #states: Fix conditionals to allow OR and XOR constructions

Much simpler: A simple OR condition can be expressed as jQuery selector like this:

        '#states' => array(
          'visible' => array(
            ':input[name="checkboxes[foo]"], :input[name="checkboxes[bar]"]' => array('checked' => TRUE),
          ),
        ),

Translating:

  1. The selector matches more than one checkbox.
  2. The element should be visible when at least one of the checkboxes is checked.
  3. When no checkbox is checked, the element should not be visible.

Attached patch makes that work.

CommentFileSizeAuthor
#20 1057748-19.patch711 bytesrpayanm
#16 drupal-1057748-16.patch1.14 KBdmitriy.trt
#14 trigger_state-1057748-d7-14.patch711 bytesalbert volkman
#14 interdiff.txt993 bytesalbert volkman
#12 trigger_state-1057748-d7-12.patch788 bytesalbert volkman
#11 trigger_state-1057748-d7-11.patch788 bytesalbert volkman
#9 drupal8.states-checked.9.patch942 bytessun
drupal.states-checked.0.patch712 bytessun

Comments

dmitriy.trt’s picture

dmitriy.trt commented

subscribing

dmitriy.trt’s picture

dmitriy.trt commented
Status: Needs review » Reviewed & tested by the community

It works, thanks a lot! Patch makes it possible to depend on "checkboxes" element with simple condition like:

'#states' => array(
  'invisible' => array(
    'input[name^="checkboxes_name"]' => array('checked' => FALSE)
  )
)

This means "element is invisible if all of checkboxes are unchecked".

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Version: 7.x-dev » 8.x-dev
Status: Reviewed & tested by the community » Needs review

D8 first. Let's get these fixed on D8 and into D7.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

drupal.states-checked.0.patch queued for re-testing.

james.williams’s picture

james.williams
Location Coventry, UK
commented

subscribe

geerlingguy’s picture

geerlingguy commented

Sub.

nagiek’s picture

nagiek commented

Patch works for me, running Drupal 7.10. Handy!

nagiek’s picture

nagiek commented
Status: Needs review » Reviewed & tested by the community
sun’s picture

sun
Primary language German
Location Karlsruhe
commented
StatusFileSize
new drupal8.states-checked.9.patch942 bytes

Re-rolled against latest HEAD.

catch’s picture

catch
he/him
Primary language English
commented
Version: 8.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)
Issue tags: +Needs backport to D7

Looks fine, has had manual testing, shame we don't have qunit for stuff like this.

Committed/pushed to 8.x, back to 7.x for backport.

albert volkman’s picture

albert volkman commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new trigger_state-1057748-d7-11.patch788 bytes

D7 backport.

albert volkman’s picture

albert volkman commented
StatusFileSize
new trigger_state-1057748-d7-12.patch788 bytes

bump (and updated patch to apply cleanly)

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

This patch looks safe to backport for me, but I wonder whether the jQuery in D7 core supports .prop()?

(I guess it does, since I originally rolled this patch for D7, but I'm not 100% sure.)

albert volkman’s picture

albert volkman commented
StatusFileSize
new interdiff.txt993 bytes
new trigger_state-1057748-d7-14.patch711 bytes

Good point. Just checked this and D7 has jQuery 1.4.4-

http://drupalcode.org/project/drupal.git/blob/refs/heads/7.x:/misc/jquer...

and .prop() wasn't introduced until jQuery 1.6-

http://api.jquery.com/prop/

Perhaps we should use .is() instead?

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Assigned: sun » nod_

Perhaps @nod_ can help out with those question.

dmitriy.trt’s picture

dmitriy.trt commented
StatusFileSize
new drupal-1057748-16.patch1.14 KB

Patch #14 doesn't work because of missing colon before "checked" pseudo-class. Updated patch attached.

jason.fisher’s picture

jason.fisher commented

@16

Are you missing the check to see if it is checked? It seems like you are always returning the state of the first checkbox?

dmitriy.trt’s picture

dmitriy.trt commented

@jason.fisher,

Innermost function breaks the jQuery.each() loop only if checked checkbox is found. Otherwise the loop will move to the next one. See jQuery API documentation:

You can stop the loop from within the callback function by returning false.

mgifford’s picture

mgifford
he/him
Primary language English
commented
Issue summary: View changes
Status: Needs review » Needs work
Issue tags: +Needs reroll

Not surprising that the patch from 2 years ago needs a reroll.

rpayanm’s picture

rpayanm
he/him
Primary language Spanish
commented
Status: Needs work » Needs review
StatusFileSize
new 1057748-19.patch711 bytes
mgifford’s picture

mgifford
he/him
Primary language English
commented
Issue tags: -Needs reroll
mgifford’s picture

mgifford
he/him
Primary language English
commented
Assigned: nod_ » Unassigned

@rpayanm reroll applies nicely in SimplyTest.me

What do we need to do to test this patch to mark it RTBC?

  • catch committed 56a7a11 on 8.3.x 
    Issue #1057748 by sun: Fixed #states selector matching multiple...

  • catch committed 56a7a11 on 8.3.x 
    Issue #1057748 by sun: Fixed #states selector matching multiple...

  • catch committed 56a7a11 on 8.4.x 
    Issue #1057748 by sun: Fixed #states selector matching multiple...

  • catch committed 56a7a11 on 8.4.x 
    Issue #1057748 by sun: Fixed #states selector matching multiple...

  • catch committed 56a7a11 on 9.1.x 
    Issue #1057748 by sun: Fixed #states selector matching multiple...

Status: Needs review » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.