Bug #21378
utilisateur <SqlUser 'Aude MONTMAYEUR' id:25> indiqué dans un mail
0%
Description
Sur une plateforme à laquelle je n'ai pas accès pour voir le squelette du mail, mais dont je reçois des mails de test :
Une nouvelle demande fait l'objet d'un traitement complexe. Elle vous a été transmise par <SqlUser 'Plastic Bertrand' id:25> pour la raison suivante : ...
Je me demande donc d'où vient ce <SqlUser 'Plastic Bertrand' id:25> et ce qu'on pourrait éventuellement faire pour l'éviter.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a plus de 6 ans
Ca semble être {{form_user}} dans le mail, pas testé encore en vrai.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Faut utiliser form_user_display_name. (on pourrait bien sûr ajouter un __unicode__).
Mis à jour par Thomas Noël il y a plus de 6 ans
Ça me va de laisser ça ainsi, pas de unicode, l'affichage «SqlUser...» est un effet de bord qu'on pourrait juger utile dans certains cas ; le seul truc un peu contre-intuitif c'est que dans la liste des variables, sur le backoffice, on l'indique :
...
Substitutions.register('form_user', category=N_('Form'), comment=N_('Form Submitter'))
Substitutions.register('form_user_display_name', category=N_('Form'), comment=N_('Form Submitter Name'))
Substitutions.register('form_user_email', category=N_('Form'), comment=N_('Form Submitter Email'))
...
et que ce premier form_user n'est donc pas si clair pour un non-informaticien (ie "à quoi ça sert donc ?", la réponse étant "c'est utile dans les conditions"). On pourrait peut-être ne pas l'afficher, ou carrément dire « comment=N_('Form Submitter (if present)') » ou « comment=N_('Form Submitter (as an object)') » mais là personne ne va nous suivre.
Ou bien rejeter ce ticket, ce qui est sans doute la meilleure option, et juste mentionner form_user dans la doc https://doc-publik.entrouvert.com/admin-fonctionnel/utiliser-les-variables/ en précisant bien qu'il s'agit juste d'une variable disponible pour les conditions.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0001-misc-only-advertise-strings-in-substitution-variable.patch 0001-misc-only-advertise-strings-in-substitution-variable.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Je pense que le patch à faire c'est dégager form_user de la liste affichée de variables de substitutions. Cette liste est vraiment supposée être des trucs utilisables dans les gabarits. (Virer également session_user). À dormir dessus pas fan du __unicode__ parce que pas fan de devoir un jour expliquer pourquoi form_user == 'Frederic Peters' ne fonctionne pas.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit c53587da68e00b1eaa018c5acfb7d4150826bc9b
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Jan 24 09:40:32 2018 +0100
misc: only advertise strings in substitution variables (#21378)
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
misc: only advertise strings in substitution variables (#21378)