Handle query parameters correctly

The options parameter is used to represent query parameters. So it must
be included to differentiate URLs with different query parameters.

Author: kzys

app/adapters/application.js

@@ -10,14 +10,14 @@ export default RESTAdapter.extend({
 
   ajax: function(url, type, options) {
     if (type === 'GET') {
-      let cache = this.fetcher.get(url);
+      let cache = this.fetcher.get(url, options);
       if (cache) {
         return cache;
       }
     }
 
     return this._super(url, type, options).then(resp => {
-      this.fetcher.put(url, resp);
+      this.fetcher.put(url, options, resp);
       return resp;
     });
   },

app/services/fetcher.js

@@ -7,24 +7,26 @@ const KEY = 'ajax-cache';
 export default class FetcherService extends Service {
   @service fastboot;
 
-  get(url) {
+  get(url, options) {
     let shoebox = this.fastboot.shoebox;
     if (!shoebox) {
       return;
     }
     let cache = shoebox.retrieve(KEY) || {};
-    return cache[url];
+    let key = cacheKey(url, options);
+    return cache[key];
   }
 
-  put(url, obj) {
+  put(url, options, obj) {
     let fastboot = this.fastboot;
     let shoebox = this.fastboot.shoebox;
     if (!(shoebox && fastboot.isFastBoot)) {
       return;
     }
 
     let cache = shoebox.retrieve(KEY) || {};
-    cache[url] = deepCopy(obj);
+    let key = cacheKey(url, options);
+    cache[key] = deepCopy(obj);
     shoebox.put(KEY, cache);
   }
 
@@ -41,6 +43,10 @@ export default class FetcherService extends Service {
   }
 }
 
+function cacheKey(url, options) {
+  return url + JSON.stringify(options);
+}
+
 function deepCopy(obj) {
   return JSON.parse(JSON.stringify(obj));
 }