Share logic for unsaved changes for "Update" CTA visibility and whether to show discard changes action sheet on back navigation.

Author: jaclync

WooCommerce/Classes/ViewRelated/Products/Edit Product/ProductFormViewController.swift

@@ -20,8 +20,7 @@ final class ProductFormViewController: UIViewController {
     private var product: Product {
         didSet {
             defer {
-                let isUpdateEnabled = product != originalProduct ||
-                    productImageActionHandler.productImageStatuses.hasImagesPendingUpload
+                let isUpdateEnabled = hasUnsavedChanges(product: product)
                 updateNavigationBar(isUpdateEnabled: isUpdateEnabled)
             }
 
@@ -409,7 +408,7 @@ private extension ProductFormViewController {
 //
 extension ProductFormViewController {
     override func shouldPopOnBackButton() -> Bool {
-        if product != originalProduct {
+        if hasUnsavedChanges(product: product) {
             presentBackNavigationActionSheet()
             return false
         }
@@ -421,6 +420,10 @@ extension ProductFormViewController {
             self?.navigationController?.popViewController(animated: true)
         })
     }
+
+    private func hasUnsavedChanges(product: Product) -> Bool {
+        return product != originalProduct || productImageActionHandler.productImageStatuses.hasImagesPendingUpload
+    }
 }