Skip to content

Commit be880a6

Browse files
committed
Update onOrderCreated closure to onFinished, and added documentation
1 parent a3f7913 commit be880a6

File tree

3 files changed

+7
-5
lines changed

3 files changed

+7
-5
lines changed

WooCommerce/Classes/ViewRelated/Orders/Order Creation/FlowCoordinator/AddOrderCoordinator.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ private extension AddOrderCoordinator {
7979
///
8080
func presentNewOrderController() {
8181
let viewModel = NewOrderViewModel(siteID: siteID)
82-
viewModel.onOrderCreated = onOrderCreated
82+
viewModel.onFinished = onOrderCreated
8383

8484
let viewController = NewOrderHostingController(viewModel: viewModel)
8585
if ServiceLocator.featureFlagService.isFeatureFlagEnabled(.splitViewInOrdersTab) {

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

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -346,7 +346,7 @@ final class NewOrderViewModel: ObservableObject {
346346

347347
switch result {
348348
case .success(let newOrder):
349-
self.onOrderCreated(newOrder)
349+
self.onFinished(newOrder)
350350
self.trackCreateOrderSuccess()
351351
case .failure(let error):
352352
self.notice = NoticeFactory.createOrderErrorNotice(error, order: self.orderSynchronizer.order)
@@ -357,9 +357,11 @@ final class NewOrderViewModel: ObservableObject {
357357
trackCreateButtonTapped()
358358
}
359359

360-
/// Assign this closure to be notified when a new order is created
360+
/// Assign this closure to be notified when the flow has finished.
361+
/// For creation it means that the order has been created.
362+
/// For edition it means that the merchant has finished editing the order.
361363
///
362-
var onOrderCreated: (Order) -> Void = { _ in }
364+
var onFinished: (Order) -> Void = { _ in }
363365

364366
/// Updates the order status & tracks its event
365367
///

WooCommerce/Classes/ViewRelated/Orders/Order Details/OrderDetailsViewController.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -354,7 +354,7 @@ private extension OrderDetailsViewController {
354354
///
355355
private func editOrder() {
356356
let viewModel = NewOrderViewModel(siteID: viewModel.order.siteID, flow: .editing(initialOrder: viewModel.order))
357-
viewModel.onOrderCreated = { [weak self] order in
357+
viewModel.onFinished = { [weak self] order in
358358
guard let self = self else { return }
359359
self.dismiss(animated: true)
360360
}

0 commit comments

Comments
 (0)