Only try to render params if they are required.

arikfr · arikfr · commit 3dde578b866a · 2015-01-29T15:33:08.000+02:00
diff --git a/rd_ui/app/scripts/services/resources.js b/rd_ui/app/scripts/services/resources.js
@@ -439,7 +439,7 @@
         return new QueryResult({job: {error: "Missing values for " + missingParams.join(', ')  + " "+paramsWord+".", status: 4}});
       }
 
-      if (parameters !== undefined) {
+      if (paramsRequired) {
         queryText = Mustache.render(queryText, parameters);
 
         // Need to clear latest results, to make sure we don't used results for different params.

Original file line number	Diff line number	Diff line change
`@@ -439,7 +439,7 @@`
`439`	`439`	`return new QueryResult({job: {error: "Missing values for " + missingParams.join(', ') + " "+paramsWord+".", status: 4}});`
`440`	`440`	`}`
`441`	`441`
`442`		`- if (parameters !== undefined) {`
	`442`	`+ if (paramsRequired) {`
`443`	`443`	`queryText = Mustache.render(queryText, parameters);`
`444`	`444`
`445`	`445`	`// Need to clear latest results, to make sure we don't used results for different params.`