Posted by deekayen on June 3, 2009 at 6:15pm
Jump to:
| Project: | Role Watchdog |
| Version: | 6.x-1.0 |
| Component: | User interface |
| Category: | feature request |
| Priority: | normal |
| Assigned: | deekayen |
| Status: | closed (fixed) |
Issue Summary
I thought this was a bug, until I saw only the subject was set in the notification function. I think it would be useful to have the body say something about what role was changed. This is just a prototype patch to demonstrate what I mean.
case 'notification':
$message['subject'] = t('Role Watchdog notification on !site', $variables, $language->language);
+ $message['body'] = t('The @role was modified on your account at @site', array('@role' => $role, '@site' => $site_name));
break;
Comments
#1
I am getting a lot of confusing because my site's users are getting emails with the subject "Role Watchdog notification on [sitename]" with no body in the text. Thus the email conveys no information other than "something changed."
I would like to add this patch. Where do I put the code?
I would really like it to say something like "John Doe changed Jane Smith's role to Super-Editor on [timestamp]."
#2
Fixed in release 1.1
#3
Automatically closed -- issue fixed for 2 weeks with no activity.