Repository: ambari
Updated Branches:
  refs/heads/branch-2.1 c20d89cde -> 4af321fcf


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/4af321fc
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/4af321fc
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/4af321fc

Branch: refs/heads/branch-2.1
Commit: 4af321fcf99e21260e5c019b95a27701b97a2cf9
Parents: c20d89c
Author: aBabiichuk <[email protected]>
Authored: Mon Sep 14 12:04:33 2015 +0300
Committer: aBabiichuk <[email protected]>
Committed: Tue Sep 15 11:25:23 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/4af321fc/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 65343ab..a5cc065 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({
 
@@ -468,12 +469,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);

Reply via email to