search

stephanekirsch / e-colle

Application web de gestion de colles en CPGE basée sur Python3/Django
GNU Affero General Public License v3.0
10 stars 16 forks source link

bug : problème recherche colleur par matière ET par classe #35

Closed alaincaignot closed 4 years ago

alaincaignot commented 4 years ago

Bonjour, La recherche par matière fonctionne, par classe aussi mais si je couple les deux le résultat est incorrect. Merci d'avance

stephanekirsch commented 4 years ago

effectivement, je vais enquêter sur le problème...

stephanekirsch commented 4 years ago

Après enquête il apparaît évident que lorsqu'on précise les deux (classe + matière), lors de la requête SQL, django intervertit l'ordre des paramètres: il met l'id de la matière au lieu de celui de la classe et inversement... du coup le résultat obtenu est effectivement absurde. Ce que je ne comprends pas c'est pourquoi cela arrive... a priori mon code est bon (de toute façon je ne peux filter les classes que sur une valeur de classe et idem pour les matières, sinon mon code planterait). Je vais voir quand j'aurai un peu plus de temps comment contourner ce problème. Le soucis c'est qu'il faut obligatoirement un queryset, pas possible de contourner le problème avec une requête brute..

stephanekirsch commented 4 years ago

bon le problème ne venait pas de Django! C'est moi qui intervertissait à un endroit du code les 2! C'est réglé!