Merge pull request #574 from yaquawa/remove_fields

Remove multiple fields with one call.
kristijanhusak · Jan 12, 2020 · 6dd28e5 · 6dd28e5
2 parents 9090e02 + 7d8d7fb
commit 6dd28e5
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 6 deletions.
diff --git a/src/Kris/LaravelFormBuilder/Form.php b/src/Kris/LaravelFormBuilder/Form.php
@@ -340,13 +340,15 @@ public function compose($class, array $options = [], $modify = false)
     /**
      * Remove field with specified name from the form.
      *
-     * @param $name
+     * @param string|string[] $names
      * @return $this
      */
-    public function remove($name)
+    public function remove($names)
     {
-        if ($this->has($name)) {
-            unset($this->fields[$name]);
+        foreach (is_array($names) ? $names : func_get_args() as $name) {
+            if ($this->has($name)) {
+                unset($this->fields[$name]);
+            }
         }
 
         return $this;

diff --git a/tests/FormTest.php b/tests/FormTest.php
@@ -448,11 +448,12 @@ public function it_can_remove_existing_fields_from_form_object()
 
         $this->assertTrue($this->plainForm->has('name'));
 
-        $this->plainForm->remove('name');
+        $this->plainForm->remove('name', 'description');
 
-        $this->assertEquals(2, count($this->plainForm->getFields()));
+        $this->assertEquals(1, count($this->plainForm->getFields()));
 
         $this->assertFalse($this->plainForm->has('name'));
+        $this->assertFalse($this->plainForm->has('description'));
     }
 
     /** @test */