Open CIAB booking products in web (#16250)

Author: adborbas

WooCommerce/Classes/Authentication/AuthenticatedWebViewController.swift

@@ -123,6 +123,11 @@ final class AuthenticatedWebViewController: UIViewController {
             viewModel.handleDismissal()
         }
     }
+
+    override func viewDidDisappear(_ animated: Bool) {
+        viewModel.handleDisappear()
+        super.viewDidDisappear(animated)
+    }
 }
 
 private extension AuthenticatedWebViewController {

WooCommerce/Classes/Authentication/AuthenticatedWebViewModel.swift

@@ -22,6 +22,9 @@ protocol AuthenticatedWebViewModel {
     /// Triggered when the web view is dismissed
     func handleDismissal()
 
+    /// Triggered when the web view is disappeared
+    func handleDisappear()
+
     /// Triggered when the web view redirects to a new URL
     func handleRedirect(for url: URL?)
 
@@ -53,6 +56,10 @@ extension AuthenticatedWebViewModel {
     func didFailProvisionalNavigation(with error: Error) {
         // NO-OP
     }
+
+    func handleDisappear () {
+        // NO-OP
+    }
 }
 
 // MARK: - Helper methods

WooCommerce/Classes/Authentication/WPAdminWebViewModel.swift

@@ -20,6 +20,10 @@ class WPAdminWebViewModel: AuthenticatedWebViewModel, WebviewReloadable {
         // no-op
     }
 
+    func handleDisappear() {
+        // no-op
+    }
+
     func handleRedirect(for url: URL?) {
         guard let url else {
             return

WooCommerce/Classes/CIAB/CIABEligibilityChecker.swift

@@ -1,5 +1,3 @@
-/// periphery: ignore:all - Will be used in upcoming PRs
-
 import Foundation
 import Yosemite
 

WooCommerce/Classes/CIAB/CIABEligibilityCheckerProtocol.swift

@@ -1,7 +1,6 @@
 import Foundation
 import Yosemite
 
-/// periphery: ignore - Will be used in upcoming changes for app feature gating
 protocol CIABEligibilityCheckerProtocol {
     var isCurrentSiteCIAB: Bool { get }
 

WooCommerce/Classes/Routing/ProductDetail/Coordinator/ProductDetailNativeCoordinator.swift

@@ -0,0 +1,18 @@
+import Yosemite
+import UIKit
+
+/// Coordinator for the **native** product detail/editor flow.
+/// Delegates VC construction to `ProductDetailsFactory` and applies the requested presentation style.
+class ProductDetailNativeCoordinator {
+
+    func viewController(
+        product: Product,
+        presentationStyle: ProductDetailNavigator.Presentation,
+        isReadOnly: Bool,
+        onDelete: (() -> Void)? = nil) -> UIViewController {
+            return ProductDetailsFactory.productDetails(product: product,
+                                                        presentationStyle: presentationStyle.asProductFormPresentationStyle,
+                                                        forceReadOnly: isReadOnly,
+                                                        onDeleteCompletion: onDelete ?? {})
+        }
+}

WooCommerce/Classes/Routing/ProductDetail/Coordinator/ProductDetailWebCoordinator.swift

@@ -0,0 +1,37 @@
+import UIKit
+import Yosemite
+
+/// Coordinator for the **admin web** product detail/editor flow.
+class ProductDetailWebCoordinator: NSObject {
+    private let site: Site?
+
+    init(site: Site?) {
+        self.site = site
+    }
+
+    func viewController(product: Product, onDismiss: @escaping () -> Void) -> UIViewController {
+        guard let url = ProductAdminURLProvider.editURL(for: product, site: site) else {
+            return UIViewController()
+        }
+
+        let viewModel = AdminWebViewModel(title: product.name, initialURL: url) { [onDismiss] in
+            onDismiss()
+        }
+        let webViewController = AuthenticatedWebViewController(viewModel: viewModel)
+        return webViewController
+    }
+}
+
+fileprivate class AdminWebViewModel: WPAdminWebViewModel {
+    let onDismiss: (() -> Void)
+
+    init(title: String, initialURL: URL, onDismiss: @escaping () -> Void) {
+        self.onDismiss = onDismiss
+        super.init(title: title, initialURL: initialURL)
+    }
+
+    override func handleDisappear() {
+        onDismiss()
+        super.handleDisappear()
+    }
+}

WooCommerce/Classes/Routing/ProductDetail/ProductAdminURLProvider.swift

@@ -0,0 +1,18 @@
+import Yosemite
+import Foundation
+
+/// Builds canonical admin edit URLs for products.
+enum ProductAdminURLProvider {
+
+    static func editURL(for product: Product, site: Site?) -> URL? {
+        guard let base = site?.adminURLWithFallback() else { return nil }
+
+        var components = URLComponents(url: base, resolvingAgainstBaseURL: false)!
+        components.queryItems = [
+            URLQueryItem(name: "page", value: "next-admin"),
+            URLQueryItem(name: "p", value: "/woocommerce/products/edit/\(product.productID)")
+        ]
+
+        return components.url
+    }
+}

WooCommerce/Classes/Routing/ProductDetail/ProductDetailCoordinatorFactoryProtocol.swift

@@ -0,0 +1,20 @@
+import Yosemite
+
+/// Factory for producing coordinators used by the navigator.
+protocol ProductDetailCoordinatorFactoryProtocol {
+    func webCoordinator(site: Site?) -> ProductDetailWebCoordinator
+    func nativeCoordinator() -> ProductDetailNativeCoordinator
+}
+
+/// Default coordinator factory that wires production dependencies.
+class ProductDetailCoordinatorFactory: ProductDetailCoordinatorFactoryProtocol {
+    static let `default` = ProductDetailCoordinatorFactory()
+
+    func webCoordinator(site: Site?) -> ProductDetailWebCoordinator {
+        return ProductDetailWebCoordinator(site: site)
+    }
+
+    func nativeCoordinator() -> ProductDetailNativeCoordinator {
+        return ProductDetailNativeCoordinator()
+    }
+}

WooCommerce/Classes/Routing/ProductDetail/ProductDetailNavigator.swift

@@ -0,0 +1,70 @@
+import UIKit
+import Yosemite
+
+/// Decides between **native** and **admin web** product detail flows and builds the destination VC.
+final class ProductDetailNavigator {
+    /// Describes how the **native** destination should be presented.
+    enum Presentation {
+        case push
+        case contained(in: () -> UIViewController?)
+
+        var asProductFormPresentationStyle: ProductFormPresentationStyle {
+            switch self {
+            case .contained(let inVC):
+                    .contained(containerViewController: inVC)
+            case .push:
+                    .navigationStack
+            }
+        }
+    }
+
+    static var shared = ProductDetailNavigator()
+
+    private let ciabChecker: CIABEligibilityCheckerProtocol
+    private let coordinatorFactory: ProductDetailCoordinatorFactoryProtocol
+    private let stores: StoresManager
+
+    init(ciabChecker: CIABEligibilityCheckerProtocol = CIABEligibilityChecker(),
+         coordinatorFactory: ProductDetailCoordinatorFactoryProtocol = ProductDetailCoordinatorFactory.default,
+         stores: StoresManager = ServiceLocator.stores,
+    ) {
+        self.ciabChecker = ciabChecker
+        self.coordinatorFactory = coordinatorFactory
+        self.stores = stores
+    }
+
+    /// Builds the destination `UIViewController` for the given product.
+    /// - Parameters:
+    ///   - product: The product to display.
+    ///   - presentationStyle: How to present **native** detail (ignored for web).
+    ///   - isReadOnly: Whether the native screen should be read-only.
+    ///   - onDelete: Optional callback invoked after a successful product delete.
+    /// - Returns: A ready-to-present view controller (native or web).
+    func makeDestination(product: Product,
+                         presentationStyle: Presentation = .push,
+                         isReadOnly: Bool,
+                         onDismissWeb: (() -> Void)? = nil,
+                         onDelete: (() -> Void)? = nil) -> UIViewController {
+
+        let viewController: UIViewController
+        if shouldOpenInWeb(product: product) {
+            let coordinator = coordinatorFactory.webCoordinator(site: stores.sessionManager.defaultSite)
+            viewController = coordinator.viewController(product: product) {
+                onDismissWeb?()
+            }
+
+        } else {
+            let coordinator = coordinatorFactory.nativeCoordinator()
+            viewController = coordinator.viewController(product: product,
+                                                        presentationStyle: presentationStyle,
+                                                        isReadOnly: isReadOnly,
+                                                        onDelete: onDelete)
+        }
+
+        return viewController
+    }
+
+    private func shouldOpenInWeb(product: Product) -> Bool {
+        return ciabChecker.isCurrentSiteCIAB && product.productType == .booking
+    }
+}