[ML] Singel Metric Viewer: Fix full refresh check. (#55726)

Fixes the check whether to do a full refresh of the page or not. lastRefresh was mistakenly part of the check. This fixes keeping the flyout open for rules editing.
elastic · Jan 24, 2020 · ef4b395 · ef4b395
1 parent d40269c
commit ef4b395
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/x-pack/legacy/plugins/ml/public/application/timeseriesexplorer/timeseriesexplorer.js b/x-pack/legacy/plugins/ml/public/application/timeseriesexplorer/timeseriesexplorer.js
@@ -965,16 +965,15 @@ export class TimeSeriesExplorer extends React.Component {
       !isEqual(previousProps.lastRefresh, this.props.lastRefresh) ||
       !isEqual(previousProps.selectedDetectorIndex, this.props.selectedDetectorIndex) ||
       !isEqual(previousProps.selectedEntities, this.props.selectedEntities) ||
-      !isEqual(previousProps.selectedForecastId, this.props.selectedForecastId) ||
+      previousProps.selectedForecastId !== this.props.selectedForecastId ||
       previousProps.selectedJobId !== this.props.selectedJobId
     ) {
       const fullRefresh =
         previousProps === undefined ||
         !isEqual(previousProps.bounds, this.props.bounds) ||
-        !isEqual(previousProps.lastRefresh, this.props.lastRefresh) ||
         !isEqual(previousProps.selectedDetectorIndex, this.props.selectedDetectorIndex) ||
         !isEqual(previousProps.selectedEntities, this.props.selectedEntities) ||
-        !isEqual(previousProps.selectedForecastId, this.props.selectedForecastId) ||
+        previousProps.selectedForecastId !== this.props.selectedForecastId ||
         previousProps.selectedJobId !== this.props.selectedJobId;
       this.loadSingleMetricData(fullRefresh);
     }