Bug #38952
génération de document : ne pas permettre les / dans le nom du fichier
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
14 janvier 2020
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
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 -.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Thomas Noël il y a plus de 4 ans
- Fichier 0001-export_to_model-remove-slashes-in-computed-filename-.patch 0001-export_to_model-remove-slashes-in-computed-filename-.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
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.
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Solution proposée à 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 ?
Mis à jour par Thomas Noël il y a plus de 4 ans
- Statut changé de Solution validée à 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)
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Frédéric Péters il y a environ 3 ans
- Lié à Development #30671: exclure certains caractères "difficiles" des noms de fichiers générés par l'action "créer un document" ajouté
export_to_model: replace slashes with dashes in computed filename (#38952)