From c84b1c656690737b1433899f967ffdf3d42701b0 Mon Sep 17 00:00:00 2001
From: Matthias Grob <maetugr@gmail.com>
Date: Mon, 22 Nov 2021 14:18:37 +0100
Subject: [PATCH] ManualControl: ignore unassigned mode slots

They were resulting in a message
"Switching to Unkown is currently not available".
---
 src/modules/manual_control/ManualControl.cpp | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/modules/manual_control/ManualControl.cpp b/src/modules/manual_control/ManualControl.cpp
index d087c37e75..acba99d9f2 100644
--- a/src/modules/manual_control/ManualControl.cpp
+++ b/src/modules/manual_control/ManualControl.cpp
@@ -320,6 +320,11 @@ void ManualControl::evaluateModeSlot(uint8_t mode_slot)
 
 void ManualControl::sendActionRequest(int8_t action, int8_t source, int8_t mode)
 {
+	// We catch default unassigned mode slots which have value -1
+	if (action == action_request_s::ACTION_SWITCH_MODE && mode < 0) {
+		return;
+	}
+
 	action_request_s action_request{};
 	action_request.action = action;
 	action_request.source = source;