Author: norman
Date: Sun Oct  9 17:40:48 2011
New Revision: 1180663

URL: http://svn.apache.org/viewvc?rev=1180663&view=rev
Log:
Skip messages in response which were not found in msn<->uid mappings. See 
IMAP-346

Modified:
    
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/AbstractMailboxProcessor.java

Modified: 
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/AbstractMailboxProcessor.java
URL: 
http://svn.apache.org/viewvc/james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/AbstractMailboxProcessor.java?rev=1180663&r1=1180662&r2=1180663&view=diff
==============================================================================
--- 
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/AbstractMailboxProcessor.java
 (original)
+++ 
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/AbstractMailboxProcessor.java
 Sun Oct  9 17:40:48 2011
@@ -234,15 +234,14 @@ abstract public class AbstractMailboxPro
             int msn = selected.msn(uid);
             if (msn == SelectedMailbox.NO_SUCH_MESSAGE) {
                 if (session.getLog().isDebugEnabled()) {
-                    if(session.getLog().isDebugEnabled()) {
-                        session.getLog().debug("No message found with uid " + 
uid + " in the uid<->msn mapping for mailbox " + 
selected.getPath().getFullName(mailboxSession.getPathDelimiter()) +" , this may 
be because it was deleted by a concurrent session. So skip it..");
-                        
-                    }
-                    // skip this as it was not found in the mapping
-                    // 
-                    // See IMAP-346
-                    continue;
-                }
+                    session.getLog().debug("No message found with uid " + uid 
+ " in the uid<->msn mapping for mailbox " + 
selected.getPath().getFullName(mailboxSession.getPathDelimiter()) +" , this may 
be because it was deleted by a concurrent session. So skip it..");
+                }  
+                    
+
+                // skip this as it was not found in the mapping
+                // 
+                // See IMAP-346
+                continue;
             }
 
             boolean qresyncEnabled = 
EnableProcessor.getEnabledCapabilities(session).contains(ImapConstants.SUPPORTS_QRESYNC);



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

Reply via email to