Development #32109
pas de miniature pour les PDF "avec un BOM au début"
0%
Description
quand un PDF commence par un BOM (ce qui n'est pas normal mais a été vu) la miniature est plantée.
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
Bizarre, Un gm convert en console fonctionne sur la prod toulouse.
bdauvergne@cutm-publik-prod-web1:/tmp$ gm convert 7349740_20190326_6aad589f-d939-428b-ac84-b05be5046d3d.pdf coin.png **** Warning: File has some garbage before %PDF- . bdauvergne@cutm-publik-prod-web1:/tmp$ ls coin*.png coin.png
Il faudrait commencer par construire un test qui foire autour de ce fichier.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Fichier 7349740_20190326_6aad589f-d939-428b-ac84-b05be5046d3d.pdf ajouté
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Fichier
7349740_20190326_6aad589f-d939-428b-ac84-b05be5046d3d.pdfsupprimé
Mis à jour par Thomas Noël il y a environ 5 ans
Le process de conversion utilisé par w.c.s. plante sur un fichier avec un BOM :
$ gm convert -geometry 500x ./007dc8b2dec2fcdb png:- gm convert: Improper image header (./007dc8b2dec2fcdb).
parce que le fichier n'a pas l'extension "pdf" et que gm n'arrive pas à deviner que c'est un pdf en se basant sur le contenu.
Mis à jour par Thomas Noël il y a environ 5 ans
Il faudrait être explicite et faire gm convert -geometry 500x pdf:./007dc8b2dec2fcdb png:-
Mis à jour par Thomas Noël il y a environ 5 ans
- Fichier 0001-misc-create-PDF-thumbnails-explicitly-32109.patch 0001-misc-create-PDF-thumbnails-explicitly-32109.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
testé avec succès sur ma machine (on n'a pas de test unitaire sur l'affaire car ça nécessite l'installation de graphicsmagick)
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Solution proposée à Solution validée
(on n'a pas de test unitaire sur l'affaire car ça nécessite l'installation de graphicsmagick)
Pour LibreOffice, on utilise @skipif, pour quand même avoir des tests, ex :
@pytest.mark.skipif(transform_to_pdf is None, reason='libreoffice not found')
Mais ok ainsi, en modifiant l'intitulé du commit, plutôt quelque chose dans le genre de : "misc: be explicit about input format passed to graphicsmagick".
Mis à jour par Thomas Noël il y a environ 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 3bf11103935edffa89e6ce57f731a7f4b3aaf835 Author: Thomas NOEL <tnoel@entrouvert.com> Date: Tue Apr 9 12:07:24 2019 +0200 misc: be explicit about input format passed to graphicsmagick (#32109)
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: be explicit about input format passed to graphicsmagick (#32109)