There is currently a silent (and un-necessary) dependency on the node module in _eu_cookie_compliance_find_privacy_policy() where it uses a node query to try and find a policy. This should only run if the node entity exists.

CommentFileSizeAuthor
#5 eu_cookie_compliance-remove_silent_node_dependency-2959617-5-7.x.patch612 bytessvenryen
#2 2959617-2-remove_silent_node_dependency.patch611 bytesandrewbelcher

Comments

andrewbelcher created an issue. See original summary.

andrewbelcher’s picture

andrewbelcher commented
Status: Active » Needs review
StatusFileSize
new 2959617-2-remove_silent_node_dependency.patch611 bytes

Here's a patch that checks the node entity exists before querying it.

svenryen’s picture

svenryen commented

Thanks Andrew! By any chance, are you also able to backport the patch to Drupal 7?

andrewbelcher’s picture

andrewbelcher commented

Happy to do it when I get a chance, but may not be this week!

svenryen’s picture

svenryen commented
StatusFileSize
new eu_cookie_compliance-remove_silent_node_dependency-2959617-5-7.x.patch612 bytes

I ported it to d7. Here's the patch.

  • svenryen committed 47e3d4a on 7.x-1.x 
    Issue #2959617 by svenryen: Remove silent dependency on node
svenryen’s picture

svenryen commented
Status: Needs review » Fixed

  • svenryen committed 47e3d4a on 7.x-2.x 
    Issue #2959617 by svenryen: Remove silent dependency on node
  • svenryen committed a44ca28 on 7.x-2.x
    Merge branch '7.x-1.x' into 7.x-2.x

* 7.x-1.x:
Issue #2965976 by...
svenryen’s picture

svenryen commented
Assigned: andrewbelcher » Unassigned

Status: Fixed » Closed (fixed)

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