Développement #69809
pre-commit-hooks, ajouter django-upgrade ?
0%
Description
Est-ce utile vu le peu de Django ?
Files
Related issues
Associated revisions
misc: apply django-upgrade (#69809)
misc: add django-upgrade files/notes (#69809)
History
Updated by Valentin Deniaud about 2 years ago
- File 0002-misc-add-django-upgrade-files-notes-69798.patch 0002-misc-add-django-upgrade-files-notes-69798.patch added
- File 0001-misc-apply-django-upgrade-69798.patch 0001-misc-apply-django-upgrade-69798.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
En tout cas ça tourne sans problème.
Updated by Valentin Deniaud about 2 years ago
- Related to Développement #69798: pre-commit-hooks, utiliser django-upgrade partout added
Updated by Valentin Deniaud about 2 years ago
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.
Updated by Thomas Noël about 2 years ago
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 ?
Updated by Valentin Deniaud about 2 years ago
- File 0002-misc-apply-django-upgrade-69798.patch 0002-misc-apply-django-upgrade-69798.patch added
- File 0003-misc-add-django-upgrade-files-notes-69798.patch 0003-misc-add-django-upgrade-files-notes-69798.patch added
- File 0001-misc-allow-new-request.headers-usage-69809.patch 0001-misc-allow-new-request.headers-usage-69809.patch added
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).
Updated by Thomas Noël about 2 years ago
- Status changed from Solution proposée to Solution validée
Updated by Valentin Deniaud about 2 years ago
- Status changed from Solution validée to 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)
Updated by Transition automatique about 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
misc: allow new request.headers usage (#69809)