[Tickets #13010] Re: Activesync on WP7.8 deleted messages reappear

[noreply at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/13010
------------------------------------------------------------------------------
  Ticket             | 13010
  Updated By         | spamstop2 at terriertech.com
  Summary            | Activesync on WP7.8 deleted messages reappear
  Queue              | Synchronization
  Version            | Git master
  Type               | Bug
  State              | Feedback
  Priority           | 1. Low
  Milestone          |
  Patch              | 1
  Owners             |
+New Attachment     | Activesync-delete-2.patch
------------------------------------------------------------------------------


spamstop2 at terriertech.com (2014-03-05 14:13) wrote:

> The correct way to workaround this via IMAP is to search for the  
> message ID header in the new mailbox.  You absolutely can't use the  
> UIDNEXT value since servers don't necessarily use the next  
> sequential UID (i.e. Gmail).  The only IMAP requirement for UIDs is  
> that they must be ascending; there is no requirement they must be  
> contiguous.

Thank you for the hints.  I've attached a new patch for  
/usr/share/pear/Horde/ActiveSync/Imap/Adapter.php which does the  
above, and seems to fix the problem (limited testing so far).

A few notes/questions:

- The submitted patch fetches the message-id from BOTH the 'from' and  
'to' folders.  If it is already known to Horde, I couldn't see where.

- If moving multiple UID's, the patch looks up each message-id in the  
'to' folder as a separate IMAP query.  Maybe it would be better to OR  
them?  I left this as an optimization to see if the solution is on the  
right track first.

- I haven't patched the base IMAP client, just ActiveSync.  I suppose  
it is an open question whether looking up the message-id's in the 'to'  
folder should be done within the copy() function.

Here is a new log with the patch:

2014-03-05T08:12:08-06:00 INFO: [30698] Handling MoveItems command.
2014-03-05T08:12:08-06:00 DEBUG: [30698] I  <Move:Moves>
2014-03-05T08:12:08-06:00 DEBUG: [30698] I   <Move:Move>
2014-03-05T08:12:08-06:00 DEBUG: [30698] I    <Move:SrcMsgId>
2014-03-05T08:12:08-06:00 DEBUG: [30698] I      95473
2014-03-05T08:12:08-06:00 DEBUG: [30698] I    </Move:SrcMsgId>
2014-03-05T08:12:08-06:00 DEBUG: [30698] I    <Move:SrcFldId>
2014-03-05T08:12:08-06:00 DEBUG: [30698] I      Fdf2bae0c
2014-03-05T08:12:08-06:00 DEBUG: [30698] I    </Move:SrcFldId>
2014-03-05T08:12:08-06:00 DEBUG: [30698] I    <Move:DstFldId>
2014-03-05T08:12:08-06:00 DEBUG: [30698] I      F861db7b0
2014-03-05T08:12:08-06:00 DEBUG: [30698] I    </Move:DstFldId>
2014-03-05T08:12:08-06:00 DEBUG: [30698] I   </Move:Move>
2014-03-05T08:12:08-06:00 DEBUG: [30698] I  </Move:Moves>
2014-03-05T08:12:08-06:00 DEBUG: [30698] O  <Move:Moves>
2014-03-05T08:12:08-06:00 DEBUG: [30698] O   <Move:Response>
2014-03-05T08:12:08-06:00 DEBUG: [30698] O    <Move:SrcMsgId>
2014-03-05T08:12:08-06:00 DEBUG: [30698] O     95473
2014-03-05T08:12:08-06:00 DEBUG: [30698] O    </Move:SrcMsgId>
2014-03-05T08:12:08-06:00 INFO: [30698]  
Horde_Core_ActiveSync_Driver::moveMessage(INBOX, [95473], Deleted Items)
2014-03-05T08:12:08-06:00 INFO: [30698] Updating state during delete
2014-03-05T08:12:08-06:00 DEBUG: [30698] O    <Move:Status>
2014-03-05T08:12:08-06:00 DEBUG: [30698] O     3
2014-03-05T08:12:08-06:00 DEBUG: [30698] O    </Move:Status>
2014-03-05T08:12:08-06:00 DEBUG: [30698] O    <Move:DstMsgId>
2014-03-05T08:12:08-06:00 DEBUG: [30698] O     63
2014-03-05T08:12:08-06:00 DEBUG: [30698] O    </Move:DstMsgId>
2014-03-05T08:12:08-06:00 DEBUG: [30698] O   </Move:Response>
2014-03-05T08:12:08-06:00 DEBUG: [30698] O  </Move:Moves>
2014-03-05T08:12:08-06:00 INFO: [30698] Maximum memory usage for  
ActiveSync request: 8908912 bytes.




spamstop2 at terriertech.com (2014-03-05 14:13) uploaded:  
Activesync-delete-2.patch

http://bugs.horde.org/h/services/download/?app=whups&actionID=download_file&file=Activesync-delete-2.patch&ticket=13010&fn=%2FActivesync-delete-2.patch