[Local Catalog] POS Settings: integrate with catalog data and full sync functionality (#16144)

Author: jaclync

Modules/Sources/Yosemite/Tools/POS/POSCatalogSettingsService.swift

@@ -1,4 +1,3 @@
-// periphery:ignore:all
 import Foundation
 import GRDB
 import protocol Storage.GRDBManagerProtocol

WooCommerce/Classes/POS/Models/PointOfSaleSettingsController.swift

@@ -7,29 +7,45 @@ import protocol Yosemite.PluginsServiceProtocol
 import protocol Yosemite.PointOfSaleSettingsServiceProtocol
 import struct Yosemite.POSReceiptInformation
 import Observation
+import protocol Storage.GRDBManagerProtocol
+import protocol Yosemite.POSCatalogSyncCoordinatorProtocol
+import class Yosemite.POSCatalogSettingsService
 
 protocol PointOfSaleSettingsControllerProtocol {
     var connectedCardReader: CardPresentPaymentCardReader? { get }
     var storeViewModel: POSSettingsStoreViewModel { get }
+    var localCatalogViewModel: POSSettingsLocalCatalogViewModel? { get }
 }
 
 @Observable final class PointOfSaleSettingsController: PointOfSaleSettingsControllerProtocol {
     private(set) var connectedCardReader: CardPresentPaymentCardReader?
     private var cancellables: AnyCancellable?
 
     let storeViewModel: POSSettingsStoreViewModel
+    let localCatalogViewModel: POSSettingsLocalCatalogViewModel?
 
     init(siteID: Int64,
          settingsService: PointOfSaleSettingsServiceProtocol,
          cardPresentPaymentService: CardPresentPaymentFacade,
          pluginsService: PluginsServiceProtocol,
          defaultSiteName: String?,
-         siteSettings: [SiteSetting]) {
+         siteSettings: [SiteSetting],
+         grdbManager: GRDBManagerProtocol?,
+         catalogSyncCoordinator: POSCatalogSyncCoordinatorProtocol?) {
         self.storeViewModel = POSSettingsStoreViewModel(siteID: siteID,
                                                         settingsService: settingsService,
                                                         pluginsService: pluginsService,
                                                         defaultSiteName: defaultSiteName,
                                                         siteSettings: siteSettings)
+        if let catalogSyncCoordinator, let grdbManager {
+            self.localCatalogViewModel = POSSettingsLocalCatalogViewModel(
+                siteID: siteID,
+                catalogSettingsService: POSCatalogSettingsService(grdbManager: grdbManager),
+                catalogSyncCoordinator: catalogSyncCoordinator
+            )
+        } else {
+            self.localCatalogViewModel = nil
+        }
 
         observeCardReader(from: cardPresentPaymentService)
     }
@@ -62,6 +78,8 @@ final class PointOfSaleSettingsPreviewController: PointOfSaleSettingsControllerP
                                                                               pluginsService: PluginsServicePreview(),
                                                                               defaultSiteName: "Sample Store",
                                                                               siteSettings: [])
+
+    var localCatalogViewModel: POSSettingsLocalCatalogViewModel?
 }
 
 final class MockPointOfSaleSettingsService: PointOfSaleSettingsServiceProtocol {

WooCommerce/Classes/POS/Presentation/Settings/POSSettingsLocalCatalogDetailView.swift

@@ -3,6 +3,11 @@ import SwiftUI
 struct POSSettingsLocalCatalogDetailView: View {
     // TODO: WOOMOB-1335 - implement full sync cellular data setting functionality
     @State private var allowFullSyncOnCellular: Bool = true
+    private let viewModel: POSSettingsLocalCatalogViewModel
+
+    init(viewModel: POSSettingsLocalCatalogViewModel) {
+        self.viewModel = viewModel
+    }
 
     var body: some View {
         NavigationStack {
@@ -21,6 +26,9 @@ struct POSSettingsLocalCatalogDetailView: View {
                 .background(Style.backgroundColor)
             }
         }
+        .task {
+            await viewModel.loadCatalogData()
+        }
     }
 }
 
@@ -31,12 +39,13 @@ private extension POSSettingsLocalCatalogDetailView {
             sectionHeaderView(title: Localization.catalogStatus)
 
             VStack(spacing: POSSpacing.medium) {
-                // TODO: WOOMOB-1100 - replace with catalog data
-                fieldRowView(label: Localization.catalogSize, value: "1,250 products, 3,420 variations")
-                fieldRowView(label: Localization.lastIncrementalUpdate, value: "5 minutes ago")
-                fieldRowView(label: Localization.lastFullSync, value: "Today at 2:34 PM")
+                fieldRowView(label: Localization.catalogSize, value: viewModel.catalogSize)
+                fieldRowView(label: Localization.lastIncrementalUpdate, value: viewModel.lastIncrementalSyncDate)
+                fieldRowView(label: Localization.lastFullSync, value: viewModel.lastFullSyncDate)
             }
             .padding(.bottom, POSPadding.medium)
+            .redacted(reason: viewModel.isLoading ? .placeholder : [])
+            .shimmering(active: viewModel.isLoading)
         }
     }
 
@@ -64,11 +73,13 @@ private extension POSSettingsLocalCatalogDetailView {
                     .frame(maxWidth: .infinity, alignment: .leading)
 
                 Button(action: {
-                    // Handle refresh catalog action
+                    Task {
+                        await viewModel.refreshCatalog()
+                    }
                 }) {
                     Text(Localization.refreshCatalog)
                 }
-                .buttonStyle(POSFilledButtonStyle(size: .normal))
+                .buttonStyle(POSFilledButtonStyle(size: .normal, isLoading: viewModel.isRefreshingCatalog))
             }
             .padding(.horizontal, POSPadding.medium)
             .padding(.bottom, POSPadding.medium)
@@ -134,20 +145,20 @@ private extension POSSettingsLocalCatalogDetailView {
         )
 
         static let managingDataUsage = NSLocalizedString(
-            "posSettingsLocalCatalogDetailView.managingDataUsage",
-            value: "Managing data usage",
+            "posSettingsLocalCatalogDetailView.managingDataUsage.1",
+            value: "Managing Data Usage",
             comment: "Section title for managing data usage in Point of Sale settings."
         )
 
         static let lastIncrementalUpdate = NSLocalizedString(
-            "posSettingsLocalCatalogDetailView.lastIncrementalUpdate",
-            value: "Last incremental update",
+            "posSettingsLocalCatalogDetailView.lastIncrementalSync",
+            value: "Last update",
             comment: "Label for last incremental update field in Point of Sale settings."
         )
 
         static let lastFullSync = NSLocalizedString(
-            "posSettingsLocalCatalogDetailView.lastFullSync",
-            value: "Last full sync",
+            "posSettingsLocalCatalogDetailView.lastFullSync.1",
+            value: "Last full update",
             comment: "Label for last full sync field in Point of Sale settings."
         )
 
@@ -159,8 +170,8 @@ private extension POSSettingsLocalCatalogDetailView {
 
 
         static let allowFullSyncOnCellular = NSLocalizedString(
-            "posSettingsLocalCatalogDetailView.allowFullSyncOnCellular",
-            value: "Allow full sync on cellular data",
+            "posSettingsLocalCatalogDetailView.allowFullSyncOnCellular.1",
+            value: "Allow full update on cellular data",
             comment: "Label for allow full sync on cellular data toggle in Point of Sale settings."
         )
 
@@ -187,6 +198,11 @@ private extension POSSettingsLocalCatalogDetailView {
 
 #if DEBUG
 #Preview {
-    POSSettingsLocalCatalogDetailView()
+    let viewModel = POSSettingsLocalCatalogViewModel(
+        siteID: 123,
+        catalogSettingsService: POSPreviewCatalogSettingsService(),
+        catalogSyncCoordinator: POSPreviewCatalogSyncCoordinator()
+    )
+    POSSettingsLocalCatalogDetailView(viewModel: viewModel)
 }
 #endif

WooCommerce/Classes/POS/Presentation/Settings/POSSettingsLocalCatalogViewModel.swift

@@ -0,0 +1,97 @@
+import Yosemite
+import Foundation
+
+@Observable
+final class POSSettingsLocalCatalogViewModel {
+    private(set) var catalogSize: String = ""
+    private(set) var lastFullSyncDate: String = ""
+    private(set) var lastIncrementalSyncDate: String = ""
+
+    private(set) var isLoading: Bool = false
+    private(set) var isRefreshingCatalog: Bool = false
+
+    private let siteID: Int64
+    private let catalogSettingsService: POSCatalogSettingsServiceProtocol
+    private let catalogSyncCoordinator: POSCatalogSyncCoordinatorProtocol
+    private let dateFormatter: RelativeDateTimeFormatter = {
+        let formatter = RelativeDateTimeFormatter()
+        formatter.dateTimeStyle = .named
+        formatter.unitsStyle = .full
+        return formatter
+    }()
+
+    init(siteID: Int64,
+         catalogSettingsService: POSCatalogSettingsServiceProtocol,
+         catalogSyncCoordinator: POSCatalogSyncCoordinatorProtocol) {
+        self.siteID = siteID
+        self.catalogSettingsService = catalogSettingsService
+        self.catalogSyncCoordinator = catalogSyncCoordinator
+    }
+
+    @MainActor
+    func loadCatalogData() async {
+        isLoading = true
+        defer { isLoading = false }
+
+        do {
+            let catalogInfo = try await catalogSettingsService.loadCatalogInfo(for: siteID)
+            catalogSize = String(format: Localization.catalogSizeFormat, catalogInfo.productCount, catalogInfo.variationCount)
+            lastFullSyncDate = formatSyncDate(catalogInfo.lastFullSyncDate)
+            lastIncrementalSyncDate = formatSyncDate(catalogInfo.lastIncrementalSyncDate)
+        } catch {
+            DDLogError("⛔️ POSSettingsLocalCatalog: Error loading catalog data: \(error)")
+            catalogSize = Localization.catalogSizeUnavailable
+            lastFullSyncDate = Localization.syncDateUnavailable
+            lastIncrementalSyncDate = Localization.syncDateUnavailable
+        }
+    }
+
+    @MainActor
+    func refreshCatalog() async {
+        isRefreshingCatalog = true
+        defer { isRefreshingCatalog = false }
+
+        do {
+            try await catalogSyncCoordinator.performFullSync(for: siteID)
+            await loadCatalogData()
+        } catch {
+            DDLogError("⛔️ POSSettingsLocalCatalog: Failed to refresh catalog: \(error)")
+        }
+    }
+}
+
+private extension POSSettingsLocalCatalogViewModel {
+    func formatSyncDate(_ date: Date?) -> String {
+        guard let date else { return Localization.neverSynced }
+        return dateFormatter.localizedString(for: date, relativeTo: Date())
+    }
+}
+
+private extension POSSettingsLocalCatalogViewModel {
+    enum Localization {
+        static let catalogSizeFormat = NSLocalizedString(
+            "posSettingsLocalCatalogViewModel.catalogSizeFormat",
+            value: "%1$d products, %2$ld variations",
+            comment: "Format string for catalog size showing product count and variation count. " +
+            "%1$d will be replaced by the product count, and %2$ld will be replaced by the variation count."
+        )
+
+        static let catalogSizeUnavailable = NSLocalizedString(
+            "posSettingsLocalCatalogViewModel.catalogSizeUnavailable",
+            value: "Catalog size unavailable",
+            comment: "Text shown when catalog size cannot be determined."
+        )
+
+        static let neverSynced = NSLocalizedString(
+            "posSettingsLocalCatalogViewModel.neverSynced",
+            value: "Not updated",
+            comment: "Text shown when no update has been performed yet."
+        )
+
+        static let syncDateUnavailable = NSLocalizedString(
+            "posSettingsLocalCatalogViewModel.syncDateUnavailable",
+            value: "Update date unavailable",
+            comment: "Text shown when update date cannot be determined."
+        )
+    }
+}

WooCommerce/Classes/POS/Presentation/Settings/PointOfSaleSettingsView.swift

@@ -55,7 +55,7 @@ extension PointOfSaleSettingsView {
                 )
 
                 // TODO: WOOMOB-1287 - integrate with local catalog feature eligibility
-                if ServiceLocator.featureFlagService.isFeatureFlagEnabled(.pointOfSaleLocalCatalogi1) {
+                if ServiceLocator.featureFlagService.isFeatureFlagEnabled(.pointOfSaleLocalCatalogi1) && settingsController.localCatalogViewModel != nil {
                     PointOfSaleSettingsCard(
                         item: .localCatalog,
                         isSelected: selection == .localCatalog,
@@ -89,7 +89,11 @@ extension PointOfSaleSettingsView {
         case .hardware:
             PointOfSaleSettingsHardwareDetailView(settingsController: settingsController)
         case .localCatalog:
-            POSSettingsLocalCatalogDetailView()
+            if let viewModel = settingsController.localCatalogViewModel {
+                POSSettingsLocalCatalogDetailView(viewModel: viewModel)
+            } else {
+                EmptyView()
+            }
         case .help:
             PointOfSaleSettingsHelpDetailView()
         default:

WooCommerce/Classes/POS/TabBar/POSTabCoordinator.swift

@@ -3,6 +3,7 @@ import UIKit
 import SwiftUI
 import Yosemite
 import class WooFoundation.CurrencySettings
+import protocol Storage.GRDBManagerProtocol
 import protocol Storage.StorageManagerType
 import class WooFoundationCore.CurrencyFormatter
 import struct NetworkingCore.JetpackSite
@@ -171,6 +172,8 @@ private extension POSTabCoordinator {
             let pluginsService = PluginsService(storageManager: storageManager)
             let siteTimezone = storesManager.sessionManager.defaultSite?.siteTimezone ?? .current
 
+            let grdbManager: GRDBManagerProtocol? = serviceAdaptor.featureFlags.isFeatureFlagEnabled(.pointOfSaleLocalCatalogi1) ? ServiceLocator.grdbManager : nil
+            let catalogSyncCoordinator = ServiceLocator.posCatalogSyncCoordinator
 
             if let receiptService = POSReceiptService(siteID: siteID,
                                                       credentials: credentials,
@@ -230,7 +233,9 @@ private extension POSTabCoordinator {
                                                                       cardPresentPaymentService: cardPresentPaymentService,
                                                                       pluginsService: pluginsService,
                                                                       defaultSiteName: storesManager.sessionManager.defaultSite?.name,
-                                                                      siteSettings: ServiceLocator.selectedSiteSettings.siteSettings),
+                                                                      siteSettings: ServiceLocator.selectedSiteSettings.siteSettings,
+                                                                      grdbManager: grdbManager,
+                                                                      catalogSyncCoordinator: catalogSyncCoordinator),
                     collectOrderPaymentAnalyticsTracker: collectPaymentAnalyticsAdaptor,
                     searchHistoryService: POSSearchHistoryService(siteID: siteID),
                     popularPurchasableItemsController: PointOfSaleItemsController(

WooCommerce/Classes/POS/Utils/PreviewHelpers.swift

@@ -28,6 +28,9 @@ import struct Yosemite.POSOrderRefund
 import typealias Yosemite.OrderItemAttribute
 import class Yosemite.POSOrderListService
 import class Yosemite.POSOrderListFetchStrategyFactory
+import protocol Yosemite.POSCatalogSyncCoordinatorProtocol
+import protocol Yosemite.POSCatalogSettingsServiceProtocol
+import struct Yosemite.POSCatalogInfo
 import struct Yosemite.Site
 
 // MARK: - PreviewProvider helpers
@@ -453,4 +456,36 @@ final class POSPreviewServices: POSDependencyProviding {
     var externalViews: POSExternalViewProviding = EmptyPOSExternalView()
 }
 
+// MARK: - Preview Catalog Services
+
+final class POSPreviewCatalogSettingsService: POSCatalogSettingsServiceProtocol {
+    func loadCatalogInfo(for siteID: Int64) async throws -> POSCatalogInfo {
+        let now = Date()
+        let lastFullSync = now.addingTimeInterval(-2 * 60 * 60) // 2 hours ago
+        let lastIncrementalSync = now.addingTimeInterval(-15 * 60) // 15 minutes ago
+        return POSCatalogInfo(
+            productCount: 247,
+            variationCount: 89,
+            lastFullSyncDate: lastFullSync,
+            lastIncrementalSyncDate: lastIncrementalSync
+        )
+    }
+}
+
+final class POSPreviewCatalogSyncCoordinator: POSCatalogSyncCoordinatorProtocol {
+    func performFullSync(for siteID: Int64) async throws {
+        // Simulates a full sync operation with a 1 second delay.
+        try await Task.sleep(nanoseconds: 1_000_000_000)
+    }
+
+    func shouldPerformFullSync(for siteID: Int64, maxAge: TimeInterval) async -> Bool {
+        true
+    }
+
+    func performIncrementalSyncIfApplicable(for siteID: Int64, forceSync: Bool) async throws {
+        // Simulates an incremental sync operation with a 0.5 second delay.
+        try await Task.sleep(nanoseconds: 500_000_000)
+    }
+}
+
 #endif