From 59191641b283ca457a0a4ab391731bec0907ea40 Mon Sep 17 00:00:00 2001
From: heuj <58398276+heuj@users.noreply.github.com>
Date: Sat, 7 Nov 2020 16:46:51 +0100
Subject: [PATCH] By default show exec history only for active plans

In larger projects (many test executions), loading the exec history window contents can take quite long time. Proposing to by default only show the results from active plans.
---
 gui/javascript/testlink_library.js | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/gui/javascript/testlink_library.js b/gui/javascript/testlink_library.js
index 6995dbf6e3..78e76e1ec6 100644
--- a/gui/javascript/testlink_library.js
+++ b/gui/javascript/testlink_library.js
@@ -1635,10 +1635,16 @@ function openExecHistoryWindow(tc_id,tplan_check) {
   var width = getCookie("execHistoryPopupWidth");
   var height = getCookie("execHistoryPopupHeight");
 
-  if(tplan_check != undefined)
+  //performance: by default show only executions from active plans
+  if (tplan_check == undefined)
   {
-    url = url + '&onlyActiveTestPlans=' + tplan_check;    
-  }  
+    tplan_check = 1;
+  }
+  
+  if (tplan_check == 1 || tplan_check == 0)
+  {
+    url = url + '&onlyActiveTestPlans=' + tplan_check;
+  } 
 
   if (width == null)
   {
@@ -1649,7 +1655,7 @@ function openExecHistoryWindow(tc_id,tplan_check) {
   {
     height = "600";
   }
-  
+
   var windowCfg = "width="+width+",height="+height+",resizable=yes,scrollbars=yes,dependent=yes";
   window.open(fRoot+url, '_blank', windowCfg);
 }