Quelques coquilles restantes, alémioration du reporting & tests toutes années confondues

admin/migration/AgrementStructure.php

@@ -46,19 +46,15 @@ class AgrementStructure extends AbstractMigration
     {
         $bdd     = $this->manager->getBdd();
         $console = $this->manager->getOseAdmin()->getConsole();
-        $console->println("Récupération des agréments conseil restreint sans structure");
         //La vue TBL_AGREMENT a changé, donc il faut d'abord recharger la vue agrement pour le traitement
-        $console->println("------------------------------------------------------------------");
-        $console->println("Nouveau calcul du tableau de bord agrement");
+        $console->println("Calcul du tableau de bord agrement");
         $bdd->exec('BEGIN unicaen_tbl.calculer(\'agrement\'); END;');
 
         $tacrId = (int)$bdd->select("SELECT id FROM type_agrement WHERE code = 'CONSEIL_RESTREINT'")[0]['ID'];
 
-        $bdd->exec("alter trigger AGREMENT_CK disable");
-
         //On récupère les agrements de type conseil restreint sans structure
         $agrementsToRecreate = [];
-        $sql                 = 'SELECT 
+        $sql                 = "SELECT 
                     a.id               agrement_id,
                     i.id               intervenant_id,
                     i.annee_id         annee_id,
@@ -69,15 +65,17 @@ class AgrementStructure extends AbstractMigration
                 FROM AGREMENT a
                 JOIN INTERVENANT i ON i.id = a.intervenant_id
                 WHERE
-                a.type_agrement_id = 1
+                a.type_agrement_id = $tacrId
                 AND a.structure_id IS NULL
                 AND a.histo_destruction IS NULL
-                AND i.annee_id = 2019';
+                ";
 
         $agrements = $bdd->select($sql);
         //Nombre d'agréments concernés
         $nbAgrement = count($agrements);
-        $console->println("Il y a $nbAgrement a corriger");
+        if ($nbAgrement > 0) {
+            $bdd->exec("alter trigger AGREMENT_CK disable");
+            $console->println("Il y a $nbAgrement agréments du conseil restreint a corriger");
             $nbAgrementCorrige = 0;
             //On commence le traitement de chaque agrément qui pose problème
             foreach ($agrements as $agrement) {
@@ -162,10 +160,10 @@ class AgrementStructure extends AbstractMigration
                 $console->println("Seulement $nbAgrementCorrige sur $nbAgrement on été corrigés", $console::COLOR_LIGHT_RED);
                 $console->println("Les intervenants suivants nécessitent une intervention manuelle de votre part : il faut saisir à nouveau leurs agréments du conseil restreint : ", $console::COLOR_LIGHT_RED);
                 foreach ($agrementsToRecreate as $value) {
-                $console->println($value['PRENOM'] . " " . $value['NOM_USUEL'] . " (id : " . $value['INTERVENANT_ID'] . " / code : " . $value['CODE_INTERVENANT'] . ")", $console::COLOR_LIGHT_RED);
+                    $console->println($value['PRENOM'] . " " . $value['NOM_USUEL'] . " (année : " . $value['ANNEE_ID'] . " / id : " . $value['INTERVENANT_ID'] . " / code : " . $value['CODE_INTERVENANT'] . ")", $console::COLOR_LIGHT_RED);
                 }
             } else {
-            $console->println("Les $nbAgrement agrements restreints sans structure on été corrigé.", $console::COLOR_LIGHT_GREEN);
+                $console->println("Les $nbAgrement agrements restreints sans structure on été corrigés.", $console::COLOR_LIGHT_GREEN);
             }
 
             $bdd->exec("alter trigger AGREMENT_CK enable");
@@ -180,5 +178,6 @@ class AgrementStructure extends AbstractMigration
 
             $console->println("Terminé");
         }
+    }
 
 }
\ No newline at end of file