ForeignKeyAutoComplete -> admin.ModelAdmin #22

New Issue

Fred · 2022-01-09T20:17:30+01:00

Fred commented

2022-01-09 20:17:30 +01:00

Dans le fichier ultron/objective/admin.py, tu utilises une classe ForeignKeyAutoComplete, mais j'ai l'impression qu'elle ne sert à rien (ou en tout cas, que tu n'utilises aucune fonctionnalité qui ne soit déjà présente dans le framework...).

Actuellement, cela donne ceci:

Et en modifiant légèrement le code, cela donne ceci:

Du coup, si ce n'est vraiment pas nécessaire, cela pourrait faire sauter la dépendance à django_extensions 😜 (et une dépendance en moins est toujours une victoire).

Pour la modification:

diff --git a/ultron/followup/admin.py b/ultron/followup/admin.py
index a4b861c..8b89a69 100644
--- a/ultron/followup/admin.py
+++ b/ultron/followup/admin.py
@@ -15,15 +15,12 @@ from .models import (
 )
 
 
-class ChronoAdmin(ForeignKeyAutocompleteAdmin):
+class ChronoAdmin(admin.ModelAdmin):
     model = Chrono
 
     list_display = ("date", "gymnast", "tof", "chrono_type")
     list_filter = ("chrono_type",)
-    # search_fields = ('gymnast', 'routine')
-    autocomplete_fields = ["gymnast"]
-
-    related_search_fields = {"gymnast": ("last_name", "first_name")}
+    autocomplete_fields = ('gymnast',)

(+ deux-trois autres endroits où tu en hérites, mais sans utiliser ses fonctionnalités non plus).

Dans le fichier `ultron/objective/admin.py`, tu utilises une classe `ForeignKeyAutoComplete`, mais j'ai l'impression qu'elle ne sert à rien (ou en tout cas, que tu n'utilises aucune fonctionnalité qui ne soit déjà présente dans le framework...). Actuellement, cela donne ceci: ![image](/attachments/d4e8c3c1-ec62-4752-904e-45ae334340ea) Et en modifiant légèrement le code, cela donne ceci: ![image](/attachments/72a1c72d-e728-4f78-9d73-b77c31ad136d) Du coup, si ce n'est vraiment pas nécessaire, cela pourrait faire sauter la dépendance à `django_extensions` 😜 (et une dépendance en moins est toujours une victoire). Pour la modification: ```diff diff --git a/ultron/followup/admin.py b/ultron/followup/admin.py index a4b861c..8b89a69 100644 --- a/ultron/followup/admin.py +++ b/ultron/followup/admin.py @@ -15,15 +15,12 @@ from .models import ( ) -class ChronoAdmin(ForeignKeyAutocompleteAdmin): +class ChronoAdmin(admin.ModelAdmin): model = Chrono list_display = ("date", "gymnast", "tof", "chrono_type") list_filter = ("chrono_type",) - # search_fields = ('gymnast', 'routine') - autocomplete_fields = ["gymnast"] - - related_search_fields = {"gymnast": ("last_name", "first_name")} + autocomplete_fields = ('gymnast',) ``` (+ deux-trois autres endroits où tu en hérites, mais sans utiliser ses fonctionnalités non plus).

image.png

19 KiB

image.png

23 KiB

Sulley was assigned by Fred

2022-01-09 20:17:30 +01:00

Sulley commented

2022-01-31 16:54:41 +01:00

Je l'ai fait mais j'avais oublié de fermé le ticket.

Sulley closed this issue

2022-01-31 16:54:41 +01:00

Sign in to join this conversation.