Closed (fixed)
Project:
Role Change Notify
Version:
6.x-2.1
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
5 Feb 2010 at 02:31 UTC
Updated:
21 Mar 2010 at 23:13 UTC
Jump to comment: Most recent file
Comments
Comment #1
rfayI'm unable to replicate your error, but I do notice an error in the code (see below).
Could you please enable "Token Actions" module and see if the problem goes away? I suspect it will. I think you have token on and token actions is not on.
The error is described here:
should be
Comment #2
rfayI fixed this issue: http://drupal.org/cvs?commit=326832
@scott859, I still look forward to your further testing on this. Please let me know if the new dev release solves this for you. You can also send me a database (or attach one) for further investigation. I'm marking this fixed, but please reopen if it's not.
Comment #3
3dloco commentedHello Rfay
I am getting this same error (now on line 112) after an update from 6.x-2.0 to 2.1, installing workflow, and rebuilding node permissions (although I am not sure whether the error was there before installing workflow).
I was not getting this error before with 6.x-2.0.
I am currently using Token 6.x-1.12 and have token actions enabled.
Thanks,
KH
Comment #4
rfay@3dloco, thanks for the detailed report.
Would it be possible for you to provide me with at least a failing database (dump) or (better) a tarball of your drupal directory *and* a db dump?
Comment #5
rfayThanks for the good description, @3dloco. It was shown easily just by enabling workflow module, and it was a flaw in role_change_notify.
Thanks to all of you for chiming up on this.
Committed: http://drupal.org/cvs?commit=331488
Please try the dev release as soon as it's built (like tomorrow morning) and see if it's fixed for you.
I should mention that there's a very straightforward way to make this happen:
1. Enable role_change_notify, token, token_actions, trigger
2. Enable workflow module - you'll see it on enabling, and on visiting the triggers page.
Comment #7
scott859 commented@rfray
My apologies for not getting back to you on this. The latest Dev version is working fine - thanks for your help on this.
Best,
Scott