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.

CommentFileSizeAuthor
#4 mailhandler-doesnt-delete.patch455 bytesz.stolar

Comments

nsciacca’s picture

I'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?

z.stolar’s picture

Title: "Delete messages" checkbox should have huge warning label » Mailhandler processes ALL messages if "Delete messages after they are processed" option is checked.
Category: feature » bug

Changed 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.

danepowell’s picture

Yes, that is correct. Sorry about the dramatic initial title- needless to say, this was a very embarrassing incident for me :)

z.stolar’s picture

Status: Active » Needs review
StatusFileSize
new455 bytes

Are 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.

danepowell’s picture

Status: Needs review » Reviewed & tested by the community

I 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.

danepowell’s picture

Version: 6.x-1.5 » 6.x-1.6

Is there a reason this patch didn't make it into the 6.x-1.6 release?

z.stolar’s picture

If it didn't - then it's a mistake.
I'll take care of couple more issues, and will release another version.
Sorry.

danepowell’s picture

No worries, I just wanted to make sure there wasn't an issue with the patch that I wasn't aware of.

Cheers

z.stolar’s picture

Status: Reviewed & tested by the community » Fixed

commited

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.