Produits Entr'ouvert » Authentic 2

Bon j'ai du mal à comprendre.

Benjamin Dauvergne a écrit :

Il suffit de renommer le champ pour que le modèle ne soit pas modifié par le ModelForm.save() standard.

Elle faisait rien d'autre que return self.instance cette méthode, je pige pas.

C'est le hook manager-change-email-request qui à la charge de faire persister quelque part les changements ? (ce serait le rôle de UserChangeEmailForm.save dans une application lambda ?)
Pourquoi fields devient vide ?

Désolé je ne suis pas précis, l'application des modifications des champs au modèle est faite pendant la validation et pas dans le save() le fait d'utiliser un champ nommé différemment évite ça, mais même si save() ne sauve pas le modèle il y a un effet de bord qui pourrait surprendre dans le futur et si par exemple un hook décide de faire une modification au user et un .save() l'effet de bord se retrouve sauvé en base. Ha et aussi comme cet effet de bord avait l'effet que c'est la nouvelle email qui se trouvait dans user.email et donc send_email_change_email mettait deux fois la même email dans son texte (en voulant citer l'ancienne email il remettait la nouvelle). Et puis surcharger save pour juste faire return self.instance c'est super moche. Là j'ai le comportement métier qui est bien encapsulé dans le Form et simplement le hook dans la vue (parce qu'on veut pouvoir passer plus de contexte au hook, request, view, etc..).

Ok, j'ai compris, à peu près, ack.
Pour ma culture quand même, t'aurais pas pu t'en sortir avec ça : https://docs.djangoproject.com/en/1.11/ref/forms/api/#django.forms.Form.has_changed ?
Et pourquoi tu as besoin de rendre fields vide ?

J'ai une contrainte que je m'impose tout seul: utiliser une vue générique d'édition de modèle et donc avoir une ModelForm.

has_changed() ça ne me sert à rien ici, je ne cherche pas à savoir si un truc a changé ou pas, je veux juste que la ModelForm ne modifie pas le champs email une fois le formulaire validé qui est sont comportement par défaut quand on appelle clean() (ou is_valid() qui fait pareil). À mon avis c'est un défaut de conception le modèle ne devrait pas bouger avant qu'on fasse save(), mais bon c'est comme ça.