POS cash payment: prefill order amount and update analytics tracking (#16058)

Author: jaclync

RELEASE-NOTES.txt

@@ -3,6 +3,7 @@
 
 23.2
 -----
+- [*] POS: Enhanced cash payment experience with pre-filled amounts. [https://github.com/woocommerce/woocommerce-ios/pull/16058]
 - [*] POS: Point of Sale settings are now accessible in POS mode [https://github.com/woocommerce/woocommerce-ios/pull/16067]
 
 23.1

WooCommerce/Classes/Analytics/TracksProvider.swift

@@ -139,6 +139,7 @@ private extension TracksProvider {
             WooAnalyticsStat.pointOfSaleViewDocsTapped,
             WooAnalyticsStat.pointOfSaleReaderReadyForCardPayment,
             WooAnalyticsStat.pointOfSaleCashCollectPaymentSuccess,
+            WooAnalyticsStat.pointOfSaleCheckoutCashPaymentTapped,
             WooAnalyticsStat.pointOfSaleCashPaymentTapped,
             WooAnalyticsStat.pointOfSaleCashPaymentFailed,
             WooAnalyticsStat.pointOfSaleItemsHeaderTapped,

WooCommerce/Classes/Analytics/WooAnalyticsStat.swift

@@ -1258,6 +1258,7 @@ enum WooAnalyticsStat: String {
     case pointOfSaleItemRemovedFromCart = "item_removed_from_cart"
     case pointOfSaleCheckoutTapped = "checkout_tapped"
     case pointOfSaleBackToCartTapped = "back_to_cart_tapped"
+    case pointOfSaleCheckoutCashPaymentTapped = "checkout_cash_payment_tapped"
     case pointOfSaleCashPaymentTapped = "cash_payment_tapped"
     case pointOfSaleCashPaymentFailed = "cash_payment_failed"
     case pointOfSaleBackToCheckoutFromCashTapped = "back_to_checkout_from_cash"

WooCommerce/Classes/POS/Models/PointOfSaleAggregateModel.swift

@@ -368,7 +368,7 @@ extension PointOfSaleAggregateModel {
     // Once we get the callback from the card service, we switch to cash collection state
     @MainActor
     func startCashPayment() async {
-        analytics.track(.pointOfSaleCashPaymentTapped)
+        analytics.track(.pointOfSaleCheckoutCashPaymentTapped)
         try? await cardPresentPaymentService.cancelPayment()
         paymentState.cash = .collectingCash
     }

WooCommerce/Classes/POS/Presentation/PointOfSaleCollectCashView.swift

@@ -25,6 +25,12 @@ struct PointOfSaleCollectCashView: View {
         String.localizedStringWithFormat(Localization.backNavigationSubtitle, orderTotal)
     }
 
+    private var isButtonEnabled: Bool {
+        viewHelper.isPaymentButtonEnabled(orderTotal: orderTotal,
+                                          textFieldAmountInput: textFieldAmountInput,
+                                          isLoading: isLoading)
+    }
+
     @StateObject private var textFieldViewModel = FormattableAmountTextFieldViewModel(size: .extraLarge,
                                                                                       locale: Locale.autoupdatingCurrent,
                                                                                       storeCurrencySettings: ServiceLocator.currencySettings,
@@ -89,7 +95,7 @@ struct PointOfSaleCollectCashView: View {
                         .buttonStyle(POSFilledButtonStyle(size: .normal, isLoading: isLoading))
                         .frame(maxWidth: .infinity)
                         .dynamicTypeSize(...DynamicTypeSize.accessibility1)
-                        .disabled(isLoading)
+                        .disabled(!isButtonEnabled)
                     }
                     .padding([.horizontal])
                     .padding(.bottom, max(keyboardFrame.height - geometry.safeAreaInsets.bottom,
@@ -110,6 +116,9 @@ struct PointOfSaleCollectCashView: View {
             }
         }
         .frame(maxWidth: .infinity, maxHeight: .infinity)
+        .onAppear {
+            prefillOrderTotal()
+        }
     }
 
     private func markComplete() async throws {
@@ -121,9 +130,7 @@ struct PointOfSaleCollectCashView: View {
 
 private extension PointOfSaleCollectCashView {
     private func submitCashAmount() async {
-        guard validateAmountOnSubmit() else {
-            return
-        }
+        ServiceLocator.analytics.track(.pointOfSaleCashPaymentTapped)
         isLoading = true
         do {
             try await markComplete()
@@ -140,14 +147,12 @@ private extension PointOfSaleCollectCashView {
             textFieldAmountInput: textFieldAmountInput)
     }
 
-    private func validateAmountOnSubmit() -> Bool {
-        viewHelper.validateAmountOnSubmit(
-            orderTotal: orderTotal,
-            textFieldAmountInput: textFieldAmountInput,
-            onError: { error in
-                errorMessage = error
-            })
+    private func prefillOrderTotal() {
+        if let orderDecimal = viewHelper.parseCurrency(orderTotal) {
+            textFieldViewModel.presetAmount(orderDecimal)
         }
+        isTextFieldFocused = true
+    }
 }
 
 private extension PointOfSaleCollectCashView {

WooCommerce/Classes/POS/ViewHelpers/CollectCashViewHelper.swift

@@ -54,25 +54,22 @@ final class CollectCashViewHelper {
         }
     }
 
-    func validateAmountOnSubmit(orderTotal: String,
+    func isPaymentButtonEnabled(orderTotal: String,
                                 textFieldAmountInput: String,
-                                onError: (String) -> Void) -> Bool {
-        let userInput = textFieldAmountInput.isNotEmpty ? textFieldAmountInput : "0"
-
-        guard let orderDecimal = parseCurrency(orderTotal),
-              let inputDecimal = parseCurrency(userInput) else {
-            onError(Localization.failedToCollectCashPayment)
+                                isLoading: Bool) -> Bool {
+        guard !isLoading else {
             return false
         }
 
-        if inputDecimal < orderDecimal {
-            onError(Localization.cashPaymentAmountNotEnough)
+        let inputAmount = textFieldAmountInput.isNotEmpty ? textFieldAmountInput : "0"
+        guard let orderDecimal = parseCurrency(orderTotal),
+              let inputDecimal = parseCurrency(inputAmount) else {
             return false
         }
-        return true
+        return inputDecimal >= orderDecimal
     }
 
-    private func parseCurrency(_ amountString: String) -> Decimal? {
+    func parseCurrency(_ amountString: String) -> Decimal? {
         // Removes all leading/trailing whitespace, if any
         let sanitized = amountString.trimmingCharacters(in: .whitespacesAndNewlines)
 
@@ -101,16 +98,6 @@ private extension CollectCashViewHelper {
             comment: "Change due when the cash amount entered exceeds the order total." +
             "Reads as 'Change due: $1.23'"
         )
-        static let failedToCollectCashPayment = NSLocalizedString(
-            "collectcashviewhelper.failedtocollectcashpayment.errormessage",
-            value: "Error trying to process payment. Try again.",
-            comment: "Error message when the system fails to collect a cash payment."
-        )
-        static let cashPaymentAmountNotEnough = NSLocalizedString(
-            "collectcashviewhelper.cashpaymentamountnotenough.errormessage",
-            value: "Amount must be more or equal to total.",
-            comment: "Error message when the cash amount entered is less than the order total."
-        )
     }
 }