Skip to content

Commit 15c27e8

Browse files
committed
Update syncEverything dependency flow with defer
1 parent 1440ac7 commit 15c27e8

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

WooCommerce/Classes/ViewModels/Order Details/OrderDetailsViewModel.swift

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -193,12 +193,13 @@ extension OrderDetailsViewModel {
193193

194194
group.enter()
195195
syncOrder { [weak self] _ in
196-
// Products require order.items data, so sync them only after the order is loaded
197-
guard let self = self else {
196+
defer {
198197
group.leave()
199-
return
200198
}
201199

200+
// Products require order.items data, so sync them only after the order is loaded
201+
guard let self = self else { return }
202+
202203
group.enter()
203204
self.syncProducts { _ in
204205
group.leave()
@@ -208,8 +209,6 @@ extension OrderDetailsViewModel {
208209
self.syncProductVariations { _ in
209210
group.leave()
210211
}
211-
212-
group.leave()
213212
}
214213

215214
group.enter()

0 commit comments

Comments
 (0)