Development #69809
pre-commit-hooks, ajouter django-upgrade ?
0%
Description
Est-ce utile vu le peu de Django ?
Fichiers
Demandes liées
Révisions associées
misc: apply django-upgrade (#69809)
misc: add django-upgrade files/notes (#69809)
Historique
Mis à jour par Valentin Deniaud il y a plus d'un an
- Fichier 0002-misc-add-django-upgrade-files-notes-69798.patch 0002-misc-add-django-upgrade-files-notes-69798.patch ajouté
- Fichier 0001-misc-apply-django-upgrade-69798.patch 0001-misc-apply-django-upgrade-69798.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
En tout cas ça tourne sans problème.
Mis à jour par Valentin Deniaud il y a plus d'un an
- Lié à Development #69798: pre-commit-hooks, utiliser django-upgrade partout ajouté
Mis à jour par Valentin Deniaud il y a plus d'un an
Build rouge parce que django-upgrade vient modifier request.META au profit de request.headers.
Note, il ne remplace que deux occurrences de request.META alors qu'il y en a cinq en tout.
Ça nécessiterait simplement le patch
--- a/wcs/qommon/http_request.py +++ b/wcs/qommon/http_request.py @@ -240,0 +241,4 @@ class HTTPRequest(quixote.http_request.HTTPRequest): @property def META(self): return self.environ + @property + def headers(self): + return self.environ
Mais encore une fois, pas sûr qu'on veuille se plier aux injonctions de django-upgrade dans une brique pas Django.
Mis à jour par Thomas Noël il y a plus d'un an
Valentin Deniaud a écrit :
pas sûr qu'on veuille se plier aux injonctions de django-upgrade dans une brique pas Django.
Mon avis : bien que w.c.s. n'utilise pas totalement Django, quand il est utilisé ça doit l'être correctement, et ces moments sont destinés à être de plus en plus fréquents selon Le Grand Plan de Migration. Je serais donc pour ajouter django-upgrade. Tu peux "finir" le patch ?
Mis à jour par Valentin Deniaud il y a plus d'un an
- Fichier 0002-misc-apply-django-upgrade-69798.patch 0002-misc-apply-django-upgrade-69798.patch ajouté
- Fichier 0003-misc-add-django-upgrade-files-notes-69798.patch 0003-misc-add-django-upgrade-files-notes-69798.patch ajouté
- Fichier 0001-misc-allow-new-request.headers-usage-69809.patch 0001-misc-allow-new-request.headers-usage-69809.patch ajouté
OK, je reprends.
Valentin Deniaud a écrit :
Note, il ne remplace que deux occurrences de request.META alors qu'il y en a cinq en tout.
Renseignements pris request.META n'est pas déprécié, il y a juste request.headers qui est introduit en Django 2 comme une simplification de certains usages de request.META, c'est donc normal qu'il en reste qui ne soient pas remplacés.
J'ajoute 0001 qui permet request.headers (ce code est testé par test_forms_new_popup
dans admin_pages/test_form.py).
Mis à jour par Thomas Noël il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Mis à jour par Valentin Deniaud il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 7ba5ff18f5390fa339653a4b2615032f6a5e424a Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Mon Oct 3 14:41:32 2022 +0200 misc: add django-upgrade files/notes (#69809) commit 77ad58bf8f16303d19d9f16352bb6ff8ca6d0e98 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Mon Oct 3 17:53:32 2022 +0200 misc: apply django-upgrade (#69809) commit 0635869a0c566a9941baf99a4d2aba58bd934acd Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Mon Oct 3 17:53:22 2022 +0200 misc: allow new request.headers usage (#69809)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
misc: allow new request.headers usage (#69809)