Posted by rfay on December 29, 2008 at 12:29am
Jump to:
| Project: | Apply for role |
| Version: | 6.x-1.8 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
The module should use user_save() or similar mechanism to update the user's roles. Currently it is directly manipulating the database. This means that other modules cannot act on role changes, because they never get notified. Role Change Notify for example, can't send out role change notification messages. Workflow can't take action, etc.
Attached is a patch that uses user_save() to do the change to roles both on approve and deny.
| Attachment | Size |
|---|---|
| user_save_mod.patch | 1018 bytes |
Comments
#1
sweet patch!
#2
Thank you!
Applied to -dev.
S.
#3
Automatically closed -- issue fixed for two weeks with no activity.