AMBARI-13546. Flume graph popups are closing only from second time. (akovalenko)

ambari-web/app/views/common/chart/linear_time.js

@@ -830,8 +830,7 @@ App.ChartLinearTimeView = Ember.View.extend(App.ExportMetricsMixin, {
 
       onPrimary: function () {
         self.setProperties({
-          currentTimeIndex: self.get('controller.isServiceWithEnhancedWidgets') === false ?
-            self.get('parentView.currentTimeRangeIndex') : self.get('parentView.parentView.currentTimeRangeIndex'),
+          currentTimeIndex: !Em.isNone(self.get('parentView.currentTimeRangeIndex')) ? self.get('parentView.currentTimeRangeIndex') : self.get('parentView.parentView.currentTimeRangeIndex'),
           isPopup: false
         });
         this._super();
@@ -898,8 +897,7 @@ App.ChartLinearTimeView = Ember.View.extend(App.ExportMetricsMixin, {
   // should be set by time range control dropdown list when create current graph
   currentTimeIndex: 0,
   setCurrentTimeIndexFromParent: function () {
-    var index = this.get('controller.isServiceWithEnhancedWidgets') === false ?
-      this.get('parentView.currentTimeRangeIndex') : this.get('parentView.parentView.currentTimeRangeIndex');
+    var index = !Em.isNone(this.get('parentView.currentTimeRangeIndex')) ? this.get('parentView.currentTimeRangeIndex') : this.get('parentView.parentView.currentTimeRangeIndex');
     this.set('currentTimeIndex', index);
   }.observes('parentView.parentView.currentTimeRangeIndex', 'parentView.currentTimeRangeIndex'),
   timeUnitSeconds: function () {

ambari-web/test/views/common/chart/linear_time_test.js

@@ -229,18 +229,22 @@ describe('App.ChartLinearTimeView', function () {
     var view,
       cases = [
       {
-        isServiceWithEnhancedWidgets: false,
-        currentTimeIndex: 1,
-        title: 'service with enhanced widgets'
+        parent: 1,
+        child: 2,
+        result: 2,
+        title: 'child and parent have currentTimeRangeIndex'
       },
       {
-        isServiceWithEnhancedWidgets: true,
-        currentTimeIndex: 2,
-        title: 'service without enhanced widgets'
+        parent: undefined,
+        child: 2,
+        result: 2,
+        title: 'only child has currentTimeRangeIndex'
       },
       {
-        currentTimeIndex: 2,
-        title: 'other view'
+        parent: 1,
+        child: undefined,
+        result: 1,
+        title: 'only parent has currentTimeRangeIndex'
       }
     ];
 
@@ -258,9 +262,10 @@ describe('App.ChartLinearTimeView', function () {
 
     cases.forEach(function (item) {
       it(item.title, function () {
-        view.set('controller.isServiceWithEnhancedWidgets', item.isServiceWithEnhancedWidgets);
+        view.set('parentView.currentTimeRangeIndex', item.child);
+        view.set('parentView.parentView.currentTimeRangeIndex', item.parent);
         view.propertyDidChange('parentView.currentTimeRangeIndex');
-        expect(view.get('currentTimeIndex')).to.equal(item.currentTimeIndex);
+        expect(view.get('currentTimeIndex')).to.equal(item.result);
       });
     });