Merge branch 'hotfix_release_5.2.11'

CHANGELOG.md

 Journal des modifications
 =========================
 
+5.3.1
+-----
+- [FIX] Donnée : ajout de garde lorsque le mail fourni par les données sources est " "
+- [FIX] hydration des justificatifs
+
 5.3.0
 -----
 - Nouvelle version du module Rapports d'activité.

doc/release-notes/v5.3.1.md

+# Version 5.3.1
+
+## 1. Sur le serveur d'application
+
+- Placez-vous dans le répertoire de l'application puis lancez la commande suivante
+  pour installer la nouvelle version :
+
+```bash
+git fetch --tags && git checkout --force 5.3.1 && bash ./install.sh
+```
+
+- Selon le moteur PHP que vous avez installé, rechargez le service, exemple :
+    - php7.4-fpm         : `service php7.4-fpm reload`
+    - apache2-mod-php7.4 : `service apache2 reload`
+
+## 2. Dans la base de données
+
+RAS
\ No newline at end of file

module/Fichier/src/Fichier/Entity/Db/NatureFichier.php

@@ -56,7 +56,7 @@ class NatureFichier
     const CODE_LANGUE_ANGLAISE = 'DEMANDE_LANGUE_ANGLAISE';
     const CODE_AUTRES_JUSTIFICATIFS = 'AUTRES_JUSTIFICATIFS';
 
-    const LABEL_FORMATION_INTEGRITE_SCIENTIFIQUE = 'Justificatif de suivi de la formation "Intégrité scientifique"';
+    const LABEL_FORMATION_INTEGRITE_SCIENTIFIQUE = 'Justificatif de suivi de la formation Intégrité scientifique';
     const LABEL_JUSTIFICATIF_HDR = "Justificatif d'habilitation à diriger des recherches";
     const LABEL_JUSTIFICATIF_EMERITAT = "Justificatif d'émeritat";
     const LABEL_JUSTIFICATIF_ETRANGER = "Justificatif dans le cas d'un membre du jury étranger";

module/Individu/src/Individu/Entity/Db/Individu.php

@@ -266,11 +266,14 @@ class Individu implements HistoriqueAwareInterface, SourceAwareInterface, Resour
     public function getEmailPro(): ?string
     {
         $complement = $this->getComplement();
-        if ($complement AND !$complement->estHistorise() AND $complement->getEmailPro() !== null) {
+        if ($complement AND !$complement->estHistorise()
+                AND $complement->getEmailPro() !== null) {
             return $complement->getEmailPro();
         }
 
-        return $this->email;
+        $email = $this->email;
+        if ($email === null or trim($this->email) === '') return null;
+        return $email;
     }
 
     /**

module/Soutenance/src/Soutenance/Controller/JustificatifController.php

@@ -88,10 +88,15 @@ class JustificatifController extends AbstractController {
             $files = ['files' => $request->getFiles()->toArray()];
             if ($nature) $data['nature'] = $nature->getCode();
 
-            $form->setData($data);
-            if ($form->isValid()) {
-                var_dump("After form");
+//            $form->setData($data);
+//            if ($form->isValid()) {
+            // <!> Attention : L'hydration echoue ...
+            if ($data['nature'] !== null AND !empty($files)) {
                 if ($nature === null) $nature = $this->fichierTheseService->fetchNatureFichier($data['nature']);
+                if ($data['membre']) {
+                    $membre = $this->getMembreService()->find($data['membre']);
+                    $justificatif->setMembre($membre);
+                }
                 $version = $this->fichierTheseService->fetchVersionFichier(VersionFichier::CODE_ORIG);
                 $fichiers = $this->fichierTheseService->createFichierThesesFromUpload($these, $files, $nature, $version);
                 $justificatif->setFichier($fichiers[0]);

module/Soutenance/src/Soutenance/Controller/PresoutenanceController.php

@@ -692,7 +692,9 @@ class PresoutenanceController extends AbstractController
         /** @var Membre $membre */
         foreach ($proposition->getMembres() as $membre) {
             if ($membre->estMembre()) {
-                $email = ($membre->getIndividu() and $membre->getIndividu()->getEmailPro()) ? $membre->getIndividu()->getEmailPro() : $membre->getEmail();
+                $email = null;
+                if ($membre->getIndividu() and $membre->getIndividu()->getEmailPro()) $email = $membre->getIndividu()->getEmailPro();
+                if ($email === null or trim($email) === '') $email = $membre->getEmail();
                 /** @see PresoutenanceController::convocationMembreAction() */
                 $url = $this->url()->fromRoute('soutenance/presoutenance/convocation-membre', ['proposition' => $proposition->getId(), 'membre' => $membre->getId()], ['force_canonical' => true], true);
                 try {

module/Soutenance/src/Soutenance/Form/Justificatif/JusticatifHydrator.php

@@ -40,6 +40,7 @@ class JusticatifHydrator implements HydratorInterface {
     {
         /** @var Membre $membre */
         $membre = $this->getMembreService()->find($data['membre']);
+
         $object->setMembre($membre);
         return $object;
     }

module/Soutenance/src/Soutenance/Form/Justificatif/JustificatifForm.php

@@ -83,7 +83,7 @@ class JustificatifForm extends Form {
         $this->setInputFilter((new Factory())->createInputFilter([
             'nature'    => [
                 'name' => 'nature',
-                'required' => true,
+                'required' => false,
             ],
             'fichier'   => [
                 'name' => 'fichier',

module/Soutenance/src/Soutenance/Service/Notification/SoutenanceNotificationFactory.php

@@ -674,6 +674,13 @@ class SoutenanceNotificationFactory extends NotificationFactory
         $these = $proposition->getThese();
         $pdcData = $this->getTheseService()->fetchInformationsPageDeCouverture($these);
 
+        if ($email === null or $email === '') {
+            throw new RuntimeException("Aucune adresse électronique pour le doctorant [".$doctorant->getIndividu()->getNomComplet()."]");
+        }
+        if (!filter_var($email, FILTER_VALIDATE_EMAIL)) {
+            throw new RuntimeException("L'adresse électronique est mal formée pour le doctorant [".$doctorant->getIndividu()->getNomComplet()."]");
+        }
+
         $notif = new Notification();
         $notif
             ->setSubject("Convocation pour la soutenance de thèse de  " . $doctorant->getNomComplet())
@@ -709,6 +716,13 @@ class SoutenanceNotificationFactory extends NotificationFactory
         $these = $proposition->getThese();
         $pdcData = $this->getTheseService()->fetchInformationsPageDeCouverture($these);
 
+        if ($email === null or $email === '') {
+            throw new RuntimeException("Aucune adresse électronique pour le membre [".$membre->getDenomination()."]");
+        }
+        if (!filter_var($email, FILTER_VALIDATE_EMAIL)) {
+            throw new RuntimeException("L'adresse électronique est mal formée pour le membre [".$membre->getDenomination()."]");
+        }
+
         $notif = new Notification();
         $notif
             ->setSubject("Convocation pour la soutenance de thèse de  " . $doctorant->getNomComplet())