"After importing feed" rules event never seems to fire, you can create a rule with this as an event fine. But then the actions are never triggers.
We have upgraded from the stable version to attempt to use this feature.
"After importing feed" rules event never seems to fire, you can create a rule with this as an event fine. But then the actions are never triggers.
We have upgraded from the stable version to attempt to use this feature.
Comments
Comment #1
specky_rum commentedAnyone got any news on this?
Comment #2
generalredneckThis is still an issue. It seems that the events are never fired anywhere in feeds. I've got a patch but it needs testing. I'm not 100% sure that the conditions used in conjunction with the event work correctly. If you need a specific version of feeds fixed, you should be able to reroll this patch fairly easily.
Comment #4
generalredneck#2: feeds-making_rules_events_fire-1886230-2.patch queued for re-testing.
Comment #6
zephirus commentedAny news about this issue? I really need to perform some actions on "After importing feed" rule.
Comment #7
generalredneckI know for a fact this works because I'm using it on a production site right now... I just need to get it stable so that it pasts tests. With that said, Use at your own risk.
I'll see what I can do about the tests this morning.
Comment #8
generalredneckThis should fix the problems. It looks like the tests weren't enabling the rules module. This did bring out the fact that I don't check to see if one had the rules module. Not having the rules module enabled proved to be fatal... as I am now posting this patch after seeing the white light of a WSOD. It wasn't pure though... as it had a twinge of black that said something about rules_invoke_event wasn't found.
Take it away testbot!
Comment #10
generalredneckok the failing tests make no sense to me... anyone want to help on that reguard? It's all stuff I haven't even bothered to touch. I can assure you though that the changes do work and seem to be stable from my testing...
Comment #11
benchesters commentedIs there any more news on this, I need this rule to fire! Thanks.
Comment #12
dexiecarla commentedSubscribing
Comment #13
zuernbernhard commentedThese hooks / rules implementations don't fire in cron but when you start the feeds-imnport manually from backend with file_upload-fetcher. there is no module_invoke_all or rule_invoke_all in case of cron usage !
Comment #14
werfu commentedI have the same issue, I had this event working and suddenly something caused it to not work anymore. I've digged through the feeds.rules.inc and seen how the feeds_rules_event_info generate the handler for each Feeds importer. However, does somebody know where the rules_invoke_event for these handler get called?
Comment #15
glynnr commentedThere is no rules_invoke_event for this event, which makes it ineffective.
My workaround is to implement hook_feeds_after_import and then invoke it myself.
Comment #16
jelo commentedAny progress on this? Would be great to have it functioning again by adding the proper call to fire the rule off. In the meantime glynnr's workaround did it for me...
Comment #17
megachrizApparently, you need to install the Feeds Rules module for the invoking event to have effect. The code was added in #1744570: Invoke Before import event before start import (I found this out by executing
git blame feeds.rules.inc). I believe there happened some miscommunication in that issue.The attached patch is a sort of reroll of #8 without adding a dependency on Rules in the automated tests. The tests in #8 failed because the testbot didn't know it needed to checkout the Rules module (you can only tell the testbot this by committing a
dependencies[]entry or atest_dependencies[]entry to the module's .info file).Comment #18
ConradFlashback commentedSub
Comment #20
megachrizNow with an automated test. And also a reroll, because the patch in #17 no longer applies.
The test only patch should fail.
Comment #22
megachrizOops.
Comment #25
megachrizCommitted #22.