From fb4bbaf6977f7f5d46075175b69453bf703e4636 Mon Sep 17 00:00:00 2001 From: Jean-Christophe Engel Date: Fri, 30 Oct 2026 10:27:20 +0100 Subject: [PATCH] =?UTF-8?q?S=C3=A9curisation=20donn=C3=A9es=20POST?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- add_asset.html | 11 +++++++++++ add_infos.html | 6 ++++++ attach_credit.html | 2 +- balance_sheet_exit.html | 11 +++++++---- modify_infos.html | 12 ++++++++++++ module.ini | 2 +- save_amort.html | 3 +++ transfer.html | 6 ++++++ write_exit.html | 2 ++ 9 files changed, 49 insertions(+), 6 deletions(-) diff --git a/add_asset.html b/add_asset.html index 727d50d..a704dea 100644 --- a/add_asset.html +++ b/add_asset.html @@ -20,6 +20,12 @@ {{* Traiter l'envoi du formulaire *}} {{#form on="add"}} {{* vérifier que la date d'achat est située dans un l'exercice sélectionné *}} + {{if $_POST.date_achat|parse_date === false}} + {{:error message="Erreur : la date d'achat (%s) est invalide"|args:$_POST.date_achat}} + {{/if}} + {{if $_POST.date_mes|parse_date === false}} + {{:error message="Erreur : la date de mise en service (%s) est invalide"|args:$_POST.date_mes}} + {{/if}} {{:assign ts_date = $_POST.date_achat|parse_date|strtotime}} {{:assign ok=false}} {{#foreach from=$years}} @@ -93,7 +99,12 @@ {{:assign da=$_POST.date_achat|date_short}} {{:error message="Erreur : la date de mise en service (%s) ne peut être antérieure à la date d'acquisition (%s)"|args:$dd:$da}} {{/if}} + {{:assign duration=$_POST.duree|intval}} + {{if $duration <= 0}} + {{:error message="Erreur : la durée d'amortissement doit être strictement positive"}} + {{/if}} + {{:assign status="managed"}} {{if $_POST.date_mes == null || $_POST.date_mes == $_POST.date_achat}} {{:assign date_mes=null}} diff --git a/add_infos.html b/add_infos.html index d2a4874..cac3115 100644 --- a/add_infos.html +++ b/add_infos.html @@ -73,6 +73,12 @@ {{:redirect force="exit_step1.html?immo_doc_id=%s&from=exit"|args:$immo_doc_id}} {{else}} {{* immobilisation amortissable *}} + {{if $_POST.date_achat|parse_date === false}} + {{:error message="Erreur : la date d'achat (%s) est invalide"|args:$_POST.date_achat}} + {{/if}} + {{if $_POST.date_mes|parse_date === false}} + {{:error message="Erreur : la date de mise en service (%s) est invalide"|args:$_POST.date_mes}} + {{/if}} {{if $_POST.date_achat|parse_date > $ligne_immo.date_achat}} {{:assign da=$ligne_immo.date_achat|date_short}} {{:error message="Erreur : la date d'achat (%s) ne peut être postérieure à la date de l'écriture d'immobilisation (%s)"|args:$_POST.date_achat:$da}} diff --git a/attach_credit.html b/attach_credit.html index b1adb99..26ba567 100644 --- a/attach_credit.html +++ b/attach_credit.html @@ -61,7 +61,7 @@ {{* Traiter l'envoi du formulaire *}} {{#form on="save"}} - {{if $_POST.montant == null}} + {{if $_POST.montant == null || $_POST.montant == 0}} {{if $montant_affecte == 0}} {{:assign saved_credit=null}} {{:assign montant_credit=$credit_line.amount}} diff --git a/balance_sheet_exit.html b/balance_sheet_exit.html index 4043c5d..b6b4d5a 100644 --- a/balance_sheet_exit.html +++ b/balance_sheet_exit.html @@ -35,9 +35,6 @@ {{:assign amort_number=$linked_amort|count}} {{#foreach from=$linked_amort|ksort item="line"}} - {{if $first_amort_date == null}} - {{:assign first_amort_date=$line.date}} - {{/if}} {{if $last_amort_date == null}} {{:assign last_amort_date=$line.date}} {{elseif $line.date > $last_amort_date}} @@ -82,11 +79,15 @@ {{* Traiter l'envoi du formulaire *}} {{#form on="proceed"}} + {{* vérifier la validité de la date de sortie *}} {{if $_POST.date_sortie == null}} {{:error message="Vous devez renseigner la date de sortie"}} {{/if}} - {{* vérifier la validité de la date de sortie *}} + {{if $_POST.date_sortie|parse_date === false}} + {{:error message="Erreur : la date de sortie (%s) est invalide"|args:$_POST.date_sortie}} + {{/if}} + {{if $info_immo.date_mes != null}} {{if $_POST.date_sortie|parse_date < $info_immo.date_mes}} {{:assign immo_date=$info_immo.date_mes|date_short}} @@ -96,6 +97,7 @@ {{:assign immo_date=$ligne_immo.date_achat|date_short}} {{:error message="La date de sortie doit être postérieure à la date d'acquisition de l'immobilisation (%s)"|args:$immo_date}} {{/if}} + {{if $amort_number != 0 && $_POST.date_sortie|parse_date <= $last_amort_date}} {{:assign last_amort_date=$last_amort_date|date_short}} {{:error message="La date de sortie doit être postérieure à la date du dernier amortissement (%s)"|args:$last_amort_date}} @@ -103,6 +105,7 @@ {{:assign ts_exit = $_POST.date_sortie|parse_date|strtotime}} {{:assign exercice_ok=false}} + {{* vérifier que la date de sortie est dans l'exercice choisi *}} {{#foreach from=$years item="current_year"}} {{if $current_year.id == $_POST.id_year}} diff --git a/modify_infos.html b/modify_infos.html index 2e056e5..4830d7c 100644 --- a/modify_infos.html +++ b/modify_infos.html @@ -34,6 +34,18 @@ {{* traiter la saisie *}} {{#form on="save"}} + {{if $_POST.date_achat == null || $_POST.date_achat|parse_date == $ligne_immo.date_achat}} + {{:assign date_achat=null}} + {{else}} + {{:assign date_achat=$_POST.date_achat|parse_date}} + {{/if}} + + {{if $_POST.date_mes == null || $_POST.date_mes == $_POST.date_achat}} + {{:assign date_mes=null}} + {{else}} + {{:assign date_mes=$_POST.date_mes|parse_date}} + {{/if}} + {{if $_POST.date_achat|parse_date > $ligne_immo.date_achat}} {{:assign da=$ligne_immo.date_achat|date_short}} {{:error message="Erreur : la date d'achat (%s) ne peut être postérieure à la date de l'écriture d'immobilisation (%s)"|args:$_POST.date_achat:$da}} diff --git a/module.ini b/module.ini index 07ac9b4..515fdfd 100644 --- a/module.ini +++ b/module.ini @@ -1,5 +1,5 @@ name="Amortissements" -description="Immobilisations et amortissements\nversion 0.42.1" +description="Immobilisations et amortissements\nversion 0.42.2" author="Jean-Christophe Engel" author_url="https://gitea.zaclys.com/lesanges" home_button=false diff --git a/save_amort.html b/save_amort.html index 0226760..3e871a7 100644 --- a/save_amort.html +++ b/save_amort.html @@ -93,6 +93,9 @@ {{* Traiter l'envoi du formulaire *}} {{#form on="save"}} {{* vérifier que la date est située dans un exercice compatible *}} + {{if $_POST.date_amort|parse_date === false}} + {{:error message="Erreur : la date d'amortissement (%s) est invalide"|args:$_POST.date_amort}} + {{/if}} {{:assign ts_date = $_POST.date_amort|parse_date|strtotime}} {{:assign ok=false}} {{#foreach from=$years}} diff --git a/transfer.html b/transfer.html index 3017036..151d4e7 100644 --- a/transfer.html +++ b/transfer.html @@ -69,6 +69,9 @@ {{* Traiter l'envoi du formulaire *}} {{#form on="transfer"}} {{* vérifier que la date de mise en service est postérieure aux dates d'acquisition *}} + {{if $_POST.date_mes|parse_date === false}} + {{:error message="Erreur : la date de mise en service (%s) est invalide"|args:$_POST.date_mes}} + {{/if}} {{if $_POST.date_mes|parse_date < $max_date}} {{:assign dacq=$max_date|date_short}} {{:error message="La date de mise en service (%s) doit être postérieure à la date d'acquisition d'immobilisation la plus récente (%s) !"|args:$_POST.date_mes:$dacq}} @@ -96,6 +99,9 @@ {{:assign status="ignored"}} {{else}} {{:assign duration=$_POST.duree|intval}} + {{if $duration <= 0}} + {{:error message="Erreur : la durée d'amortissement doit être strictement positive"}} + {{/if}} {{:assign status="managed"}} {{/if}} diff --git a/write_exit.html b/write_exit.html index e9033d0..a62fb9c 100644 --- a/write_exit.html +++ b/write_exit.html @@ -320,6 +320,8 @@ :year_id = $_GET.year }} {{:assign chart_id=$id}} +{{else}} + {{:error message="Pas de plan comptable pour l'exercice %s"|args:$_GET.year}} {{/select}} {{#select