Update status parameter to use WCOrderStatusModel instead.

wordpress-mobile · Dec 16, 2024 · 3ba4738 · 3ba4738
1 parent f9bad88
commit 3ba4738
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/example/src/test/java/org/wordpress/android/fluxc/wc/order/WCOrderStoreTest.kt b/example/src/test/java/org/wordpress/android/fluxc/wc/order/WCOrderStoreTest.kt
@@ -641,7 +641,11 @@ class WCOrderStoreTest {
                 .thenReturn(BulkUpdateOrderStatusResponsePayload(successResponses))
 
             // When
-            val result = orderStore.batchUpdateOrdersStatus(site, orderIds, newStatus)
+            val result = orderStore.batchUpdateOrdersStatus(
+                site,
+                orderIds,
+                WCOrderStatusModel(COMPLETED.value)
+            )
 
             // Then
             assertThat(result.isError).isFalse()
@@ -687,7 +691,11 @@ class WCOrderStoreTest {
                 .thenReturn(BulkUpdateOrderStatusResponsePayload(mixedResponses))
 
             // When
-            val result = orderStore.batchUpdateOrdersStatus(site, orderIds, newStatus)
+            val result = orderStore.batchUpdateOrdersStatus(
+                site,
+                orderIds,
+                WCOrderStatusModel(COMPLETED.value)
+            )
 
             // Then
             assertThat(result.isError).isFalse()

diff --git a/plugins/woocommerce/src/main/kotlin/org/wordpress/android/fluxc/store/WCOrderStore.kt b/plugins/woocommerce/src/main/kotlin/org/wordpress/android/fluxc/store/WCOrderStore.kt
@@ -1174,9 +1174,9 @@ class WCOrderStore @Inject constructor(
     suspend fun batchUpdateOrdersStatus(
         site: SiteModel,
         orderIds: List<Long>,
-        newStatus: String
+        newStatus: WCOrderStatusModel
     ): WooResult<UpdateOrdersStatusResult> {
-        val result = wcOrderRestClient.batchUpdateOrdersStatus(site, orderIds, newStatus)
+        val result = wcOrderRestClient.batchUpdateOrdersStatus(site, orderIds, newStatus.statusKey)
 
         return if (!result.isError) {
             val orders = result.response