Amélioration navigation ; correction erreur date

FossilOrigin-Name: 21a10381cd0e6acfae85483ee95ca78652bf5671d731f7fb289e225626c26fdb

templates/recu_html.tpl

@@ -1,5 +1,8 @@
+<?php
+  $nav = $_GET['type'];
+?>
 <!-- nav bar -->
-{include file="%s/templates/_nav.tpl"|args:$plugin_root current_nav="activite"}
+{include file="%s/templates/_nav.tpl"|args:$plugin_root current_nav="%s"|args:$nav}
 
 <?php
     $fmt = new \NumberFormatter('fr_FR', \NumberFormatter::SPELLOUT);
@@ -74,11 +77,23 @@
             ({$libelle})
             {/if}
             <br /><span id="date_versements">date des versements&nbsp;:
+            <?php
+              $dmin = date("d/m/Y", $versement->dateMin);
+              $dmax = date("d/m/Y", $versement->dateMax);
+            ?>
+              {if $versement.dateMin == $versement.dateMax}
+              le {$dmin}
+              {else}
+              du {$dmin} au {$dmax}
+              {/if}
+{*
+  Erreur : dates décalées d'un jour (en arrière)
               {if $versement.dateMin == $versement.dateMax}
               le {$versement.dateMin|date_format:"%d/%m/%Y"}
               {else}
               du {$versement.dateMin|date_format:"%d/%m/%Y"} au {$versement.dateMax|date_format:"%d/%m/%Y"}
               {/if}
+*}
               </span>
           </li>
           {/foreach}

www/admin/versements_activites.php

@@ -9,39 +9,56 @@ use Garradin\Plugin\RecusFiscaux\Utils;
 // récupérer les infos du formulaire
 // ------------------------------------------------------------
 
-// vérifier qu'on a bien sélectionné une activité ou un compe
+// vérifier qu'on a bien sélectionné une activité ou un compte
-if (null === f('tarifs') && null === f('comptes'))
+if (! isset($_SESSION['tauxSelectionnes'])
+    &&
+    null === f('tarifs')
+    &&
+    null === f('comptes'))
 {
     \Garradin\Utils::redirect(PLUGIN_URL . 'index.php');
 }
 
 // tarifs sélectionnés
-$tarifsSelectionnes = f('tarifs') ?: [];
+if (null !== f('tarifs')) {
+    $tarifsSelectionnes = f('tarifs');
+} else if (! isset($_SESSION['tauxSelectionnes'])) {
+    $tarifsSelectionnes = [];
+}
 
 // comptes sélectionnés
-$comptesSelectionnes = f('comptes') ?: [];
+if (null !== f('comptes')) {
+    $_SESSION['comptesSelectionnes'] = f('comptes');
+} /*
+else if (! isset($_SESSION['tauxSelectionnes'])) {
+    $_SESSION['comptesSelectionnes'] = [];
+}
+  */
 
 // taux de réduction associés
+if (isset($tarifsSelectionnes) && isset($_SESSION['comptesSelectionnes']))
+{
     $tauxSelectionnes = array();
     foreach ($tarifsSelectionnes as $idTarif)
     {
         $nomRadio = "taux_reduction_" . $idTarif;
-    $valRadio = f("$nomRadio");
+        $tauxSelectionnes[$idTarif] = f("$nomRadio");
-    $tauxSelectionnes[$idTarif] = $valRadio;
     }
-foreach ($comptesSelectionnes as $idCompte)
+    foreach ($_SESSION['comptesSelectionnes'] as $idCompte)
     {
         $nomRadio = "taux_reduction_" . $idCompte;
-    $valRadio = f("$nomRadio");
+        $tauxSelectionnes[$idCompte] = f("$nomRadio");
-    $tauxSelectionnes[$idCompte] = $valRadio;
     }
     $_SESSION['tauxSelectionnes'] = $tauxSelectionnes;
+}
+
+$lesTarifs = array_map(fn($elem) : string =>
+                       strpos($elem, '_') !== false ? substr($elem, 0, strpos($elem, '_')) : "",
+                       array_keys($_SESSION['tauxSelectionnes']));
+$lesComptes = array_map(fn($elem) : string =>
+                        strpos($elem, '_') !== false ? substr($elem, 1 + strpos($elem, '_')) : "",
+                        array_keys($_SESSION['tauxSelectionnes']));
 
-// versements correspondants à la sélection, triés par tarif, nom, compte, date
-$lesTarifs = array_map(fn($elem) : string => substr($elem, 0, strpos($elem, '_')),
-                       $tarifsSelectionnes);
-$lesComptes = array_map(fn($elem) : string => substr($elem, 1 + strpos($elem, '_')),
-                       $tarifsSelectionnes);
 $_SESSION['lesVersements'] =
                            Utils::getVersementsTarifsComptes(
                                $_SESSION['annee_recu'],
@@ -51,15 +68,15 @@ $_SESSION['lesVersements'] =
 
 // ajouter les versements sans tarif (tri par nom, compte, date)
 $versementsSansTarif = Utils::getVersementsComptes($_SESSION['annee_recu'],
-                                                   $comptesSelectionnes,
+                                                   $_SESSION['comptesSelectionnes'],
                                                    $champsNom);
 foreach ($versementsSansTarif as $versement)
 {
     $_SESSION['lesVersements'][] = $versement;
 }
-
 // préparation de l'affichage
 $tpl->assign('lesVersements', $_SESSION['lesVersements']);
+$tpl->assign('annee_recu', $_SESSION['annee_recu']);
 $tpl->assign('plugin_css', ['style.css']);
 
 // envoyer au template

www/admin/versements_personnes.php

@@ -6,11 +6,16 @@ use Garradin\Plugin\RecusFiscaux\Personne;
 use Garradin\Plugin\RecusFiscaux\Utils;
 
 // vérifier si le taux de réduction a été sélectionné au préalable
-$_SESSION['taux_reduction'] = f('taux_reduction');
+$taux = f('taux_reduction');
-if (! isset($_SESSION['taux_reduction']) || $_SESSION['taux_reduction'] == "")
+if (! isset($_SESSION['taux_reduction'])
+    &&
+    null === $taux)
 {
     \Garradin\Utils::redirect(PLUGIN_URL . 'index.php');
 }
+if (null !== $taux) {
+    $_SESSION['taux_reduction'] = $taux;
+}
 
 // versements par personne
 $_SESSION['lesVersements'] = Utils::getVersementsPersonnes(
@@ -21,6 +26,7 @@ $_SESSION['lesVersements'] = Utils::getVersementsPersonnes(
 
 // préparation de l'affichage
 $tpl->assign('lesVersements', $_SESSION['lesVersements']);
+$tpl->assign('annee_recu', $_SESSION['annee_recu']);
 $tpl->assign('plugin_css', ['style.css']);
 
 // envoyer au template