HostServiceTermValidator: $testConfig requires backend for `Monitor…

…ingState::apply()` call If the `$testConfig` has no backend set and icingadb is in use, `BpConfig::getBackend()` returns an icingadb backend that is not suitable for the `MonitoringState` class
Icinga · Mar 27, 2024 · 49b62ad · 49b62ad
1 parent 4428d78
commit 49b62ad
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/library/Businessprocess/Web/Form/Validator/HostServiceTermValidator.php b/library/Businessprocess/Web/Form/Validator/HostServiceTermValidator.php
@@ -45,7 +45,8 @@ public function isValid($terms)
         }
 
         $isValid = true;
-        $testConfig = new BpConfig();
+        $testConfig = (new BpConfig())
+            ->setBackend($this->parent->getBpConfig()->getBackend());
 
         foreach ($terms as $term) {
             /** @var Term $term */
@@ -68,7 +69,7 @@ public function isValid($terms)
             }
         }
 
-        if ($this->parent->getBpConfig()->getBackend() instanceof MonitoringBackend) {
+        if ($testConfig->getBackend() instanceof MonitoringBackend) {
             MonitoringState::apply($testConfig);
         } else {
             IcingaDbState::apply($testConfig);