You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by no...@apache.org on 2010/06/23 15:47:05 UTC

svn commit: r957203 - /james/imap/trunk/store/src/main/java/org/apache/james/imap/store/StoreMessageManager.java

Author: norman
Date: Wed Jun 23 13:47:04 2010
New Revision: 957203

URL: http://svn.apache.org/viewvc?rev=957203&view=rev
Log:
Only call save on the Message if we need to update the recent flag (IMAP-161)

Modified:
    james/imap/trunk/store/src/main/java/org/apache/james/imap/store/StoreMessageManager.java

Modified: james/imap/trunk/store/src/main/java/org/apache/james/imap/store/StoreMessageManager.java
URL: http://svn.apache.org/viewvc/james/imap/trunk/store/src/main/java/org/apache/james/imap/store/StoreMessageManager.java?rev=957203&r1=957202&r2=957203&view=diff
==============================================================================
--- james/imap/trunk/store/src/main/java/org/apache/james/imap/store/StoreMessageManager.java (original)
+++ james/imap/trunk/store/src/main/java/org/apache/james/imap/store/StoreMessageManager.java Wed Jun 23 13:47:04 2010
@@ -420,8 +420,10 @@ public abstract class StoreMessageManage
                     results.add(member.getUid());
                     if (reset) {
                         member.unsetRecent();
+                        
+                        // only call save if we need to
+                        messageMapper.save(getMailboxId(), member);
                     }
-                    messageMapper.save(getMailboxId(), member);
                 }
             }
             



---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org