#1688 fix: do not crash when inserting key maps if they already exist

Author: sds100

app/src/main/java/io/github/sds100/keymapper/actions/Action.kt

@@ -33,11 +33,7 @@ data class Action(
 
     val multiplier: Int? = null,
     val delayBeforeNextAction: Int? = null,
-) {
-    companion object {
-        const val REPEAT_DELAY_MIN = 0
-    }
-}
+)
 
 object ActionEntityMapper {
     fun fromEntity(entity: ActionEntity): Action? {
@@ -108,7 +104,7 @@ object ActionEntityMapper {
         )
     }
 
-    fun toEntity(keyMap: KeyMap): List<ActionEntity> = keyMap.actionList.mapNotNull { action ->
+    fun toEntity(keyMap: KeyMap): List<ActionEntity> = keyMap.actionList.map { action ->
         val base = ActionDataEntityMapper.toEntity(action.data)
 
         val extras = mutableListOf<EntityExtra>().apply {
@@ -187,7 +183,7 @@ object ActionEntityMapper {
             flags = flags.withFlag(ActionEntity.ACTION_FLAG_HOLD_DOWN)
         }
 
-        return@mapNotNull ActionEntity(
+        return@map ActionEntity(
             type = base.type,
             data = base.data,
             extras = base.extras.plus(extras),

app/src/main/java/io/github/sds100/keymapper/mappings/keymaps/ConfigKeyMapUseCase.kt

@@ -1,5 +1,6 @@
 package io.github.sds100.keymapper.mappings.keymaps
 
+import android.database.sqlite.SQLiteConstraintException
 import io.github.sds100.keymapper.actions.Action
 import io.github.sds100.keymapper.actions.ActionData
 import io.github.sds100.keymapper.actions.RepeatMode
@@ -851,7 +852,12 @@ class ConfigKeyMapUseCaseController(
         val keyMap = keyMap.value.dataOrNull() ?: return
 
         if (keyMap.dbId == null) {
-            keyMapRepository.insert(KeyMapEntityMapper.toEntity(keyMap, 0))
+            val entity = KeyMapEntityMapper.toEntity(keyMap, 0)
+            try {
+                keyMapRepository.insert(entity)
+            } catch (e: SQLiteConstraintException) {
+                keyMapRepository.update(entity)
+            }
         } else {
             keyMapRepository.update(KeyMapEntityMapper.toEntity(keyMap, keyMap.dbId))
         }