From fe9f14c48fe1c4a352f318109baf491936f7f1b3 Mon Sep 17 00:00:00 2001 From: "bienvenu%netscape.com" Date: Tue, 30 Apr 2002 14:46:44 +0000 Subject: [PATCH] fix 140555 r/sr=sspitzer downloading two copies of every news msg for offline use when syncing newsgroups, also don't allow multiple downloads to the same group git-svn-id: svn://10.0.0.236/trunk@120300 18797224-902f-48f8-a5cc-f745e15eee43 --- mozilla/mailnews/news/src/nsNewsDownloader.cpp | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/mozilla/mailnews/news/src/nsNewsDownloader.cpp b/mozilla/mailnews/news/src/nsNewsDownloader.cpp index 67db83c9b66..e18db5f62b9 100644 --- a/mozilla/mailnews/news/src/nsNewsDownloader.cpp +++ b/mozilla/mailnews/news/src/nsNewsDownloader.cpp @@ -303,7 +303,8 @@ NS_IMETHODIMP nsNewsDownloader::OnSearchDone(nsresult status) if (m_listener) return m_listener->OnStopRunningUrl(nsnull, NS_OK); } - nsresult rv = DownloadArticles(m_window, m_folder, &m_keysToDownload); + nsresult rv = DownloadArticles(m_window, m_folder, + /* we've already set m_keysToDownload, so don't pass it in */ nsnull); if (NS_FAILED(rv)) if (m_listener) m_listener->OnStopRunningUrl(nsnull, rv); @@ -380,8 +381,16 @@ nsMsgDownloadAllNewsgroups::OnStopRunningUrl(nsIURI* url, nsresult exitCode) { if (m_downloadedHdrsForCurGroup) { - rv = DownloadMsgsForCurrentGroup(); + PRBool savingArticlesOffline = PR_FALSE; + nsCOMPtr newsFolder = do_QueryInterface(m_currentFolder); + if (newsFolder) + newsFolder->GetSaveArticleOffline(&savingArticlesOffline); + m_downloadedHdrsForCurGroup = PR_FALSE; + if (savingArticlesOffline) // skip this group - we're saving to it already + rv = ProcessNextGroup(); + else + rv = DownloadMsgsForCurrentGroup(); } else {