Comments

socketwench’s picture

This issue is postponed until event support is added to Rules. See https://www.drupal.org/node/2145343

shabana.navas’s picture

Am looking to get on it once the above issue is cleared up in Rules.

dasjo’s picture

Issue tags: +d8rules

While events aren't there yet, you can already implement actions & conditions for Rules. You can take a look at the core integrations that have been ported already:
https://github.com/fago/rules
#2269561: [META] Port all conditions to 8.x
#2269577: [META] Port all actions to 8.x

I think it would be great to create sub-tickets for creating some first action & condition integrations for the flag module to have a proof of concept for a first contrib module integrating with Rules 8.x :)

czigor’s picture

Issue summary: View changes
czigor’s picture

Issue summary: View changes
czigor’s picture

Issue summary: View changes
dasjo’s picture

Issue summary: View changes
dasjo’s picture

Status: Postponed » Active

Setting to active as some actions are already being ported. See their sub-tickets

martin107’s picture

triggering summary update

dasjo’s picture

note that rules events are now ready for porting as well #2513476: [META] Port all events to 8.x

dasjo’s picture

Issue summary: View changes
socketwench’s picture

We should start breaking down this meta into solvable issues now.

dasjo’s picture

I have pushed a few fixes to my flag fork: for example the Action namespace has been renamed to RulesAction within rules:
https://github.com/dasjo/flag/commits/8.x-4.x-rules

dasjo’s picture

Issue summary: View changes
Anonymous’s picture

Issue summary: View changes
Anonymous’s picture

Issue summary: View changes
Anonymous’s picture

Issue summary: View changes
Anonymous’s picture

Issue summary: View changes
emclaughlin’s picture

Question: do all of the Action ones now depend on #2470661: Port "Flag a [entity type]" Rules action to 8.x being approved and merged? It creates a FlagActionBase class that I assume all of the actions would need to use. I'd really like to dive into this.

joachim’s picture

That sounds quite likely. The patch on that issue needs review from someone who's familiar with Rules.

dasjo’s picture

#19 is correct, you would want to use the FlagActionBase from there

TR’s picture

Patch which implements "Flag is flagged" AND "Flag is unflagged" events, with tests, may be found in #2540032-21: Port "Flag is flagged" Rules event to 8.x

If you're interested in getting Rules functionality into Flag, please review that patch!

I will work on the Conditions after these two Events are committed.

TR’s picture

As I stated 1.5 years ago, I am willing to do the work to port the Rules functionality for this module from D7 to D8. This is all predicated on getting the events ported first. Please review #2540032-21: Port "Flag is flagged" Rules event to 8.x if this is of interest to you.

CraigBertrand’s picture

Hey folks, what is the status on this? I have installed rules and flag and am having issues getting things to work.

I would be happy to test and perhaps do a how-to video from a user perspective as commits are made.

mobius_time’s picture

Hi all, what is the status of this? What does the road forward look like?

I am developing a site which I may have to do in D7 if Flags and Rules don't work nicely together.

I need to create a rule so that when particular flag is set on a comment or a node, I can then take action.

Thanks to the maintainers for all their hard work on this D8 port.