Imagine my consternation when every registered user began getting notifications of every change of every existing or new user! At it looked like every user was getting them. I didn't do much checking, obviously, I just stopped it ASAP.
When I finally shut down my mail server I had over 8,000 notices in the out queue! I stopped most of them, but I still have a number of miffed users. Yikes!
Well, the notes to say the email notification hasn't been tested. I had just installed the thing and hadn't even looked at the settings yet. PLEASE set the default as NO notifications.
-- Cronin
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | advuser.5.x-2.x-dev.patch | 8.3 KB | Anonymous (not verified) |
Comments
Comment #1
Anonymous (not verified) commentedNASTY!!! Sorry for the consternation. I hope that by the end of my day today I can upload a patch. I'll keep you posted.
Comment #2
Anonymous (not verified) commentedIssue title change.
Comment #3
Anonymous (not verified) commented@ztnews: You perhaps chose ``authenticated user'' under the ``Notification Roles''? Please confirm.
Comment #4
cvining commentedI hadn't touched the "Notification Roles" at all. Nor any other module settings. I installed the module and was doing some searching/filtering to identify a few users (which worked well, BTW). Before I knew it these notification emails were going out.
Usually I test new modules on a small test site, but this time I went straight with a live site. That was my bad.
-- Cronin
Comment #5
perlgal commentedYes, I also had the same problem. Just installed this module and the next time change in profile email was sent to all the registered users!
Can you tell me where to disable Notification Roles!
Thanks
Comment #6
Anonymous (not verified) commentedI've found the bug. It definitely is due to the fact that no role is selected. If a role is selected the correct list of users will be given. I'll be revamping that piece of code sometime soon. I'll provide a patch to fix this issue sometime later today. The patch will also set the default notification to false as it should have been from day one. Notification settings are set at admin/settings/advuser; uncheck the boxes and hit save.
Comment #7
Anonymous (not verified) commentedThe attached patch resolves this issue. I have removed "authenticated users" from the selectable roles. I have added a pretend "super user" in the selectable roles but at present this doesn't work. I have reformatted the default email notifications and extended the token variables. I have tested with the devel module intercepting the emailing the user function. I have tested with the devel module intercepting the email that no notification is sent with no role selected. I have not tested delivering email for real. I have not tested if email notification is sent with a role selected. I have modified the default notifications request to FALSE. I have modified the permissions to change "email users advuser" to "send email advuser" and have added "receive email advuser". These two permission settings at the moment do nothing. I will be modifying the administration page to remove the role selection and instead use the "receive email advuser" to control distribution of notifications.
Comment #8
Anonymous (not verified) commentedWatch for the updated file during the next cron occurrence.
Comment #9
cvining commentedearnie,
Thanks for the fast work.
One of my site users affected by this was very Zen: the only way to avoid all mistakes is by doing nothing at all.
-- Cronin
Comment #10
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.