Support #53236
journaliser la désactivation d'un compte faite ailleurs que dans backoffice
0%
Description
Cela peut être fait au niveau de la méthode mark_as_inactive.
Demandes liées
Historique
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
Il y a déjà un ticket pour le coté LDAP #52671. Je pense que c'est mieux de ne pas le faire dans mark_as_inactive() mais au point où la désactivation est faite car il y a des détails à noter (notamment la raison). Je disais justement qu'il fallait noter la raison pour que dans le cas LDAP on puisse réactiver le compte automatiquement.
Mis à jour par Serghei Mihai il y a environ 3 ans
Benjamin Dauvergne a écrit :
Il y a déjà un ticket pour le coté LDAP #52671. Je pense que c'est mieux de ne pas le faire dans mark_as_inactive() mais au point où la désactivation est faite car il y a des détails à noter (notamment la raison).
Et cette raison ne pourrait pas être un paramètre de mark_as_inactive ?
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
Si mais je suis de plus en plus frileux des méthodes de modèle à tout faire (un modèle ne devrait pas tapper des logs, faire des requêtes HTTP, etc. une vue ou un cron oui), le jour où on veut marque un truc inactif sans reporter ça dans les logs on devra écrire le code à la main ou on fera une bêtise en remplissant les logs. Maintenant si tu vois un grain à faire comme cela.
Mais pour l'instant il n'y a que 2 usages de cette méthode, le besoin de factorisation ne me parait donc pas imminent :- admin / manager
- ldap
En gros pour moi la règle c'est : à partir de 3 usages effectifs ou qu'on sent imminents on peut réfléchir à factoriser.
Mis à jour par Valentin Deniaud il y a presque 3 ans
- Duplique Development #52671: ldap: journaliser la désactivation des comptes LDAP orphelins ajouté