Hi,
When using "Mass Mail by User" the drop down lists shows all users in the system - even those without permission to read Private Messages. If by accident a PM is sent to one of these users who doesn't yet have PM permissions when they finally do, that previously sent message is visible in their Inbox.
Would it be possible for the drop down list in "Mass Mail by User" to NOT include users who don't have PM permissions? hopefully this will prevent them receiving email which at the current time they can't possibly read.
Thank you.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | readMsgPerm.patch | 1014 bytes | kriskd |
| #3 | readMsgPerm.patch | 1006 bytes | kriskd |
Comments
Comment #1
McGo commentedNice idea! I'll take it on my "do those things when there is time, hopefully before winter"-list. If you want to hack that thing, feel free to file a patch for this feature. If you'd like me to implement it a bit earlier, get in contact.
Comment #2
victoria_b commentedMcGo,
Ha! Np - I'm not a coder otherwise I would have a crack at it. I appreciate that you're well busy... so winter it is!
Comment #3
kriskd commentedHere's a quick and dirty patch I just did to address this. All it does is adjust the SQL statements to get users that have the 'read privatemsg' permission. No guarantees. Please test and I will try to fix if it doesn't address the problem correctly.
Comment #4
kriskd commentedArgh... forgot braces around the table names in the query for realname. Here is a correction.
Comment #5
victoria_b commentedHi,
Patch worked fine on my dev site version. Thank you for the quick turn around!
Victoria
Comment #6
kriskd commentedCool! Glad I was able to contribute. I hope it was a help to McGo as well.