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 2018/04/12 15:31:32 UTC
[ambari] 02/02: AMBARI-23527 Slash not unescaped in property name
This is an automated email from the ASF dual-hosted git repository.
atkach pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ambari.git
commit 06424bd6c6003145f48378d09714db3f1fff5a6a
Author: Andrii Tkach <at...@apache.org>
AuthorDate: Wed Apr 11 17:16:14 2018 +0300
AMBARI-23527 Slash not unescaped in property name
---
ambari-web/app/mixins/common/configs/configs_comparator.js | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/ambari-web/app/mixins/common/configs/configs_comparator.js b/ambari-web/app/mixins/common/configs/configs_comparator.js
index 0af60ce..4dbcfc1 100644
--- a/ambari-web/app/mixins/common/configs/configs_comparator.js
+++ b/ambari-web/app/mixins/common/configs/configs_comparator.js
@@ -88,9 +88,10 @@ App.ConfigsComparator = Em.Mixin.create({
if (serviceName === 'YARN' && configuration.type === 'capacity-scheduler') {
this.addCompareCSConfigs(configuration, serviceVersionMap, item);
} else {
- for (var prop in configuration.properties) {
- serviceVersionMap[item.service_config_version][prop + '-' + configuration.type] = {
- name: prop,
+ for (const prop in configuration.properties) {
+ const name = JSON.parse('"' + prop + '"');
+ serviceVersionMap[item.service_config_version][name + '-' + configuration.type] = {
+ name: name,
value: configuration.properties[prop],
type: configuration.type,
tag: configuration.tag,
@@ -98,8 +99,8 @@ App.ConfigsComparator = Em.Mixin.create({
service_config_version: item.service_config_version,
filename: App.config.getOriginalFileName(configuration.type)
};
- if (Em.isNone(configNamesMap[prop])) {
- allConfigs.push(this.getMockConfig(prop, serviceName, App.config.getOriginalFileName(configuration.type)));
+ if (Em.isNone(configNamesMap[name])) {
+ allConfigs.push(this.getMockConfig(name, serviceName, App.config.getOriginalFileName(configuration.type)));
}
}
}
--
To stop receiving notification emails like this one, please contact
atkach@apache.org.