This post is probably not what you are thinking - I am not sad that all of my emails accidentally got deleted or something. I would actually be quite happy if that had been the case. My Mailhandler mailbox had a few dozen emails in it, and I decided I'd rather have Mailhandler simply delete them after processing rather than marking them as read, so I checked the "Delete messages after they are processed" box on admin/content/mailhandler/edit. Big mistake. It would seem that when this option is enabled, Mailhandler no longer checks to see if messages are unread before processing them - thus, every email in the box was processed a second time. This is very bad because on my site Mailhandler is integrated with Notifications to create a listserv, so all of the emails were also resent to all subscribers of the site.
This functionality should either be changed - so that Mailhandler won't process a message unless it is unread - or there should be a very large warning under the option advising users to purge their mailbox before enabling it.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | mailhandler-doesnt-delete.patch | 455 bytes | z.stolar |
Comments
Comment #1
nsciaccaI'm experiencing the same issue -- it appears that the "won't process unless it's unread" isn't actually working. Does anybody have a fix for this or have an idea of whether it will be addressed in the next release?
Comment #2
z.stolar commentedChanged the title. Is that correct?
I am planning a new release in the coming weeks (need to find the right time to process the issue queue). I'll test this bug and try to fix it.
Of course, if you already found a solution, please post it here.
Comment #3
danepowell commentedYes, that is correct. Sorry about the dramatic initial title- needless to say, this was a very embarrassing incident for me :)
Comment #4
z.stolar commentedAre you working in POP3 or IMAP mode?
It seems that some php's IMAP functions are not compatible with POP3.
Also, please test this patch.
Comment #5
danepowell commentedI am using IMAP.
The patch seems to correct the behavior - duplicate messages are no longer sent if the "Delete messages..." checkbox is checked while read messages remain in the mailbox.
Comment #6
danepowell commentedIs there a reason this patch didn't make it into the 6.x-1.6 release?
Comment #7
z.stolar commentedIf it didn't - then it's a mistake.
I'll take care of couple more issues, and will release another version.
Sorry.
Comment #8
danepowell commentedNo worries, I just wanted to make sure there wasn't an issue with the patch that I wasn't aware of.
Cheers
Comment #9
z.stolar commentedcommited