Merge pull request getredash#2352 from kravets-levko/bug/query-result…

…-column-names

Cast column names to string

client/app/services/query-result.js

@@ -31,8 +31,7 @@ function getColumnNameWithoutType(column) {
 }
 
 export function getColumnCleanName(column) {
-  const name = getColumnNameWithoutType(column);
-  return name;
+  return getColumnNameWithoutType(column);
 }
 
 function getColumnFriendlyName(column) {
@@ -120,6 +119,7 @@ function QueryResultService($resource, $timeout, $q) {
         });
 
         each(this.query_result.data.columns, (column) => {
+          column.name = '' + column.name;
           if (columnTypes[column.name]) {
             if (column.type == null || column.type === 'string') {
               column.type = columnTypes[column.name];
@@ -183,9 +183,7 @@ function QueryResultService($resource, $timeout, $q) {
         return null;
       }
 
-      const data = this.query_result.data.rows;
-
-      return data;
+      return this.query_result.data.rows;
     }
 
     getData() {
@@ -262,12 +260,11 @@ function QueryResultService($resource, $timeout, $q) {
         let sizeValue = null;
 
         each(row, (v, definition) => {
-          const name = definition.split('::')[0] || definition.split('__')[0];
+          definition = '' + definition;
+          const definitionParts = definition.split('::') || definition.split('__');
+          const name = definitionParts[0];
+          const type = mapping ? mapping[definition] : definitionParts[1];
           let value = v;
-          let type = definition.split('::')[1] || definition.split('__')[1];
-          if (mapping) {
-            type = mapping[definition];
-          }
 
           if (type === 'unused') {
             return;