Project

General

Profile

Development #26049

gestion des ressources : miniature image svg

Added by Frédéric Péters 12 months ago. Updated 8 months ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
03 Sep 2018
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Sur l'installation gnm ça fait cannot identify image file <cStringIO.StringI object at 0x7f6245a5d2d8>, on pourrait sans doute simplement zapper le côté thumbnail pour du svg. (à défaut de trouver dans sorl.thumbnail comment lui faire accepter ça).

0001-assets-don-t-pass-svg-to-sorl.thumbnail-26049.patch View (933 Bytes) Frédéric Péters, 04 Sep 2018 04:16 PM

0001-assets-don-t-pass-svg-to-sorl.thumbnail-26049.patch View (941 Bytes) Frédéric Péters, 04 Sep 2018 04:41 PM


Related issues

Related to Combo - Bug #25885: crash "PIL/Image: cannot identify image file" Rejeté 27 Aug 2018
Related to Combo - Bug #30533: ressources: erreur lors de la création d'une vignette pour un SVG Solution déployée 11 Feb 2019

Associated revisions

Revision 82bade20 (diff)
Added by Frédéric Péters 12 months ago

assets: don't pass svg to sorl.thumbnail (#26049)

History

#1 Updated by Frédéric Péters 12 months ago

#2 Updated by Frédéric Péters 12 months ago

  • Related to Bug #25885: crash "PIL/Image: cannot identify image file" added

#3 Updated by Thomas Noël 12 months ago

  • Status changed from Solution proposée to Solution validée

L'idée du thumbnail pour moi c'était aussi celle que le fichier était petit et donc pas méchant à afficher (en perf cpu). Sur un gros SVG ça va tasser un peu, non ? Bon, ceci dit, normalement, rares devraient être les gros SVG uploadés dans combo.

Sinon, quid des "TRUC.SVG", ajouter un "lower()" : if self.asset.asset.path.lower().endswith('.svg'): ?

#4 Updated by Frédéric Péters 12 months ago

Yep, pour le moment je ne me sens pas trop concerné par les gros svg, ce n'est pas vraiment l'usage.

#5 Updated by Thomas Noël 12 months ago

  • Status changed from Solution proposée to Solution validée

impec

#6 Updated by Frédéric Péters 12 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 82bade20aad5de3ed19b2defbde0489f802b7cb7
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Sep 4 16:15:24 2018 +0200

    assets: don't pass svg to sorl.thumbnail (#26049)

#7 Updated by Frédéric Péters 8 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

#8 Updated by Frédéric Péters 6 months ago

  • Related to Bug #30533: ressources: erreur lors de la création d'une vignette pour un SVG added

Also available in: Atom PDF