feat(pat ajax): Add accept parameter with default text/html for ajax requests. We are mainly using text/html in Patternslib - if a JSON response is needed you need to configure it when using pat-ajax.

Author: thet

src/pat/ajax/ajax.js

@@ -12,12 +12,10 @@ import registry from "../../core/registry";
 const log = logging.getLogger("pat.ajax");
 
 export const parser = new Parser("ajax");
+parser.addArgument("accept", "text/html");
 parser.addArgument("url", function ($el) {
-    return ($el.is("a")
-        ? $el.attr("href")
-        : $el.is("form")
-        ? $el.attr("action")
-        : ""
+    return (
+        $el.is("a") ? $el.attr("href") : $el.is("form") ? $el.attr("action") : ""
     ).split("#")[0];
 });
 
@@ -104,10 +102,15 @@ var _ = {
             args = {
                 context: $el,
                 data: [$el.serialize(), clickedData].filter(Boolean).join("&"),
+                headers: {},
                 url: cfg.url,
                 method: $el.attr("method") ? $el.attr("method") : "GET",
             };
 
+        if (cfg.accept) {
+            args.headers.Accept = cfg.accept;
+        }
+
         if (
             $el.is("form") &&
             $el.attr("method") &&

src/pat/ajax/ajax.test.js

@@ -135,5 +135,23 @@ describe("pat-ajax", function () {
             const ajaxargs = $.ajax.mock.calls[$.ajax.mock.calls.length - 1][0];
             expect(ajaxargs.url).toEqual("else.html");
         });
+
+        // Accept
+        it("Default accept header", function () {
+            document.body.innerHTML = `<a class="pat-ajax" />`;
+            registry.scan(document.body);
+            jest.spyOn($, "ajax");
+            document.body.querySelector(".pat-ajax").click();
+            const ajaxargs = $.ajax.mock.calls[$.ajax.mock.calls.length - 1][0];
+            expect(ajaxargs.headers).toEqual({ Accept: "text/html" });
+        });
+        it("Can set accept header", function () {
+            document.body.innerHTML = `<a class="pat-ajax" data-pat-ajax="accept: */*"/>`;
+            registry.scan(document.body);
+            jest.spyOn($, "ajax");
+            document.body.querySelector(".pat-ajax").click();
+            const ajaxargs = $.ajax.mock.calls[$.ajax.mock.calls.length - 1][0];
+            expect(ajaxargs.headers).toEqual({ Accept: "*/*" });
+        });
     });
 });