Handle the case where the order can be an empty string

wordpress-mobile · Jan 23, 2020 · 73fb317 · 73fb317
1 parent f3b4d1b
commit 73fb317
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/example/src/test/java/org/wordpress/android/fluxc/wc/gateways/GatewayFixtures.kt b/example/src/test/java/org/wordpress/android/fluxc/wc/gateways/GatewayFixtures.kt
@@ -7,7 +7,7 @@ val GATEWAYS_RESPONSE = listOf(
             "cod",
             "Cash on Delivery",
             "Pay cash",
-            0,
+            "",
             false,
                 "COD: Method title",
                 "COD: Method description",
@@ -17,7 +17,7 @@ val GATEWAYS_RESPONSE = listOf(
                 "stripe",
                 "Credit card",
                 "Pay with CC",
-                1,
+                "1",
                 true,
                 "Stripe: Method title",
                 "Stripe: Method description",

diff --git a/...s/woocommerce/src/main/kotlin/org/wordpress/android/fluxc/model/gateways/GatewayMapper.kt b/...s/woocommerce/src/main/kotlin/org/wordpress/android/fluxc/model/gateways/GatewayMapper.kt
@@ -10,7 +10,7 @@ class GatewayMapper
                 response.gatewayId,
                 response.title ?: "",
                 response.description ?: "",
-                response.order ?: 0,
+                response.order?.toIntOrNull() ?: 0,
                 response.enabled ?: false,
                 response.methodTitle ?: "",
                 response.methodDescription ?: "",

diff --git a/...in/kotlin/org/wordpress/android/fluxc/network/rest/wpcom/wc/gateways/GatewayRestClient.kt b/...in/kotlin/org/wordpress/android/fluxc/network/rest/wpcom/wc/gateways/GatewayRestClient.kt
@@ -75,7 +75,7 @@ constructor(
         @SerializedName("id") val gatewayId: String,
         @SerializedName("title") val title: String?,
         @SerializedName("description") val description: String?,
-        @SerializedName("order") val order: Int?,
+        @SerializedName("order") val order: String?,
         @SerializedName("enabled") val enabled: Boolean?,
         @SerializedName("method_title") val methodTitle: String?,
         @SerializedName("method_description") val methodDescription: String?,