diff --git a/application/controllers/ConfigController.php b/application/controllers/ConfigController.php index e56a8cdbe..2205bc4b2 100644 --- a/application/controllers/ConfigController.php +++ b/application/controllers/ConfigController.php @@ -366,23 +366,25 @@ class ConfigController extends BaseConfigController $configArray = IcingaConfig::app('authentication', true)->toArray(); $authBackend = $this->getParam('auth_backend'); - if (!isset($configArray[$authBackend])) { - Notification::error('Can\'t perform removal: Unknown Authentication Backend Provided'); - $this->render('authentication/remove'); - return; + if (false === array_key_exists($authBackend, $configArray)) { + $this->addErrorMessage( + $this->translate('Can\'t perform removal: Unknown authentication backend provided') + ); + $this->redirectNow('config/configurationerror'); } $form = new ConfirmRemovalForm(); - $form->setRequest($this->getRequest()); - $form->setRemoveTarget('auth_backend', $authBackend); + $request = $this->getRequest(); - if ($form->isSubmittedAndValid()) { + if ($request->isPost() && $form->isValid($request->getPost())) { unset($configArray[$authBackend]); if ($this->writeAuthenticationFile($configArray)) { - Notification::success('Authentication Backend "' . $authBackend . '" Removed'); - $this->redirectNow("config/authentication"); + Notification::success(sprintf( + $this->translate('Authentication Backend "%s" Removed'), + $authBackend + )); + $this->redirectNow('config/authentication'); } - return; } $this->view->form = $form; diff --git a/application/views/scripts/config/authentication/remove.phtml b/application/views/scripts/config/authentication/remove.phtml index ed478d24b..23b85f6d7 100644 --- a/application/views/scripts/config/authentication/remove.phtml +++ b/application/views/scripts/config/authentication/remove.phtml @@ -1,10 +1,8 @@

- - Remove Backend "escape($this->name); ?>" + translate('Remove Backend "%s"'), + $this->escape($name) + ); ?>

- -messageBox)): ?> - messageBox->render() ?> - - -form ?> \ No newline at end of file + +