From 970e68da35be93e5de145d0f9225ddb741269d93 Mon Sep 17 00:00:00 2001 From: Maurice Parker Date: Fri, 20 Nov 2020 16:00:31 -0600 Subject: [PATCH] Remove extraneous complete task functions that were throwing off the progress indicator. Fixes #2543 --- .../NewsBlur/Internals/NewsBlurAccountDelegate+Internal.swift | 3 +-- Account/Sources/Account/NewsBlur/NewsBlurAccountDelegate.swift | 1 - 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/Account/Sources/Account/NewsBlur/Internals/NewsBlurAccountDelegate+Internal.swift b/Account/Sources/Account/NewsBlur/Internals/NewsBlurAccountDelegate+Internal.swift index 1548d7c67..7608e0431 100644 --- a/Account/Sources/Account/NewsBlur/Internals/NewsBlurAccountDelegate+Internal.swift +++ b/Account/Sources/Account/NewsBlur/Internals/NewsBlurAccountDelegate+Internal.swift @@ -16,6 +16,7 @@ import SyncDatabase import os.log extension NewsBlurAccountDelegate { + func refreshFeeds(for account: Account, completion: @escaping (Result) -> Void) { os_log(.debug, log: log, "Refreshing feeds...") @@ -27,8 +28,6 @@ extension NewsBlurAccountDelegate { self.syncFeeds(account, feeds) self.syncFeedFolderRelationship(account, folders) } - - self.refreshProgress.completeTask() completion(.success(())) case .failure(let error): completion(.failure(error)) diff --git a/Account/Sources/Account/NewsBlur/NewsBlurAccountDelegate.swift b/Account/Sources/Account/NewsBlur/NewsBlurAccountDelegate.swift index e239a6706..6776c90a1 100644 --- a/Account/Sources/Account/NewsBlur/NewsBlurAccountDelegate.swift +++ b/Account/Sources/Account/NewsBlur/NewsBlurAccountDelegate.swift @@ -246,7 +246,6 @@ final class NewsBlurAccountDelegate: AccountDelegate { caller.retrieveUnreadStoryHashes { result in switch result { case .success(let storyHashes): - self.refreshProgress.completeTask() if let count = storyHashes?.count, count > 0 { self.refreshProgress.addToNumberOfTasksAndRemaining((count - 1) / 100 + 1)