bug fix

- prevent other click until when complete to load transaction detail page.

Author: denzelsN

web/src/main/webapp/features/transactionTable/transaction-table.directive.js

@@ -16,6 +16,7 @@
 
 	            // define private variables of methods
 	            var clear, appendTransactionList, resetIndexToTransactionList;
+				var bIsDetailPageLoading = false;
 
 	            // initialize scope variables
 	            scope.transactionList = [];
@@ -63,6 +64,9 @@
 	             * @param transaction
 	             */
 	            scope.traceByApplication = function (transaction) {
+					if ( bIsDetailPageLoading === true ) {
+						return;
+					}
 					AnalyticsService.send(AnalyticsService.CONST.CALLSTACK, AnalyticsService.CONST.CLK_TRANSACTION);
 	                scope.currentTransaction = transaction;
 	                scope.$emit('transactionTableDirective.applicationSelected', transaction);
@@ -109,6 +113,9 @@
 	            scope.$on('transactionTableDirective.appendTransactionList', function (event, transactionList) {
 	                appendTransactionList(transactionList);
 	            });
+				scope.$on('transactionTableDirective.completedDetailPageLoad', function () {
+					bIsDetailPageLoading = false;
+				});
 
 	            /**
 	             * scope event on transactionTableDirective.clear

web/src/main/webapp/pages/transactionDetail/transaction-detail.controller.js

@@ -198,6 +198,11 @@
 	        });
 
 			tooltipService.init( "callTree" );
+
+			// invoke top frame angular method
+			try {
+				top.angular.element( top.document.getElementById("main-container") ).scope().completedDetailPageLoad();
+			} catch(e) {}
 	    }
 	]);
 })();

web/src/main/webapp/pages/transactionList/transaction-list.controller.js

@@ -1,4 +1,4 @@
-(function() {
+(function( $ ) {
 	'use strict';
 	/**
 	 * (en)TransactionListCtrl 
@@ -18,7 +18,7 @@
 			analyticsService.send(analyticsService.CONST.TRANSACTION_LIST_PAGE);
 	        // define private variables
 	        var nFetchCount, nLastFetchedIndex, htTransactionInfo, htTransactionData, oTimeSliderVoService;
-			var aParamTransactionInfo;
+			var aParamTransactionInfo, beforeTransactionDetailUrl = "";
 
 	        // define private variables of methods
 	        var fetchStart, fetchNext, fetchAll, emitTransactionListToTable, getQuery, getTransactionList, changeTransactionDetail,
@@ -300,14 +300,19 @@
 	         * @param transaction
 	         */
 	        changeTransactionDetail = function (transaction) {
-				$location.path( "/transactionList/" + $routeParams.application + "/" + $routeParams.readablePeriod + "/" + $routeParams.queryEndDateTime + "/" + transaction.traceId + "-" + transaction.collectorAcceptTime + "-" + transaction.elapsed , false );
-	            var transactionDetailUrl = 'index.html#/transactionDetail'; // the filename should be existing, if not it's doesn't work on ie and firefox
-	            if (transaction.traceId && transaction.collectorAcceptTime) {
-	                transactionDetailUrl += '/' + $window.encodeURIComponent(transaction.traceId) + '/' + transaction.collectorAcceptTime;
-					$timeout(function() {
+				var transactionDetailUrl = 'index.html#/transactionDetail';
+				if (transaction.traceId && transaction.collectorAcceptTime) {
+					transactionDetailUrl += '/' + $window.encodeURIComponent(transaction.traceId) + '/' + transaction.collectorAcceptTime;
+				}
+				if ( beforeTransactionDetailUrl == transactionDetailUrl ) {
+					$scope.$emit( "transactionTableDirective.completedDetailPageLoad" );
+				} else {
+					beforeTransactionDetailUrl = transactionDetailUrl;
+					$location.path( "/transactionList/" + $routeParams.application + "/" + $routeParams.readablePeriod + "/" + $routeParams.queryEndDateTime + "/" + transaction.traceId + "-" + transaction.collectorAcceptTime + "-" + transaction.elapsed , false );
+					$timeout(function () {
 						$scope.transactionDetailUrl = transactionDetailUrl;
 					});
-	            }
+				}
 	        };
 
 	        /**
@@ -326,8 +331,10 @@
 	            $timeout(function () {
 	                fetchNext();
 	            }, 1000);
-	
 	        });
+			$scope.completedDetailPageLoad = function() {
+				$scope.$emit( "transactionTableDirective.completedDetailPageLoad" );
+			};
 	    }
 	]);
-})();
+})( jQuery );

web/src/main/webapp/pages/transactionList/transactionList.html

@@ -4,5 +4,5 @@
 <!-- Add your site or application content here -->
 <div id="main-container" class="main-container">
     <transaction-table-directive ng-style="table" class="ui-layout-north" loading-directive="sidebarLoading" loading-message="Loading..."></transaction-table-directive>
-    <iframe class="ui-layout-center" ng-src="{{transactionDetailUrl}}" style="border:none;padding:0;margin:0;width:100%;height:100%;"></iframe>
+    <iframe class="ui-layout-center" ng-src="{{transactionDetailUrl}}" style="border:none;padding:0;margin:0;width:100%;height:100%;" id="transactionDetailIframe"></iframe>
 </div>