fix(types): update Route query type definition to support null values

Author: Yash Dayama

flow/declarations.js

@@ -84,7 +84,7 @@ declare type Location = {
   name?: string;
   path?: string;
   hash?: string;
-  query?: Dictionary<string>;
+  query?: Dictionary<string | null | Array<string | null>>;
   params?: Dictionary<string>;
   append?: boolean;
   replace?: boolean;
@@ -96,7 +96,7 @@ declare type Route = {
   path: string;
   name: ?string;
   hash: string;
-  query: Dictionary<string>;
+  query: Dictionary<string | null | Array<string | null>>;
   params: Dictionary<string>;
   fullPath: string;
   matched: Array<RouteRecord>;

types/router.d.ts

@@ -411,7 +411,7 @@ export interface Route {
   path: string
   name?: string | null
   hash: string
-  query: Dictionary<string | (string | null)[]>
+  query: Dictionary<string | null | (string | null)[]>
   params: Dictionary<string>
   fullPath: string
   matched: RouteRecord[]

types/test/query-types.test.ts

@@ -0,0 +1,53 @@
+import VueRouter from '../index'
+import { Route } from '../index'
+
+const router = new VueRouter()
+const route: Route = router.currentRoute
+
+const stringQuery: string | null | (string | null)[] = route.query['stringParam']
+if (typeof stringQuery === 'string') {
+  console.log(stringQuery.toLowerCase())
+}
+
+const nullQuery: string | null | (string | null)[] = route.query['paramWithoutValue']
+if (nullQuery === null) {
+  console.log('Parameter exists but has no value')
+}
+
+const arrayQuery: string | null | (string | null)[] = route.query['arrayParam']
+if (Array.isArray(arrayQuery)) {
+  arrayQuery.forEach(value => {
+    if (value === null) {
+      console.log('Array contains null value')
+    } else {
+      console.log(value.toLowerCase())
+    }
+  })
+}
+
+router.push({
+  path: '/test',
+  query: {
+    string: 'value',              // string value
+    empty: null,                  // null value
+    array: ['value1', 'value2'],  // array of strings
+    mixedArray: ['value', null],  // array with null
+    noValue: null                 // parameter without value
+  }
+})
+
+const routeWithQueries: Route = {
+  path: '/test',
+  name: null,
+  hash: '',
+  query: {
+    param1: 'string',            // string value
+    param2: null,               // null value
+    param3: ['val1', 'val2'],   // array of strings
+    param4: ['val', null],      // array with null
+    param5: null                // parameter without value
+  },
+  params: {},
+  fullPath: '/test',
+  matched: []
+}