Once the "winning formula" of #298109: Rules integration is reached, we can convert it to wfng.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | flag_workflow_ng.info.txt | 111 bytes | cyu |
| #7 | flag_workflow_ng.module.txt | 2.24 KB | cyu |
Once the "winning formula" of #298109: Rules integration is reached, we can convert it to wfng.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | flag_workflow_ng.info.txt | 111 bytes | cyu |
| #7 | flag_workflow_ng.module.txt | 2.24 KB | cyu |
Comments
Comment #1
cyu commentedWill it even be possible for both workflow-ng and actions2 to be supported by a module? In this case, both workflow-ng and actions2 will be looking for flag_action_info() to define their respective actions. I had commented out Flag's actions support when I started messing with workflow-ng integration because the latest release throws errors if actions2 is not installed, but once I grabbed the CVS copy of Flag and saw that it gracefully degraded for actions1 I started getting errors because I was redefining flag_action_info().
Replacing
with
should temporarily fix that for me, but if I upgrade my site to actions2 it will break it again.
If there is going to be workflow-ng support added eventually, it seems like you'll need a settings page where a choice must be made between actions2 support or workflow-ng support.
Comment #2
cyu commentedHere is a little bit of related history on this issue:
#235612: Fatal error: Cannot redeclare user_action_info
Comment #3
cyu commentedI guess one option is to actually have a separate module like flag_workflow_ng.module which would then have hooks that would not clash with the actions 2 hooks of flag.module
Comment #4
mooffie commentedI'm marking this 'postponed' till the other issue gets settled:
#298109: Rules integration
PHP has a function to inspect the call-stack, so the hook can return different things depending on who's calling.
But I think the issue you linked to says the problem was solved already.
Comment #5
quicksketchI dare say this will never happen. I'm not working on Rules nor am are any of the maintainers working on Drupal 5, so Workflow-NG support is all but out of the question. Patches for this feature will probably not be considered, it's time to move to Drupal 6.
Comment #6
crea commentedAny idea how can I implement this backport myself ?
Is this backporting really hard, in case i'm not using Actions 2.0 ?
Comment #7
cyu commentedI wrote some code which worked for me in 5 to integrate workflow-ng with flag module. I believe it takes a different approach than the rules integration in 6, but I'll post it here in case anyone stumbles across this issue and can make use of it. I agree about moving towards Drupal 6 so am leaving this as won't fix.
Comment #8
crea commentedMoving to 6 is only worth it for new sites.
I have one deeply developed community site with 5.x and moving it to 6 won't give anything worth the effort...only headache.
For established site it is easier to backport missing module when its greatly needed (which is very rare).