admin/action.php

@@ -71,8 +71,8 @@ $tpl->register_function('afficher_debut_tarif', function ($params)
         <summary class="activite">
             <div class="activite">
                 <input type="checkbox" id="check_%1$s"
-                  onclick="cocherDecocherTarif(check_%1$s, %2$s)" />',
+                  onclick="cocherDecocherTarif(check_%1$s)" />',
-                       $idTarif, "total_general");
+                       $idTarif);
     if ($idTarif == 0) {
         // versement sur un compte non rattaché à une activité
         $out .= sprintf('
@@ -122,7 +122,7 @@ $tpl->register_function('afficher_debut_personne', function ($params)
             <summary class="personne">
                 <div class="personne">
                     <input type="checkbox" id="check_%1$s"
-                      onclick="cocherDecocherPersonne(check_%1$s, total_%1$s, %3$s)" />
+                      onclick="cocherDecocherPersonne(check_%1$s, total_%1$s)" />
                     <label for="check_%1$s">
                       %2$s : <span class="total" id="total_%1$s">0,00 €</span>
                     </label>
@@ -130,8 +130,7 @@ $tpl->register_function('afficher_debut_personne', function ($params)
             </summary>
             <div class="versements">',
                    $idVersement,
-                   $personne->nomPrenom,
+                   $personne->nomPrenom
-				   "total_general"
     );
     return $out;
 });
@@ -164,7 +163,7 @@ $tpl->register_function('afficher_versement', function ($params)
                 id="check_%1$s_%2$s"
                 name="selected[]"
                 value="%2$s"
-                onclick="cocherDecocherCase(check_%1$s_%2$s, total_%1$s, %5$s)" />
+                onclick="cocherDecocherVersement(check_%1$s_%2$s, total_%1$s)" />
             <label for="check_%1$s_%2$s"><span class="montant">%3$s</span>
                 <span>%4$s</span>
             </label>
@@ -177,8 +176,7 @@ $tpl->register_function('afficher_versement', function ($params)
                         ",",
                         "&nbsp;"
                     ),
-                    date_format(date_create($versement->date),"d/m/Y"),
+                    date_format(date_create($versement->date),"d/m/Y")
-					"total_general"
     );
     return $out;
 });

admin/script.js

@@ -1,26 +1,5 @@
 "use strict";
 
-/**
- * renvoyer la valeur numérique d'un montant formaté en €
- * @param texte qui représente nu nombre
- */
-function getNumber(texte) {
-	return Number(texte.replace(/[^0-9,]/g, '').replace(/,/, '.'));
-}
-
-/**
- * afficher un montant au format monétaire
- * @param montant à afficher
- * @param idElem : élément où faire l'affichage
- */
-function displayNumber(montant, idElem) {
-	idElem.innerHTML =
-        montant.toLocaleString('fr-FR', {
-            style: 'currency', currency: 'EUR',
-            minimumFractionDigits: 2
-        });
-}
-
 // ------------------------------------------------------------------------
 // actions sur la liste des versements
 // ------------------------------------------------------------------------
@@ -29,15 +8,14 @@ function displayNumber(montant, idElem) {
  * Fonction appelée quand on (dé)coche la case globale
  * (dé)sélectionner toutes les cases de toutes les activités
  * @param {HTMLInputElement} idCaseGlobale id de la case globale
- * @param {HTMLSpanElement} idTotalGeneral id du total général
  */
-function cocherDecocherTout(idCaseGlobale, idTotalGeneral) {
+function cocherDecocherTout(idCaseGlobale) {
     // itérer sur la liste des éléments détails : 1 par couple <activité, tarif>
     let lesDetails = document.querySelectorAll("details.activite");
     for (let i = 0; i < lesDetails.length; ++i) {
         let idCase = lesDetails[i].querySelector("input[type=checkbox]");
         idCase.checked = idCaseGlobale.checked;
-        cocherDecocherTarif(idCase, idTotalGeneral);
+        cocherDecocherTarif(idCase);
     }
     // changer le message
     changerMessage(idCaseGlobale.nextElementSibling, idCaseGlobale);
@@ -47,37 +25,35 @@ function cocherDecocherTout(idCaseGlobale, idTotalGeneral) {
  * Fonction appelée quand on (dé)coche la case d'activité
  * (dé)sélectionner toutes les cases de cette activité
  * @param {HTMLInputElement} idCaseGlobale id de la case d'activité
- * @param {HTMLSpanElement} idTotalGeneral id du total général
  */
-function cocherDecocherTarif(idCaseGlobale, idTotalGeneral) {
+function cocherDecocherTarif(idCaseGlobale) {
     let lesPersonnes = idCaseGlobale.closest("details").querySelectorAll("div.personne");
-    cocherDecocherLesPersonnes(idCaseGlobale, lesPersonnes, idTotalGeneral);
+    cocherDecocherLesPersonnes(idCaseGlobale, lesPersonnes);
 }
 
 /**
  * idem dans le cas des versements des personnes
  * @param {HTMLInputElement} idCaseGlobale id case à cocher d'une personne
  */
-function cocherDecocherToutesLesPersonnes(idCaseGlobale, idTotalGeneral) {
+function cocherDecocherToutesLesPersonnes(idCaseGlobale) {
-    let lesPersonnes = document.querySelectorAll("summary.personne");
+    let lesPersonnes = document.querySelectorAll("div.personne");
-    cocherDecocherLesPersonnes(idCaseGlobale, lesPersonnes, idTotalGeneral);
+    cocherDecocherLesPersonnes(idCaseGlobale, lesPersonnes);
     changerMessage(idCaseGlobale.nextElementSibling, idCaseGlobale);
 }
 
 /**
  * @param {HTMLInputElement} idCaseGlobale
  * @param {NodeListOf<Element>} lesPersonnes
-  * @param {HTMLSpanElement} idTotalGeneral id du total général
+ */
-*/
+function cocherDecocherLesPersonnes(idCaseGlobale, lesPersonnes) {
-function cocherDecocherLesPersonnes(idCaseGlobale, lesPersonnes, idTotalGeneral) {
     for (let j = 0; j < lesPersonnes.length; ++j) {
         // trouver l'élément total de la personne
-        let idTotal = lesPersonnes[j].querySelector("span.total");
+        let idTotal = lesPersonnes[j].querySelector("span");
         // puis la case à cocher
-        let idCase = lesPersonnes[j].querySelector("input[type=checkbox]");
+        let idCase = lesPersonnes[j].closest("summary").querySelector("input");
         idCase.checked = idCaseGlobale.checked;
         // puis traiter toutes les cases de la personne
-        cocherDecocherPersonne(idCase, idTotal, idTotalGeneral);
+        cocherDecocherPersonne(idCase, idTotal);
     }
 }
 
@@ -86,45 +62,51 @@ function cocherDecocherLesPersonnes(idCaseGlobale, lesPersonnes, idTotalGeneral)
  * - (dé)sélectionner toutes les cases à cocher
  * - faire le total des cases cochées et l'afficher
  * @param {HTMLInputElement} idCase id de la case qui a été cochée
- * @param {HTMLSpanElement} idTotal id de l'élément où afficher le total de la personne
+ * @param {HTMLSpanElement} idTotal id de l'élément où afficher le total
- * @param {HTMLSpanElement} idTotalGeneral id de l'élément où afficher le total général
  */
-function cocherDecocherPersonne(idCase, idTotal, idTotalGeneral) {
+function cocherDecocherPersonne(idCase, idTotal) {
-	let conteneur = idCase.closest("details").querySelector("div.versements");
+    // chercher le fieldset des versements
-	let listeCases = conteneur.querySelectorAll("input[type=checkbox]");
+    let fieldset = idCase.closest("details").querySelector("div.versements");
-
+    let listeCases = fieldset.querySelectorAll("input[type=checkbox]");
-	for (let i = 0; i < listeCases.length; ++i) {
+    for (let i = 0; i < listeCases.length; ++i) {
-		if (listeCases[i].checked != idCase.checked) {
+        listeCases[i].checked = idCase.checked;
-			listeCases[i].checked = idCase.checked;
+        cocherDecocherVersement(listeCases[i], idTotal);
-			cocherDecocherVersement(listeCases[i], idTotal, idTotalGeneral);
+    }
-		}
-	}
 }
 
 /**
  * Fonction appelée quand on (dé)coche la case d'un versement
- * Mettre à jour le total des cases cochées et le total global et les afficher
+ * Faire le total des cases cochées et l'afficher
  *
- * @param {HTMLInputElement} idCase id de la case qui a été (dé)cochée
+ * @param {HTMLInputElement} idCase id de la case qui a été cochée
- * @param {HTMLSpanElement} idTotal id du total de la personne
+ * @param {HTMLSpanElement} idTotal id de l'élément où afficher le total
- * @param {HTMLSpanElement} idTotalGeneral id du total général
  */
+function cocherDecocherVersement(idCase, idTotal) {
+    let fieldset = idCase.closest("div.versements");
+    let listeCases = fieldset.querySelectorAll("input[type=checkbox]");
+    let listeMontants = fieldset.querySelectorAll("span.montant");
+    calculerTotal(listeCases, listeMontants, idTotal);
+}
 
-function cocherDecocherVersement(idCase, idTotal, idTotalGeneral) {
+/**
-	let div = idCase.closest("div");
+ * Faire le total des cases cochées et l'afficher
-	let idmontant = div.querySelector("span.montant");
+ * @param {NodeListOf<Element>} listeCases liste des cases
-	let montant = getNumber(idmontant.textContent);
+ * @param {NodeListOf<Element>} listeMontants liste des montants associés
-	let totalPersonne = getNumber(idTotal.textContent);
+ * @param {HTMLSpanElement} idTotal id de l'élément où afficher le total
-	let totalGeneral = getNumber(idTotalGeneral.textContent);
+ */
-	if (idCase.checked) {
+function calculerTotal(listeCases, listeMontants, idTotal) {
-		totalPersonne += montant;
+    let total = 0;
-		totalGeneral += montant;
+    for (let i = 0; i < listeCases.length; ++i) {
-	} else {
+        if (listeCases[i].checked) {
-		totalPersonne -= montant;
+            total += parseFloat(listeMontants[i].textContent.replace(/\s/g, "").replace(",", "."));
-		totalGeneral -= montant;
+        }
-	}
+    }
-	displayNumber(totalPersonne, idTotal);
+    // afficher le total
-	displayNumber(totalGeneral, idTotalGeneral);
+    idTotal.innerHTML =
+        total.toLocaleString('fr-FR', {
+            style: 'currency', currency: 'EUR',
+            minimumFractionDigits: 2
+        });
 }
 
 /**
@@ -291,4 +273,4 @@ function changerStyle(document) {
         if (sheet.href.includes('imprimer_recu.css')) { sheet.media = 'print'; }
     }
     // console.log(styles);
-}
+}

admin/style.css

@@ -2,10 +2,6 @@
  *  liste des versements
  */
 
-label.strong {
-	font-weight : bold;
-}
-
 div.pair {
     background-color: rgba(var(--gSecondColor), 0.15);
 }

admin/upload.php

@@ -15,8 +15,8 @@ if (!File::checkCreateAccess($parent, $session)) {
 
 $csrf_key = 'upload_file_' . md5($parent);
 
-$form->runIf('upload', function () use ($parent, $session) {
+$form->runIf('upload', function () use ($parent) {
-	$_SESSION['sig_file'] = \Paheko\Files\Files::uploadMultiple($parent, 'file', $session);
+	$_SESSION['sig_file'] = \Paheko\Files\Files::uploadMultiple($parent, 'file');
 }, $csrf_key, PLUGIN_ROOT . '/admin/config.php');
 
 $tpl->assign(compact('parent', 'csrf_key'));

templates/versements_activites.tpl

@@ -4,24 +4,15 @@
 <h2>Année {$annee_recu} : versements par activité et tarif</h2>
 
 <fieldset class="noprint">
-	<dl>
+	<input type="checkbox" class="check_global" id="check_global" onclick="cocherDecocherTout(check_global)" />
-		<dd>
-			<label class="strong">Total des cases cochées : </label>
-			<span class="total" id="total_general">0,00 €</span>
-		</dd>
-		<dd>
-	<input type="checkbox" class="check_global" id="check_global" onclick="cocherDecocherTout(check_global, total_general)" />
 	<label for="check_global">Cliquer pour cocher toutes les lignes</label>
-		</dd>
-		<dd>
 	<button type="button" data-icon="↑" class="icn-btn" id="close_details_activite"
 		onclick="montrerMasquerDetails(this.id, 'details.activite', 'toutes les activités')">
 		Replier toutes les activités</button>
 	<button type="button" data-icon="↑" class="icn-btn" id="close_details_personne"
 		onclick="montrerMasquerDetails(this.id, 'details.personne', 'toutes les personnes')">
 		Replier toutes les personnes</button>
-		</dd>
+		<br />
-		<dd>
 	{button type="submit" label="Télécharger les reçus au format PDF" shape="download"
 			form="versements_activites"
 			formaction="generer_recus.php?type=activite&format=pdf"
@@ -30,8 +21,6 @@
 			form="versements_activites"
 			formaction="generer_recus.php?type=activite&format=print"
 			onclick="return verifierChoix(this.form)"}
-		</dd>
-	</dl>
 </fieldset>
 
 <form method="post" target="_blank" id="versements_activites">

templates/versements_personnes.tpl

@@ -4,33 +4,21 @@
 <h2>Année {$annee_recu} : versements par personne</h2>
 
 <fieldset class="noprint">
-	<dl>
+	<input type="checkbox" class="check_global" id="check_global"
-
+		onclick="cocherDecocherToutesLesPersonnes(check_global)" />
-		<dd>
+	<label for="check_global">Cliquer pour cocher toutes les lignes</label>
-			<label class="strong">Total des cases cochées : </label>
+	<button type="button" data-icon="↑" class="icn-btn" id="close_details_personne"
-			<span class="total" id="total_general">0,00 €</span>
+		onclick="montrerMasquerDetails(this.id, 'details.personne', 'toutes les personnes')">
-		</dd>
+		Replier toutes les personnes</button>
-		<dd>
+		<br />
-			<input type="checkbox" class="check_global" id="check_global"
+	{button type="submit" label="Télécharger les reçus au format PDF" shape="download"
-				   onclick="cocherDecocherToutesLesPersonnes(check_global, total_general)" />
+			form="versements_personnes"
-			<label for="check_global">Cliquer pour cocher toutes les lignes</label>
+			formaction="generer_recus.php?type=personne&format=pdf"
-		</dd>
+			onclick="return verifierChoix(this.form)"}
-		<dd>
+	{button type="submit" label="Imprimer les reçus" shape="print"
-			<button type="button" data-icon="↑" class="icn-btn" id="close_details_personne"
+			form="versements_personnes"
-				onclick="montrerMasquerDetails(this.id, 'details.personne', 'toutes les personnes')">
+			formaction="generer_recus.php?type=personne&format=print"
-				Replier toutes les personnes</button>
+			onclick="return verifierChoix(this.form)"}
-		</dd>
-		<dd>
-			{button type="submit" label="Télécharger les reçus au format PDF" shape="download"
-				form="versements_personnes"
-				formaction="generer_recus.php?type=personne&format=pdf"
-				onclick="return verifierChoix(this.form)"}
-			{button type="submit" label="Imprimer les reçus" shape="print"
-				form="versements_personnes"
-				formaction="generer_recus.php?type=personne&format=print"
-				onclick="return verifierChoix(this.form)"}
-		</dd>
-	</dl>
 </fieldset>
 
 <form method="post" target="_dialog" id="versements_personnes">