The for..in loop used to loop over the array in Drupal.behaviors.facetapi.attach() fails if Drupal.settings.facetapi.facets has an enumerable property that does not represent a facet.
In our case, some component extends the Array.prototype with an enumerable property toJSON, making the loop fail. The looping construct used here must be better suited to array looping.

We will submit a patch very soon.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cussack created an issue. See original summary.

cussack’s picture

cussack’s picture

Status: Active » Needs review
eugene.ilyin’s picture

Status: Needs review » Postponed (maintainer needs more info)

In our case, some component extends the Array.prototype with an enumerable property toJSON, making the loop fail.

Mhh, it sounds like your special case. How does it happen?