You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by ab...@apache.org on 2015/09/15 10:24:43 UTC

ambari git commit: AMBARI-13095 hadoop.proxyuser.*.hosts properties changed when adding services. (ababiichuk)

Repository: ambari
Updated Branches:
  refs/heads/trunk 89e239cae -> 53878959a


AMBARI-13095 hadoop.proxyuser.*.hosts properties changed when adding services.  (ababiichuk)


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

Branch: refs/heads/trunk
Commit: 53878959a618dd0e27b545c7f809f3b105bb4b4b
Parents: 89e239c
Author: aBabiichuk <ab...@cybervisiontech.com>
Authored: Mon Sep 14 12:04:33 2015 +0300
Committer: aBabiichuk <ab...@cybervisiontech.com>
Committed: Tue Sep 15 11:21:45 2015 +0300

----------------------------------------------------------------------
 ambari-web/app/mixins/common/configs/enhanced_configs.js | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/53878959/ambari-web/app/mixins/common/configs/enhanced_configs.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/mixins/common/configs/enhanced_configs.js b/ambari-web/app/mixins/common/configs/enhanced_configs.js
index e1fd598..69cc7e5 100644
--- a/ambari-web/app/mixins/common/configs/enhanced_configs.js
+++ b/ambari-web/app/mixins/common/configs/enhanced_configs.js
@@ -18,6 +18,7 @@
 
 var App = require('app');
 var blueprintUtils = require('utils/blueprint');
+var validator = require('utils/validator');
 
 App.EnhancedConfigsMixin = Em.Mixin.create({
 
@@ -469,12 +470,9 @@ App.EnhancedConfigsMixin = Em.Mixin.create({
 
           var isNewProperty = (!notDefaultGroup && Em.isNone(cp)) || (notDefaultGroup && group && Em.isNone(override));
 
-          var parsedInit = parseFloat(initialValue);
-          var parsedRecommended = parseFloat(recommendedValue);
-          if (!isNaN(parsedInit) && !isNaN(parsedRecommended)) {
-            initialValue = parsedInit.toString();
-            recommendedValue = parsedRecommended.toString();
-          }
+          initialValue = validator.isValidFloat(initialValue) ? parseFloat(initialValue).toString() : initialValue;
+          recommendedValue = validator.isValidFloat(recommendedValue) ? parseFloat(recommendedValue).toString() : recommendedValue;
+
           if (!updateOnlyBoundaries && !parentPropertiesNames.contains(propertyName) && initialValue != recommendedValue) { //on first initial request we don't need to change values
             if (dependentProperty) {
               Em.set(dependentProperty, 'value', initialValue);