From 38a3556dbd058a92fdf1d0d566f0d8fbd0d989b5 Mon Sep 17 00:00:00 2001
From: engel <>
Date: Fri, 18 Feb 2022 09:37:39 +0000
Subject: [PATCH] =?UTF-8?q?v=C3=A9rification=20s=C3=A9lections=20cases=20e?=
 =?UTF-8?q?t=20radios?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

FossilOrigin-Name: ba0d1089be4acbc5c7cfba5ab180a19e6a66bb1b466e8ed238337c239d326a49
---
 lib/Personne.php    | 12 +++++--
 templates/index.tpl | 12 +++----
 www/admin/script.js | 79 +++++++++++++++++++++++++++++++++------------
 3 files changed, 75 insertions(+), 28 deletions(-)

diff --git a/lib/Personne.php b/lib/Personne.php
index 2497586..02c73ec 100644
--- a/lib/Personne.php
+++ b/lib/Personne.php
@@ -32,6 +32,10 @@ class Personne
         $this->versements = array();
     }
 
+    /**
+     * return copie d'une personne
+     * @param $p
+     */
     public static function copier($p)
     {
         return new Personne(
@@ -44,8 +48,12 @@ class Personne
     }
 
 
-    /*
-     * ajouter un versement pour une activité et un tarif donnés
+    /**
+     * ajouter un versement
+     * @param $idActivite
+     * @param $idTarif
+     * @param $montant
+     * @param $tauxReduction
      */
     public function ajouterVersement(
         $idActivite,
diff --git a/templates/index.tpl b/templates/index.tpl
index c7df99a..92a8be0 100644
--- a/templates/index.tpl
+++ b/templates/index.tpl
@@ -2,7 +2,7 @@
 {include file="%s/templates/_nav.tpl"|args:$plugin_root current_nav="index"}
 
 <h2>Choisir l'année fiscale</h2>
-<form id="formulaire_saisie" method="post" action="">
+<form id="formulaire_saisie" method="post" action="action.php">
     <fieldset>
         {* <legend>Choisir l'année fiscale</legend> *}
         <select id="annee_recu" name="annee_recu">
@@ -48,7 +48,7 @@
         </fieldset>
     </div>
 
-    <div id="taux_reduc" class="tous hidden">
+    <div id="div_taux_reduc" class="tous hidden">
         <h2>Choisir le taux de réduction</h2>
         <fieldset>
             {foreach from=$plugin_config->reduction item="reduc"}
@@ -64,7 +64,7 @@
     <div id="generer_tous" class="tous hidden">
         <p class=" submit">
             {csrf_field key="generer_tous_recus"}
-            {button type="submit" name="generer_tous" label="Poursuivre" shape="right" class="main" onclick="aiguiller(this.form, 'versements_personnes');" }
+            {button type="submit" name="generer_tous" label="Poursuivre" shape="right" class="main" onclick="return verifierRadio('div_taux_reduc');" }
         </p>
     </div>
 
@@ -105,14 +105,14 @@
                 {/foreach}
             </tbody>
         </table>
-        <p class="help">Si vous ne savez pas quel taux de réduction utiliser, n'en choisissez aucun</p>
+        {*<p class="help">Si vous ne savez pas quel taux de réduction utiliser, n'en choisissez aucun</p>*}
         </fieldset>
     </div>
 
     <div id="generer_activites" class="activites hidden">
         <p class=" submit">
             {csrf_field key="generer_recus_activites"}
-            {button type="submit" name="generer_activites" label="Poursuivre" shape="right" class="main" onclick="aiguiller(this.form, 'versements_activites');" }
+            {button type="submit" name="generer_activites" label="Poursuivre" shape="right" class="main" onclick="return verifierCases('liste_activites_tarifs');" }
         </p>
     </div>
 </form>
@@ -120,7 +120,7 @@
 <script type="text/javascript" src="script.js" defer="defer"></script>
 {literal}
 <script type="text/javascript">
-        // activer/désactiver les radios
+        // activer/désactiver les radios des activités/tarifs
         for (var laCase of document.querySelectorAll("input[type=checkbox]")) {
             laCase.addEventListener('change', (evt) => {
                 var idCase = evt.target;
diff --git a/www/admin/script.js b/www/admin/script.js
index db6ea6d..4c18d31 100644
--- a/www/admin/script.js
+++ b/www/admin/script.js
@@ -3,7 +3,8 @@
  * (dé)sélectionner toutes les cases à cocher de toutes les activités
  * @param id de la case globale
  */
-function cocherDecocherTout(idCaseGlobale) {
+function cocherDecocherTout(idCaseGlobale)
+{
     // chercher le formulaire englobant
     var formulaire = idCaseGlobale.closest("form");
     // itérer sur la liste des éléments détails : 1 par couple <activité, tarif>
@@ -39,7 +40,8 @@ function cocherDecocherTout(idCaseGlobale) {
  * @param id de la case qui a été cochée
  * @param id de l'élément où afficher le total
  */
-function cocherDecocherPersonne(idCase, idTotal) {
+function cocherDecocherPersonne(idCase, idTotal)
+{
     // chercher le fieldset englobant
     var fieldset = idCase.closest("fieldset");
     var listeCases = fieldset.querySelectorAll("input[type=checkbox]");
@@ -67,7 +69,8 @@ function cocherDecocherPersonne(idCase, idTotal) {
  * @param id de la case qui a été cochée
  * @param id de l'élément où afficher le total
  */
-function cocherDecocherVersement(idCase, idTotal) {
+function cocherDecocherVersement(idCase, idTotal)
+{
     var fieldset = idCase.closest("fieldset");
     var listeCases = fieldset.querySelectorAll("input[type=checkbox]");
     var listeMontants = fieldset.querySelectorAll("span.montant");
@@ -80,7 +83,8 @@ function cocherDecocherVersement(idCase, idTotal) {
  * @param listes des montants associés
  * @param id de l'élément où afficher le total
 */
-function calculerTotal(listeCases, listeMontants, idTotal) {
+function calculerTotal(listeCases, listeMontants, idTotal)
+{
     var total = 0;
     for (var i = 1; i < listeCases.length; ++i)
     {
@@ -118,7 +122,11 @@ function verifierChoix(formulaire)
     return ok;
 }
 
-function afficherMasquer(formulaire, nomClasse1, nomClasse2) {
+/**
+ * fonction appelée pour afficher et masquer des portions de formulaire
+*/
+function afficherMasquer(formulaire, nomClasse1, nomClasse2)
+{
     for (var elem of formulaire.querySelectorAll(nomClasse1)) {
         elem.classList.remove('hidden');
     }
@@ -127,13 +135,53 @@ function afficherMasquer(formulaire, nomClasse1, nomClasse2) {
     }
 }
 
-// aiguiller la suite vers la page passée en paramètre
-function aiguiller(formulaire, pageSuivante) {
-    formulaire.action = pageSuivante + ".php";
+// vérifier 
+// - qu'au moins une activité/tarif est sélectionnée
+// - qu'un radio de chaque activité/tarif sélectionné a été sélectionné :)
+function verifierCases(idElem)
+{
+    var div = document.getElementById(idElem);
+    var nbChoix = 0;
+    // parcourir les cases à cocher
+    for (var idCase of div.querySelectorAll("input[type=checkbox]"))
+    {
+        if (idCase.checked) {
+            ++nbChoix;
+            // vérifier qu'un radio de la même ligne est sélectionné
+            var ligneCorrecte = false;
+            // trouver la ligne englobante
+            var ligne = idCase.closest("tr");
+            for (var idRadio of ligne.querySelectorAll('input[type=radio]'))
+            {
+                if (idRadio.checked) { ligneCorrecte = true; break; }
+            }
+            if (! ligneCorrecte) {
+                alert("Erreur : il faut sélectionner un taux de réduction dans chaque ligne cochée");
+                return false;
+            }
+        }
+    }
+    if (nbChoix == 0) {
+        alert("Erreur : il faut sélectionner au moins une activité/tarif");
+    }
+    return nbChoix != 0;
+}
+
+// vérifier qu'un radio a été sélectionné dans la div paramètre
+function verifierRadio(idElem)
+{
+    var div = document.getElementById(idElem);
+    for (var idRadio of div.querySelectorAll('input[type=radio]'))
+    {
+        if (idRadio.checked) { return true; }
+    }
+    alert("Erreur : il faut sélectionner un taux de réduction");
+    return false;    
 }
 
 // inutilisé
-function activerDesactiverRadio(evt) {
+function activerDesactiverRadio(evt)
+{
     var idCase = evt.target;
     // checher la ligne englobante ( <tr>)
     var ligne = idCase.closest("tr");
@@ -148,21 +196,12 @@ function activerDesactiverRadio(evt) {
     }
 }
 
-/**
- * indiquer le nom du script php à activer
- * @param formulaire
- * @param script php
- */
-function activer(formulaire, script) {
-    formulaire.action = script;
-}
-
-
 /**
  * Associer un écouteur à la première case à cocher de chaque table
  * @remarks : n'est plus utile
  */
-function activerListener() {
+function activerListener()
+{
     // parcourir les tables
     const lesTables = document.querySelectorAll("table.list");
     for (let i = 0; i < lesTables.length; ++i) {