corrected display of arrays in the metric vis

elastic · Dec 17, 2016 · eb70e84 · eb70e84
1 parent 2a2fa67
commit eb70e84
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 8 deletions.
diff --git a/src/core_plugins/metric_vis/public/__tests__/metric_vis_controller.js b/src/core_plugins/metric_vis/public/__tests__/metric_vis_controller.js
@@ -21,7 +21,7 @@ describe('metric vis', function () {
     $scope.processTableGroups({
       tables: [{
         columns: [{ title: 'Count' }],
-        rows: [[4301021]],
+        rows: [[ { toString: () => formatter(4301021) } ]],
         aggConfig: function () {
           return {
             fieldFormatter: function () {
@@ -44,7 +44,7 @@ describe('metric vis', function () {
           { title: '1st percentile of bytes' },
           { title: '99th percentile of bytes' }
         ],
-        rows: [[182, 445842.4634666484]],
+        rows: [[ { toString: () => formatter(182) }, { toString: () => formatter(445842.4634666484) } ]],
         aggConfig: function () {
           return {
             fieldFormatter: function () {

diff --git a/src/core_plugins/metric_vis/public/metric_vis_controller.js b/src/core_plugins/metric_vis/public/metric_vis_controller.js
@@ -17,23 +17,24 @@ module.controller('KbnMetricVisController', function ($scope, $element, Private)
   $scope.processTableGroups = function (tableGroups) {
     tableGroups.tables.forEach(function (table) {
       table.columns.forEach(function (column, i) {
-        const fieldFormatter = table.aggConfig(column).fieldFormatter();
-        let value = table.rows[0][i];
-
-        value = isInvalid(value) ? '?' : fieldFormatter(value);
+        const value = table.rows[0][i];
 
         metrics.push({
           label: column.title,
-          value: value
+          value: value.toString('html')
         });
       });
     });
   };
 
   $scope.$watch('esResponse', function (resp) {
     if (resp) {
+      const options = {
+        asAggConfigResults: true
+      };
+
       metrics.length = 0;
-      $scope.processTableGroups(tabifyAggResponse($scope.vis, resp));
+      $scope.processTableGroups(tabifyAggResponse($scope.vis, resp, options));
       $element.trigger('renderComplete');
     }
   });