Repository: ambari
Updated Branches:
  refs/heads/trunk 65b59e51f -> 9d833ee2b


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/9d833ee2
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/9d833ee2
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/9d833ee2

Branch: refs/heads/trunk
Commit: 9d833ee2ba49a20b995281ec9c9d73dd0fe51da2
Parents: 65b59e5
Author: Oleg Nechiporenko <onechipore...@apache.org>
Authored: Fri Jun 26 13:29:09 2015 +0300
Committer: Oleg Nechiporenko <onechipore...@apache.org>
Committed: Fri Jun 26 13:30:39 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/9d833ee2/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/9d833ee2/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 = [

Reply via email to