Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#2 | facetapi_bonus-1388644-1.patch | 1.83 KB | cpliakas |
#1 | facetapi_extra-1388630-1.patch | 710 bytes | cpliakas |
Comments
Comment #1
cpliakas CreditAttribution: cpliakas commentedThe attached patch makes the suggested change.
Comment #2
cpliakas CreditAttribution: cpliakas commentedScratch that... completely wrong patch! Attached patch is the correct one.
Comment #3
danielnolde CreditAttribution: danielnolde commentedDependency logic/return value: Didn't know that, corrected via your patch - thanks, Chris!