ambari-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From onechipore...@apache.org
Subject ambari git commit: AMBARI-12159. Config error alert missing after navigate back and forth between "Customize Services" and "Configure Identity" (onechiporenko)
Date Fri, 26 Jun 2015 10:33:34 GMT
Repository: ambari
Updated Branches:
  refs/heads/branch-2.1 0f0358b87 -> d7fb687f7


AMBARI-12159. Config error alert missing after navigate back and forth between "Customize
Services" and "Configure Identity" (onechiporenko)


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

Branch: refs/heads/branch-2.1
Commit: d7fb687f7bc8bd6d390b3b5d9d47b4d4a760df18
Parents: 0f0358b
Author: Oleg Nechiporenko <onechiporenko@apache.org>
Authored: Fri Jun 26 13:29:09 2015 +0300
Committer: Oleg Nechiporenko <onechiporenko@apache.org>
Committed: Fri Jun 26 13:32:43 2015 +0300

----------------------------------------------------------------------
 .../app/controllers/main/admin/kerberos/step4_controller.js       | 3 ---
 ambari-web/app/mixins/common/configs/toggle_isrequired.js         | 2 +-
 2 files changed, 1 insertion(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/d7fb687f/ambari-web/app/controllers/main/admin/kerberos/step4_controller.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/controllers/main/admin/kerberos/step4_controller.js b/ambari-web/app/controllers/main/admin/kerberos/step4_controller.js
index f5f232f..6746f0a 100644
--- a/ambari-web/app/controllers/main/admin/kerberos/step4_controller.js
+++ b/ambari-web/app/controllers/main/admin/kerberos/step4_controller.js
@@ -40,9 +40,6 @@ App.KerberosWizardStep4Controller = App.WizardStep7Controller.extend(App.AddSecu
     this.getDescriptorConfigs().then(function(properties) {
       self.setStepConfigs(properties);
       self.set('isRecommendedLoaded', true);
-      self.get('stepConfigs').forEach(function(service) {
-        self.overrideConfigIsRequired(service);
-      })
     });
   },
 

http://git-wip-us.apache.org/repos/asf/ambari/blob/d7fb687f/ambari-web/app/mixins/common/configs/toggle_isrequired.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/mixins/common/configs/toggle_isrequired.js b/ambari-web/app/mixins/common/configs/toggle_isrequired.js
index 85d8f1d..cc2cc35 100644
--- a/ambari-web/app/mixins/common/configs/toggle_isrequired.js
+++ b/ambari-web/app/mixins/common/configs/toggle_isrequired.js
@@ -21,7 +21,7 @@ var App = require('app');
 App.ToggleIsRequiredMixin = Em.Mixin.create({
   /**
    * Override isRequired property of the configurations in given situation
-   * @param serviceName - make changes only for properties from this service
+   * @param serviceConfigs - make changes only for properties from this service
    */
   overrideConfigIsRequired: function (serviceConfigs) {
     var excludeProperties = [


Mime
View raw message