ref#2195 - META - change PDF export code

[Gaetanbrl]

PR dans le cadre d'une contribution financée par MNHN / PatriNat
ref #2195

Cette PR modifie la génération des PDF de métadonnées :

• Aucun fichier (temporaire) n'est déposé sur le serveur (suppression de la méthode upload_canvas)
• L'image est ajoutée en tant que data URL b64 dans la source de l'image du template/HTML
• La création du PDF se réalise via un seul appel
• Réutilisation de la méthode subscribeAndDownload
• Correction du graph qui n'était pas ajouté dans l'export PDF d'une AF - META / AF - Graphique non présent dans l'export PDF #2231
• Modification de la route (POST à la place de GET). L'envoi d'un objet/json dans le body permet de se soustraire des problèmes d'encodage que l'on a pu voir lorsque l'image était directement passée dans le body en tant que type data et qui empêché la bonne représentation dune image type data URL dans le template HTML du PDF (voir méthode render Weasyprint).

En cours sur cette PR :

• Test backend
• Prettier / Linter
• Bonus : Analyse issue [Métadonnées] Les fiches PDF des jeux de données ne remontent pas les acteurs/contacts #2034
• Revue

[codecov]

Codecov Report

Base: 67.72% // Head: 67.85% // Increases project coverage by +0.13% 🎉

Coverage data is based on head (c3e1af5) compared to base (6bd8ab2).
Patch coverage: 81.81% of modified lines in pull request are covered.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #2232      +/-   ##
===========================================
+ Coverage    67.72%   67.85%   +0.13%     
===========================================
  Files           83       83              
  Lines         7358     7367       +9     
===========================================
+ Hits          4983     4999      +16     
+ Misses        2375     2368       -7     

Flag	Coverage Δ
pytest	67.85% <81.81%> (+0.13%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
backend/geonature/core/gn_meta/mtd/mtd_utils.py	14.74% <ø> (ø)
backend/geonature/core/gn_meta/routes.py	78.53% <77.77%> (+0.99%)	⬆️
backend/geonature/utils/filemanager.py	62.74% <100.00%> (-1.41%)	⬇️
backend/geonature/core/gn_meta/mtd/__init__.py	35.71% <0.00%> (-7.15%)	⬇️
backend/geonature/core/users/routes.py	54.08% <0.00%> (-0.56%)	⬇️
backend/geonature/middlewares/__init__.py	100.00% <0.00%> (ø)
backend/geonature/core/gn_permissions/models.py	93.97% <0.00%> (+0.07%)	⬆️
backend/geonature/utils/config_schema.py	91.86% <0.00%> (+0.10%)	⬆️
backend/geonature/app.py	83.33% <0.00%> (+0.15%)	⬆️
... and 3 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[bouttier]

Lorsque l’on clique sur le bouton téléchargé, le navigateur ne comprend pas qu’il s’agit d’un PDF et ne propose pas de l’ouvrir avec un lecteur PDF. À priori cela provient du fait que le backend répond avec un Content-Type text/html plutôt que application/pdf. Est-il possible de corriger ce soucis ? Merci !

[Gaetanbrl]

Est-il possible de corriger ce soucis ? Merci !

Testé avec application/pdf et ca permet seulement de le sauvegarder en l'état.

Il faut aussi potentiellement l'ouvrir côté front et pas seulement changer le content-type.

[Gaetanbrl]

Il faut aussi potentiellement l'ouvrir côté front et pas seulement changer le content-type.

Modifications faites.

[bouttier]

Supersed by #2241