You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by al...@apache.org on 2015/07/03 14:12:12 UTC

ambari git commit: AMBARI-12090. Hive configs: save and discard buttons are blinking all the time (alexantonenko)

Repository: ambari
Updated Branches:
  refs/heads/trunk 35dc8d858 -> b74ba5182


AMBARI-12090. Hive configs: save and discard buttons are blinking all the time (alexantonenko)


Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/b74ba518
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/b74ba518
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/b74ba518

Branch: refs/heads/trunk
Commit: b74ba518298813fd6113c62a3ec54e6f86bcfcea
Parents: 35dc8d8
Author: Alex Antonenko <hi...@gmail.com>
Authored: Fri Jul 3 15:07:56 2015 +0300
Committer: Alex Antonenko <hi...@gmail.com>
Committed: Fri Jul 3 15:12:07 2015 +0300

----------------------------------------------------------------------
 .../views/common/configs/widgets/slider_config_widget_view.js | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/b74ba518/ambari-web/app/views/common/configs/widgets/slider_config_widget_view.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/views/common/configs/widgets/slider_config_widget_view.js b/ambari-web/app/views/common/configs/widgets/slider_config_widget_view.js
index e37a565..877d2b5 100644
--- a/ambari-web/app/views/common/configs/widgets/slider_config_widget_view.js
+++ b/ambari-web/app/views/common/configs/widgets/slider_config_widget_view.js
@@ -542,7 +542,12 @@ App.SliderConfigWidgetView = App.ConfigWidgetView.extend({
         this.initIncompatibleWidgetAsTextBox();
         this.initSlider();
         this.toggleWidgetState();
-        this.refreshSliderObserver();
+        // arguments exists - means method is called as observer and not directly from other method
+        // so, no need to call <code>refreshSliderObserver</code> and this prevent recursive calls
+        // like changeBoundariesOnce -> refreshSliderObserver -> changeBoundariesOnce -> ...
+        if (arguments.length) {
+          this.refreshSliderObserver();
+        }
       } catch (e) {
         console.error('error while rebuilding slider for config: ' + this.get('config.name'));
       }