From 244a9c2a0a0daa66fa054bc4ec55d3573c347e77 Mon Sep 17 00:00:00 2001
From: Maykel Geerdink <maykel.geerdink@aareon.nl>
Date: Mon, 4 Dec 2017 12:19:09 +0000
Subject: [PATCH] FSN#51437: Gereedmelden van periodieke taak geeft Javascript
 fout - groupby is not defined.

svn path=/Website/branches/v2017.2/; revision=36193
---
 APPL/INS/ins_edit_inspect.asp  | 59 +++++++++++++++++-----------------
 APPL/INS/ins_inspect_save.asp  |  7 ++--
 APPL/INS/ins_inspect_start.asp |  2 +-
 APPL/INS/ins_list.js           | 54 ++++++++++++++++++-------------
 APPL/MJB/mjb_search_list.asp   |  2 +-
 5 files changed, 68 insertions(+), 56 deletions(-)

diff --git a/APPL/INS/ins_edit_inspect.asp b/APPL/INS/ins_edit_inspect.asp
index 69bb1534bc..42bea76564 100644
--- a/APPL/INS/ins_edit_inspect.asp
+++ b/APPL/INS/ins_edit_inspect.asp
@@ -38,7 +38,9 @@ var insPlan = getQParamInt("plan", 0) == 1;      // Inplannen inspectie.
 var insClose = getQParamInt("close", 0) == 1;    // Afmelden of afronden inspectie.
 var insEdit = !insPlan && !insClose; // Wijzigen inspectie.
 var fromfinish = getQParamInt("fromfinish", 0) == 1; // Alleen Voltooien bij bulkactie (multi).
+var finishmjob = getQParamInt("finishmjob", 0) == 1;
 var groupby = getQParamInt("groupby", 7);
+var catclose = (finishmjob && groupby < 7); // Bulk closen per categorie.
 
 if (insEdit && multi)
   abort_with_warning(L("lcl_ins_allow_multi_edit"));
@@ -63,15 +65,14 @@ var mjb_start_year = S("mjb_start_year");
 var mjb_freeze_year = S("mjb_freeze_year");
 
 var transitParam = "";
-if (groupby < 7)
+if (catclose)
+{
   // Bulk closen per categorie.
-  transitParam = buildTransitParam(["close", "ins_keys", "srtcont_keys", "deelsrtcont_keys", "fromfinish",
+  transitParam = buildTransitParam(["close", "ins_keys", "srtcont_keys", "deelsrtcont_keys", "fromfinish", "finishmjob",
                                     "dist_key", "loc_key", "bld_key", "disc_key", "srtgroep_kye", "srtdeel_key",
                                     "groep", "categori_key", "srtcontrole", "frequentie", "incbtw", "groupby",
                                     "deel", "mjbMoved", "mjbFreezed", "mjbXcped"]);
 
-if (groupby < 7)
-{
   var dist_key = getQParamInt("dist_key", -1); // District
   var loc_key = getQParamInt("loc_key", -1); // Locatie
   var bld_key = getQParamInt("bld_key", -1); // Gebouw
@@ -223,7 +224,7 @@ lcl.set_dialect(ctrdisc_key, "INS_TAB_DISCIPLINE_KEY");
 user.anything_todo_or_abort(tobehandled > 0); // We klagen niet over enkele wel en enkele niet
 
 var desc = "";
-if (groupby == 7 && ins_key_arr.length == 1)
+if (!catclose && ins_key_arr.length == 1)
 {
     sql = "SELECT d.ins_deel_omschrijving"      // identificatie
         + "     , s.ins_srtdeel_code"
@@ -257,7 +258,7 @@ if (groupby == 7 && ins_key_arr.length == 1)
     var startdatum;
     var eersteinspectie;
     var info = "";
-    if (groupby == 7)
+    if (!catclose)
     {
       sql = "SELECT *"
           + "  FROM (SELECT DISTINCT xcp.ins_srtcontrole_eenheid eenheid"
@@ -315,7 +316,7 @@ if (groupby == 7 && ins_key_arr.length == 1)
     <script type="text/javascript">
       var eenheid = -1;
 
-<%    if (groupby == 7 && insClose)
+<%    if (!catclose && insClose)
       { %>
         var inspected_srtcontroles = new Array();
 <%
@@ -352,7 +353,7 @@ if (groupby == 7 && ins_key_arr.length == 1)
         }
       }
 
-      if (groupby == 7)
+      if (!catclose)
       {
         var insstatus = -1;
         var status_str = "";
@@ -391,18 +392,18 @@ if (groupby == 7 && ins_key_arr.length == 1)
       function ins_submit(finish)
       {
         // Controle verplichte velden.
-<%      if (groupby == 7 && insPlan || (insEdit && insstatus == 0)) // Alleen de datum verplicht bij inplannen en bij het wijzigen van ingeplande inspectie.
+<%      if (!catclose && insPlan || (insEdit && insstatus == 0)) // Alleen de datum verplicht bij inplannen en bij het wijzigen van ingeplande inspectie.
         { %>
           if (!validateForm("u2", { checkOnly: ["show_plandatum", "time_from_plandatum"] }))
             return false;
 <%      }
-        else if (groupby == 7 && !(insEdit && insstatus == 2)) // Bij wijzigen in status 2 is er geen enkel veld verplicht.
+        else if (!catclose && !(insEdit && insstatus == 2)) // Bij wijzigen in status 2 is er geen enkel veld verplicht.
         { %>
           if (!validateForm("u2"))
             return false;
 <%      } %>
 
-<%      if (groupby == 7 && insPlan)
+<%      if (!catclose && insPlan)
         { %>
           // Controleren voor uren inspectie of plandatum niet in het verleden ligt.
           // Dit kan alleen voorkomen bij uren omdat de uren voor vandaag de hele dag ingevuld kunnen worden.
@@ -413,7 +414,7 @@ if (groupby == 7 && ins_key_arr.length == 1)
             return false;
           }
 <%      }
-        else if (groupby == 7 && insClose)
+        else if (!catclose && insClose)
         { %>
           // Controleren voor uren inspectie of inspectie uitvoerdatum na de laatste uitvoerdatum ligt.
           // Bij multi geldt de laatste uitvoerdatum van alle objecten.
@@ -472,7 +473,7 @@ if (groupby == 7 && ins_key_arr.length == 1)
           }
           $("#finish").val(finish);
 <%      }
-        else if (groupby == 7 && insEdit)
+        else if (!catclose && insEdit)
         { %>
           if (dsc_is_afgemeld)
           {
@@ -497,7 +498,7 @@ if (groupby == 7 && ins_key_arr.length == 1)
           FcltMgr.closeDetail(window, { cancel: true } );
       }
 
-<%    if (groupby == 7 && (insClose || insEdit))
+<%    if (!catclose && (insClose || insEdit))
       { %>
         function onChangeDeelsrtcdatum(id)
         {
@@ -530,7 +531,7 @@ if (groupby == 7 && ins_key_arr.length == 1)
   <body id="editbody">
 <%  // Melding aanmaakbaar? Alleen meldingen die enigzins beperkt zijn tot onze object(soort/groep)
     var canMelding = false;
-    if (groupby == 7 && !multi && ingesloten[0] > 0)
+    if (!catclose && !multi && ingesloten[0] > 0)
     {
       var sql = "SELECT COUNT(*) aantal"
               + "  FROM mld_stdmelding msm"
@@ -573,7 +574,7 @@ if (groupby == 7 && ins_key_arr.length == 1)
       buttons.push({title: (insPlan? L("lcl_ins_schedule") : L("lcl_submit")), icon: "opslaan.png", action:"ins_submit(0)"});
     if (insClose)
     { // Bij bulkacties (multi) alleen kijken naar de bulkactie die uitgevoerd moet worden.
-      if (groupby == 7 && canClose && !(multi && fromfinish))
+      if (!catclose && canClose && !(multi && fromfinish))
         buttons.push({title: L("lcl_ins_close"), icon: "inspfinish.png", action:"ins_submit(0)"});
       var finishAfterClose = (nrCanFinishAfterClose > 0 && multi && !fromfinish); // Na het gereed melden kan voor minstens 1 periodieke taak worden afgerond.
       if ((canFinish && (!multi || fromfinish)) || finishAfterClose)
@@ -600,21 +601,21 @@ if (groupby == 7 && ins_key_arr.length == 1)
       <input type="hidden" name="savemode" id="savemode" value="<%=insPlan? "P" : (insClose? "C" : "E")%>">
       <input type="hidden" name="ins_keys" id="ins_keys" value="<%=ingesloten.join(",")%>">
       <input type="hidden" name="srtcont_key_arr" id="srtcont_key_arr" value="<%=ingeslotensc.join(",")%>">
-<%    if (groupby == 7 && insEdit)
+<%    if (!catclose && insEdit)
       { %>
         <input type="hidden" name="deelsrtcont_key" id="deelsrtcont_key" value="<%=ingeslotendsc[0]%>">
 <%    } %>
       <input type="hidden" name="finish" id="finish" value="0">
 <%    var blocktext = (insPlan? L("lcl_ins_schedule") : (insClose? L("lcl_ins_stop") : L("lcl_details") ))
-                    + (multi || groupby < 7
+                    + (multi || catclose
                       ? " (" + L("lcl_count") + ":" + tobehandled + ")"
                       : "");
       BLOCK_START("insMultiEdit", blocktext);
-        if (groupby == 7 && ins_key_arr.length == 1)
+        if (!catclose && ins_key_arr.length == 1)
         {
           ROFIELDTR("fld", L("lcl_obj_sort"), sort );
         }
-        if (groupby == 7 && !multi)
+        if (!catclose && !multi)
         { // Inspecteren van één object met één of meerdere soortcontroles
           var sql = "SELECT xcp.ins_srtcontrole_key"
                   + "     , " + lcl.xsqla('xcp.ins_srtcontrole_omschrijving', 'xcp.ins_srtcontrole_key')
@@ -653,7 +654,7 @@ if (groupby == 7 && ins_key_arr.length == 1)
 
         // Status tonen.
         var params = {};
-        if (groupby == 7 && !multi && ingeslotendsc[0] > 0)
+        if (!catclose && !multi && ingeslotendsc[0] > 0)
         {
           params = { infoPointer: { Url: "appl/shared/status_info.asp?insc_key=" + ingeslotendsc[0] + "&urole=bo",
                                     Title: L("lcl_status_details") + " " + ingeslotendsc[0]
@@ -664,7 +665,7 @@ if (groupby == 7 && ins_key_arr.length == 1)
         ROFIELDTR("fld", L("lcl_ins_status"), status_str, params);
 
 
-        if (groupby == 7)
+        if (!catclose)
         {
           // Men mag alleen een inspectie-uitvoerdatum invoeren die later is dan de laatste inspectie-uitvoerdatum.
           // Bij multi ook de laatste uitvoerdatum bepalen.
@@ -768,7 +769,7 @@ if (groupby == 7 && ins_key_arr.length == 1)
           }
         }
 
-        if (groupby == 7 && (insPlan || (insEdit && (insstatus == 0 || insstatus == 2))))
+        if (!catclose && (insPlan || (insEdit && (insstatus == 0 || insstatus == 2))))
           FCLTcalendar("plandatum",
                        { label: L("lcl_ins_controle_plandate"),
                          datum: minDate > inspect_data.deelsrtcontrole_plandatum? minDate : inspect_data.deelsrtcontrole_plandatum,
@@ -783,7 +784,7 @@ if (groupby == 7 && ins_key_arr.length == 1)
                          suppressEmpty: true
                        });
 
-        if (groupby == 7 && (insClose || (insEdit && insstatus >= 5)))
+        if (!catclose && (insClose || (insEdit && insstatus >= 5)))
         {
           FCLTcalendar("deelsrtcontrole_plandatum",
                        { label: L("lcl_ins_controle_plandate"),
@@ -915,7 +916,7 @@ if (groupby == 7 && ins_key_arr.length == 1)
                          });
           }
         }
-        else (groupby != 7)
+        else (catclose)
         {
           FCLTcalendar("deelsrtcontrole_datum",
                        { label: L("lcl_ins_controle_datum"),
@@ -935,7 +936,7 @@ if (groupby == 7 && ins_key_arr.length == 1)
 %>        <input type="hidden" id="deelsrtcontrole_datum" name="deelsrtcontrole_datum" value="<%=new Date().getTime()%>">
 <%      }
 
-        if (groupby == 7)
+        if (!catclose)
         {
           sql = "SELECT ins_controlemode_key"
               + "     , " + lcl.xsqla('ins_controlemode_oms', 'ins_controlemode_key')
@@ -1005,13 +1006,13 @@ if (groupby == 7 && ins_key_arr.length == 1)
         RWTEXTAREATR("opmerk",
                      "fldremark",
                      L("lcl_remark"),
-                     (groupby == 7? inspect_data.deelsrtcontrole_opmerking : ""),
+                     (!catclose? inspect_data.deelsrtcontrole_opmerking : ""),
                      { html: "rows='8'",
                        trclass: "invoerinsp",
                        maxlength: 4000
                      }
                     );
-      if (groupby == 7)
+      if (!catclose)
       { %>
         <script type="text/javascript">
           var insstatus = <%=insstatus%>;
@@ -1032,7 +1033,7 @@ if (groupby == 7 && ins_key_arr.length == 1)
       <tr>
         <td>
 <%
-      if (groupby == 7 && !multi)
+      if (!catclose && !multi)
       {
         BLOCK_START("insFlex2", ""); //block over gehele breedte
         %><div id='insFlexList'><% // Wrapper div voor dynamisch herladen
diff --git a/APPL/INS/ins_inspect_save.asp b/APPL/INS/ins_inspect_save.asp
index 169290e1c2..cda58c2edb 100644
--- a/APPL/INS/ins_inspect_save.asp
+++ b/APPL/INS/ins_inspect_save.asp
@@ -40,10 +40,13 @@ var savemode = getFParam("savemode");
 
 var finish = getFParamInt("finish", 0) == 1; // Inspectie direct afronden.
 var groupby = getFParamInt("groupby", 7);
+var finishmjob = getFParamInt("finishmjob", 0) == 1;
+var catclose = (finishmjob && groupby < 7); // Bulk closen per categorie.
+
 var mjob = getFParamInt("mjob", 0) == 1;
 
 // Als multi dan moet de ins_deel array even groot zijn als de srtcontrole array en deelsrtcontrole array.
-user.auth_required_or_abort(groupby < 7 || !multi || (ins_key_arr.length == srtcont_key_arr.length));
+user.auth_required_or_abort(catclose || !multi || (ins_key_arr.length == srtcont_key_arr.length));
 
 var deelsrtcontrole_datum;
 var plandatum, moveyears;
@@ -95,7 +98,7 @@ var inflatiefactor = 1 + (inflatiepct / 100);
 var mjb_start_year = S("mjb_start_year");
 var mjb_freeze_year = S("mjb_freeze_year");
 
-if (groupby < 7)
+if (catclose)
 {
   var dist_key = getFParamInt("dist_key", -1); // District
   var loc_key = getFParamInt("loc_key", -1); // Locatie
diff --git a/APPL/INS/ins_inspect_start.asp b/APPL/INS/ins_inspect_start.asp
index ae96e0970f..893bb3802f 100644
--- a/APPL/INS/ins_inspect_start.asp
+++ b/APPL/INS/ins_inspect_start.asp
@@ -42,7 +42,7 @@ var mjb_start_year = S("mjb_start_year");
 var mjb_freeze_year = S("mjb_freeze_year");
 
 var ingesloten = [];
-if (groupby < 7)
+if (freeze && groupby < 7)
 {
   var dist_key = getFParamInt("dist_key", -1); // District
   var loc_key = getFParamInt("loc_key", -1); // Locatie
diff --git a/APPL/INS/ins_list.js b/APPL/INS/ins_list.js
index 2f013fe791..cfed5fbca2 100644
--- a/APPL/INS/ins_list.js
+++ b/APPL/INS/ins_list.js
@@ -262,7 +262,7 @@ function insStart(rowArray, isMulti, pfreeze)
                       // Scenario's kunnen niet gestart worden. Hier de scenario's er al uit filteren.
                       if (!rowdata.insScenKey || rowdata.insScenKey == 1)
                       { // Het is geen scenario. De periodieke taak kan gestart worden. De inspectie aan het array toevoegen.
-                        if (groupby == 7)
+                        if (!pfreeze || (pfreeze && groupby == 7))
                           srtcontrolekeyArray[i - deleted] = rowdata.insSrtControleKey;
                       }
                       else
@@ -302,7 +302,7 @@ function insStart(rowArray, isMulti, pfreeze)
                   });
 }
 
-function insClose(rowArray, isMulti, fromfinish)
+function insClose(rowArray, isMulti, fromfinish, pfinishmjob)
 {
   var insKeyString = getKeyString(rowArray);
   var srtcontrolekeyArray = new Array();
@@ -314,7 +314,7 @@ function insClose(rowArray, isMulti, fromfinish)
     // Scenario's kunnen niet gestart worden. Hier de scenario's er al uit filteren.
     if (!rowdata.insScenKey || rowdata.insScenKey == 1)
     { // Het is geen scenario. De periodieke taak kan gestart worden. De inspectie aan het array toevoegen.
-      if (groupby == 7)
+      if (!pfinishmjob || (pfinishmjob && groupby == 7))
       {
         srtcontrolekeyArray[i - deleted] = rowdata.insSrtControleKey;
         deelsrtcontrkeyArray[i - deleted] = rowdata.insDeelSrtContrKey;
@@ -340,27 +340,35 @@ function insClose(rowArray, isMulti, fromfinish)
             + (fromfinish
               ? "&fromfinish=1"
               : "")
-            + "&dist_key=" + dist_key
-            + "&loc_key=" + loc_key
-            + "&bld_key=" + bld_key
-            + "&disc_key=" + disc_key
-            + "&srtgroep_key=" + srtgroep_key
-            + "&srtdeel_key=" + srtdeel_key
-            + "&groep=" + groep
-            + "&categorie_key=" + categorie_key
-            + "&srtcontrole=" + srtcontrole
-            + "&frequentie=" + frequentie
-            + "&incbtw=" + incbtw
-            + "&groupby=" + groupby
-            + "&deel=" + deel
-            + "&actsit=" +
-            + "&mjbMoved=" + mjbMoved
-            + "&mjbFreezed=" + mjbFreezed
-            + "&mjbXcped=" + mjbXcped;
+            + (pfinishmjob
+              ? "&finishmjob=1"
+                + "&dist_key=" + dist_key
+                + "&loc_key=" + loc_key
+                + "&bld_key=" + bld_key
+                + "&disc_key=" + disc_key
+                + "&srtgroep_key=" + srtgroep_key
+                + "&srtdeel_key=" + srtdeel_key
+                + "&groep=" + groep
+                + "&categorie_key=" + categorie_key
+                + "&srtcontrole=" + srtcontrole
+                + "&frequentie=" + frequentie
+                + "&incbtw=" + incbtw
+                + "&groupby=" + groupby
+                + "&deel=" + deel
+                + "&actsit=" +
+                + "&mjbMoved=" + mjbMoved
+                + "&mjbFreezed=" + mjbFreezed
+                + "&mjbXcped=" + mjbXcped
+              : "");
   FcltMgr.openDetail(url, L("lcl_ins_stop"));
 }
 
-function insFinish(rowArray, isMulti)
+function insFinishMJOB(rowArray, isMulti)
+{
+  insFinish(rowArray, isMulti, true)
+}
+
+function insFinish(rowArray, isMulti, pfinishmjob)
 {
   FcltMgr.confirm(rowArray.length == 1 ? L("lcl_ins_finishing") : L("lcl_ins_finishing_m"),
                   { autoconfirm: isMulti },
@@ -378,7 +386,7 @@ function insFinish(rowArray, isMulti)
                       // Scenario's kunnen niet gestart worden. Hier de scenario's er al uit filteren.
                       if (!rowdata.insScenKey || rowdata.insScenKey == 1)
                       { // Het is geen scenario. De periodieke taak kan gestart worden. De inspectie aan het array toevoegen.
-                        if (groupby == 7)
+                        if (!pfinishmjob || (pfinishmjob && groupby == 7))
                           deelsrtcontrkeyArray[i - deleted] = rowdata.insDeelSrtContrKey;
                         isc_status = rowdata.ins_status;
                         mustClose = mustClose || (isc_status != 5);
@@ -394,7 +402,7 @@ function insFinish(rowArray, isMulti)
 
                     if (mustClose)
                     { // Via insClose scherm de gegevens invullen.
-                      insClose(rowArray, isMulti, true);
+                      insClose(rowArray, isMulti, true, (pfinishmjob? true: false));
                     }
                     else
                     {
diff --git a/APPL/MJB/mjb_search_list.asp b/APPL/MJB/mjb_search_list.asp
index c046eea608..2ae6e66c8e 100644
--- a/APPL/MJB/mjb_search_list.asp
+++ b/APPL/MJB/mjb_search_list.asp
@@ -792,7 +792,7 @@ var transitParam = buildTransitParam(["disc", "srtgroep", "srtdeel", "srtcontrol
     if (scenario == 1)
     {
       rst.addAction({ action: "insFreeze",  caption: L("lcl_mjb_freeze"), multi: true, multiOnce: true, enabler: "eFreeze", onlyMulti: true });
-      rst.addAction({ action: "insFinish", caption: L("lcl_ins_finish"), multi: true, multiOnce: true, enabler: "eFinishMjb", onlyMulti: true });
+      rst.addAction({ action: "insFinishMJOB", caption: L("lcl_ins_finish"), multi: true, multiOnce: true, enabler: "eFinishMjb", onlyMulti: true });
       if (fulldetails)
         rst.addAction({ action: "insAddScen", caption: L("lcl_mjb_incl_scen"), multi: true, multiOnce: true, enabler: "eAddScen", onlyMulti: true });
     }