Add comments

ealeksandrov · ealeksandrov · commit 9fd2816d656d · 2023-01-17T20:32:42.000+03:00
diff --git a/WooCommerce/Classes/ViewRelated/Products/ProductsListViewModel.swift b/WooCommerce/Classes/ViewRelated/Products/ProductsListViewModel.swift
@@ -43,6 +43,8 @@ class ProductListViewModel {
         selectedProducts.removeAll()
     }
 
+    /// Check if selected products share the same common ProductStatus. Returns `nil` otherwise.
+    ///
     var commonStatusForSelectedProducts: ProductStatus? {
         let status = selectedProducts.first?.productStatus
         if selectedProducts.allSatisfy({ $0.productStatus == status }) {
@@ -52,6 +54,8 @@ class ProductListViewModel {
         }
     }
 
+    /// Update selected products with new ProductStatus and trigger Network action to save the change remotely.
+    ///
     func updateSelectedProducts(with newStatus: ProductStatus, completion: @escaping (Result<Void, Error>) -> Void ) {
         guard selectedProductsCount > 0 else {
             completion(.failure(BulkEditError.noProductsSelected))

Original file line number	Diff line number	Diff line change
`@@ -43,6 +43,8 @@ class ProductListViewModel {`
`43`	`43`	`selectedProducts.removeAll()`
`44`	`44`	`}`
`45`	`45`
	`46`	+ /// Check if selected products share the same common ProductStatus. Returns `nil` otherwise.
	`47`	`+ ///`
`46`	`48`	`var commonStatusForSelectedProducts: ProductStatus? {`
`47`	`49`	`let status = selectedProducts.first?.productStatus`
`48`	`50`	`if selectedProducts.allSatisfy({ $0.productStatus == status }) {`
`@@ -52,6 +54,8 @@ class ProductListViewModel {`
`52`	`54`	`}`
`53`	`55`	`}`
`54`	`56`
	`57`	`+ /// Update selected products with new ProductStatus and trigger Network action to save the change remotely.`
	`58`	`+ ///`
`55`	`59`	`func updateSelectedProducts(with newStatus: ProductStatus, completion: @escaping (Result<Void, Error>) -> Void ) {`
`56`	`60`	`guard selectedProductsCount > 0 else {`
`57`	`61`	`completion(.failure(BulkEditError.noProductsSelected))`