bug fix.

- break chart layout when click to link node again.
pinpoint-apm · Nov 4, 2016 · 0519801 · 0519801
1 parent 1079064
commit 0519801
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 9 deletions.
diff --git a/web/src/main/webapp/features/linkInfoDetails/link-info-details.directive.js b/web/src/main/webapp/features/linkInfoDetails/link-info-details.directive.js
@@ -150,12 +150,11 @@
 	                    var className = $filter('applicationNameToClassName')(toApplicationName);
 						namespace = namespace || 'forLink_' + className;
 	                    if (namespace === 'forLink' && bResponseSummaryForLinkRendered) {
-	                        scope.$broadcast('responseTimeChartDirective.updateData.' + namespace, histogram);
+	                        // scope.$broadcast('responseTimeChartDirective.updateData.' + namespace, histogram);
 	                    } else {
 	                        if (namespace === 'forLink') {
 	                            bResponseSummaryForLinkRendered = true;
 	                        }
-	                        scope.$broadcast('responseTimeChartDirective.initAndRenderWithData.' + namespace, histogram, w, h, true, true);
 	                        scope.$on('responseTimeChartDirective.itemClicked.' + namespace, function (event, data) {
 	                            var label = data.responseTime,
 	                                values = data.count;
@@ -199,6 +198,7 @@
 	                            scope.$emit('linkInfoDetailsDirective.ResponseSummary.barClicked', oServerMapFilterVoService, oServerMapHintVoService);
 	                        });
 	                    }
+						scope.$broadcast('responseTimeChartDirective.initAndRenderWithData.' + namespace, histogram, w, h, true, true);
 	                };
 
 	                renderResponseSummaryWithLink = function (namespace, link, w, h) {

diff --git a/web/src/main/webapp/features/responseTimeChart/response-time-chart.directive.js b/web/src/main/webapp/features/responseTimeChart/response-time-chart.directive.js
@@ -1,12 +1,5 @@
 (function() {
 	'use strict';
-	/**
-	 * (en)responseTimeChartDirective 
-	 * @ko responseTimeChartDirective
-	 * @group Directive
-	 * @name responseTimeChartDirective
-	 * @class
-	 */	
 	pinpointApp.constant('responseTimeChartDirectiveConfig', {
 	    myColors: ["#2ca02c", "#3c81fa", "#f8c731", "#f69124", "#f53034"]
 	});