From 9b53c72826d71c80e9b0882111668c2d62376b0c Mon Sep 17 00:00:00 2001
From: Green Sky <green@g-s.xyz>
Date: Sat, 21 Aug 2021 22:30:01 +0200
Subject: [PATCH] fix overwriting tasks

---
 .../mm/update_strategies/sequential_strategy.cpp  | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

diff --git a/framework/engine/src/mm/update_strategies/sequential_strategy.cpp b/framework/engine/src/mm/update_strategies/sequential_strategy.cpp
index 97b7a4d..d68d0f2 100644
--- a/framework/engine/src/mm/update_strategies/sequential_strategy.cpp
+++ b/framework/engine/src/mm/update_strategies/sequential_strategy.cpp
@@ -5,6 +5,7 @@
 #include <cassert>
 #include <functional>
 
+#include <mm/logger.hpp>
 #include <tracy/Tracy.hpp>
 
 namespace MM::UpdateStrategies {
@@ -34,6 +35,20 @@ bool Sequential::enableService(const entt::id_type service_id, std::vector<TaskI
 	auto& service = _service_tasks[service_id];
 
 	for (const auto& task : task_array) {
+		if (_tasks.count(task._key)) {
+			entt::id_type other_service = 0;
+			for (const auto& it : _service_tasks) {
+				if (it.second.count(task._key)) {
+					other_service = it.first;
+				}
+			}
+
+			// TODO: this is cryptic
+			SPDLOG_ERROR("service '{}' adding task with taken id '{}'/'{}'. taken by '{}'", service_id, task._key, task._name, other_service);
+
+			continue;
+		}
+
 		_tasks.emplace(task._key, task);
 		service.emplace(task._key);
 	}