Merge remote-tracking branch 'origin/master'

CHANGELOG.md

@@ -90,6 +90,7 @@ Objectif : Plafonds personnalisables & refonte gestion des statuts
 * Correction sur la suppression de service lorsque la clôture de service a été historisé (#42046)
 * Le calcul des choix minimum/maximum est de nouveau fiable (#42080)
 * Liens inactifs lors du changement d'année universitaire (#40992)
+* Dans certains cas avec des motifs de non paiements, le détail des services n'affichait pas toutes les heures
 
 ## Notes de mise à jour
 
@@ -102,7 +103,7 @@ Objectif : Plafonds personnalisables & refonte gestion des statuts
 * Pour les instances qui utilisent le module Export Siham, vous devez renseigner un nouveau paramètre dans Administration > paramètres généraux > Gestion export RH, il vous faut sélectionner l'étape de la feuille de route franchie à partir de laquelle l'intervenant peut être exporté dans SIHAM.
 * La vue source [SRC_INTERVENANT](doc/Connecteurs-Import/Générique/SRC_INTERVENANT.sql) doit être mise à jour. 
 * Par précaution, la mise à jour désactive la synchronisation sur la table INTERVENANT. Vous devrez manuellement remettre en place cette synchronisation après avoir mis à jour votre vue source SRC_INTERVENANT.
-
+* ** ATTENTION** Les indicateurs portant sur les anciens plafonds ayant été supprimés et remplacés par de tous nouveaux indicateurs, les notifications par mail et abonnements correspondants seront résiliés
 
 # OSE 17.3 (17/03/2022)
 

admin/actions/maj-public-links.php

+<?php
+
+// à supprimer pour la v19
\ No newline at end of file

admin/migration/v18Divers.php

@@ -26,6 +26,11 @@ class v18Divers extends AbstractMigration
         $bdd = $this->manager->getBdd();
         $c   = $this->manager->getOseAdmin()->getConsole();
 
+        // test pour savoir si on est bien en V17 minimum
+        if (!$this->manager->hasColumn('INTERVENANT', 'EXPORT_DATE')) {
+            $c->printDie('Attention : vous devez d\'abord mettre à jour en version 17.3 AVANT de mettre à jour en version 18');
+        }
+
         try {
             $c->msg('Coupure forcée de la synchronisation sur la table INTERVENANT');
             $bdd->exec("UPDATE IMPORT_TABLES SET SYNC_ENABLED = 0 WHERE TABLE_NAME = 'INTERVENANT'");
@@ -36,7 +41,7 @@ class v18Divers extends AbstractMigration
         $this->sauvegardes();
 
         try {
-            $c->msg('Suppression de la contrainte TYPE_INTERVENANT_CODE_UN en prévision de sa recréation');
+            $c->msg('Suppression des affectations de recherche ayant des structures invalides');
             $bdd->exec("DELETE FROM AFFECTATION_RECHERCHE WHERE structure_id NOT IN (SELECT ID FROM STRUCTURE)");
         } catch (\Exception $e) {
             // rien à faire : la contrainte a déjà du être supprimée

admin/migration/v18Indicateurs.php

+<?php
+
+
+
+
+
+class v18Indicateurs extends AbstractMigration
+{
+
+    public function description(): string
+    {
+        return "Migration des indicateurs de OSE 17 vers OSE 18";
+    }
+
+
+
+    public function utile(): bool
+    {
+        return true;
+
+        return $this->manager->hasNew('table', 'TYPE_INDICATEUR');
+    }
+
+
+
+    public function before()
+    {
+        $bdd = $this->manager->getBdd();
+        $c   = $this->manager->getOseAdmin()->getConsole();
+
+        $c->begin('Préparation à la mise à jour des indicateurs');
+
+        $bdd->exec('ALTER TABLE INDICATEUR ADD (TYPE_INDICATEUR_ID NUMBER)');
+        $bdd->exec('CREATE TABLE TYPE_INDICATEUR (  
+              ID NUMBER NOT NULL ENABLE,
+              LIBELLE VARCHAR2(60 CHAR) NOT NULL ENABLE,
+              ORDRE NUMBER DEFAULT 1 NOT NULL ENABLE
+            )');
+
+        $indicateurs = require $this->manager->getOseAdmin()->getOseDir() . '/data/indicateurs.php';
+        foreach ($indicateurs as $libelle => $type) {
+            $data = ['ID' => $type['id'], 'LIBELLE' => $libelle];
+            $bdd->getTable('TYPE_INDICATEUR')->insert($data);
+            foreach ($type['indicateurs'] as $numero => $indicateur) {
+                $bdd->getTable('INDICATEUR')->update(['TYPE_INDICATEUR_ID' => $type['id']], ['NUMERO' => $numero]);
+            }
+        }
+
+        $bdd->exec('DELETE FROM INDICATEUR WHERE TYPE_INDICATEUR_ID IS NULL');
+
+        $c->end('Préparation à la migration des indicateurs terminée');
+    }
+}
\ No newline at end of file

admin/migration/v18Statuts.php

@@ -26,12 +26,6 @@ class v18Statuts extends AbstractMigration
         $bdd = $this->manager->getBdd();
         $c   = $this->manager->getOseAdmin()->getConsole();
 
-        try {
-            $this->preMigrationIndicateurs();
-        } catch (\Exception $e) {
-            $c->println($e->getMessage(), $c::COLOR_RED);
-        }
-
         try {
             $this->preMigrationStatuts();
         } catch (\Exception $e) {
@@ -124,27 +118,6 @@ class v18Statuts extends AbstractMigration
 
 
 
-    public function preMigrationIndicateurs()
-    {
-        $bdd = $this->manager->getBdd();
-        $c   = $this->manager->getOseAdmin()->getConsole();
-
-        $c->begin('Préparation à la mise à jour des indicateurs');
-        if (!$this->manager->hasTable('TYPE_INDICATEUR')) {
-            $bdd->exec('ALTER TABLE INDICATEUR ADD (TYPE_INDICATEUR_ID NUMBER)');
-            $bdd->exec('CREATE TABLE TYPE_INDICATEUR (  
-              ID NUMBER NOT NULL ENABLE,
-              LIBELLE VARCHAR2(60 CHAR) NOT NULL ENABLE,
-              ORDRE NUMBER DEFAULT 1 NOT NULL ENABLE
-            )');
-            $bdd->exec('INSERT INTO TYPE_INDICATEUR (ID, LIBELLE, ORDRE) VALUES (1,\'provisoire\', 1)');
-            $bdd->exec('UPDATE INDICATEUR SET TYPE_INDICATEUR_ID = 1');
-        }
-        $c->end('Préparation à la migration des indicateurs terminée');
-    }
-
-
-
     public function preMigrationStatuts()
     {
         $bdd = $this->manager->getBdd();

module/Application/src/View/Helper/VolumeHoraire/Liste.php

@@ -158,7 +158,15 @@ class Liste extends AbstractViewHelper
         $out      .= "</tr>\n";
         $periodes = $this->getPeriodes();
         foreach ($periodes as $periode) {
-            $vhl               = $this->getVolumeHoraireListe()->setPeriode($periode)->setTypeIntervention(false);
+            $vhl = $this->getVolumeHoraireListe()->createChild()->setPeriode($periode)->setTypeIntervention(false);
+
+            /* Gestion des mauvaises périodes */
+            $forbiddenPeriode = $vhl->getService()?->getElementPedagogique()?->getPeriode() ?? $periode !== $periode;
+            if ($forbiddenPeriode) {
+                $this->hasForbiddenPeriodes = true;
+            }
+
+            /*Listage des motifs de non paiement */
             $motifsNonPaiement = [];
             if ($canViewMNP) {  // découpage par motif de non paiement
                 $motifsNonPaiement = $vhl->getMotifsNonPaiement();
@@ -169,18 +177,13 @@ class Liste extends AbstractViewHelper
             if (empty($motifsNonPaiement)) {
                 $motifsNonPaiement = [0 => false];
             }
+
+            /* Affichage par motif de non paiement */
             foreach ($motifsNonPaiement as $motifNonPaiement) {
+                $vhl->setMotifNonPaiement($motifNonPaiement);
+                if ($vhl->getHeures() == 0) continue; // rien à afficher
+
                 $readOnly = $motifNonPaiement instanceof MotifNonPaiement && !$canEditMNP;
-                $forbiddenPeriode = false;
-                if (
-                    $this->getVolumeHoraireListe()->getService()
-                    && $this->getVolumeHoraireListe()->getService()->getElementPedagogique()
-                    && $this->getVolumeHoraireListe()->getService()->getElementPedagogique()->getPeriode()
-                    && $this->getVolumeHoraireListe()->getService()->getElementPedagogique()->getPeriode() !== $periode
-                ) {
-                    $forbiddenPeriode           = true;
-                    $this->hasForbiddenPeriodes = true;
-                }
                 if ($forbiddenPeriode) {
                     $out .= '<tr class="bg-danger">';
                     $out .= "<td><abbr title=\"La période n'est pas conforme à l'enseignement\">" . $this->renderPeriode($periode) . "</abbr></td>\n";
@@ -190,8 +193,7 @@ class Liste extends AbstractViewHelper
                 }
 
                 foreach ($this->typesIntervention as $typeIntervention) {
-                    $vhl->setMotifNonPaiement($motifNonPaiement)
-                        ->setTypeIntervention($typeIntervention);
+                    $vhl->setTypeIntervention($typeIntervention);
                     if ($vhl->getHeures() == 0) {
                         $class = "heures-empty";
                     } else {