[Home]

Summary:ASTERISK-12517: [patch] Incorrect behavior when moving messages to/from New/Old folders
Reporter:James Rothenberger (jaroth)Labels:
Date Opened:2008-08-04 10:43:02Date Closed:2008-08-07 17:17:30
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_voicemail/IMAP
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) markread.patch
Description:When a message is moved from a folder (such as Work) to the New folder (IMAP INBOX) it is not marked as unread, thus it looks as though it is in the Old folder. Moving a message from the New folder (UNREAD) to the Old folder (or vice versa) does not change its read/unread status so in effect the move does not happen.

This patch adds a method imap_mark_read() that can be called to set a message to either read or unread, based on the flag used.  This method is called when moving a message into the New or Old folder to be sure that the message flag is set correctly.
Comments:By: Mark Michelson (mmichelson) 2008-08-06 11:48:28

This issue is actually addressed in my imap_consistency_trunk branch and so once it's ready for merging, the attached patch should not be necessary. I'll leave this bug open until that branch gets merged in, though.
By: Digium Subversion (svnbot) 2008-08-07 17:17:29

Repository: asterisk
Revision: 136715

_U  trunk/
U   trunk/apps/app_voicemail.c

------------------------------------------------------------------------
r136715 | mmichelson | 2008-08-07 17:17:28 -0500 (Thu, 07 Aug 2008) | 18 lines

Merging the imap_consistency_trunk branch to
trunk.

For an explanation of what "imap_consistency" is,
please see svn revision 134223 to the 1.4 branch.

Coincidentally, this also fixes a recent bug report
regarding the inability to save messages to the new
folder when using IMAP storage since they will would
be flagged as "seen" and not be recognized as new
messages.


(closes issue ASTERISK-12517)
Reported by: jaroth



------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=136715