Ensure that all forms have a name

refs #5525
Icinga · Aug 22, 2014 · 75a0c17 · 75a0c17
1 parent 2b879b3
commit 75a0c17
Show file tree

Hide file tree

Showing 9 changed files with 48 additions and 1 deletion.
diff --git a/application/forms/Config/Authentication/AutologinBackendForm.php b/application/forms/Config/Authentication/AutologinBackendForm.php
@@ -11,6 +11,14 @@
  */
 class AutologinBackendForm extends BaseBackendForm
 {
+    /**
+     * Initialize this form
+     */
+    public function init()
+    {
+        $this->setName('form_config_authentication_autologin');
+    }
+
     public function isValidAuthenticationBackend()
     {
         return true;

diff --git a/application/forms/Config/Authentication/DbBackendForm.php b/application/forms/Config/Authentication/DbBackendForm.php
@@ -30,6 +30,8 @@ class DbBackendForm extends BaseBackendForm
      */
     public function init()
     {
+        $this->setName('form_config_authentication_db');
+
         $dbResources = array_keys(
             ResourceFactory::getResourceConfigs('db')->toArray()
         );

diff --git a/application/forms/Config/Authentication/LdapBackendForm.php b/application/forms/Config/Authentication/LdapBackendForm.php
@@ -30,6 +30,8 @@ class LdapBackendForm extends BaseBackendForm
      */
     public function init()
     {
+        $this->setName('form_config_authentication_ldap');
+
         $ldapResources = array_keys(
             ResourceFactory::getResourceConfigs('ldap')->toArray()
         );

diff --git a/application/forms/Dashboard/AddUrlForm.php b/application/forms/Dashboard/AddUrlForm.php
@@ -13,6 +13,14 @@
  */
 class AddUrlForm extends Form
 {
+    /**
+     * Initialize this form
+     */
+    public function init()
+    {
+        $this->setName('form_dashboard_addurl');
+    }
+
     /**
      * @see Form::createElements()
      */

diff --git a/application/forms/Preference/GeneralForm.php b/application/forms/Preference/GeneralForm.php
@@ -19,7 +19,7 @@ class GeneralForm extends Form
      */
     public function init()
     {
-        $this->setName('form_preference_set');
+        $this->setName('form_config_preferences');
     }
 
     /**

diff --git a/modules/monitoring/application/controllers/ConfigController.php b/modules/monitoring/application/controllers/ConfigController.php
@@ -251,6 +251,9 @@ protected function writeConfiguration($config, $file = null)
         return true;
     }
 
+    /**
+     * Display a form to adjust security relevant settings
+     */
     public function securityAction()
     {
         $this->view->tabs = $this->Module()->getConfigTabs()->activate('security');

diff --git a/modules/monitoring/application/forms/Config/BackendForm.php b/modules/monitoring/application/forms/Config/BackendForm.php
@@ -12,6 +12,14 @@
  */
 class BackendForm extends Form
 {
+    /**
+     * Initialize this form
+     */
+    public function init()
+    {
+        $this->setName('form_config_monitoring_backends');
+    }
+
     /**
      * @see Form::createElements()
      */

diff --git a/modules/monitoring/application/forms/Config/InstanceForm.php b/modules/monitoring/application/forms/Config/InstanceForm.php
@@ -14,6 +14,14 @@
  */
 class InstanceForm extends Form
 {
+    /**
+     * Initialize this form
+     */
+    public function init()
+    {
+        $this->setName('form_config_monitoring_instances');
+    }
+
     /**
      * @see Form::createElements()
      */

diff --git a/modules/monitoring/application/forms/Config/SecurityForm.php b/modules/monitoring/application/forms/Config/SecurityForm.php
@@ -17,6 +17,14 @@ class SecurityForm extends Form
      */
     protected $config;
 
+    /**
+     * Initialize this form
+     */
+    public function init()
+    {
+        $this->setName('form_config_monitoring_security');
+    }
+
     /**
      * @see Form::createElements()
      */