From ae091afb183b4b3336e572d708ec7dd1ea26fee9 Mon Sep 17 00:00:00 2001 From: Maurice Parker Date: Tue, 7 Apr 2020 19:02:25 -0500 Subject: [PATCH] Allow for error on query when looking for existing starred records. --- .../Account/CloudKit/CloudKitArticlesZone.swift | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/Frameworks/Account/CloudKit/CloudKitArticlesZone.swift b/Frameworks/Account/CloudKit/CloudKitArticlesZone.swift index 080799ed8..8a11fd951 100644 --- a/Frameworks/Account/CloudKit/CloudKitArticlesZone.swift +++ b/Frameworks/Account/CloudKit/CloudKitArticlesZone.swift @@ -147,7 +147,6 @@ private extension CloudKitArticlesZone { func makeArticleRecordsIfNecessary(_ articles: Set
, completion: @escaping ((Result<[CKRecord], Error>) -> Void)) { let group = DispatchGroup() - var errorOccurred = false var records = [CKRecord]() for article in articles { @@ -164,9 +163,8 @@ private extension CloudKitArticlesZone { if !recordFound { records.append(contentsOf: self.makeArticleRecords(article)) } - case .failure(let error): - errorOccurred = true - os_log(.error, log: self.log, "Error occurred while checking for existing articles: %@", error.localizedDescription) + case .failure: + records.append(contentsOf: self.makeArticleRecords(article)) } group.leave() } @@ -174,11 +172,7 @@ private extension CloudKitArticlesZone { } group.notify(queue: DispatchQueue.main) { - if errorOccurred { - completion(.failure(CloudKitZoneError.unknown)) - } else { - completion(.success(records)) - } + completion(.success(records)) } }