From 871c81ac1aa6b61e8af1cbbcf7c04de7b6659248 Mon Sep 17 00:00:00 2001
From: vaxerski <43317083+vaxerski@users.noreply.github.com>
Date: Wed, 3 Aug 2022 17:15:45 +0200
Subject: [PATCH] erase instead of overwriting monitor rules

---
 src/config/ConfigManager.cpp | 16 ++--------------
 1 file changed, 2 insertions(+), 14 deletions(-)

diff --git a/src/config/ConfigManager.cpp b/src/config/ConfigManager.cpp
index de8e87af..b8c0ab6e 100644
--- a/src/config/ConfigManager.cpp
+++ b/src/config/ConfigManager.cpp
@@ -433,13 +433,7 @@ void CConfigManager::handleMonitor(const std::string& command, const std::string
             return;
         }
 
-        // overwrite if exists
-        for (auto& r : m_dMonitorRules) {
-            if (r.name == newrule.name) {
-                r = newrule;
-                return;
-            }
-        }
+        m_dMonitorRules.erase(std::remove_if(m_dMonitorRules.begin(), m_dMonitorRules.end(), [&](const auto& other) { return other.name == newrule.name; }));
 
         m_dMonitorRules.push_back(newrule);
 
@@ -474,13 +468,7 @@ void CConfigManager::handleMonitor(const std::string& command, const std::string
         return;
     }
 
-    // overwrite if exists
-    for (auto& r : m_dMonitorRules) {
-        if (r.name == newrule.name) {
-            r = newrule;
-            return;
-        }
-    }
+    m_dMonitorRules.erase(std::remove_if(m_dMonitorRules.begin(), m_dMonitorRules.end(), [&](const auto& other) { return other.name == newrule.name; }));
 
     m_dMonitorRules.push_back(newrule);
 }