Merge pull request #47 from JerseyMilker/select-multiple

New method for setting multiple on selects.

src/AdamWathan/Form/Elements/Element.php

@@ -91,6 +91,7 @@ protected function renderAttributes()
 
     public function __call($method, $params)
     {
+        $params = count($params) ? $params : array($method);
         $params = array_merge(array($method), $params);
         call_user_func_array(array($this, 'attribute'), $params);
         return $this;

src/AdamWathan/Form/Elements/Select.php

@@ -96,4 +96,16 @@ public function defaultValue($value)
         $this->select($value);
         return $this;
     }
+
+    public function multiple()
+    {
+        $name = $this->attributes['name'];
+        if (substr($name, -2) != '[]') {
+            $name .= '[]';
+        }
+
+        $this->setName($name);
+        $this->setAttribute('multiple', 'multiple');
+        return $this;
+    }
 }

tests/SelectTest.php

@@ -241,4 +241,19 @@ public function testSelectCanBeCreatedWithIntegerKeyValueOptions()
 		$result = $select->render();
 		$this->assertEquals($expected, $result);
 	}
+
+	public function testSelectCanBeMultiple()
+	{
+		$select = new Select('people');
+		$expected = '<select name="people[]" multiple="multiple"></select>';
+		$result = $select->multiple()->render();
+
+		$this->assertEquals($expected, $result);
+
+		$select = new Select('people[]');
+		$expected = '<select name="people[]" multiple="multiple"></select>';
+		$result = $select->multiple()->render();
+
+		$this->assertEquals($expected, $result);
+	}
 }

tests/TextTest.php

@@ -260,4 +260,14 @@ public function testCanAddAttributesThroughMagicMethods()
 		$result = $text->render();
 		$this->assertEquals($expected, $result);
 	}
+
+	public function testCanAddAttributesThroughMagicMethodsWithOptionalParameter()
+	{
+		$text = new Text('cow');
+		$text = $text->moo();
+
+		$expected = '<input type="text" name="cow" moo="moo">';
+		$result = $text->render();
+		$this->assertEquals($expected, $result);
+	}
 }