Bug #59815
MultiValueDictKeyError: 'theme'
0%
Description
https://sentry.entrouvert.org/entrouvert/publik/issues/54149/
KeyError: 'theme'
File "django/utils/datastructures.py", line 78, in __getitem__
list_ = super().__getitem__(key)
MultiValueDictKeyError: 'theme'
(5 additional frame(s) were not displayed)
...
File "django/views/generic/base.py", line 97, in dispatch
return handler(request, *args, **kwargs)
File "django/views/generic/base.py", line 205, in post
return self.get(request, *args, **kwargs)
File "django/views/generic/base.py", line 188, in get
url = self.get_redirect_url(*args, **kwargs)
File "hobo/theme/views.py", line 62, in get_redirect_url
set_theme(self.request.POST['theme'])
File "django/utils/datastructures.py", line 80, in __getitem__
raise MultiValueDictKeyError(key)
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Paul Marillonnet il y a plus de 2 ans
Je ne sais pas comment ça a pu survenir, mais toujours est-il que la vue de sélection du thème trace lorsqu’elle reçoit un POST sans dico précisant le thème dans le payload.
Mis à jour par Frédéric Péters il y a plus de 2 ans
Ça arrive quand repost après se reconnecter.
Il y a eu #41079 mais visiblement pas bon/suffisant.
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Lié à Bug #41079: MultiValueDictKeyError: "'theme'" ajouté
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Fichier 0001-misc-check-a-theme-is-selected-before-applying-it-59.patch 0001-misc-check-a-theme-is-selected-before-applying-it-59.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Perdons pas de temps avec ça.
Mis à jour par Paul Marillonnet il y a plus de 2 ans
- Fichier 0001-theme-do-not-be-overly-confident-about-select-post-p.patch 0001-theme-do-not-be-overly-confident-about-select-post-p.patch ajouté
- Statut changé de Solution proposée à Solution validée
La version avec deux lignes de test. Pousse celle que tu veux, peu importe.
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
Je ne suis pas bien sûr que le POST de ton test n'envoie pas de données, j'envoie mon commit pour ne pas me poser la question plus tard.
commit 2c2f779a5c64f336252f09a6a8715db088a19f19
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Thu Dec 16 20:23:16 2021 +0100
misc: check a theme is selected before applying it (#59815)
Mis à jour par Paul Marillonnet il y a plus de 2 ans
Frédéric Péters a écrit :
Je ne suis pas bien sûr que le POST de ton test n'envoie pas de données, j'envoie mon commit pour ne pas me poser la question plus tard.
[...]
D’ac pas de souci (ça traçait bien comme dans sentry, mais le test envoyait peut-être quand même quelque chose, j’ai pas vérifié que le POST était complètement vide).
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: check a theme is selected before applying it (#59815)