Projet

Général

Profil

Development #21997

solis: ajouter un convertisseur PDF

Ajouté par Thomas Noël il y a environ 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Thomas Noël
Version cible:
-
Début:
20 février 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

On reçoit des jpeg ou png dans le payload, mais la GED Solis n'accepte que des PDF.

Dnas un premier temps on a pensé à un convertisseur dans le workflow wcs lié, mais c'est infernal (beaucoup de documents).

Donc, plutôt simplifier et utiliser Pillow dans le connecteur pour convertir les "image/*" en PDF

Fichiers

0001-solis-try-to-convert-files-to-PDF-21997.patch (7,24 ko) 0001-solis-try-to-convert-files-to-PDF-21997.patch Thomas Noël, 20 février 2018 15:52
0001-solis-try-to-convert-files-to-PDF-21997.patch (7,96 ko) 0001-solis-try-to-convert-files-to-PDF-21997.patch Thomas Noël, 20 février 2018 17:19
0001-solis-try-to-convert-files-to-PDF-21997.patch (7,61 ko) 0001-solis-try-to-convert-files-to-PDF-21997.patch Thomas Noël, 20 février 2018 19:09

Révisions associées

Révision 685e947e (diff)
Ajouté par Thomas Noël il y a environ 6 ans

solis: try to convert files to PDF (#21997)

Historique

#2

Mis à jour par Benjamin Dauvergne il y a environ 6 ans

Ça m'a l'air bien, ack.

#3

Mis à jour par Christophe Siraut il y a environ 6 ans

Je ne comprend pas l'utilité des fichiers etat_civil_004.pdf et 005, il y a en commentaire '# bad content, conversion will fail'

#4

Mis à jour par Benjamin Dauvergne il y a environ 6 ans

Allez je fais des remarques quand même:
  • si tu peux mettre ça sur deux lignes:
                            pdf_content = convert_to_pdf(base64.b64decode(value['content']),
                                                 value['content_type'])

    ça mange pas de pain et dans les traces on aura la valeur décodée de value['content'].
  • mettre hors du try (dans un else):
                            files.append(('files', (filename, pdf_content, 'application.pdf')))
  • except ValueError puisqu'on n'attend que ça, ou alors faut voir si python-pil a une classe d'exception de base, mais dans ce cas je catcherai ça dans convert_to_pdf pour convertir en une ValueError
  • je trouve toujours qu'il n'y a pas assez de contrôle des formats (t'es sympa de continuer à traiter une demande avec des champs fichiers vides ou mal formés) je mettrai ça aussi dans files_failed_pdf_conversion
  • à ce titre je factoriserai tout le traitement des clés 'file:*' ça simplifiera la gestion des erreurs
#5

Mis à jour par Thomas Noël il y a environ 6 ans

Christophe Siraut a écrit :

Je ne comprend pas l'utilité des fichiers etat_civil_004.pdf et 005, il y a en commentaire '# bad content, conversion will fail'

C'est pour justement tester que ça fait pas planter le connecteur.

#6

Mis à jour par Thomas Noël il y a environ 6 ans

Version avec les bonnes remarques de Benj, mais sans la "factorisation" finale parce que c'est assez rangé comme ça pour moi (et pour solis)

#7

Mis à jour par Benjamin Dauvergne il y a environ 6 ans

Allez ack.

#8

Mis à jour par Thomas Noël il y a environ 6 ans

Et parce que Frédéric m'a dit d'être plus KISS encore, on ne fait plus confiance ici au content_type envoyé. Si le contenu commence par %PDF alors ça roule, sinon on tente de convertir en PDF, quelque soit le cas.

#9

Mis à jour par Frédéric Péters il y a environ 6 ans

Ouaip ack.

#10

Mis à jour par Thomas Noël il y a environ 6 ans

  • Statut changé de Nouveau à Résolu (à déployer)
commit 685e947efd074383a1ab7d1d56fb2645cd6cd503
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Tue Feb 20 15:52:36 2018 +0100

    solis: try to convert files to PDF (#21997)
#11

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF