Development #21814
Ne pas envoyer de message quand l'adresse n'est pas changée
100%
Description
De la page de changement d'adresse dans le /manage/ si le formulaire est validé il y a un message qui est envoyé à l'usager, même si l'agent n'a finalement pas modifié l'adresse.
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
Mis à jour par Emmanuel Cazenave il y a environ 6 ans
Ack.
Pas spécifique à ce ticket mais pour info : je sais pas si c'est normal ou pas, mais j'ai lancé uniquement ton nouveau test en mettant un pdb dans get_success_message
et je constate que cette méthode est appelée deux fois.
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
Emmanuel Cazenave a écrit :
Ack.
Pas spécifique à ce ticket mais pour info : je sais pas si c'est normal ou pas, mais j'ai lancé uniquement ton nouveau test en mettant un pdb dansget_success_message
et je constate que cette méthode est appelée deux fois.
Bizarre, d'après le code dans django.contrib.messages.views.SuccessMessageMixing
ça n'est appelé que dans form_valid() et donc sur un POST, tu n'as pas pensé à regarder la pile pour voir d'où venait l'appel ?
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
Et donc j'ai testé moi même, c'est parce que le test s'exécute deux fois à cause d'une fixture paramétrique:
tests/test_user_manager.py::test_manager_user_change_email_no_change[superuser] PASSED tests/test_user_manager.py::test_manager_user_change_email_no_change[admin] PASSED
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
- Statut changé de Nouveau à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit authentic2|d645e3ee27d782e1296f33846bb639957668dc39.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Fermé
manager: don't show a message if email is not changed (fixes #21814)