83831 r=bienveu sr=mscott; don;t update the count if the copy has failed

and also continue to copy messsages even if one fails inbetween, for
multiple messages.

mailnews/imap/src/nsImapMailFolder.cpp

@@ -3886,17 +3886,20 @@ nsImapMailFolder::OnStopRunningUrl(nsIURI *aUrl, nsresult aExitCode)
         case nsIImapUrl::nsImapAppendDraftFromFile:
             if (m_copyState)
             {
-              if (folderOpen)
-                UpdateFolder(aWindow);
-              else
-                UpdatePendingCounts(PR_TRUE, PR_FALSE);
+              if (NS_SUCCEEDED(aExitCode))
+              {
+                if (folderOpen)
+                  UpdateFolder(aWindow);
+                else
+                  UpdatePendingCounts(PR_TRUE, PR_FALSE);
+              }
               m_copyState->m_curIndex++;
               if (m_copyState->m_curIndex >= m_copyState->m_totalCount)
               {
-                  if (m_transactionManager && m_copyState->m_undoMsgTxn)
-                      m_transactionManager->DoTransaction(m_copyState->m_undoMsgTxn);
-                  ClearCopyState(aExitCode);
-                  sendEndCopyNotification = PR_TRUE;
+                if (m_transactionManager && m_copyState->m_undoMsgTxn)
+                    m_transactionManager->DoTransaction(m_copyState->m_undoMsgTxn);
+                ClearCopyState(aExitCode);
+                sendEndCopyNotification = PR_TRUE;
               }
               else
                 NS_ASSERTION(PR_FALSE, "not clearing copy state");

mailnews/imap/src/nsImapProtocol.cpp

@@ -1158,8 +1158,7 @@ PRBool nsImapProtocol::ProcessCurrentURL()
     }
   m_lastActiveTime = PR_Now(); // ** jt -- is this the best place for time stamp
   SetFlag(IMAP_CLEAN_UP_URL_STATE);
-  if (NS_SUCCEEDED(rv) && GetConnectionStatus() >= 0 && GetServerStateParser().LastCommandSuccessful() 
-	&& m_imapMiscellaneousSink && m_runningUrl)
+  if (GetConnectionStatus() >= 0 && m_imapMiscellaneousSink && m_runningUrl)
   {
       m_imapMiscellaneousSink->CopyNextStreamMessage(this, m_runningUrl);
       WaitForFEEventCompletion();