Repository: ambari
Updated Branches:
  refs/heads/branch-2.4 d6d66b30e -> 72da7c5ed


AMBARI-17915 User principals not showing in configs tab in UI. (ababiichuk)


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

Branch: refs/heads/branch-2.4
Commit: 72da7c5ed90d3edcad54a88a4b502a2370c6c925
Parents: d6d66b3
Author: ababiichuk <[email protected]>
Authored: Wed Jul 27 14:45:26 2016 +0300
Committer: ababiichuk <[email protected]>
Committed: Wed Jul 27 16:25:46 2016 +0300

----------------------------------------------------------------------
 ambari-web/app/mappers/configs/stack_config_properties_mapper.js | 3 ---
 ambari-web/app/models/configs/objects/service_config_property.js | 3 +++
 2 files changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/72da7c5e/ambari-web/app/mappers/configs/stack_config_properties_mapper.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/mappers/configs/stack_config_properties_mapper.js 
b/ambari-web/app/mappers/configs/stack_config_properties_mapper.js
index 9eda557..2c8959d 100644
--- a/ambari-web/app/mappers/configs/stack_config_properties_mapper.js
+++ b/ambari-web/app/mappers/configs/stack_config_properties_mapper.js
@@ -141,9 +141,6 @@ App.stackConfigPropertiesMapper = 
App.QuickDataMapper.create({
           }
 
           var staticConfigInfo = this.parseIt(config, 
this.get('configToPlain'));
-          if (Em.isNone(staticConfigInfo.value)) {
-            staticConfigInfo.isVisible = false;
-          }
           var v = Em.isNone(staticConfigInfo.recommendedValue) ? 
staticConfigInfo.recommendedValue : staticConfigInfo.value;
           staticConfigInfo.value = staticConfigInfo.recommendedValue = 
App.config.formatPropertyValue(staticConfigInfo, v);
           staticConfigInfo.isSecureConfig = 
App.config.getIsSecure(staticConfigInfo.name);

http://git-wip-us.apache.org/repos/asf/ambari/blob/72da7c5e/ambari-web/app/models/configs/objects/service_config_property.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/models/configs/objects/service_config_property.js 
b/ambari-web/app/models/configs/objects/service_config_property.js
index 861d599..e9ed8d7 100644
--- a/ambari-web/app/models/configs/objects/service_config_property.js
+++ b/ambari-web/app/models/configs/objects/service_config_property.js
@@ -282,6 +282,9 @@ App.ServiceConfigProperty = Em.Object.extend({
         this.set('value', this.get('recommendedValue'));
       }
     }
+    if (this.get('value') === null) {
+      this.set('isVisible', false);
+    }
     if (this.get("displayType") === "password") {
       this.set('retypedPassword', this.get('value'));
       this.set('recommendedValue', '');

Reply via email to