Repository: ambari
Updated Branches:
  refs/heads/branch-1.7.0 55fea90a4 -> 2f519d361


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

Branch: refs/heads/branch-1.7.0
Commit: 2f519d361b5411e5166d7ab49b7be9da942f4b7e
Parents: 55fea90
Author: atkach <[email protected]>
Authored: Sun Nov 2 22:31:39 2014 +0200
Committer: atkach <[email protected]>
Committed: Sun Nov 2 22:31:39 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/2f519d36/ambari-web/app/controllers/wizard.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/controllers/wizard.js 
b/ambari-web/app/controllers/wizard.js
index 01f3ea2..46b224a 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