Problem/Motivation

On click disableFacet() is called to prevent further clicking before the ajax completes. Attribute "disabled" is not properly assigned due to incorrect selector 'input.facetapi-checkbox'.
On the other hand, the click handler is applied to all 'input.facets-checkbox', instead of items in the current facet.

Proposed resolution

Change the selectors to be correc according to the function documentation.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

RumyanaRuseva created an issue. See original summary.

RumyanaRuseva’s picture

Status: Active » Needs review
FileSize
554 bytes

Here is a patch.

Status: Needs review » Needs work

The last submitted patch, 2: checkboxes_not_disabled-2950884-2.patch, failed testing. View results

borisson_’s picture

Status: Needs work » Reviewed & tested by the community

Looks great, there seems to be a general problem with the tests, but I think we can rtbc this one already.

StryKaizer’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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