Merge pull request #2 from jgrossi/add-group-id

Add the possibility to set the group id from the element
glhd · Nov 13, 2018 · c4d5945 · c4d5945
2 parents 9adb5a1 + 97d8973
commit c4d5945
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/src/Elements/GroupWrapper.php b/src/Elements/GroupWrapper.php
@@ -57,6 +57,17 @@ public function addGroupClass($class)
         return $this;
     }
 
+    /**
+     * @param mixed $id
+     * @return $this
+     */
+    public function setGroupId($id)
+    {
+        $this->formGroup->id($id);
+
+        return $this;
+    }
+
     /**
      * @param $class
      * @return $this

diff --git a/tests/BasicFormBuilderTest.php b/tests/BasicFormBuilderTest.php
@@ -546,6 +546,13 @@ public function testCanAddGroupClass()
         $this->assertEquals($expected, $result);
     }
 
+    public function testCanSetGroupId()
+    {
+        $expected = '<div class="form-group" id="test-id"><label class="control-label" for="email">Email</label><input type="text" name="email" id="email" class="form-control"></div>';
+        $result = $this->form->text('Email', 'email')->setGroupId('test-id')->render();
+        $this->assertEquals($expected, $result);
+    }
+
     public function testCanRemoveGroupClass()
     {
         $expected = '<div><label class="control-label" for="email">Email</label><input type="text" name="email" id="email" class="form-control"></div>';