Projet

Général

Profil

Development #40051

manager : ne pas avoir de message d'erreur en dur dans le template manager_confirm_delete.html

Ajouté par Emmanuel Cazenave il y a environ 4 ans. Mis à jour il y a environ 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
20 février 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Plutôt le tirer du contexte, ce qui permettra de réutiliser le template.


Fichiers

Révisions associées

Révision 13ee8457 (diff)
Ajouté par Emmanuel Cazenave il y a environ 4 ans

manager: grab error message from context (#40051)

Historique

#1

Mis à jour par Emmanuel Cazenave il y a environ 4 ans

#2

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)

#3

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ï.

#4

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 :))

#5

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

Formats disponibles : Atom PDF