Project

General

Profile

Development #69809

pre-commit-hooks, ajouter django-upgrade ?

Added by Valentin Deniaud over 1 year ago. Updated over 1 year ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
03 October 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Est-ce utile vu le peu de Django ?


Files


Related issues

Related to Publik - Development #69798: pre-commit-hooks, utiliser django-upgrade partoutFermé03 October 2022

Actions

Associated revisions

Revision 0635869a (diff)
Added by Valentin Deniaud over 1 year ago

misc: allow new request.headers usage (#69809)

Revision 77ad58bf (diff)
Added by Valentin Deniaud over 1 year ago

misc: apply django-upgrade (#69809)

Revision 7ba5ff18 (diff)
Added by Valentin Deniaud over 1 year ago

misc: add django-upgrade files/notes (#69809)

History

#1

Updated by Valentin Deniaud over 1 year ago

En tout cas ça tourne sans problème.

#2

Updated by Valentin Deniaud over 1 year ago

#3

Updated by Valentin Deniaud over 1 year 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.

#4

Updated by Thomas Noël over 1 year 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 ?

#5

Updated by Valentin Deniaud over 1 year ago

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).

#6

Updated by Thomas Noël over 1 year ago

  • Status changed from Solution proposée to Solution validée
#7

Updated by Valentin Deniaud over 1 year 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)
#8

Updated by Transition automatique over 1 year ago

  • Status changed from Résolu (à déployer) to Solution déployée
#9

Updated by Transition automatique about 1 year ago

Automatic expiration

Also available in: Atom PDF