[ML] Make sure anomalyChartRecords gets initialized as an Array. (#24021

) (#24061) Fixes an issue where resizing the Anomaly Explorer window could trigger an error if the only job being loaded so far didn't have any anomalyChartRecords.
elastic · Oct 16, 2018 · 4826063 · 4826063
1 parent fad44f9
commit 4826063
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/x-pack/plugins/ml/public/explorer/__tests__/explorer_controller.js b/x-pack/plugins/ml/public/explorer/__tests__/explorer_controller.js
@@ -19,6 +19,7 @@ describe('ML - Explorer Controller', () => {
       const scope = $rootScope.$new();
       $controller('MlExplorerController', { $scope: scope });
 
+      expect(Array.isArray(scope.anomalyChartRecords)).to.be(true);
       expect(scope.loading).to.be(true);
     });
   });

diff --git a/x-pack/plugins/ml/public/explorer/explorer_controller.js b/x-pack/plugins/ml/public/explorer/explorer_controller.js
@@ -81,6 +81,7 @@ module.controller('MlExplorerController', function (
   mlSelectIntervalService,
   mlSelectSeverityService) {
 
+  $scope.anomalyChartRecords = [];
   $scope.timeFieldName = 'timestamp';
   $scope.loading = true;
   timefilter.enableTimeRangeSelector();