agallou
commented
2 years ago Espace membre :
- ajouter une contrainte unique sur le champ email : https://github.com/afup/web/blob/fc75ffda495699bc600094cfb609f90024005c14/sources/AppBundle/Association/Form/ContactDetailsType.php#L35
- ajouter une contrainte unique sur le champ username : https://github.com/afup/web/blob/fc75ffda495699bc600094cfb609f90024005c14/sources/AppBundle/Association/Form/ContactDetailsType.php#L72
Admin :
- ajouter une contrainte unique sur le champ email https://github.com/afup/web/blob/78cd242cf523c7aa5b6713a0d72aec0952b601bb/sources/AppBundle/Association/Form/UserEditType.php#L63
- ajouter une contrainte uniqiue sur le champ login : https://github.com/afup/web/blob/78cd242cf523c7aa5b6713a0d72aec0952b601bb/sources/AppBundle/Association/Form/UserEditType.php#L169
Base :
- Dans la table
afup_personnes_physiques, on a déjà une unicité sur le mail :UNIQUE KEYidx_email_unique(email), - ajouter via une migration une unicité sur le champ
logindeafup_personnes_physiques.
Nous n'avons pas deux fois le mêm login en base donc, il n'y aura pas de rattrapage à faire là dessus :
MariaDB [afup]> select count(*) from afup_personnes_physiques group by login having count(*) > 1;
Empty set (0.00 sec)Adhésion personne physique
- on a déjà une vérification sur l'unicité du login : https://github.com/afup/web/blob/baea20261e4f8b31a699ae30719ad2246f087cae/sources/AppBundle/Controller/LegacyController.php#L140
- ajouter une règle identique sur l'email : https://github.com/afup/web/blob/baea20261e4f8b31a699ae30719ad2246f087cae/sources/AppBundle/Controller/LegacyController.php#L143
Adhésion personne morale
- vérifier que le mail n'existe pas déjà : https://github.com/afup/web/blob/149f660eaa96dde327da2ac0aad14b00bab7f725/sources/AppBundle/Association/Form/CompanyMemberInvitationType.php#L18
Invitation sur personne morale
- ajouter les controles sur l'email/login sur le formulaire de création de user après d'invitation : https://github.com/afup/web/blob/fb1569d4b87a0528d10e93aa5e463e702fd24099/sources/AppBundle/Association/Form/UserType.php#L24
Sur l'espace membre, lorsqu'on modifie son login, nous n'avons pas de notification si la nouvelle valeur est un doublon par rapport aux autres membres.
Dans le BO, il y a bien une vérification par contre, on n'a pas un "joli" message d'erreur sur le formulaire, mais juste un message comme quoi il y eut une erreur. Il faudrait à la fois changer l'unicité en base du login, et améliorer le message dans le BO (et voir les impacts du changement d'unicité sur les autres formulaires comme le formulaire d'inscription)