Merge pull request #8698 from woocommerce/issue/8524-select-all

Bulk Editing: Add select all action

Author: ealeksandrov

WooCommerce/Classes/ViewRelated/Products/ProductsListViewModel.swift

@@ -31,12 +31,16 @@ class ProductListViewModel {
         return selectedProducts.contains(productToCheck)
     }
 
-    func selectProduct(_ selectedProduct: Product) {
-        selectedProducts.insert(selectedProduct)
+    func selectProduct(_ product: Product) {
+        selectedProducts.insert(product)
     }
 
-    func deselectProduct(_ selectedProduct: Product) {
-        selectedProducts.remove(selectedProduct)
+    func selectProducts(_ products: [Product]) {
+        selectedProducts.formUnion(products)
+    }
+
+    func deselectProduct(_ product: Product) {
+        selectedProducts.remove(product)
     }
 
     func deselectAll() {

WooCommerce/Classes/ViewRelated/Products/ProductsViewController.swift

@@ -73,7 +73,7 @@ final class ProductsViewController: UIViewController, GhostableViewController {
         didSet {
             bottomToolbar.isHidden = true
             bottomToolbar.backgroundColor = .systemColor(.secondarySystemGroupedBackground)
-            bottomToolbar.setSubviews(leftViews: [], rightViews: [bulkEditButton])
+            bottomToolbar.setSubviews(leftViews: [selectAllButton], rightViews: [bulkEditButton])
             bottomToolbar.addDividerOnTop()
         }
     }
@@ -105,6 +105,18 @@ final class ProductsViewController: UIViewController, GhostableViewController {
         return button
     }()
 
+    /// The select all CTA in the bottom toolbar.
+    private lazy var selectAllButton: UIButton = {
+        let button = UIButton(frame: .zero)
+        button.setTitle(Localization.selectAllToolbarButtonTitle, for: .normal)
+        button.addTarget(self, action: #selector(selectAllProducts), for: .touchUpInside)
+        button.applyLinkButtonStyle()
+        var configuration = UIButton.Configuration.plain()
+        configuration.contentInsets = Constants.toolbarButtonInsets
+        button.configuration = configuration
+        return button
+    }()
+
     /// Container of the top banner that shows that the Products feature is still work in progress.
     ///
     private lazy var topBannerContainerView: SwappableSubviewContainerView = SwappableSubviewContainerView()
@@ -333,6 +345,12 @@ private extension ProductsViewController {
         bulkEditButton.isEnabled = viewModel.bulkEditActionIsEnabled
     }
 
+    @objc func selectAllProducts() {
+        viewModel.selectProducts(resultsController.fetchedObjects)
+        updatedSelectedItems()
+        tableView.reloadRows(at: tableView.indexPathsForVisibleRows ?? [], with: .none)
+    }
+
     @objc func openBulkEditingOptions(sender: UIButton) {
         let actionSheet = UIAlertController(title: nil, message: nil, preferredStyle: .actionSheet)
 
@@ -1305,6 +1323,10 @@ private extension ProductsViewController {
             comment: "VoiceOver accessibility hint, informing the user the button can be used to bulk edit products"
         )
 
+        static let selectAllToolbarButtonTitle = NSLocalizedString(
+            "Select all",
+            comment: "Title of a button that selects all products for bulk update"
+        )
         static let bulkEditingToolbarButtonTitle = NSLocalizedString(
             "Bulk update",
             comment: "Title of a button that presents a menu with possible products bulk update options"

WooCommerce/WooCommerceTests/ViewRelated/Products/ProductListViewModelTests.swift

@@ -38,6 +38,25 @@ final class ProductListViewModelTests: XCTestCase {
         XCTAssertFalse(viewModel.productIsSelected(sampleProduct1))
     }
 
+    func test_selecting_multiple_products_works() {
+        // Given
+        let viewModel = ProductListViewModel(siteID: sampleSiteID, stores: storesManager)
+        let sampleProduct1 = Product.fake().copy(productID: 1)
+        let sampleProduct2 = Product.fake().copy(productID: 2)
+        let sampleProduct3 = Product.fake().copy(productID: 3)
+        XCTAssertEqual(viewModel.selectedProductsCount, 0)
+
+        // When
+        viewModel.selectProduct(sampleProduct1)
+        viewModel.selectProducts([sampleProduct2, sampleProduct3])
+
+        // Then
+        XCTAssertEqual(viewModel.selectedProductsCount, 3)
+        XCTAssertTrue(viewModel.productIsSelected(sampleProduct1))
+        XCTAssertTrue(viewModel.productIsSelected(sampleProduct2))
+        XCTAssertTrue(viewModel.productIsSelected(sampleProduct3))
+    }
+
     func test_deselecting_not_selected_product_does_nothing() {
         // Given
         let viewModel = ProductListViewModel(siteID: sampleSiteID, stores: storesManager)