renun/NetNewsWire
1
0
Fork 0
mirror of https://github.com/Ranchero-Software/NetNewsWire synced 2025-08-12 06:26:36 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Remove batch update blocks that were causing more harm than good.

Browse Source
This commit is contained in:
Maurice Parker
2020-04-01 15:39:29 -05:00
parent 39aecd84fe
commit b3cf7ccdb7
1 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
Frameworks/Account/CloudKit/CloudKitAccountDelegate.swift
View File

@@ -59,7 +59,6 @@ final class CloudKitAccountDelegate: AccountDelegate {
func receiveRemoteNotification(for account: Account, userInfo: [AnyHashable : Any], completion: @escaping () -> Void) {
let group = DispatchGroup()
BatchUpdate.shared.start()
zones.forEach { zone in
group.enter()
@@ -69,7 +68,6 @@ final class CloudKitAccountDelegate: AccountDelegate {
}
group.notify(queue: DispatchQueue.main) {
BatchUpdate.shared.end()
completion()
}
}
@@ -77,9 +75,10 @@ final class CloudKitAccountDelegate: AccountDelegate {
func refreshAll(for account: Account, completion: @escaping (Result<Void, Error>) -> Void) {
BatchUpdate.shared.start()
accountZone.fetchChangesInZone() { result in
BatchUpdate.shared.end()
switch result {
case .success:
self.sendArticleStatus(for: account) { result in
switch result {
case .success:
@@ -89,7 +88,6 @@ final class CloudKitAccountDelegate: AccountDelegate {
case .success:
self.refresher.refreshFeeds(account.flattenedWebFeeds()) {
BatchUpdate.shared.end()
account.metadata.lastArticleFetchEndTime = Date()
completion(.success(()))
}