PCHD#36752 Factuur goedkeurder verplicht kunnen maken

svn path=/Website/trunk/; revision=30843

APPL/FAC/fac_fiattering_list.asp

@@ -47,7 +47,7 @@
                                                                     ik heb write rechten op de locatie van het afleveradres (in collegakeylist) OR
                                                                     ik heb write rechten op de plaats van de bijbehorende melding (in collegakeylist) OR
                                                                     ik ben contracteigenaar (in collegakeylist))
-                     Goedkeuren) fin_factuur_statuses_key == 5 AND prs_perslid_key_goedkeur in Collegakeylist (AND S("fin_enable_goedkeuren") == 1 geldt al omdat anders de status niet "Ter goedkeuring(5)" kan zijn)
+                     Goedkeuren) fin_factuur_statuses_key == 5 AND prs_perslid_key_goedkeur in Collegakeylist (AND S("fin_enable_goedkeuren") != 0 geldt al omdat anders de status niet "Ter goedkeuring(5)" kan zijn)
             Ik heb gefiatteerd of afgewezen als er tracking is geweest
                 BES) gefiateerd:  'BESFIT' (kostenplaatsverantwoordelijke) of 'BESACP'" (kostenplaatsgroepverantwoordelijke)
                      afgewezen:   'BESREJ'

APPL/FIN/fin.inc

@@ -126,7 +126,7 @@ fin = { func_enabled_factuur: // Wat mag ik zoal op deze specifieke factuur?
                          fresult.canDateChange     = fresult.canFEwrite || fresult.canWrite("WEB_FINFOF");
                          fresult.canRemarkChange   = fresult.canFEwrite || fresult.canWrite("WEB_FINFOF");
                          fresult.canInvLinChange   = fresult.canFEwrite || fresult.canWrite("WEB_FINFOF");
-                         fresult.canGoedkeur       = fresult.canWrite("WEB_FINGOE") && user.isCollega(goedkeurder) && S("fin_enable_goedkeuren") == 1;
+                         fresult.canGoedkeur       = fresult.canWrite("WEB_FINGOE") && user.isCollega(goedkeurder) && S("fin_enable_goedkeuren") != 0;
                          break;
                 case 6:  // Akkoord
                          fresult.canUnapprove      = (fresult.canWrite("WEB_FINFOF") || fresult.canWrite("WEB_FINBOF") || user.isCollega(kpverantwoordelijke)) && (kpverantwoordelijke > 0);

APPL/FIN/fin_approve.asp

@@ -76,7 +76,7 @@ if (tobeapproved > 0)
 { // Fiatteren of afkeuren.
   if (accept)
   { // Fiatteren.
-    var fin_status_key = S("fin_enable_goedkeuren") == 1
+    var fin_status_key = S("fin_enable_goedkeuren") != 0
                      ? "CASE WHEN prs_perslid_key_goedkeur IS NOT NULL THEN 5 ELSE 6 END"
                      : "6";
     sql = "UPDATE fin_factuur"
@@ -85,7 +85,7 @@ if (tobeapproved > 0)
         + " AND (mld_opdr_key IS NOT NULL OR cnt_contract_key IS NOT NULL OR bes_bestelopdr_key IS NOT NULL)";
     for (f = 0 ; f < ingesloten.length; f++)
     {
-      if (S("fin_enable_goedkeuren") == 1 && ingesloten[f].prs_key_goedkeur > 0)
+      if (S("fin_enable_goedkeuren") != 0 && ingesloten[f].prs_key_goedkeur > 0)
       { // Factuur is naar status Ter goedkeuring(5) gegaan.
         shared.trackaction("FINGOE", ingesloten[f].factuur_key);
         // Notificatie aan de goedkeurder.

APPL/FIN/fin_edit_factuur.asp

@@ -364,13 +364,17 @@ var fin_btwtabel_key = oRs("fin_btwtabel_key").value; // Standaard btw tabel key
                                  });
         </script>
 <%
+        if (S("fin_enable_goedkeuren") != 0)
+        {
         FCLTpersoonselector("goedkeurder",
                             "sgGoedkeurder",
                             { perslidKey: fin_factuur.perslid_key_goedkeur,
                               label: L("lcl_fin_goedkeurder"),
                               readonly: !this_fin.canChange,
-                              filtercode: "FING" // Goedkeurder.
+                              filtercode: "FING", // Goedkeurder.
+                              required: S("fin_enable_goedkeuren") == 2
                             });
+        }                            
         ROFIELDTR("fld", L("lcl_shared_created_by"), fin_factuur.fin_perslid_user); // Aangemaakt door
         ROFIELDTR("fld", L("lcl_fin_invoice") + " " + L("lcl_fin_fin_status"), fin_factuur.fin_status_oms); // Factuur status
         ROFIELDTR("fld", L("lcl_fin_invoice_source"), fin_factuur.fin_bron_oms, {suppressEmpty: true});

APPL/FIN/fin_goedkeur.asp

@@ -28,7 +28,7 @@ var reject = getQParamInt("r", -1) == 1;
 if (accept && reject) eval("INTERNAL ERROR or ABUSE");
 
 // Accepteren en afkeuren moet wel mogelijk zijn.
-user.anything_todo_or_abort((S("fin_enable_goedkeuren") == 1 && accept) || (S("fin_enable_afwijzen") == 1 && reject));
+user.anything_todo_or_abort((S("fin_enable_goedkeuren") != 0 && accept) || (S("fin_enable_afwijzen") == 1 && reject));
 
 /***** End get webform parameters *****/
 var message = "";

APPL/FIN/fin_list.inc

@@ -713,7 +713,7 @@ function fin_list (params)
                          !deleted;
 
         var eGoedkeur =!embedded &&
-                        S("fin_enable_goedkeuren") == 1 &&
+                        S("fin_enable_goedkeuren") != 0 &&
                         authparamsFINGOE &&
                         authparamsFINGOE.PRSwritelevel < 9 &&
                         user.isCollega(oRs("prs_perslid_key_goedkeur").Value) && // Goedkeurder IN (user_key, vervangers).

APPL/FIN/fin_search.asp

@@ -211,7 +211,7 @@ var isContactpersoon = user.isContactpersoon();
                                     emptyOption: "",
                                     whenEmpty: L("lcl_search_generic")
                                   });
-              if (S("fin_enable_goedkeuren") == 1)
+              if (S("fin_enable_goedkeuren") != 0)
               {
 %>              <!-- Goedkeurder -->
                 <tr class="primsearch onlyxd hidden">

APPL/FIN/fin_unapprove.asp

@@ -49,7 +49,7 @@ while (!oRs.eof)
 {
   var this_fin = fin.func_enabled_factuur(oRs("fin_factuur_key").Value);
   if ((this_fin.canFOwrite || this_fin.canBOwrite || user.isCollega(oRs("kpverantwoordelijke").Value)) ||
-      (this_fin.canGOEwrite && user.isCollega(oRs("prs_perslid_key_goedkeur").Value) && S("fin_enable_goedkeuren") == 1))
+      (this_fin.canGOEwrite && user.isCollega(oRs("prs_perslid_key_goedkeur").Value) && S("fin_enable_goedkeuren") != 0))
   {
     ingesloten.push ({ factuur_key: oRs("fin_factuur_key").Value,
                        fin_status: oRs("fin_factuur_statuses_key").Value