Merge pull request #100 from scoutforpets/fix-query-param-issue

ignores stringifying query params when content is jsonapi (fixes #99)

addon/ajax-request.js

@@ -75,7 +75,7 @@ export default class AjaxRequest {
       url: hash.url
     };
 
-    if (isJSONAPIContentType(hash.headers['Content-Type'])) {
+    if (isJSONAPIContentType(hash.headers['Content-Type']) && requestData.type !== 'GET') {
       if (typeof hash.data === 'object') {
         hash.data = JSON.stringify(hash.data);
       }

tests/unit/ajax-request-test.js

@@ -615,6 +615,30 @@ test('it JSON encodes JSON:API request data automatically', function(assert) {
   });
 });
 
+test('it does not JSON encode query parameters when JSON:API headers are present', function(assert) {
+  assert.expect(1);
+
+  server.get('/test', ({ queryParams }) => {
+    const { foo } = queryParams;
+    assert.equal(foo, 'bar', 'Correctly received query param');
+    return jsonResponse();
+  });
+
+  class RequestWithHeaders extends AjaxRequest {
+    get headers() {
+      return {
+        'Content-Type': 'application/vnd.api+json'
+      };
+    }
+  }
+  const service = new RequestWithHeaders();
+  return service.request('/test', {
+    data: {
+      foo: 'bar'
+    }
+  });
+});
+
 test('it JSON encodes JSON:API "extension" request data automatically', function(assert) {
   assert.expect(1);