Development #25502
road to django 1.11 : éviter les lookup 'in' implicites
0%
Description
Quelques endroits où l'on se repose sur une fonctionnalité qui disparaît en 1.9 :
https://docs.djangoproject.com/en/2.0/releases/1.9/#implicit-queryset-in-lookup-removed
https://code.djangoproject.com/ticket/25284
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Lié à Development #21489: Fonctionner avec Django 1.11 (et 1.8) ajouté
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Sujet changé de road to django 1.11 : éviter les lookup 'in' implicite à road to django 1.11 : éviter les lookup 'in' implicites
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
Et je note quelques occurrences :
/src/authentic2/attributes_ng/sources/service_roles.py::44
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Fichier 0001-use-explicit-in-lookup-25502.patch 0001-use-explicit-in-lookup-25502.patch ajouté
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
Avec une suppression de fonction non utilisée qui utilise un in lookup implicite.
Mis à jour par Anonyme il y a plus de 5 ans
Emmanuel Cazenave a écrit :
Avec une suppression de fonction non utilisée qui utilise un in lookup implicite.
Je me demande si le filter de la ligne 153 de src/authentic2/custom_user/models.py
rp_qs = RoleParenting.objects.filter(child=qs1)
n'est pas un in implicite que tu souhaite supprimer ici. mais je suis incapable de dire s'il est nécessaire de changer ce bout de code...
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
Je ne sais pas non plus, l'exemple dans le changelog de Django est <fk>_id=queryset
ici c'est <fk>=queryset
, je ne sais pas si le problème est le même, il faudrait tester sur un exemple en Django 1.11.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
Benjamin Dauvergne a écrit :
Je ne sais pas non plus, l'exemple dans le changelog de Django est
<fk>_id=queryset
ici c'est<fk>=queryset
, je ne sais pas si le problème est le même, il faudrait tester sur un exemple en Django 1.11.
Où alors corriger de toute façon, avec __in
ça marchera toujours.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
Good catch, new patch.
Mis à jour par Anonyme il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Emmanuel Cazenave a écrit :
Good catch, new patch.
Tests OK, patch acké
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit d02451702543ba2b44b4f25ab2b104c88aa4ec80 Author: Emmanuel Cazenave <ecazenave@entrouvert.com> Date: Fri Jul 27 15:37:45 2018 +0200 use explicit in lookup (#25502) Delete unused function using implicit in lookup.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Lié à Development #25557: road to django 1.11 : encore un 'in' lookup ajouté
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Solution déployée à Fermé
use explicit in lookup (#25502)
Delete unused function using implicit in lookup.