Closed (fixed)
Project:
Flag
Version:
7.x-3.x-dev
Component:
Flag core
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
9 Aug 2012 at 08:05 UTC
Updated:
24 Aug 2012 at 07:01 UTC
Jump to comment: Most recent file
Comments
Comment #1
joachim commentedAs seen on http://drupal.org/node/884554:
> Usually one also provides a hook for modules - in that case first invoke the hook, then the rules event.
Comment #2
joachim commentedAnd in EntityAPI, which is by the same author as Rules:
That clinches it :)
Comment #3
joachim commentedNote that this preserves the bug in #1719942: RulesEvaluationException with node unflagged event in the code that's moved.
Comment #4
joachim commentedI'm also tempted to replace this test:
with something based on the class of the current flag object being a child class of flag_entity, but that can wait.
Comment #5
joachim commentedTestbot is happy, patch is trivial, go go go.
Issue #1720196 by joachim: Changed Rules event to be invoked after hook_flag() rather than within it.