[Bug 166150] GMail filtering does not work if the message is not moved

[Thomas McGuire]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=166150         




------- Additional Comments From Thomas.McGuire gmx net  2008-07-16 23:08 -------
SVN commit 833475 by tmcguire:

Some fixes for online IMAP filtering:

- In the action scheduler, don't error out when the filtered message
  couldn't be moved back to the target folder. Instead, ignore the error
  (but don't delete the orginal message). This fixes filtering 
  stopping on GMail accounts once a message was encountered that was not
  moved to another folder, but stayed in the same folder (GMail prevents moving
  in this case, since it thinks it is a duplicate message)

- When moving the filtered message with the action scheduler from the
  temporary filter folder to the target folder, the original message wouldn't
  get deleted properly.
  The reason for this was that the move command thought the move failed because
  a message with another serial number arrived.
  Fix this by remembering the serial number (based on the message ID) when using
  the action scheduler for filtering.
  This fixes filters which move messages to other folders: Now the message is properly
  removed from the source folder again.

- add comments and kDebug output

This does _not_ solve the following problem:
When using GMail and online IMAP, filter actions which modify the message will not have
any effect, since GMails duplicate message prevention prevents the filtered message to
be moved back to the IMAP folder.
This will not be fixed, I see no way to work around this. GMail should fix their IMAP
server instead.

Please test, I'm not sure if this is safe to backport.

CCBUG: 166150


 M  +80 -38    actionscheduler.cpp  
 M  +26 -1     actionscheduler.h  
 M  +20 -2     imapjob.cpp  
 M  +7 -0      kmfolderimap.cpp  
 M  +8 -0      kmfolderimap.h  
 M  +1 -1      kmmessage.h  
 M  +1 -1      kmmsgbase.cpp  
 M  +1 -1      kmmsgbase.h  
 M  +1 -1      kmmsginfo.cpp  
 M  +1 -1      kmmsginfo.h  
 M  +20 -0     messageproperty.cpp  
 M  +23 -1     messageproperty.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=833475