[ML] Adding missed applyAsync calls (#28814)

* [ML] Adding missed applyAsync calls * fixing data recognizer results panel * adding extra applySync calls to job saving in advanced job page * fix for forcast results display issue
elastic · Jan 24, 2019 · ad48848 · ad48848
1 parent 735cc82
commit ad48848
Show file tree

Hide file tree

Showing 5 changed files with 24 additions and 3 deletions.
diff --git a/x-pack/plugins/ml/public/components/data_recognizer/data_recognizer.js b/x-pack/plugins/ml/public/components/data_recognizer/data_recognizer.js
@@ -41,6 +41,9 @@ export class DataRecognizer extends Component {
         ));
         if (typeof this.results === 'object') {
           this.results.count = results.length;
+          if (typeof this.results.onChange === 'function') {
+            this.results.onChange();
+          }
         }
 
         this.setState({

diff --git a/x-pack/plugins/ml/public/datavisualizer/datavisualizer_controller.js b/x-pack/plugins/ml/public/datavisualizer/datavisualizer_controller.js
@@ -95,7 +95,12 @@ module
     $scope.metricFieldFilter = '';
     $scope.fieldFilterIcon = 0;
     $scope.fieldFilter = '';
-    $scope.recognizerResults = { count: 0 };
+    $scope.recognizerResults = {
+      count: 0,
+      onChange() {
+        $scope.$applyAsync();
+      }
+    };
 
     $scope.showSidebar = isFullLicense();
 

diff --git a/x-pack/plugins/ml/public/jobs/new_job/advanced/new_job_controller.js b/x-pack/plugins/ml/public/jobs/new_job/advanced/new_job_controller.js
@@ -595,6 +595,7 @@ module.controller('MlNewJob',
                     );
                     // update status
                     $scope.ui.saveStatus.job = 2;
+                    $scope.$applyAsync();
 
                     // save successful, attempt to open the job
                     mlJobService.openJob($scope.job.job_id)
@@ -631,10 +632,14 @@ module.controller('MlNewJob',
                               resp
                             );
                             $scope.saveLock = false;
+                          })
+                          .then(() => {
+                            $scope.$applyAsync();
                           });
                       } else {
                         // no datafeed, so save is complete
                         $scope.saveLock = false;
+                        $scope.$applyAsync();
                       }
                     }
 
@@ -650,6 +655,7 @@ module.controller('MlNewJob',
                         values: { message: result.resp.message }
                       })
                     );
+                    $scope.$applyAsync();
                   }
                 }).catch((result) => {
                   $scope.ui.saveStatus.job = -1;
@@ -660,6 +666,7 @@ module.controller('MlNewJob',
                       values: { message: result.resp.message }
                     })
                   );
+                  $scope.$applyAsync();
                 });
             }
           })
@@ -670,11 +677,13 @@ module.controller('MlNewJob',
               })
             );
             console.log('save(): job validation failed. Jobs list could not be loaded.');
+            $scope.$applyAsync();
           });
       }
       else {
         msgs.error(jobValid.message);
         console.log('save(): job validation failed');
+        $scope.$applyAsync();
       }
     };
 
@@ -689,6 +698,7 @@ module.controller('MlNewJob',
       })
         .then(() => {
           msgs.clear();
+          $scope.$applyAsync();
           $location.path('jobs');
         });
     };

diff --git a/...k/plugins/ml/public/jobs/new_job/simple/single_metric/create_job/create_job_controller.js b/...k/plugins/ml/public/jobs/new_job/simple/single_metric/create_job/create_job_controller.js
@@ -475,6 +475,7 @@ module
             msgs.error(i18n('xpack.ml.newJob.simple.singleMetric.saveDatafeedFailedErrorMessage', {
               defaultMessage: 'Save datafeed failed: '
             }), resp);
+            $scope.$applyAsync();
           });
       }
     };

diff --git a/...ns/ml/public/timeseriesexplorer/components/timeseries_chart/timeseries_chart_directive.js b/...ns/ml/public/timeseriesexplorer/components/timeseries_chart/timeseries_chart_directive.js
@@ -32,7 +32,7 @@ import { I18nProvider } from '@kbn/i18n/react';
 import chrome from 'ui/chrome';
 const mlAnnotationsEnabled = chrome.getInjected('mlAnnotationsEnabled', false);
 
-module.directive('mlTimeseriesChart', function () {
+module.directive('mlTimeseriesChart', function ($timeout) {
 
   function link(scope, element) {
     // Key dimensions for the viz and constituent charts.
@@ -85,7 +85,9 @@ module.directive('mlTimeseriesChart', function () {
     renderReactComponent();
 
     scope.$on('render', () => {
-      renderReactComponent();
+      $timeout(() => {
+        renderReactComponent();
+      });
     });
 
     function renderFocusChart() {