Added input naming support for multiple selection.

Author: srtab

src/Widgets/ChoiceWidget.php

@@ -33,10 +33,20 @@ public function setChoices(array $choices)
         $this->choices = $choices;
     }
 
+    public function buildName($name)
+    {
+        if ($this->allow_multiple_selected && substr($name, -2, 2) !== "[]") {
+            $name = $name . "[]";
+        }
+
+        return $name;
+    }
+
     public function getContext(string $name, $value, array $attrs = null)
     {
         $context = parent::getContext($name, $value, $attrs);
 
+        $context["name"] = $this->buildName($name);
         $context["options"] = implode($this->getSubWidgets($name, $value, $attrs));
 
         return $context;
@@ -91,7 +101,7 @@ public function getSubWidgetContext(
         return array(
             "for" => $this->buildAutoId($name, $index),
             "type" => $this->input_type,
-            "name" => htmlentities($name),
+            "name" => htmlentities($this->buildName($name)),
             "value" => htmlentities($value),
             "label" => htmlentities($label),
             "attrs" => Attributes::flatatt($attrs),

tests/unit/Widgets/CheckboxSelectMultipleTest.php

@@ -17,10 +17,10 @@ public function testRender()
         $expected =
             '<div>' .
                 '<label for="id_name_1">' .
-                    '<input id="id_name_1" name="name" type="checkbox" value="option1"/> option1' .
+                    '<input id="id_name_1" name="name[]" type="checkbox" value="option1"/> option1' .
                 '</label>' .
                 '<label for="id_name_2">' .
-                    '<input id="id_name_2" name="name" type="checkbox" value="option2"/> option2'.
+                    '<input id="id_name_2" name="name[]" type="checkbox" value="option2"/> option2'.
                 '</label>' .
             '</div>';
 
@@ -32,10 +32,10 @@ public function testRenderChecked()
         $expected =
             '<div>' .
                 '<label for="id_name_1">' .
-                    '<input id="id_name_1" name="name" type="checkbox" checked="checked" value="option1"/> option1' .
+                    '<input id="id_name_1" name="name[]" type="checkbox" checked="checked" value="option1"/> option1' .
                 '</label>' .
                 '<label for="id_name_2">' .
-                    '<input id="id_name_2" name="name" type="checkbox" value="option2"/> option2'.
+                    '<input id="id_name_2" name="name[]" type="checkbox" value="option2"/> option2'.
                 '</label>' .
             '</div>';
 
@@ -54,16 +54,16 @@ public function testRenderMultipleChecked()
         $expected =
             '<div>' .
                 '<label for="id_name_1">' .
-                    '<input id="id_name_1" name="name" type="checkbox" value="option1"/> option1' .
+                    '<input id="id_name_1" name="name[]" type="checkbox" value="option1"/> option1' .
                 '</label>' .
                 '<label for="id_name_2">' .
-                    '<input id="id_name_2" name="name" type="checkbox" checked="checked" value="option2"/> option2'.
+                    '<input id="id_name_2" name="name[]" type="checkbox" checked="checked" value="option2"/> option2'.
                 '</label>' .
                 '<label for="id_name_3">' .
-                    '<input id="id_name_3" name="name" type="checkbox" checked="checked" value="option3"/> option3'.
+                    '<input id="id_name_3" name="name[]" type="checkbox" checked="checked" value="option3"/> option3'.
                 '</label>' .
                 '<label for="id_name_4">' .
-                    '<input id="id_name_4" name="name" type="checkbox" value="option4"/> option4'.
+                    '<input id="id_name_4" name="name[]" type="checkbox" value="option4"/> option4'.
                 '</label>' .
             '</div>';
 

tests/unit/Widgets/SelectMultipleTest.php

@@ -14,7 +14,7 @@ public function setUp()
 
     public function testRender()
     {
-        $expected = '<select id="id_name" multiple="multiple" name="name">' .
+        $expected = '<select id="id_name" multiple="multiple" name="name[]">' .
                         '<option value="option1">option1</option>' .
                         '<option value="option2">option2</option>' .
                     '</select>';
@@ -24,7 +24,7 @@ public function testRender()
 
     public function testRenderSelected()
     {
-        $expected = '<select id="id_name" multiple="multiple" name="name">' .
+        $expected = '<select id="id_name" multiple="multiple" name="name[]">' .
                         '<option value="option1" selected="selected">option1</option>' .
                         '<option value="option2">option2</option>' .
                     '</select>';
@@ -41,7 +41,7 @@ public function testRenderMultipleSelected()
             "option4" => "option4",
         ]);
 
-        $expected = '<select id="id_name" multiple="multiple" name="name">' .
+        $expected = '<select id="id_name" multiple="multiple" name="name[]">' .
                         '<option value="option1">option1</option>' .
                         '<option value="option2" selected="selected">option2</option>' .
                         '<option value="option3" selected="selected">option3</option>' .