Skip to content

Commit eb8763f

Browse files
committed
Updates create order method to use local synchronizer
1 parent 74901b8 commit eb8763f

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

WooCommerce/Classes/ViewRelated/Orders/Order Creation/NewOrderViewModel.swift

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -225,7 +225,7 @@ final class NewOrderViewModel: ObservableObject {
225225
func createOrder() {
226226
performingNetworkRequest = true
227227

228-
let action = OrderAction.createOrder(siteID: siteID, order: orderDetails.toOrder()) { [weak self] result in
228+
orderSynchronizer.commitAllChanges { [weak self] result in
229229
guard let self = self else { return }
230230
self.performingNetworkRequest = false
231231

@@ -239,7 +239,6 @@ final class NewOrderViewModel: ObservableObject {
239239
DDLogError("⛔️ Error creating new order: \(error)")
240240
}
241241
}
242-
stores.dispatch(action)
243242
trackCreateButtonTapped()
244243
}
245244

@@ -527,9 +526,9 @@ private extension NewOrderViewModel {
527526
/// or figure out a better way to get the product count.
528527
///
529528
func trackCreateButtonTapped() {
530-
let hasCustomerDetails = orderDetails.billingAddress != nil || orderDetails.shippingAddress != nil
531-
analytics.track(event: WooAnalyticsEvent.Orders.orderCreateButtonTapped(status: orderDetails.status,
532-
productCount: orderDetails.items.count,
529+
let hasCustomerDetails = orderSynchronizer.order.billingAddress != nil || orderSynchronizer.order.shippingAddress != nil
530+
analytics.track(event: WooAnalyticsEvent.Orders.orderCreateButtonTapped(status: orderSynchronizer.order.status,
531+
productCount: orderSynchronizer.order.items.count,
533532
hasCustomerDetails: hasCustomerDetails))
534533
}
535534

0 commit comments

Comments
 (0)