Project

General

Profile

Bug #30533

ressources: erreur lors de la création d'une vignette pour un SVG

Added by Serghei Mihai 5 months ago. Updated 5 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
11 Feb 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Sorl n'arrive pas à créer des vignettes à partir d'un SVG.

Il faudrait soit interdire l'upload des SVG, avec un message d'erreur explicite à l'usager soit que l'appel à asset_url retourne le SVG lui même.

0001-assets-don-t-use-get_thumbnail-on-SVG-files-30533.patch View (1.69 KB) Frédéric Péters, 11 Feb 2019 04:03 PM


Related issues

Related to Combo - Development #26049: gestion des ressources : miniature image svg Solution déployée 03 Sep 2018

Associated revisions

Revision 016d3371 (diff)
Added by Frédéric Péters 5 months ago

assets: don't use get_thumbnail on SVG files (#30533)

History

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

Mmm, #26049 ?

#3 Updated by Frédéric Péters 5 months ago

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

Et vu le commentaire en #30514 la situation a été modifiée pour ne plus se produire en mettant des PNG, mais où donc alors reproduire la situation ?

#5 Updated by Serghei Mihai 5 months ago

Dans #30514, l'erreur se produisait lors de l'appel au templatetag asset_url (dans http://git.entrouvert.org/publik-base-theme.git/tree/templates/variants/fontenay-sous-bois-2018/combo/wcs/forms_of_category.html#n7) qui appele get_thumbnail dans vérification du type.

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

#7 Updated by Serghei Mihai 5 months ago

  • Status changed from Solution proposée to Solution validée
  • Assignee set to Frédéric Péters

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

  • Status changed from Solution validée to Résolu (à déployer)
commit 016d33712e7f71ca39a3292da74ad1764a3d5a36
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Feb 11 16:03:29 2019 +0100

    assets: don't use get_thumbnail on SVG files (#30533)

#9 Updated by Frédéric Péters 5 months ago

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

Also available in: Atom PDF