Add canBeDismissed property on NewOrderViewModel

Author: ealeksandrov

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

@@ -44,7 +44,7 @@ final class NewOrderHostingController: UIHostingController<NewOrder> {
 ///
 extension NewOrderHostingController {
     override func shouldPopOnBackButton() -> Bool {
-        guard !viewModel.hasChanges else {
+        guard viewModel.canBeDismissed || !viewModel.hasChanges else {
             presentDiscardChangesActionSheet(onDiscard: { [weak self] in
                 self?.discardOrderAndPop()
             })
@@ -62,7 +62,7 @@ extension NewOrderHostingController {
 ///
 extension NewOrderHostingController: UIAdaptivePresentationControllerDelegate {
     func presentationControllerShouldDismiss(_ presentationController: UIPresentationController) -> Bool {
-        return !viewModel.hasChanges
+        return viewModel.canBeDismissed || !viewModel.hasChanges
     }
 
     func presentationControllerDidAttemptToDismiss(_ presentationController: UIPresentationController) {

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

@@ -28,7 +28,18 @@ final class NewOrderViewModel: ObservableObject {
     /// Indicates whether user has made any changes
     ///
     var hasChanges: Bool {
-        orderSynchronizer.order != OrderFactory.emptyNewOrder && flow == .creation
+        switch flow {
+        case .creation:
+            return orderSynchronizer.order != OrderFactory.emptyNewOrder
+        case .editing(let initialOrder):
+            return orderSynchronizer.order != initialOrder
+        }
+    }
+
+    /// Indicates whether view can be dismissed. If the value is `false` - confirmation alert will be displayd depending on `hasChanges` value.
+    ///
+    var canBeDismissed: Bool {
+        flow == .creation
     }
 
     /// Indicates whether the cancel button is visible.