Implemented persisting new password only if changed database password has been successful

persistence/database/RoomBankingPersistence/src/main/java/net/dankito/banking/persistence/RoomBankingPersistence.kt

@@ -41,12 +41,14 @@ open class RoomBankingPersistence(protected open val applicationContext: Context
         return openDatabase(password)
     }
 
-    override fun changePassword(newPassword: String?) {
+    override fun changePassword(newPassword: String?): Boolean {
         if (this::database.isInitialized) {
-            database.query("PRAGMA rekey = '$newPassword';", emptyArray())
+            val cursor = database.query("PRAGMA rekey = '$newPassword';", emptyArray())
+
+            return cursor.count == 1 // TODO: also check if first column content is 'ok' ?
         }
         else { // database hasn't been opened yet, that means we're on the first app run
-            openDatabase(newPassword)
+            return openDatabase(newPassword)
         }
     }
 

persistence/json/BankingPersistenceJson/src/main/kotlin/net/dankito/banking/persistence/BankingPersistenceJson.kt

@@ -40,8 +40,9 @@ open class BankingPersistenceJson(
         return true
     }
 
-    override fun changePassword(newPassword: String?) {
+    override fun changePassword(newPassword: String?): Boolean {
         // TODO: may implement data decryption. But then we have to store newPassword to be able to encrypt data
+        return true
     }
 
 

ui/BankingAndroidApp/src/main/java/net/dankito/banking/ui/android/authentication/AuthenticationService.kt

@@ -173,12 +173,13 @@ open class AuthenticationService(
             settings.salt = encodeToBase64(salt)
         }
 
-        if (saveAuthenticationSettings(settings)) {
-            this.authenticationType = type
-            this.encryptionCipherForBiometric = null
+        if (persistence.changePassword(newPassword)) {
+            if (saveAuthenticationSettings(settings)) {
+                this.authenticationType = type
+                this.encryptionCipherForBiometric = null
 
-            persistence.changePassword(newPassword) // TODO: actually this is bad. If changing password fails then password is saved in AuthenticationSettings but DB has a different password
-            return true
+                return true
+            }
         }
 
         return false

ui/BankingUiCommon/src/commonMain/kotlin/net/dankito/banking/persistence/IBankingPersistence.kt

@@ -8,7 +8,7 @@ interface IBankingPersistence {
 
     fun decryptData(password: String?): Boolean
 
-    fun changePassword(newPassword: String?)
+    fun changePassword(newPassword: String?): Boolean
 
 
     fun saveOrUpdateBank(bank: TypedBankData, allBanks: List<TypedBankData>)

ui/BankingUiCommon/src/commonMain/kotlin/net/dankito/banking/persistence/NoOpBankingPersistence.kt

@@ -10,8 +10,8 @@ open class NoOpBankingPersistence : IBankingPersistence {
         return true
     }
 
-    override fun changePassword(newPassword: String?) {
-
+    override fun changePassword(newPassword: String?): Boolean {
+        return true
     }