Merge branch 'hotfix/4996'

Close #4996
zendframework · Aug 21, 2013 · 24970da · 24970da
2 parents 85829ec + 65b5c6a
commit 24970da
Show file tree

Hide file tree

Showing 5 changed files with 78 additions and 9 deletions.
diff --git a/library/Zend/Form/Form.php b/library/Zend/Form/Form.php
@@ -717,13 +717,6 @@ public function attachInputFilterDefaults(InputFilterInterface $inputFilter, Fie
         $formFactory  = $this->getFormFactory();
         $inputFactory = $formFactory->getInputFilterFactory();
 
-        if ($fieldset === $this && $fieldset instanceof InputFilterProviderInterface) {
-            foreach ($fieldset->getInputFilterSpecification() as $name => $spec) {
-                $input = $inputFactory->createInput($spec);
-                $inputFilter->add($input, $name);
-            }
-        }
-
         if ($fieldset instanceof Collection && $fieldset->getTargetElement() instanceof FieldsetInterface) {
             $elements = $fieldset->getTargetElement()->getElements();
         } else {
@@ -752,6 +745,13 @@ public function attachInputFilterDefaults(InputFilterInterface $inputFilter, Fie
                 $input = $inputFactory->createInput($spec);
                 $inputFilter->add($input, $name);
             }
+
+            if ($fieldset instanceof InputFilterProviderInterface) {
+                foreach ($fieldset->getInputFilterSpecification() as $name => $spec) {
+                    $input = $inputFactory->createInput($spec);
+                    $inputFilter->add($input, $name);
+                }
+            }
         }
 
         foreach ($fieldset->getFieldsets() as $childFieldset) {

diff --git a/library/Zend/InputFilter/BaseInputFilter.php b/library/Zend/InputFilter/BaseInputFilter.php
@@ -73,9 +73,12 @@ public function add($input, $name = null)
         }
 
         if (isset($this->inputs[$name]) && $this->inputs[$name] instanceof InputInterface) {
-            // The element already exists, so merge the config. Please note that the order is important (already existing
+            // The element already exists, so merge the config. Please note
+            // that the order is important (already existing
             // input is merged with the parameter given)
-            $input->merge($this->inputs[$name]);
+            $original = $this->inputs[$name];
+            $original->merge($input);
+            return $this;
         }
 
         $this->inputs[$name] = $input;

diff --git a/tests/ZendTest/Form/FormTest.php b/tests/ZendTest/Form/FormTest.php
@@ -1542,4 +1542,18 @@ public function testNestedFormElementNameWrapping()
         $rootForm->prepare();
         $this->assertEquals('form[element]', $element->getName());
     }
+
+    /**
+     * @group 4996
+     */
+    public function testCanOverrideDefaultInputSettings()
+    {
+        $myFieldset = new TestAsset\MyFieldset();
+        $myFieldset->setUseAsBaseFieldset(true);
+        $form = new Form();
+        $form->add($myFieldset);
+
+        $inputFilter = $form->getInputFilter()->get('my-fieldset');
+        $this->assertFalse($inputFilter->get('email')->isRequired());
+    }
 }
diff --git a/tests/ZendTest/Form/TestAsset/MyFieldset.php b/tests/ZendTest/Form/TestAsset/MyFieldset.php
@@ -0,0 +1,34 @@
+<?php
+/**
+ * Zend Framework (http://framework.zend.com/)
+ *
+ * @link      http://github.com/zendframework/zf2 for the canonical source repository
+ * @copyright Copyright (c) 2005-2013 Zend Technologies USA Inc. (http://www.zend.com)
+ * @license   http://framework.zend.com/license/new-bsd New BSD License
+ */
+
+namespace ZendTest\Form\TestAsset;
+
+use Zend\InputFilter\InputFilterProviderInterface;
+use Zend\Form\Fieldset;
+
+class MyFieldset extends Fieldset implements InputFilterProviderInterface
+{
+    public function __construct()
+    {
+        parent::__construct('my-fieldset');
+        $this->add(array(
+            'type' => 'Email',
+            'name' => 'email',
+        ));
+    }
+
+    public function getInputFilterSpecification()
+    {
+        return array(
+            'email' => array(
+                'required' => false,
+            ),
+        );
+    }
+}
diff --git a/tests/ZendTest/InputFilter/BaseInputFilterTest.php b/tests/ZendTest/InputFilter/BaseInputFilterTest.php
@@ -765,4 +765,22 @@ public function testGetInputs()
         $this->assertEquals('foo', $filters['foo']->getName());
         $this->assertEquals('bar', $filters['bar']->getName());
     }
+
+    /**
+     * @group 4996
+     */
+    public function testAddingExistingInputWillMergeIntoExisting()
+    {
+        $filter = new InputFilter();
+
+        $foo1    = new Input('foo');
+        $foo1->setRequired(true);
+        $filter->add($foo1);
+
+        $foo2    = new Input('foo');
+        $foo2->setRequired(false);
+        $filter->add($foo2);
+
+        $this->assertFalse($filter->get('foo')->isRequired());
+    }
 }