Admin: ajout du bouton enregistrer en haut du formulaire pour les utilisateurs et 2 autres modèles - #6463
Conversation
xavfernandez
changed the title
| return super().get_object(request, object_id, from_field) | ||
|
|
||
|
|
||
| class ItouModelAdmin(ItouModelMixin, ModelAdmin): |
There was a problem hiding this comment.
J'ai l'impression que ItouModelAdmin et ItouModelMixin sont assez équivalents. Pourquoi as-tu choisi de renommer l'existant pour recréer une méthode du nom d'origine qui en hérite ?
There was a problem hiding this comment.
Prévois tu un remplacement de masse de tous les ItouModelAdmin par ItouModelMixin dans un commit ultérieur ?
There was a problem hiding this comment.
Parce que mon Mixin n'en était pas un dans la première version, c'est maintenant bien le cas.
|
|
||
| @admin.register(Email) | ||
| class EmailAdmin(ReadonlyMixin, admin.ModelAdmin): | ||
| class EmailAdmin(ReadonlyMixin, ItouModelAdmin): |
There was a problem hiding this comment.
Je m'attendais à voir ItouModelMixin ici comme pour QPVAdmin. Est-ce que je loupe un point ?
There was a problem hiding this comment.
admin.ModelAdmin c'est la modèle de base donc ReadonlyMixin, ItouModelMixin, admin.ModelAdmin est équivalent à ReadonlyMixin, ItouModelAdmin, dans les fait on pourrais supprimer ItouModelAdmin et ne prendre que la première forme mais vu que dans 98% des cas on n'a pas besoin d'hériter d'une autre classe que celle par défaut ça évite de trop se prendre la tête.
8e1122a to
429d211
Compare
to allow its reuse on other base classes
429d211 to
043e54d
Compare
|
|
||
| @admin.register(Email) | ||
| class EmailAdmin(ReadonlyMixin, admin.ModelAdmin): | ||
| class EmailAdmin(ReadonlyMixin, ItouModelAdmin): |
There was a problem hiding this comment.
admin.ModelAdmin c'est la modèle de base donc ReadonlyMixin, ItouModelMixin, admin.ModelAdmin est équivalent à ReadonlyMixin, ItouModelAdmin, dans les fait on pourrais supprimer ItouModelAdmin et ne prendre que la première forme mais vu que dans 98% des cas on n'a pas besoin d'hériter d'une autre classe que celle par défaut ça évite de trop se prendre la tête.
3 participants
🤔 Pourquoi ?
Pour homogénéiser l'admin (et faciliter de futur développement: #6259)
🍰 Comment ?
🚨 À vérifier
🏝️ Comment tester ?
💻 Captures d'écran