You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by sr...@apache.org on 2014/12/09 23:31:35 UTC

ambari git commit: AMBARI-8622. Alerts UI: Enabling/Disabling alert-definitions takes too long

Repository: ambari
Updated Branches:
  refs/heads/trunk 1a04f7c7d -> abff00010


AMBARI-8622. Alerts UI: Enabling/Disabling alert-definitions takes too long


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

Branch: refs/heads/trunk
Commit: abff00010279aa64cf36d489f260702206ba4444
Parents: 1a04f7c
Author: Srimanth Gunturi <sg...@hortonworks.com>
Authored: Tue Dec 9 14:09:57 2014 -0800
Committer: Srimanth Gunturi <sg...@hortonworks.com>
Committed: Tue Dec 9 14:09:57 2014 -0800

----------------------------------------------------------------------
 ambari-web/app/controllers/main/alert_definitions_controller.js  | 4 +++-
 .../app/controllers/main/alerts/definition_details_controller.js | 4 +++-
 2 files changed, 6 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/abff0001/ambari-web/app/controllers/main/alert_definitions_controller.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/controllers/main/alert_definitions_controller.js b/ambari-web/app/controllers/main/alert_definitions_controller.js
index 876b515..2a27f5a 100644
--- a/ambari-web/app/controllers/main/alert_definitions_controller.js
+++ b/ambari-web/app/controllers/main/alert_definitions_controller.js
@@ -71,13 +71,15 @@ App.MainAlertDefinitionsController = Em.ArrayController.extend({
    * @method toggleDefinitionState
    */
   toggleDefinitionState: function(alertDefinition) {
+    var newState = !alertDefinition.get('enabled');
+    alertDefinition.set('enabled', newState);
     return App.ajax.send({
       name: 'alerts.update_alert_definition',
       sender: this,
       data: {
         id: alertDefinition.get('id'),
         data: {
-          "AlertDefinition/enabled": !alertDefinition.get('enabled')
+          "AlertDefinition/enabled": newState
         }
       }
     });

http://git-wip-us.apache.org/repos/asf/ambari/blob/abff0001/ambari-web/app/controllers/main/alerts/definition_details_controller.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/controllers/main/alerts/definition_details_controller.js b/ambari-web/app/controllers/main/alerts/definition_details_controller.js
index ab23487..1197ac1 100644
--- a/ambari-web/app/controllers/main/alerts/definition_details_controller.js
+++ b/ambari-web/app/controllers/main/alerts/definition_details_controller.js
@@ -216,13 +216,15 @@ App.MainAlertDefinitionDetailsController = Em.Controller.extend({
    */
   toggleState: function () {
     var alertDefinition = this.get('content');
+    var newState = !alertDefinition.get('enabled');
+    alertDefinition.set('enabled', newState);
     return App.ajax.send({
       name: 'alerts.update_alert_definition',
       sender: this,
       data: {
         id: alertDefinition.get('id'),
         data: {
-          "AlertDefinition/enabled": !alertDefinition.get('enabled')
+          "AlertDefinition/enabled": newState
         }
       }
     });