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 <[email protected]> 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 [email protected].
