Development #40051
manager : ne pas avoir de message d'erreur en dur dans le template manager_confirm_delete.html
0%
Description
Plutôt le tirer du contexte, ce qui permettra de réutiliser le template.
Fichiers
Révisions associées
Historique
Mis à jour par Emmanuel Cazenave il y a environ 4 ans
- Fichier 0001-manager-grab-error-message-from-context-40051.patch 0001-manager-grab-error-message-from-context-40051.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a environ 4 ans
Je ne suis jamais bien à l'aise avec ça : on va bien passer par une traduction avec un « _('chose') » ainsi posé ?
Mais en fait même sans ça, je trouve que poser une constante comme ça juste pour éviter une répétition, bon, bof... Ça sera plus explicite d'avoir directement, même deux fois :
context['cannot_delete_msg'] = _('This cannot be removed as there are bookings for a future date.')
(on pourrait même penser que cannot_delete pourrait contenir le message et pas juste un booléen, et éviter ainsi le cannot_delete_msg, mais bon, bref)
Mis à jour par Thomas Noël il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Thomas Noël a écrit :
Je ne suis jamais bien à l'aise avec ça : on va bien passer par une traduction avec un « _('chose') » ainsi posé ?
En fait je pense que ça marchera car on me rappelle ailleurs que le _() de Django est lazy (et je suis habitué à un autre logiciel pas tout à fait Djangoisé).
Mais bon, quand même, bref, avec ou sans ta constante, alleï.
Mis à jour par Emmanuel Cazenave il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 13ee8457150f7eb8ae9f5ec9a9189c947c8abce7 Author: Emmanuel Cazenave <ecazenave@entrouvert.com> Date: Thu Feb 20 11:34:13 2020 +0100 manager: grab error message from context (#40051)
(je n'ai pas pu me résoudre à volontairement dupliquer le message :))
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
manager: grab error message from context (#40051)