Adapt Order Details to new card eligibility approach.

Author: toupper

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

@@ -46,11 +46,7 @@ final class OrderDetailsDataSource: NSObject {
 
     /// Whether the order is eligible for card present payment.
     ///
-    var isEligibleForCardPresentPayment: Bool {
-        return cardPresentPaymentsConfiguration.isSupportedCountry &&
-        order.isEligibleForCardPresentPayment(cardPresentPaymentsConfiguration: cardPresentPaymentsConfiguration,
-                                              products: resultsControllers.products)
-    }
+    var isEligibleForCardPresentPayment: Bool = false
 
     var isEligibleForRefund: Bool {
         guard !isRefundedStatus,

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

@@ -233,7 +233,13 @@ extension OrderDetailsViewModel {
         }
 
         group.enter()
-        refreshCardPresentPaymentEligibility()
+        checkCardPresentPaymentEligibility() {
+            onReloadSections?()
+            group.leave()
+        }
+
+        group.enter()
+        loadPaymentGatewayAccounts()
         group.leave()
 
         group.enter()
@@ -601,7 +607,33 @@ extension OrderDetailsViewModel {
         stores.dispatch(action)
     }
 
-    func refreshCardPresentPaymentEligibility() {
+    func checkCardPresentPaymentEligibility(onCompletion: (() -> Void)? = nil) {
+        let configuration = configurationLoader.configuration
+
+        guard configuration.isSupportedCountry else {
+            dataSource.isEligibleForCardPresentPayment = false
+            onCompletion?()
+            return
+        }
+
+        let action = OrderCardPresentPaymentEligibilityAction
+            .orderIsEligibleForCardPresentPayment(orderID: order.orderID,
+                                                  siteID: order.siteID,
+                                                  cardPresentPaymentsConfiguration: configurationLoader.configuration) { [weak self] result in
+            switch result {
+            case .success(let eligible):
+                self?.dataSource.isEligibleForCardPresentPayment = eligible
+            case .failure(_):
+                self?.dataSource.isEligibleForCardPresentPayment = false
+            }
+
+            onCompletion?()
+        }
+
+        stores.dispatch(action)
+    }
+
+    func loadPaymentGatewayAccounts() {
         /// No need for a completion here. The VC will be notified of changes to the stored paymentGatewayAccounts
         /// by the viewModel (after passing up through the dataSource and originating in the resultsControllers)
         ///

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

@@ -600,7 +600,7 @@ private extension OrderDetailsViewController {
             // Refresh date & view once payment has been collected.
             if result.isSuccess {
                 self.viewModel.syncOrderAfterPaymentCollection {
-                    self.viewModel.refreshCardPresentPaymentEligibility()
+                    self.viewModel.loadPaymentGatewayAccounts()
                 }
             }
         }