Development #23281
possibilité de surcharger le rendu des widgets fichier
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
19 avril 2018
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 6 ans
- Fichier 0001-forms-use-a-template-to-render-file-fields-23281.patch 0001-forms-use-a-template-to-render-file-fields-23281.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a presque 6 ans
Je ne suis pas sûr du
htmltext('<img alt="" src="tempfile?t=%s&thumbnail=1" />' % self.get('token'))
qui devient
<img alt="" src="tempfile?t={{ widget.tempfile.token }}&thumbnail=1"/>
Ça serait pas widget.token au lieu de widget.tempfile.token ?
Mis à jour par Frédéric Péters il y a presque 6 ans
C'est la même valeur :
(Pdb) p self.get('token')
'e2a09cf921736e8c:2l1qeUcSWh3HQlHaueP2YV-8CHA'
(Pdb) get_session().get_tempfile(self.get('token'))
{'orig_filename': 'spinozad-partout.jpg', 'unsigned_token': 'e2a09cf921736e8c', 'charset': None, 'token': 'e2a09cf921736e8c:2l1qeUcSWh3HQlHaueP2YV-8CHA',
'session': 'd320ca43a5c53f52', 'base_filename': 'spinozad-partout.jpg', 'content_type': 'image/jpeg', 'size': 817981}
(Pdb) get_session().get_tempfile(self.get('token'))['token']
'e2a09cf921736e8c:2l1qeUcSWh3HQlHaueP2YV-8CHA'
Mais le self.get() c'est une méthode de CompositeWidget, qui est un détail d'implémentation dont petit à petit j'aimerais me défaire, du coup je préfère la version passant par le fichier associé. (même si c'est un peu plus coûteux en traitement)
Mis à jour par Thomas Noël il y a presque 6 ans
Ok, j'avais un peu raté ça. C'est effectivement couteux mais le code semble plus cohérent ainsi : ack.
Mis à jour par Frédéric Péters il y a presque 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit 64bad2d27e24d3320e9564de3f086025607b1055
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue May 1 14:33:01 2018 +0200
forms: use a template to render file fields (#23281)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
forms: use a template to render file fields (#23281)