Integrate GenerateAllVariations Presenter

Author: Ecarrion

WooCommerce/Classes/ViewRelated/Products/Variations/ProductVariationsViewController.swift

@@ -565,12 +565,7 @@ private extension ProductVariationsViewController {
 
     @objc func addButtonTapped() {
         analytics.track(event: WooAnalyticsEvent.Variations.addMoreVariationsButtonTapped(productID: product.productID))
-
-        if featureFlagService.isFeatureFlagEnabled(.generateAllVariations) {
-            presentGenerateVariationOptions()
-        } else {
-            createVariation()
-        }
+        presentGenerateVariationOptions()
     }
 
     /// More Options Action Sheet
@@ -616,83 +611,6 @@ private extension ProductVariationsViewController {
         }
     }
 
-    /// Informs the merchant about errors that happen during the variation generation
-    ///
-    private func presentGenerationError(_ error: ProductVariationsViewModel.GenerationError) {
-        let notice = Notice(title: error.errorTitle, message: error.errorDescription)
-        noticePresenter.enqueue(notice: notice)
-    }
-
-    /// Asks the merchant for confirmation before generating all variations.
-    ///
-    private func presentGenerationConfirmation(numberOfVariations: Int, onCompletion: @escaping (_ confirmed: Bool) -> Void) {
-        let controller = UIAlertController(title: Localization.confirmationTitle,
-                                           message: Localization.confirmationDescription(variationCount: numberOfVariations),
-                                           preferredStyle: .alert)
-        controller.addDefaultActionWithTitle(Localization.ok) { _ in
-            onCompletion(true)
-        }
-        controller.addCancelActionWithTitle(Localization.cancel) { _ in
-            onCompletion(false)
-        }
-
-        // There should be an `inProgressViewController` being presented. Fallback to `self` in case there isn't one.
-        let baseViewController = presentedViewController ?? self
-        baseViewController.present(controller, animated: true)
-    }
-
-    /// Presents a blocking view controller while variations are being fetched.
-    ///
-    private func presentFetchingIndicator() {
-        let inProgressViewController = InProgressViewController(viewProperties: .init(title: Localization.fetchingVariations, message: ""))
-        present(inProgressViewController, animated: true)
-    }
-
-    /// Presents a blocking view controller while variations are being created.
-    ///
-    private func presentCreatingIndicator() {
-        let newViewProperties = InProgressViewProperties(title: Localization.creatingVariations, message: "")
-
-        // There should be already a presented `InProgressViewController`. But we present one in case there isn;t.
-        guard let inProgressViewController = self.presentedViewController as? InProgressViewController else {
-            let inProgressViewController = InProgressViewController(viewProperties: newViewProperties)
-            return present(inProgressViewController, animated: true)
-        }
-
-        // Update the already presented view controller with the "Creating..." copy.
-        inProgressViewController.updateViewProperties(newViewProperties)
-    }
-
-    /// Dismiss any `InProgressViewController` being presented.
-    /// By default retires the dismissal one time to overcome UIKit presentation delays.
-    ///
-    private func dismissBlockingIndicator(retry: Bool = true) {
-        if let inProgressViewController = self.presentedViewController as? InProgressViewController {
-            inProgressViewController.dismiss(animated: true)
-        } else {
-            // When this method is invoked right after `InProgressViewController` is presented, chances are that it won't exists in the view controller
-            // hierarchy yet.
-            // Here we are retrying it with a small delay to give UIKit APIs time to finish its presentation.
-            DispatchQueue.main.asyncAfter(deadline: .now() + 0.3) {
-                self.dismissBlockingIndicator(retry: false)
-            }
-        }
-    }
-
-    /// Informs the merchant that no variations were created.
-    ///
-    private func presentNoGenerationNotice() {
-        let notice = Notice(title: Localization.noVariationsCreatedTitle, message: Localization.noVariationsCreatedDescription)
-        noticePresenter.enqueue(notice: notice)
-    }
-
-    /// Informs the merchant that some variations were created.
-    ///
-    private func presentVariationsCreatedNotice() {
-        let notice = Notice(title: Localization.variationsCreatedTitle)
-        noticePresenter.enqueue(notice: notice)
-    }
-
     /// Updates the current product with the up-to-date list of variations IDs.
     /// This is needed in order to reflect variations count changes back to this and to other screens.
     ///
@@ -775,28 +693,18 @@ extension ProductVariationsViewController: SyncingCoordinatorDelegate {
     /// Generates all possible variations for the product attributes.
     ///
     private func generateAllVariations() {
-        viewModel.generateAllVariations(for: product) { [weak self] currentState in
+        let presenter = GenerateAllVariationsPresenter(baseViewController: self)
+        viewModel.generateAllVariations(for: product) { [weak self, presenter] currentState in
+            // Perform Presentation Actions
+            presenter.handleStateChanges(state: currentState)
+
+            // Perform other side effects
             switch currentState {
-            case .fetching:
-                self?.presentFetchingIndicator()
-            case .confirmation(let variationCount, let onCompletion):
-                self?.presentGenerationConfirmation(numberOfVariations: variationCount, onCompletion: onCompletion)
-            case .creating:
-                self?.presentCreatingIndicator()
-            case .canceled:
-                self?.dismissBlockingIndicator()
             case .finished(let variationsCreated):
-                self?.dismissBlockingIndicator()
                 if variationsCreated {
                     self?.updateProductVariationCount()
-                    self?.presentVariationsCreatedNotice()
-                } else {
-                    self?.presentNoGenerationNotice()
                 }
-                break
-            case .error(let error):
-                self?.dismissBlockingIndicator()
-                self?.presentGenerationError(error)
+            default: break
             }
         }
     }
@@ -905,27 +813,6 @@ private extension ProductVariationsViewController {
         static let generateVariationError = NSLocalizedString("The variation couldn't be generated.",
                                                               comment: "Error title when failing to generate a variation.")
         static let variationCreated = NSLocalizedString("Variation created", comment: "Text for the notice after creating the first variation.")
-
-        static let confirmationTitle = NSLocalizedString("Generate all variations?",
-                                                         comment: "Alert title to allow the user confirm if they want to generate all variations")
-        static func confirmationDescription(variationCount: Int) -> String {
-            let format = NSLocalizedString("This will create a variation for each and every possible combination of variation attributes (%d variations).",
-                                           comment: "Alert description to allow the user confirm if they want to generate all variations")
-            return String.localizedStringWithFormat(format, variationCount)
-        }
-        static let ok = NSLocalizedString("OK", comment: "Button text to confirm that we want to generate all variations")
-        static let cancel = NSLocalizedString("Cancel", comment: "Button text to confirm that we don't want to generate all variations")
-        static let fetchingVariations = NSLocalizedString("Fetching Variations...",
-                                                          comment: "Blocking indicator text when fetching existing variations prior generating them.")
-        static let creatingVariations = NSLocalizedString("Creating Variations...",
-                                                          comment: "Blocking indicator text when creating multiple variations remotely.")
-        static let noVariationsCreatedTitle = NSLocalizedString("No variations to generate",
-                                                                comment: "Title for the notice when there were no variations to generate")
-        static let noVariationsCreatedDescription = NSLocalizedString("All variations are already generated.",
-                                                                      comment: "Message for the notice when there were no variations to generate")
-        static let variationsCreatedTitle = NSLocalizedString("Variations created successfully",
-                                                              comment: "Title for the notice when after variations were created")
-
     }
 
     /// Localizated strings for the  action sheet options