Skip to content

Commit 395a295

Browse files
committed
Do not use the async API
1 parent b30d7b5 commit 395a295

3 files changed

Lines changed: 21 additions & 24 deletions

File tree

WordPress/Classes/Services/BloggingPromptsService.swift

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -330,13 +330,14 @@ private extension BloggingPromptsService {
330330
/// - Parameters:
331331
/// - remoteSettings: The blogging prompt settings from the remote.
332332
/// - completion: Closure to be called on completion.
333-
func saveSettings(_ remoteSettings: RemoteBloggingPromptsSettings, completion: @MainActor @escaping () -> Void) {
334-
Task {
335-
await contextManager.save { derivedContext in
336-
let settings = self.loadSettings(context: derivedContext) ?? BloggingPromptSettings(context: derivedContext)
337-
settings.configure(with: remoteSettings, siteID: self.siteID.int32Value, context: derivedContext)
333+
func saveSettings(_ remoteSettings: RemoteBloggingPromptsSettings, completion: @escaping () -> Void) {
334+
contextManager.save { derivedContext in
335+
let settings = self.loadSettings(context: derivedContext) ?? BloggingPromptSettings(context: derivedContext)
336+
settings.configure(with: remoteSettings, siteID: self.siteID.int32Value, context: derivedContext)
337+
} completion: {
338+
DispatchQueue.main.async {
339+
completion()
338340
}
339-
await completion()
340341
}
341342
}
342343

WordPress/Classes/Services/CommentService+Likes.swift

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -94,17 +94,15 @@ private extension CommentService {
9494
return
9595
}
9696

97-
Task {
98-
await ContextManager.shared.save { derivedContext in
99-
let likers = remoteLikeUsers.map { remoteUser in
100-
LikeUserHelper.createOrUpdateFrom(remoteUser: remoteUser, context: derivedContext)
101-
}
102-
103-
if purgeExisting {
104-
self.deleteExistingUsersFor(commentID: commentID, siteID: siteID, from: derivedContext, likesToKeep: likers)
105-
}
97+
ContextManager.shared.save { derivedContext in
98+
let likers = remoteLikeUsers.map { remoteUser in
99+
LikeUserHelper.createOrUpdateFrom(remoteUser: remoteUser, context: derivedContext)
106100
}
107101

102+
if purgeExisting {
103+
self.deleteExistingUsersFor(commentID: commentID, siteID: siteID, from: derivedContext, likesToKeep: likers)
104+
}
105+
} completion: {
108106
DispatchQueue.main.async {
109107
onComplete()
110108
}

WordPress/Classes/Services/PostService+Likes.swift

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -94,17 +94,15 @@ private extension PostService {
9494
return
9595
}
9696

97-
Task {
98-
await ContextManager.shared.save { derivedContext in
99-
let likers = remoteLikeUsers.map { remoteUser in
100-
LikeUserHelper.createOrUpdateFrom(remoteUser: remoteUser, context: derivedContext)
101-
}
102-
103-
if purgeExisting {
104-
self.deleteExistingUsersFor(postID: postID, siteID: siteID, from: derivedContext, likesToKeep: likers)
105-
}
97+
ContextManager.shared.save { derivedContext in
98+
let likers = remoteLikeUsers.map { remoteUser in
99+
LikeUserHelper.createOrUpdateFrom(remoteUser: remoteUser, context: derivedContext)
106100
}
107101

102+
if purgeExisting {
103+
self.deleteExistingUsersFor(postID: postID, siteID: siteID, from: derivedContext, likesToKeep: likers)
104+
}
105+
} completion: {
108106
DispatchQueue.main.async {
109107
onComplete()
110108
}

0 commit comments

Comments
 (0)