You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by on...@apache.org on 2017/02/02 12:33:11 UTC

ambari git commit: AMBARI-19465. JS error when user click Alert Link in the popup on the service summary page (onechiporenko)

Repository: ambari
Updated Branches:
  refs/heads/branch-2.5 877ab571b -> 8075f5eb3


AMBARI-19465. JS error when user click Alert Link in the popup on the service summary page (onechiporenko)


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

Branch: refs/heads/branch-2.5
Commit: 8075f5eb3dae5738227c72a8a80e64db522910d0
Parents: 877ab57
Author: Oleg Nechiporenko <on...@apache.org>
Authored: Wed Jan 11 15:50:11 2017 +0200
Committer: Oleg Nechiporenko <on...@apache.org>
Committed: Thu Feb 2 14:33:02 2017 +0200

----------------------------------------------------------------------
 ambari-web/app/controllers/main/service/info/summary.js | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/8075f5eb/ambari-web/app/controllers/main/service/info/summary.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/controllers/main/service/info/summary.js b/ambari-web/app/controllers/main/service/info/summary.js
index 09f3429..9f61058 100644
--- a/ambari-web/app/controllers/main/service/info/summary.js
+++ b/ambari-web/app/controllers/main/service/info/summary.js
@@ -408,13 +408,14 @@ App.MainServiceInfoSummaryController = Em.Controller.extend(App.WidgetSectionMix
         }.property('controller.content'),
         onToggleBlock: function (alert) {
           this.$('#' + alert.context.clientId).toggle('blind', 500);
-          alert.context.set("isCollapsed", !alert.context.get("isCollapsed"));
+          alert.context.toggleProperty('isCollapsed');
         },
-        gotoAlertDetails: function (event) {
-          if (event && event.context) {
+        gotoAlertDetails: function (e) {
+          if (e && e.context) {
             this.get('parentView').hide();
-            App.router.transitionTo('main.alerts.alertDetails', event.context);
+            App.router.transitionTo('main.alerts.alertDetails', e.context);
           }
+          return false;
         },
         closePopup: function () {
           this.get('parentView').hide();