From eb343009bb696e155897dda4decef532242cf9fa Mon Sep 17 00:00:00 2001
From: Johannes Meyer <johannes.meyer@icinga.com>
Date: Thu, 13 Jul 2017 13:53:48 +0200
Subject: [PATCH] Form: Don't prevent disabled elements from getting
 overwritten..

..if someone just utilizes populate() programatically.

refs #2509
---
 library/Icinga/Web/Form.php | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/library/Icinga/Web/Form.php b/library/Icinga/Web/Form.php
index df98e6a92f..27450b3440 100644
--- a/library/Icinga/Web/Form.php
+++ b/library/Icinga/Web/Form.php
@@ -1102,21 +1102,22 @@ public function populate(array $defaults)
      *
      * @param   Zend_Form   $form
      * @param   array       $defaults
+     * @param   bool        $ignoreDisabled
      */
-    protected function preserveDefaults(Zend_Form $form, array & $defaults)
+    protected function preserveDefaults(Zend_Form $form, array & $defaults, $ignoreDisabled = true)
     {
         foreach ($form->getElements() as $name => $element) {
             if ((array_key_exists($name, $defaults)
                     && array_key_exists($name . static::DEFAULT_SUFFIX, $defaults)
                     && $defaults[$name] === $defaults[$name . static::DEFAULT_SUFFIX])
-                || $element->getAttrib('disabled')
+                || (! $ignoreDisabled && $element->getAttrib('disabled'))
             ) {
                 unset($defaults[$name]);
             }
         }
 
         foreach ($form->getSubForms() as $_ => $subForm) {
-            $this->preserveDefaults($subForm, $defaults);
+            $this->preserveDefaults($subForm, $defaults, $ignoreDisabled);
         }
     }
 
@@ -1143,6 +1144,11 @@ public function handleRequest(Request $request = null)
             if (($frameUpload = (bool) $request->getUrl()->shift('_frameUpload', false))) {
                 $this->getView()->layout()->setLayout('wrapped');
             }
+
+            // To prevent a BC, this is here. The proper fix is to extend populate()
+            // and pass $ignoreDisabled through to preserveDefaults()
+            $this->create($formData)->preserveDefaults($this, $formData, false);
+
             $this->populate($formData); // Necessary to get isSubmitted() to work
             if (! $this->getSubmitLabel() || $this->isSubmitted()) {
                 if ($this->isValid($formData)