(Option) Allow empty selection for OptionSelection multiple

maxkeppeler · maxkeppeler · commit 8b20004b0bdb · 2023-03-16T12:27:33.000+01:00
diff --git a/option/src/main/java/com/maxkeppeler/sheets/option/OptionState.kt b/option/src/main/java/com/maxkeppeler/sheets/option/OptionState.kt
@@ -90,8 +90,7 @@ internal class OptionState(
     private fun isValid(): Boolean = when (selection) {
         is OptionSelection.Single -> selectedOptions.isNotEmpty()
         is OptionSelection.Multiple -> {
-            selectedOptions.isNotEmpty()
-                    && (selection.minChoices?.let { selectedOptions.size >= it } ?: true)
+            (selection.minChoices?.let { selectedOptions.size >= it } ?: true)
                     && (selection.maxChoices?.let { selectedOptions.size <= it } ?: true)
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -90,8 +90,7 @@ internal class OptionState(`
`90`	`90`	`private fun isValid(): Boolean = when (selection) {`
`91`	`91`	`is OptionSelection.Single -> selectedOptions.isNotEmpty()`
`92`	`92`	`is OptionSelection.Multiple -> {`
`93`		`- selectedOptions.isNotEmpty()`
`94`		`- && (selection.minChoices?.let { selectedOptions.size >= it } ?: true)`
	`93`	`+ (selection.minChoices?.let { selectedOptions.size >= it } ?: true)`
`95`	`94`	`&& (selection.maxChoices?.let { selectedOptions.size <= it } ?: true)`
`96`	`95`	`}`
`97`	`96`	`}`