Fix type for multicheckbox

Fix type for multicheckbox and add unit tests
zendframework · Aug 30, 2012 · 6ba750f · 6ba750f
1 parent 89eaf41
commit 6ba750f
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/library/Zend/Form/Element/MultiCheckbox.php b/library/Zend/Form/Element/MultiCheckbox.php
@@ -21,6 +21,15 @@
  */
 class MultiCheckbox extends Checkbox
 {
+    /**
+     * Seed attributes
+     *
+     * @var array
+     */
+    protected $attributes = array(
+        'type' => 'multi_checkbox'
+    );
+
     /**
      * @var bool
      */

diff --git a/tests/ZendTest/Form/View/Helper/FormElementTest.php b/tests/ZendTest/Form/View/Helper/FormElementTest.php
@@ -112,10 +112,13 @@ public function testRendersMultiElementsAsExpected($type, $inputType, $additiona
     {
         if ($type === 'radio') {
             $element = new Element\Radio('foo');
+            $this->assertEquals('radio', $element->getAttribute('type'));
         } elseif ($type === 'multi_checkbox') {
             $element = new Element\MultiCheckbox('foo');
+            $this->assertEquals('multi_checkbox', $element->getAttribute('type'));
         } elseif ($type === 'select') {
             $element = new Element\Select('foo');
+            $this->assertEquals('select', $element->getAttribute('type'));
         } else {
             $element = new Element('foo');
         }