Fix query string parsing (#518)

tonymeng · web-flow · commit 2e147ff61da8 · 2018-02-16T13:53:30.000-08:00
* Fix query string parsing

* [AUTOMATED]: Prettier Code Styling

* Feedback
diff --git a/packages/database/src/core/util/libs/parser.ts b/packages/database/src/core/util/libs/parser.ts
@@ -47,11 +47,14 @@ function decodeQuery(queryString: string): { [key: string]: string } {
     queryString = queryString.substring(1);
   }
   for (const segment of queryString.split('&')) {
+    if (segment.length === 0) {
+      continue;
+    }
     const kv = segment.split('=');
     if (kv.length === 2) {
       results[decodeURIComponent(kv[0])] = decodeURIComponent(kv[1]);
     } else {
-      warn('Invalid query string segment: ' + segment);
+      warn(`Invalid query segment '${segment}' in query '${queryString}'`);
     }
   }
   return results;

Original file line number	Diff line number	Diff line change
`@@ -47,11 +47,14 @@ function decodeQuery(queryString: string): { [key: string]: string } {`
`47`	`47`	`queryString = queryString.substring(1);`
`48`	`48`	`}`
`49`	`49`	`for (const segment of queryString.split('&')) {`
	`50`	`+ if (segment.length === 0) {`
	`51`	`+ continue;`
	`52`	`+ }`
`50`	`53`	`const kv = segment.split('=');`
`51`	`54`	`if (kv.length === 2) {`
`52`	`55`	`results[decodeURIComponent(kv[0])] = decodeURIComponent(kv[1]);`
`53`	`56`	`} else {`
`54`		`- warn('Invalid query string segment: ' + segment);`
	`57`	+ warn(`Invalid query segment '${segment}' in query '${queryString}'`);
`55`	`58`	`}`
`56`	`59`	`}`
`57`	`60`	`return results;`