This isn't really a bug per-se, however the FacetapiDependency::execute() method employs the hook_node_access() approach where a plugin can either actively satisfy (TRUE), actively reject (FALSE), or ignore (NULL) to let other dependency plugins take action. Instead of returning TRUE, this module should return NULL so it works nicely with other dependency plugins.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cpliakas’s picture

Status: Active » Needs review
FileSize
710 bytes

The attached patch makes the suggested change.

cpliakas’s picture

Scratch that... completely wrong patch! Attached patch is the correct one.

danielnolde’s picture

Status: Needs review » Fixed

Dependency logic/return value: Didn't know that, corrected via your patch - thanks, Chris!

Status: Fixed » Closed (fixed)

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