Project

General

Profile

Bug #38952

génération de document : ne pas permettre les / dans le nom du fichier

Added by Thomas Noël 7 days ago. Updated 6 days ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
14 Jan 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Sur une action de génération de document selon un moèdle, si on met par exemple dans "Nom de fichier" :

fichier_{{ form_receipt_date }}

on va se retrouver en français avec un fichier nommé fichier_14/01/2020.pdf

Le nom du fichier contient des / : il n'est de fait pas accessible.

Il faudrait ne retenir que les caractères autorisés, ou, au moins, remplacer les / par des -.

0001-export_to_model-remove-slashes-in-computed-filename-.patch View (1.59 KB) Thomas Noël, 14 Jan 2020 02:42 PM

Associated revisions

Revision 6e00660c (diff)
Added by Thomas Noël 7 days ago

export_to_model: replace slashes with dashes in computed filename (#38952)

History

#2 Updated by Thomas Noël 7 days ago

Je ne remplace que / car c'est a priori le seul caractère qui va vraiment nous bloquer dans les URL générées.

#3 Updated by Frédéric Péters 7 days ago

Ou ignorer et taper un slugify dessus ?

#4 Updated by Frédéric Péters 7 days ago

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

Bon vu le patch présent, n'en faisons pas plus. Mais tu veux pas un tiret plutôt que l'underscore ?

#5 Updated by Thomas Noël 7 days ago

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

Poussé, avec un tiret, oui.

commit 6e00660c3962811588ad3a06c49da57d3721d58e
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Tue Jan 14 14:41:52 2020 +0100

    export_to_model: replace slashes with dashes in computed filename (#38952)

(J'ai pensé à slugify mais ça risquait de changer l'existant, mais ça sera une solution plus "fiable" si le problème revient par ailleurs)

#6 Updated by Frédéric Péters 6 days ago

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

Also available in: Atom PDF