Merge pull request #148 from soramitsu/fix/change_pin_crash

fix ArrayIndexOutOfBoundsException on pincode screen
novasamatech · Jan 28, 2021 · aaf845b · aaf845b
2 parents ea6d500 + b4d24e3
commit aaf845b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/...n/java/jp/co/soramitsu/feature_account_impl/presentation/pincode/view/DotsProgressView.kt b/...n/java/jp/co/soramitsu/feature_account_impl/presentation/pincode/view/DotsProgressView.kt
@@ -50,7 +50,7 @@ class DotsProgressView @JvmOverloads constructor(
         for (circle in circles) {
             circle?.background = emptyDrawable
         }
-        if (currentProgress == 0) {
+        if (currentProgress == 0 || currentProgress > MAX_PROGRESS) {
             return
         }
         for (i in 0 until currentProgress) {

diff --git a/...c/main/java/jp/co/soramitsu/feature_account_impl/presentation/pincode/view/PinCodeView.kt b/...c/main/java/jp/co/soramitsu/feature_account_impl/presentation/pincode/view/PinCodeView.kt
@@ -88,7 +88,7 @@ class PinCodeView @JvmOverloads constructor(
     }
 
     private fun pinNumberAdded(number: String) {
-        if (inputCode.length > DotsProgressView.MAX_PROGRESS) {
+        if (inputCode.length >= DotsProgressView.MAX_PROGRESS) {
             return
         } else {
             inputCode += number