Repository: ambari
Updated Branches:
  refs/heads/trunk 49e562109 -> b95615065


AMBARI-18268 - Compare versions does not work in some cases (rzang)


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

Branch: refs/heads/trunk
Commit: b956150659ba25dac9e8f5e388c3c3d84b858521
Parents: 49e5621
Author: Richard Zang <[email protected]>
Authored: Fri Aug 26 13:41:07 2016 -0700
Committer: Richard Zang <[email protected]>
Committed: Fri Aug 26 13:43:47 2016 -0700

----------------------------------------------------------------------
 ambari-web/app/mixins/common/configs/configs_comparator.js | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/b9561506/ambari-web/app/mixins/common/configs/configs_comparator.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/mixins/common/configs/configs_comparator.js 
b/ambari-web/app/mixins/common/configs/configs_comparator.js
index 79820c0..b619aca 100644
--- a/ambari-web/app/mixins/common/configs/configs_comparator.js
+++ b/ambari-web/app/mixins/common/configs/configs_comparator.js
@@ -107,7 +107,10 @@ App.ConfigsComparator = Em.Mixin.create({
         }
         if (configuration.properties_attributes && 
configuration.properties_attributes.final) {
           for (var final in configuration.properties_attributes.final) {
-            serviceVersionMap[item.service_config_version][final + '-' + 
configuration.type].isFinal = (configuration.properties_attributes.final[final] 
=== 'true');
+            var config = serviceVersionMap[item.service_config_version][final 
+ '-' + configuration.type];
+            if (config) {
+              config.isFinal = 
(configuration.properties_attributes.final[final] === 'true');
+            }
           }
         }
       }, this);

Reply via email to