Projet

Général

Profil

Bug #30740

modifier une adresse email "vérifiée" en backoffice ne réinitialise pas son statut

Ajouté par Frédéric Péters il y a environ 5 ans. Mis à jour il y a environ 5 ans.

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Elle reste "vérifiée" alors que ce n'est plus vrai.


Fichiers

0001-manager-unset-verified-flag-on-a-modified-email-addr.patch (2,66 ko) 0001-manager-unset-verified-flag-on-a-modified-email-addr.patch Paul Marillonnet, 18 février 2019 17:08
0001-manager-unset-verified-flag-on-a-modified-email-addr.patch (2,66 ko) 0001-manager-unset-verified-flag-on-a-modified-email-addr.patch Paul Marillonnet, 18 février 2019 18:31
0002-profile-view-unset-verified-flag-on-a-modified-email.patch (1,87 ko) 0002-profile-view-unset-verified-flag-on-a-modified-email.patch Paul Marillonnet, 18 février 2019 18:31
0003-users-api-unset-verified-flag-on-a-modified-email-ad.patch (2,61 ko) 0003-users-api-unset-verified-flag-on-a-modified-email-ad.patch Paul Marillonnet, 18 février 2019 18:31
0004-email-users-api-unset-verified-flag-on-a-modified-em.patch (2 ko) 0004-email-users-api-unset-verified-flag-on-a-modified-em.patch Paul Marillonnet, 18 février 2019 18:31
0001-manager-unset-verified-flag-on-a-modified-email-addr.patch (2,66 ko) 0001-manager-unset-verified-flag-on-a-modified-email-addr.patch Paul Marillonnet, 19 février 2019 10:15
0002-users-api-unset-verified-flag-on-a-modified-email-ad.patch (3,62 ko) 0002-users-api-unset-verified-flag-on-a-modified-email-ad.patch Paul Marillonnet, 19 février 2019 10:15
0001-manager-unset-verified-flag-on-a-modified-email-addr.patch (2,08 ko) 0001-manager-unset-verified-flag-on-a-modified-email-addr.patch Paul Marillonnet, 07 mars 2019 13:41
0002-users-api-unset-verified-flag-on-a-modified-email-ad.patch (3,88 ko) 0002-users-api-unset-verified-flag-on-a-modified-email-ad.patch Paul Marillonnet, 07 mars 2019 13:41
0002-users-api-unset-verified-flag-on-a-modified-email-ad.patch (5,81 ko) 0002-users-api-unset-verified-flag-on-a-modified-email-ad.patch Paul Marillonnet, 11 mars 2019 18:10

Révisions associées

Révision 13766a37 (diff)
Ajouté par Paul Marillonnet il y a environ 5 ans

manager: unset verified flag on a modified email address (#30740)

Révision 01d49edb (diff)
Ajouté par Paul Marillonnet il y a environ 5 ans

users api: unset verified flag on a modified email address (#30740)

Historique

#1

Mis à jour par Paul Marillonnet il y a environ 5 ans

#2

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.

#4

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

#6

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

#7

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

  • Assigné à mis à Paul Marillonnet
#8

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

  • Statut changé de Solution proposée à En cours
#9

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

  • Statut changé de En cours à Information nécessaire
#11

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.

#13

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

  • Statut changé de Solution proposée à Solution validée

Ack.

#14

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)

#15

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

Formats disponibles : Atom PDF