ConfigForm: Only render valid configurations in the ui

refs #3648 Signed-off-by: Eric Lippmann <eric.lippmann@icinga.com>
Icinga · Feb 5, 2019 · 1e0a2cd · 1e0a2cd
1 parent ed7ecb3
commit 1e0a2cd
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/application/forms/ConfigForm.php b/application/forms/ConfigForm.php
@@ -4,6 +4,7 @@
 namespace Icinga\Forms;
 
 use Exception;
+use Icinga\Exception\ConfigurationError;
 use Zend_Form_Decorator_Abstract;
 use Icinga\Application\Config;
 use Icinga\Web\Form;
@@ -99,6 +100,10 @@ public function save()
     {
         try {
             $this->writeConfig($this->config);
+        } catch (ConfigurationError $e) {
+            $this->addError($e->getMessage());
+
+            return false;
         } catch (Exception $e) {
             $this->addDecorator('ViewScript', array(
                 'viewModule'    => 'default',