Bug #30740
modifier une adresse email "vérifiée" en backoffice ne réinitialise pas son statut
0%
Description
Elle reste "vérifiée" alors que ce n'est plus vrai.
Fichiers
Révisions associées
users api: unset verified flag on a modified email address (#30740)
Historique
Mis à jour par Paul Marillonnet il y a environ 5 ans
- Fichier 0001-manager-unset-verified-flag-on-a-modified-email-addr.patch 0001-manager-unset-verified-flag-on-a-modified-email-addr.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Paul Marillonnet il y a environ 5 ans
- Statut changé de Solution proposée à En cours
Hmm, je réalise maintenant qu'on parle sans doute aussi de la page d'édition du compte par l'usager, ainsi que de l'API.
Mis à jour par Paul Marillonnet il y a environ 5 ans
- Fichier 0001-manager-unset-verified-flag-on-a-modified-email-addr.patch 0001-manager-unset-verified-flag-on-a-modified-email-addr.patch ajouté
- Fichier 0002-profile-view-unset-verified-flag-on-a-modified-email.patch 0002-profile-view-unset-verified-flag-on-a-modified-email.patch ajouté
- Fichier 0003-users-api-unset-verified-flag-on-a-modified-email-ad.patch 0003-users-api-unset-verified-flag-on-a-modified-email-ad.patch ajouté
- Fichier 0004-email-users-api-unset-verified-flag-on-a-modified-em.patch 0004-email-users-api-unset-verified-flag-on-a-modified-em.patch ajouté
- Statut changé de En cours à Solution proposée
Mis à jour par Frédéric Péters il y a environ 5 ans
de la page d'édition du compte par l'usager
Non car sur la page d'édition en elle-même il n'y a pas l'adresse email et sur la page dédiée à au changement d'email, il n'y a pas d'enregistrement immédiat du changement, il y a envoi d'un email pour faire confirmer (ce qui donne donc une adresse vérifiée).
Mis à jour par Paul Marillonnet il y a environ 5 ans
- Fichier 0001-manager-unset-verified-flag-on-a-modified-email-addr.patch 0001-manager-unset-verified-flag-on-a-modified-email-addr.patch ajouté
- Fichier 0002-users-api-unset-verified-flag-on-a-modified-email-ad.patch 0002-users-api-unset-verified-flag-on-a-modified-email-ad.patch ajouté
Oui, c'est vrai, merci. Corrigé ici.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
Paul Marillonnet a écrit :
Oui, c'est vrai, merci. Corrigé ici.
Je ne ferai pas de fixture pour un cas unique, utilise simple_user et pose email_verified dans le test.
Coté API je ferai attention au cas où l'appelant passe 'email_verified' dans son payload.
Je ne ferai pas de instance.save(), ce sera normalement fait plus loin dans super().update()
, on gagne un aller/retour avec la DB.
Sinon c'est tout bon pour moi (faut rebaser avant de pousser ta branche pour de nouveaux tests).
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Solution proposée à En cours
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de En cours à Information nécessaire
Mis à jour par Paul Marillonnet il y a environ 5 ans
- Fichier 0001-manager-unset-verified-flag-on-a-modified-email-addr.patch 0001-manager-unset-verified-flag-on-a-modified-email-addr.patch ajouté
- Fichier 0002-users-api-unset-verified-flag-on-a-modified-email-ad.patch 0002-users-api-unset-verified-flag-on-a-modified-email-ad.patch ajouté
- Statut changé de Information nécessaire à Solution proposée
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
Le 0001 ack.
Pour le 0002 on s'est mal compris, email_verified
ne doit pas passer en read-only
pour l'instant un utilisateur de l'API a parfaitement le droit de poser email_verified=True
, je disais de vérifier que cette possibilité ne disparaissait pas, aussi ton deuxième test a un problème de nommage.
Il faut deux/quatre tests : test_api_users_{create/update}_{with/without}_email_verified
.
Mis à jour par Paul Marillonnet il y a environ 5 ans
- Fichier 0002-users-api-unset-verified-flag-on-a-modified-email-ad.patch 0002-users-api-unset-verified-flag-on-a-modified-email-ad.patch ajouté
Ok, j'avais mal compris.
Corrigé ici.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Solution proposée à Solution validée
Ack.
Mis à jour par Paul Marillonnet il y a environ 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 01d49edb56b6c30378e0d32abcdc822ec8595f2f Author: Paul Marillonnet <pmarillonnet@entrouvert.com> Date: Mon Feb 18 18:22:05 2019 +0100 users api: unset verified flag on a modified email address (#30740) commit 13766a37fdb179d5661f9e055b7b15c442de6675 Author: Paul Marillonnet <pmarillonnet@entrouvert.com> Date: Mon Feb 18 17:05:41 2019 +0100 manager: unset verified flag on a modified email address (#30740)
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
manager: unset verified flag on a modified email address (#30740)