#1507 fix more

Author: sds100

app/src/free/java/io/github/sds100/keymapper/mappings/keymaps/detection/KeyMapController.kt

@@ -903,14 +903,6 @@ class KeyMapController(
                             }
 
                             detectedShortPressTriggers.add(triggerIndex)
-
-                            val vibrateDuration = when {
-                                trigger.vibrate -> vibrateDuration(trigger)
-                                forceVibrate.value -> defaultVibrateDuration.value
-                                else -> -1L
-                            }
-
-                            vibrateDurations.add(vibrateDuration)
                         }
                     } else {
                         performActionsAfterSequenceTriggerTimeout[triggerIndex]?.cancel()
@@ -1023,11 +1015,13 @@ class KeyMapController(
             useCase.showTriggeredToast()
         }
 
-        if (forceVibrate.value) {
-            useCase.vibrate(defaultVibrateDuration.value)
-        } else {
-            vibrateDurations.maxOrNull()?.let {
-                useCase.vibrate(it)
+        if (vibrateDurations.isNotEmpty()) {
+            if (forceVibrate.value) {
+                useCase.vibrate(defaultVibrateDuration.value)
+            } else {
+                vibrateDurations.maxOrNull()?.let {
+                    useCase.vibrate(it)
+                }
             }
         }
 
@@ -1385,12 +1379,6 @@ class KeyMapController(
                     if (trigger.showToast) {
                         showToast = true
                     }
-
-                    triggerActions[triggerIndex].forEachIndexed { _, _ ->
-                        if (trigger.vibrate) {
-                            vibrateDurations.add(vibrateDuration(trigger))
-                        }
-                    }
                 }
 
                 iterator.remove()
@@ -1422,14 +1410,6 @@ class KeyMapController(
             }
         }
 
-        if (forceVibrate.value) {
-            useCase.vibrate(defaultVibrateDuration.value)
-        } else {
-            vibrateDurations.maxOrNull()?.let {
-                useCase.vibrate(it)
-            }
-        }
-
         if (showToast) {
             useCase.showTriggeredToast()
         }
@@ -1558,14 +1538,6 @@ class KeyMapController(
             useCase.showTriggeredToast()
         }
 
-        if (forceVibrate.value) {
-            useCase.vibrate(defaultVibrateDuration.value)
-        } else {
-            vibrateDurations.maxOrNull()?.let {
-                useCase.vibrate(it)
-            }
-        }
-
         detectedTriggerIndexes.forEach { triggerIndex ->
             parallelTriggerActionPerformers[triggerIndex]?.onTriggered(
                 calledOnTriggerRelease = true,