Repository: ambari
Updated Branches:
  refs/heads/trunk ec9462989 -> 9b8251797


AMBARI-8110 Properties which added from Add Service Wizard NOT exist after 
deployed. (atkach)


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

Branch: refs/heads/trunk
Commit: 9b82517977ac8baa7c3b65abca1a1f2c0a885d27
Parents: ec94629
Author: atkach <[email protected]>
Authored: Sun Nov 2 22:32:32 2014 +0200
Committer: atkach <[email protected]>
Committed: Sun Nov 2 22:32:32 2014 +0200

----------------------------------------------------------------------
 ambari-web/app/controllers/wizard.js | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/9b825179/ambari-web/app/controllers/wizard.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/controllers/wizard.js 
b/ambari-web/app/controllers/wizard.js
index f29e8fa..59d1833 100644
--- a/ambari-web/app/controllers/wizard.js
+++ b/ambari-web/app/controllers/wizard.js
@@ -859,9 +859,12 @@ App.WizardController = 
Em.Controller.extend(App.LocalStorage, {
       // check for configs that need to update for installed services
       if (stepController.get('installedServiceNames') && 
stepController.get('installedServiceNames').contains(_content.get('serviceName')))
 {
         // get only modified configs
-        var configs = 
_content.get('configs').filterProperty('isNotDefaultValue').filter(function 
(config) {
-          var notAllowed = ['masterHost', 'masterHosts', 'slaveHosts', 
'slaveHost'];
-          return !notAllowed.contains(config.get('displayType')) && 
!!config.filename;
+        var configs = _content.get('configs').filter(function (config) {
+          if (config.get('isNotDefaultValue') || (config.get('defaultValue') 
=== null)) {
+            var notAllowed = ['masterHost', 'masterHosts', 'slaveHosts', 
'slaveHost'];
+            return !notAllowed.contains(config.get('displayType')) && 
!!config.filename;
+          }
+          return false;
         });
         // if modified configs detected push all service's configs for update
         if (configs.length) {

Reply via email to