Fix issue TumblThreeApp#135 Wrong calculation of downloaded items

- For every crawl one statistics item was added. - Crawls itself are not changing the statistics. - Old (wrong) statistics cannot be fixed.
thomas694 · Apr 4, 2021 · ceacd44 · ceacd44
1 parent 08d2ace
commit ceacd44
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/src/TumblThree/TumblThree.Applications/Controllers/CrawlerController.cs b/src/TumblThree/TumblThree.Applications/Controllers/CrawlerController.cs
@@ -258,7 +258,7 @@ private async Task StartSiteSpecificDownloaderAsync(QueueListItem queueListItem,
 
             ICrawler crawler = _crawlerFactory.GetCrawler(blog, progress, pt, ct);
             await crawler.CrawlAsync();
-            blog.UpdateProgress();
+            blog.UpdateProgress(true);
             crawler.Dispose();
 
             Monitor.Enter(_lockObject);

diff --git a/src/TumblThree/TumblThree.Applications/Downloader/AbstractDownloader.cs b/src/TumblThree/TumblThree.Applications/Downloader/AbstractDownloader.cs
@@ -345,7 +345,7 @@ private void DownloadTextPost(TumblrPost downloadItem)
         private void UpdateBlogDB(string postType)
         {
             blog.UpdatePostCount(postType);
-            blog.UpdateProgress();
+            blog.UpdateProgress(false);
         }
 
         protected void SetFileDate(string fileLocation, DateTime postDate)

diff --git a/src/TumblThree/TumblThree.Domain/Models/Blogs/Blog.cs b/src/TumblThree/TumblThree.Domain/Models/Blogs/Blog.cs
@@ -888,11 +888,11 @@ public bool GroupPhotoSets
             }
         }
 
-        public void UpdateProgress()
+        public void UpdateProgress(bool calcOnly)
         {
             lock (lockObjectProgress)
             {
-                DownloadedImages++;
+                if (!calcOnly) DownloadedImages++;
                 Progress = (int)(DownloadedImages / (double)TotalCount * 100);
             }
         }

diff --git a/src/TumblThree/TumblThree.Domain/Models/Blogs/IBlog.cs b/src/TumblThree/TumblThree.Domain/Models/Blogs/IBlog.cs
@@ -198,7 +198,7 @@ public interface IBlog : INotifyPropertyChanged
 
         List<string> Links { get; }
 
-        void UpdateProgress();
+        void UpdateProgress(bool calcOnly);
 
         void UpdatePostCount(string propertyName);