Unique names for query parameters (re getredash#164)

Author: Allen Short

client/app/services/query.js

@@ -144,7 +144,7 @@ class Parameter {
       };
     }
     return {
-      [`p_${this.name}`]: this.value,
+      [`p_${this.name}_${this.queryId}`]: this.value,
     };
   }
 
@@ -156,7 +156,7 @@ class Parameter {
         this.setValue([query[keyStart], query[keyEnd]]);
       }
     } else {
-      const key = `p_${this.name}`;
+      const key = `p_${this.name}_${this.query.id}`;
       if (has(query, key)) {
         this.setValue(query[key]);
       }
@@ -219,7 +219,9 @@ class Parameters {
     });
 
     const parameterExists = p => includes(parameterNames, p.name);
-    this.query.options.parameters = this.query.options.parameters.filter(parameterExists).map(p => new Parameter(p));
+    this.query.options.parameters = this.query.options.parameters
+      .filter(parameterExists)
+      .map(p => new Parameter(Object.assign({ queryId: this.query.id }, p)));
   }
 
   initFromQueryString(query) {
@@ -484,7 +486,7 @@ function QueryResource(
           params += '&';
         }
 
-        params += `p_${encodeURIComponent(name)}=${encodeURIComponent(value)}`;
+        params += `p_${encodeURIComponent(name)}_${this.id}=${encodeURIComponent(value)}`;
       });
     }