From f7d2bdbc78d25c1ef2c3c92d7893f75163904dd2 Mon Sep 17 00:00:00 2001
From: Thomas Gelf <thomas@gelf.net>
Date: Tue, 24 Jun 2014 21:11:48 +0200
Subject: [PATCH] ConfigController: notifications before redirection

That's how they reach the browser in a nice way
---
 application/controllers/ConfigController.php | 22 +++++++-------------
 1 file changed, 7 insertions(+), 15 deletions(-)

diff --git a/application/controllers/ConfigController.php b/application/controllers/ConfigController.php
index e18cd2b1b..48a2f92c3 100644
--- a/application/controllers/ConfigController.php
+++ b/application/controllers/ConfigController.php
@@ -121,7 +121,7 @@ class ConfigController extends BaseConfigController
             if (!$this->writeConfigFile($form->getConfig(), 'config'))  {
                 return;
             }
-            $this->addSuccessMessage("Configuration Sucessfully Updated");
+            Notification::success("Configuration Sucessfully Updated");
             $form->setConfiguration(IcingaConfig::app(), true);
             $this->redirectNow('config/index');
         }
@@ -143,7 +143,7 @@ class ConfigController extends BaseConfigController
             if (!$this->writeConfigFile($form->getConfig(), 'config')) {
                 return;
             }
-            $this->addSuccessMessage("Configuration Sucessfully Updated");
+            Notification::success("Configuration Sucessfully Updated");
             $form->setConfiguration(IcingaConfig::app(), true);
             $this->redirectNow('config/logging');
         }
@@ -241,7 +241,7 @@ class ConfigController extends BaseConfigController
 
             if ($form->isSubmittedAndValid()) {
                 if ($this->writeAuthenticationFile($form->getReorderedConfig($config))) {
-                    $this->addSuccessMessage('Authentication Order Updated');
+                    Notification::success('Authentication Order Updated');
                     $this->redirectNow('config/authentication');
                 }
 
@@ -291,7 +291,7 @@ class ConfigController extends BaseConfigController
             }
             if ($this->writeAuthenticationFile($backendCfg)) {
                 // redirect to overview with success message
-                $this->addSuccessMessage('Backend Modification Written.');
+                Notification::success('Backend Modification Written.');
                 $this->redirectNow("config/authentication");
             }
             return;
@@ -356,7 +356,7 @@ class ConfigController extends BaseConfigController
             }
             if ($this->writeAuthenticationFile($backendCfg)) {
                 // redirect to overview with success message
-                $this->addSuccessMessage('Backend "' . $authBackend . '" created');
+                Notification::success('Backend "' . $authBackend . '" created');
                 $this->redirectNow("config/authentication");
             }
             return;
@@ -380,7 +380,7 @@ class ConfigController extends BaseConfigController
         $configArray = IcingaConfig::app('authentication', true)->toArray();
         $authBackend =  $this->getParam('auth_backend');
         if (!isset($configArray[$authBackend])) {
-            $this->addSuccessMessage('Can\'t perform removal: Unknown Authentication Backend Provided');
+            Notification::error('Can\'t perform removal: Unknown Authentication Backend Provided');
             $this->render('authentication/remove');
             return;
         }
@@ -392,7 +392,7 @@ class ConfigController extends BaseConfigController
         if ($form->isSubmittedAndValid()) {
             unset($configArray[$authBackend]);
             if ($this->writeAuthenticationFile($configArray)) {
-                $this->addSuccessMessage('Authentication Backend "' . $authBackend . '" Removed');
+                Notification::success('Authentication Backend "' . $authBackend . '" Removed');
                 $this->redirectNow("config/authentication");
             }
             return;
@@ -403,14 +403,6 @@ class ConfigController extends BaseConfigController
         $this->render('authentication/remove');
     }
 
-    /**
-     * Show developer tools
-     */
-    public function devtoolsAction()
-    {
-        $this->view->tabs = null;
-    }
-
     public function resourceAction($showOnly = false)
     {
         $this->view->tabs->activate('resources');