From 7c5c1a9c21c5ee49415912241870e7827a8a3bc3 Mon Sep 17 00:00:00 2001 From: Ruud Lipper Date: Fri, 23 Sep 2016 14:48:04 +0000 Subject: [PATCH] PCHD#36752 Factuur goedkeurder verplicht kunnen maken svn path=/Website/trunk/; revision=30843 --- APPL/FAC/fac_fiattering_list.asp | 2 +- APPL/FIN/fin.inc | 2 +- APPL/FIN/fin_approve.asp | 4 ++-- APPL/FIN/fin_edit_factuur.asp | 6 +++++- APPL/FIN/fin_goedkeur.asp | 2 +- APPL/FIN/fin_list.inc | 2 +- APPL/FIN/fin_search.asp | 2 +- APPL/FIN/fin_unapprove.asp | 2 +- 8 files changed, 13 insertions(+), 9 deletions(-) diff --git a/APPL/FAC/fac_fiattering_list.asp b/APPL/FAC/fac_fiattering_list.asp index 863cc8d681..a987d0ab6d 100644 --- a/APPL/FAC/fac_fiattering_list.asp +++ b/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' diff --git a/APPL/FIN/fin.inc b/APPL/FIN/fin.inc index 217866fed2..ffe94dc11d 100644 --- a/APPL/FIN/fin.inc +++ b/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); diff --git a/APPL/FIN/fin_approve.asp b/APPL/FIN/fin_approve.asp index 541e5b7f52..e77181ab3a 100644 --- a/APPL/FIN/fin_approve.asp +++ b/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. diff --git a/APPL/FIN/fin_edit_factuur.asp b/APPL/FIN/fin_edit_factuur.asp index 245cd07605..42efa74dc8 100644 --- a/APPL/FIN/fin_edit_factuur.asp +++ b/APPL/FIN/fin_edit_factuur.asp @@ -364,13 +364,17 @@ var fin_btwtabel_key = oRs("fin_btwtabel_key").value; // Standaard btw tabel key }); <% + 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}); diff --git a/APPL/FIN/fin_goedkeur.asp b/APPL/FIN/fin_goedkeur.asp index 52fda1fc01..ead5e22de5 100644 --- a/APPL/FIN/fin_goedkeur.asp +++ b/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 = ""; diff --git a/APPL/FIN/fin_list.inc b/APPL/FIN/fin_list.inc index 94495f5dde..e4477659b9 100644 --- a/APPL/FIN/fin_list.inc +++ b/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). diff --git a/APPL/FIN/fin_search.asp b/APPL/FIN/fin_search.asp index 451adc757f..81fb81da0b 100644 --- a/APPL/FIN/fin_search.asp +++ b/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) { %> diff --git a/APPL/FIN/fin_unapprove.asp b/APPL/FIN/fin_unapprove.asp index b14d07b419..691d730ab0 100644 --- a/APPL/FIN/fin_unapprove.asp +++ b/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