Merge pull request #163 from yajra/multiple-form-values

Add postAjaxWithForm, fix multiple form values for select and checkbox

Author: yajra

src/Html/Options/HasAjax.php

@@ -44,6 +44,46 @@ public function ajax(array|string $attributes = ''): static
         return $this;
     }
 
+    /**
+     * @param  string  $url
+     * @param  string  $formSelector
+     * @return $this
+     */
+    public function postAjaxWithForm(string $url, string $formSelector): static
+    {
+        $attributes = ['url' => $url];
+
+        Arr::set($attributes, 'type', 'POST');
+        Arr::set($attributes, 'headers.X-HTTP-Method-Override', 'GET');
+
+        $script = $this->getScriptWithFormSelector($formSelector);
+
+        $attributes['data'] = "function(data) { $script }";
+
+        return $this->ajax($attributes);
+    }
+
+    /**
+     * @param  string  $formSelector
+     * @return string
+     */
+    protected function getScriptWithFormSelector(string $formSelector): string
+    {
+        return <<<CDATA
+var formData = _.groupBy($("$formSelector").find("input, select").serializeArray(), function(d) { return d.name; } );
+$.each(formData, function(i, group){
+    if (group.length > 1) {
+        data[group[0].name] = [];
+        $.each(group, function(i, obj) {
+            data[obj.name].push(obj.value)
+        })
+    } else {
+        data[group[0].name] = group[0].value;
+    }
+});
+CDATA;
+    }
+
     /**
      * Setup ajax parameter for datatables pipeline plugin.
      *
@@ -80,14 +120,7 @@ public function getAjaxUrl(): string
      */
     public function ajaxWithForm(string $url, string $formSelector): static
     {
-        $script = <<<CDATA
-var formData = $("$formSelector").find("input, select").serializeArray();
-$.each(formData, function(i, obj){
-    data[obj.name] = obj.value;
-});
-CDATA;
-
-        return $this->minifiedAjax($url, $script);
+        return $this->minifiedAjax($url, $this->getScriptWithFormSelector($formSelector));
     }
 
     /**

tests/BuilderOptionsTest.php

@@ -141,6 +141,10 @@ public function it_has_ajax_options()
         $this->assertStringContainsString('custom_script', $builder->getAjax('data'));
         $this->assertStringContainsString("data.id = 123", $builder->getAjax('data'));
         $this->assertStringContainsString("data.name = 'yajra'", $builder->getAjax('data'));
+
+        $builder->postAjaxWithForm('/test', '#formId');
+        $this->assertStringContainsString('find("input, select").serializeArray()', $builder->getAjax()['data']);
+        $this->assertStringContainsString('#formId', $builder->getAjax('data'));
     }
 
     /** @test */