Bug #60554
Erreur 500 sur la création d'une fiche avec session_user_id
0%
Description
J'utilise l'action de création de fiche et je souhaite rattacher la fiche à l'utilisateur courant (qui est un agent, en back-office).
J'ai mon action ici :
https://demarches-mkuntz.test.entrouvert.org/backoffice/workflows/174/status/1/items/1/
Pour "Utilisateur associé à la fiche", je sélectionne "Gabarit personnalisé" et j'indique {{ session_user_id }}
.
Pour tester, je vais là : https://agents-mkuntz.test.entrouvert.org/annuaire/, je clique sur le bouton "Ajouter un favori" (à droite), je sélectionne un nom (par ex. prat, gros, legrand...). Je ne choisis pas d'usager associé à droite.
Je valide et j'ai une erreur 500 sur l'url https://demarches-mkuntz.test.entrouvert.org/backoffice/submission/ajouter-un-favori/
La fiche n'a pas été enregistrée (par ex. https://demarches-mkuntz.test.entrouvert.org/backoffice/management/ajouter-un-favori/4/).
Ça marche si j'indique {{session_user_email}}
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a plus de 2 ans
C'est https://sentry.entrouvert.org/entrouvert/publik/issues/54263/
TypeError: argument of type 'int' is not iterable ... File "wcs/users.py", line 245, in lookup_by_string if '@' in user_value:
parce que {{ session_user_id }} est un nombre entier. Il faut éviter le 500 là dessus et remonter l'erreur normalement.
Ceci étant, ça ne marchera pas avec session_user_id, la doc de l'action précise qu'il faut indiquer l'utilisateur « via son adresse électronique ou son UUID ». Donc ça sera {{ session_user_email }} ou {{ session_user_nameid }} (plus "sûr" avec nameid, aucun risque de doublon)
Mis à jour par Thomas Noël il y a plus de 2 ans
Et mieux encore : {{ session_user }} (parce que si on envoie objet User, c'est encore plus direct)
Mis à jour par Thomas Noël il y a plus de 2 ans
- Fichier 0001-wf-don-t-crash-on-bad-user-type-in-form-data-creatio.patch 0001-wf-don-t-crash-on-bad-user-type-in-form-data-creatio.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Lauréline Guérin il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 1b578af73fb9c34227eda664f2823f07aedf956d Author: Thomas Noël <tnoel@entrouvert.com> Date: Wed Jan 12 13:21:10 2022 +0100 wf: don't crash on bad user type in form/data creation action (#60554)
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
wf: don't crash on bad user type in form/data creation action (#60554)