fixed a bug where game size could be any number, thanks to @ariantron

yamin8000 · Nov 20, 2022 · 3e9da35 · 3e9da35
1 parent 066e284
commit 3e9da35
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/app/build.gradle b/app/build.gradle
@@ -32,8 +32,8 @@ android {
         applicationId "io.github.yamin8000.dooz"
         minSdk 21
         targetSdk 33
-        versionCode 1
-        versionName "1.0.0"
+        versionCode 2
+        versionName "1.0.1"
         vectorDrawables {
             useSupportLibrary true
         }

diff --git a/app/src/main/java/io/github/yamin8000/dooz/content/settings/SettingsState.kt b/app/src/main/java/io/github/yamin8000/dooz/content/settings/SettingsState.kt
@@ -66,8 +66,10 @@ class SettingsState(
     var gameSize: Int
         get() = _gameSize.value
         set(value) {
-            _gameSize.value = value
-            scope.launch { dataStore.setInt(Constants.gameSize, value) }
+            if (value in Constants.gameSizeRange) {
+                _gameSize.value = value
+                scope.launch { dataStore.setInt(Constants.gameSize, value) }
+            }
         }
 
     var firstPlayerName: String

diff --git a/app/src/main/java/io/github/yamin8000/dooz/util/Constants.kt b/app/src/main/java/io/github/yamin8000/dooz/util/Constants.kt
@@ -27,6 +27,7 @@ object Constants {
     const val theme = "theme"
 
     const val gameSize = "gameSize"
+    val gameSizeRange = 3..7
 
     const val gamePlayersType = "gamePlayersType"
     const val firstPlayerName = "firstPlayerName"