Development #15857
utiliser sorl-thumbnail pour la création des vignettes
100%
Description
Pour le moment il y a appel direct à graphicsmagick pour la création des vignettes des documents, on pourrait plutôt utiliser sorl-thumbnail (qui est dispo dans debian); notamment ça offrirait une souplesse plus grande aux templates, qui pourraient faire :
{% thumbnail item.image "100x100" crop="center" as im %}
<img src="{{ im.url }}" width="{{ im.width }}" height="{{ im.height }}">
{% endthumbnail %}
plutôt que devoir se satisfaire du 200 pixels de largeur hardcodé actuellement.
Fichiers
Révisions associées
debian: add dependency on graphicsmagick-imagemagick-compat (#15857)
Historique
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
J'ai implémenté sorl-thumbnail.
D'abord avec le backend GraphicsMagick mais il ne détecte pas tout seul qu'un fichier n'est pas vignettable, donc je passe à PIL mais là il n'est pas capable de vignetter du PDF, finalement j'essaie le backend pgmagick mais il faut télécharger des kilos de libboost* pour que ça compile, finalement je suis revenu au backend GraphicsMagick de sorl-thumbnail en ajoutant un check explicite sur le type MIME pour que ce soit image/* ou application/pdf; je suis pas revenu bien loin de mon point de départ, enfin...
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
- Fichier 0001-use-sorl-thumbnail-fixes-15857.patch 0001-use-sorl-thumbnail-fixes-15857.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a environ 6 ans
Mais l'ambition posée dans la description du ticket c'était d'offrir la souplesse au template, plutôt qu'avoir ça dans le Python.
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
Ce n'est pas vraiment possible avec le templatetag de sorl-thumbnail, c'est uniquement prévu pour des images publiques (distribution des vignettes via le répertoire media/). Le mieux que je puisse faire c'est soit un endpoint qui du pk d'un modèle publie sa vignette, soit une data-url. Si je garde mes data_url mais que je ne construis pas le tag img en python ça irait ? J'ai construit directement le tag par facilité mais je peux exporter un dico {data-url, width, height} à la place.
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
Nouvelle méthode thumbnail_image qui renvoie le dico sus-mentionné.
Au passage j'ajoute un try/except sauvage autour de get_thumbnail() qui de toute façon crashera sans crier gare, le fait que libmagic considère qu'un fichier semble être un PDF n'est pas suffisant pour garantir que convert s'en sortira (exemple, le faux fichier PDF des tests qui contient juste l'entête des PDFs).
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
Ajout d'un appel à la commande de gestion thumbnail cleanup, dans la commande fargo-cleanup.
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
- Statut changé de En cours à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit 48b598000f5f5bf73a22893572e1cccb3ad66208.
Mis à jour par Frédéric Péters il y a presque 6 ans
- Statut changé de Résolu (à déployer) à Fermé
use sorl-thumbnail (fixes #15857)
It replaces GraphicsMagick, we do not use the templatetag as the files
are private.