You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by at...@apache.org on 2014/11/02 21:32:10 UTC
git commit: AMBARI-8110 Properties which added from Add Service
Wizard NOT exist after deployed. (atkach)
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 <at...@hortonworks.com>
Authored: Sun Nov 2 22:31:39 2014 +0200
Committer: atkach <at...@hortonworks.com>
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) {