Merge branch 'refactoring' of ssh://git.v0id.ovh:5022/n3wt-innov/n3wt-school into refactoring

Back-End/Subscriptions/models.py

@@ -182,6 +182,10 @@ class Student(models.Model):
             return self.birth_date.strftime('%d-%m-%Y')
         return None
 
+def registration_file_path(instance, filename):
+    # Génère le chemin : registration_files/dossier_rf_{student_id}/filename
+    return f'registration_files/dossier_rf_{instance.student_id}/{filename}'
+
 class RegistrationForm(models.Model):
     """
     Gère le dossier d’inscription lié à un élève donné.
@@ -201,7 +205,11 @@ class RegistrationForm(models.Model):
     last_update = models.DateTimeField(auto_now=True)
     notes = models.CharField(max_length=200, blank=True)
     registration_link_code = models.CharField(max_length=200, default="", blank=True)
-    registration_file = models.FileField(upload_to=settings.DOCUMENT_DIR, default="", blank=True)
+    registration_file = models.FileField(
+        upload_to=registration_file_path,
+        null=True,
+        blank=True
+    )
     associated_rf = models.CharField(max_length=200, default="", blank=True)
 
     # Many-to-Many Relationship

Back-End/Subscriptions/util.py

@@ -93,31 +93,57 @@ def getArgFromRequest(_argument, _request):
 
 def merge_files_pdf(filenames, output_filename):
     """
-    Insère plusieurs fichiers PDF dans un seul document de sortie.
+    Fusionne plusieurs fichiers PDF en un seul document.
+    Vérifie l'existence des fichiers sources avant la fusion.
     """
     merger = pymupdf.open()
+    valid_files = []
+
+    # Vérifier l'existence des fichiers et ne garder que ceux qui existent
     for filename in filenames:
+        if os.path.exists(filename):
+            valid_files.append(filename)
+
+    # Fusionner les fichiers valides
+    for filename in valid_files:
         merger.insert_file(filename)
+
+    # S'assurer que le dossier de destination existe
+    os.makedirs(os.path.dirname(output_filename), exist_ok=True)
+
+    # Sauvegarder le fichier fusionné
     merger.save(output_filename)
     merger.close()
 
-def rfToPDF(registerForm,filename):
+    return output_filename
+
+def rfToPDF(registerForm, filename):
     """
-    Génère le PDF d’un dossier d’inscription et l’associe au RegistrationForm.
+    Génère le PDF d'un dossier d'inscription et l'associe au RegistrationForm.
     """
-    # Ajout du fichier d'inscriptions
     data = {
-        'pdf_title': "Dossier d'inscription de %s"%registerForm.student.first_name,
+        'pdf_title': f"Dossier d'inscription de {registerForm.student.first_name}",
         'signatureDate': convertToStr(_now(), '%d-%m-%Y'),
         'signatureTime': convertToStr(_now(), '%H:%M'),
-        'student':registerForm.student,
+        'student': registerForm.student,
     }
-    PDFFileName = filename
+
+    # S'assurer que le dossier parent existe
+    os.makedirs(os.path.dirname(filename), exist_ok=True)
+
+    # Générer le PDF
     pdf = renderers.render_to_pdf('pdfs/dossier_inscription.html', data)
-    pathFichier = Path(filename)
-    if os.path.exists(str(pathFichier)):
-        print(f'File exists : {str(pathFichier)}')
-        os.remove(str(pathFichier))
-    receipt_file = BytesIO(pdf.content)
-    registerForm.fichierInscription = File(receipt_file, PDFFileName)
-    registerForm.fichierInscription.save()
+
+    # Écrire le fichier directement
+    with open(filename, 'wb') as f:
+        f.write(pdf.content)
+
+    # Mettre à jour le champ registration_file du registerForm
+    with open(filename, 'rb') as f:
+        registerForm.registration_file.save(
+            os.path.basename(filename),
+            File(f),
+            save=True
+        )
+
+    return registerForm.registration_file

Back-End/Subscriptions/views.py

@@ -179,20 +179,37 @@ class RegisterFormView(APIView):
         registerForm = bdd.getObject(_objectName=RegistrationForm, _columnName='student__id', _value=_id)
 
         if _status == RegistrationForm.RegistrationFormStatus.RF_UNDER_REVIEW:
-            # Le parent a complété le dossier d'inscription, il est soumis à validation par l'école
-            json.dumps(studentForm_data)
-            #Génération de la fiche d'inscription au format PDF
-            PDFFileName = "rf_%s_%s.pdf"%(registerForm.student.last_name, registerForm.student.first_name)
-            path = Path(f"registration_files/dossier_rf_{registerForm.pk}/{PDFFileName}")
-            registerForm.fichierInscription = util.rfToPDF(registerForm, path)
-            # Récupération des fichiers d'inscription
-            fileNames = RegistrationFile.get_files_from_rf(registerForm.pk)
-            fileNames.insert(0,path)
-            # Création du fichier PDF Fusionné avec le dossier complet
-            output_path = f"registration_files/dossier_rf_{registerForm.pk}/dossier_{registerForm.pk}.pdf"
-            util.merge_files_pdf(fileNames, output_path)
-            # Mise à jour de l'automate
-            updateStateMachine(registerForm, 'saisiDI')
+            try:
+                # Génération de la fiche d'inscription au format PDF
+                base_dir = f"registration_files/dossier_rf_{registerForm.pk}"
+                os.makedirs(base_dir, exist_ok=True)
+
+                # Fichier PDF initial
+                initial_pdf = f"{base_dir}/rf_{registerForm.student.last_name}_{registerForm.student.first_name}.pdf"
+                registerForm.registration_file = util.rfToPDF(registerForm, initial_pdf)
+                registerForm.save()
+
+                # Récupération des fichiers d'inscription
+                fileNames = RegistrationFile.get_files_from_rf(registerForm.pk)
+                if registerForm.registration_file:
+                    fileNames.insert(0, registerForm.registration_file.path)
+
+                # Création du fichier PDF Fusionné
+                merged_pdf = f"{base_dir}/dossier_complet_{registerForm.pk}.pdf"
+                util.merge_files_pdf(fileNames, merged_pdf)
+
+                # Mise à jour du champ registration_file avec le fichier fusionné
+                with open(merged_pdf, 'rb') as f:
+                    registerForm.registration_file.save(
+                        os.path.basename(merged_pdf),
+                        File(f),
+                        save=True
+                    )
+
+                # Mise à jour de l'automate
+                updateStateMachine(registerForm, 'saisiDI')
+            except Exception as e:
+                return JsonResponse({'error': str(e)}, status=status.HTTP_500_INTERNAL_SERVER_ERROR)
         elif _status == RegistrationForm.RegistrationFormStatus.RF_VALIDATED:
             # L'école a validé le dossier d'inscription
             # Mise à jour de l'automate