moving hard coded names to events.h (will be used elsewhere as well...)

source/armarx/navigation/core/events.h

@@ -10,6 +10,18 @@
 namespace armarx::navigation::core
 {
 
+    namespace event_names
+    {
+        inline const std::string GlobalPlanningFailed = "GlobalPlanningFailedEvent";
+        inline const std::string MovementStarted = "MovementStartedEvent";
+        inline const std::string GoalReached = "GoalReachedEvent";
+        inline const std::string WaypointReached = "WaypointReachedEvent";
+        inline const std::string SafetyThrottlingTriggered = "SafetyThrottlingTriggeredEvent";
+        inline const std::string SafetyStopTriggered = "SafetyStopTriggeredEvent";
+        inline const std::string UserAbortTriggered = "UserAbortTriggeredEvent";
+        inline const std::string InternalError = "InternalErrorEventEvent";
+    }  // namespace event_names
+
     struct Event
     {
         armarx::core::time::DateTime timestamp;

source/armarx/navigation/memory/client/events/Writer.cpp

@@ -64,55 +64,57 @@ namespace armarx::navigation::memory::client::events
     bool
     Writer::store(const core::GoalReachedEvent& event, const std::string& clientID)
     {
-        return storeImpl<core::arondto::GoalReachedEvent>(event, "GoalReachedEvent", clientID);
+        return storeImpl<core::arondto::GoalReachedEvent>(
+            event, core::event_names::GoalReached, clientID);
     }
 
     bool
     Writer::store(const core::WaypointReachedEvent& event, const std::string& clientID)
     {
         return storeImpl<core::arondto::WaypointReachedEvent>(
-            event, "WaypointReachedEvent", clientID);
+            event, core::event_names::WaypointReached, clientID);
     }
 
     bool
     Writer::store(const core::InternalErrorEvent& event, const std::string& clientID)
     {
-        return storeImpl<core::arondto::InternalErrorEvent>(event, "InternalErrorEvent", clientID);
+        return storeImpl<core::arondto::InternalErrorEvent>(
+            event, core::event_names::InternalError, clientID);
     }
 
     bool
     Writer::store(const core::GlobalPlanningFailedEvent& event, const std::string& clientID)
     {
         return storeImpl<core::arondto::GlobalPlanningFailedEvent>(
-            event, "GlobalPlanningFailedEvent", clientID);
+            event, core::event_names::GlobalPlanningFailed, clientID);
     }
 
     bool
     Writer::store(const core::MovementStartedEvent& event, const std::string& clientID)
     {
         return storeImpl<core::arondto::MovementStartedEvent>(
-            event, "MovementStartedEvent", clientID);
+            event, core::event_names::MovementStarted, clientID);
     }
 
     bool
     Writer::store(const core::UserAbortTriggeredEvent& event, const std::string& clientID)
     {
         return storeImpl<core::arondto::UserAbortTriggeredEvent>(
-            event, "UserAbortTriggered", clientID);
+            event, core::event_names::UserAbortTriggered, clientID);
     }
 
     bool
     Writer::store(const core::SafetyThrottlingTriggeredEvent& event, const std::string& clientID)
     {
         return storeImpl<core::arondto::SafetyThrottlingTriggeredEvent>(
-            event, "SafetyThrottlingTriggeredEvent", clientID);
+            event, core::event_names::SafetyThrottlingTriggered, clientID);
     }
 
     bool
     Writer::store(const core::SafetyStopTriggeredEvent& event, const std::string& clientID)
     {
         return storeImpl<core::arondto::SafetyStopTriggeredEvent>(
-            event, "SafetyStopTriggeredEvent", clientID);
+            event, core::event_names::SafetyStopTriggered, clientID);
     }