Bug #30629
'<type 'exceptions.UnboundLocalError'>', value = 'local variable 'page' referenced before assignment'
0%
Description
Stack trace (most recent call first): File "/usr/lib/python2.7/dist-packages/wcs/forms/root.py", line 1065, in live 1063 page=page, 1064 displayed_fields=displayed_fields, > 1065 transient_formdata=formdata) 1066 formdata.data.update(self.formdef.get_data(form)) 1067
Fichiers
Révisions associées
root: abort live refresh on unknown page_id (#30629)
Historique
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Fichier 0001-root-define-page-before-the-loop-30629.patch 0001-root-define-page-before-the-loop-30629.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Fichier 0002-root-define-page-before-the-loop-30629.patch 0002-root-define-page-before-the-loop-30629.patch ajouté
- Fichier 0001-tests-non-regression-test-30629.patch 0001-tests-non-regression-test-30629.patch ajouté
avec un test de non regression
Mis à jour par Thomas Noël il y a environ 5 ans
Ça me paraissait un cas anormal, de pas trouver le page_id, et j'aurais presque eu envie de répondre par un brutal « return result_error('bad page_id') » ... mais peut-être me manque-t-il un peu de contexte de l'obtention de la trace.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
Thomas Noël a écrit :
Ça me paraissait un cas anormal, de pas trouver le page_id, et j'aurais presque eu envie de répondre par un brutal « return result_error('bad page_id') » ... mais peut-être me manque-t-il un peu de contexte de l'obtention de la trace.
Ben comme on prévoit déjà le cas de pas avoir du tout de paramètre page_id je me suis dit que ça marcherait, mais bon, on peut faire une erreur propre aussi, c'est le chef qui dira, je ne comprends pas assez le contrat entre la partie cliente et le web-service pour le dire.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Fichier 0002-root-abort-live-refresh-on-unknown-page_id-30629.patch 0002-root-abort-live-refresh-on-unknown-page_id-30629.patch ajouté
- Fichier 0001-tests-non-regression-test-30629.patch 0001-tests-non-regression-test-30629.patch ajouté
En suivant la suggestion de Thomas.
Mis à jour par Frédéric Péters il y a environ 5 ans
Go ainsi sur l'idée qu'on ne trouvera de toute façon pas le cas précis.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Solution proposée à Résolu (à déployer)
commit 0ccb209d28d9a0c3c44e51e5e8527bd4e0fb8d9a Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Wed Feb 13 18:31:54 2019 +0100 root: abort live refresh on unknown page_id (#30629) commit 6d21955c25d8147c05d3f4842b45872e1d7a8e22 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Wed Feb 13 18:37:14 2019 +0100 tests: non regression test (#30629)
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
tests: non regression test (#30629)