Development #40104
dataviz - KeyError: 'data'
Début:
24 février 2020
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
https://sentry.entrouvert.org/entrouvert/publik/issues/596/
KeyError: 'data' File "django/core/handlers/exception.py", line 41, in inner response = get_response(request) File "django/core/handlers/base.py", line 187, in _get_response response = self.process_exception_by_middleware(e, request) File "django/core/handlers/base.py", line 185, in _get_response response = wrapped_callback(request, *callback_args, **callback_kwargs) File "combo/apps/dataviz/views.py", line 42, in dataviz_graph height=int(request.GET.get('height', 0)) or int(cell.height) File "combo/apps/dataviz/models.py", line 199, in get_chart data = response['data'] Internal Server Error: /api/dataviz/graph/1/
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a environ 4 ans
- Fichier 0001-dataviz-handle-missing-visualization-40104.patch 0001-dataviz-handle-missing-visualization-40104.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
C'est étrange que cette trace ne soit pas plus commune, ça arrive à chaque fois qu'une visualisation n'existe plus dans bijoe (facile à reproduire, je l'avais déjà sur de vielles pages en local).
Mis à jour par Frédéric Péters il y a environ 4 ans
except Exception as e: if isinstance(e, UnsupportedDataSet): error_text = _('Unsupported dataset.')
Je préfère des except dédiées plutôt qu'une série de isinstance() (de manière générale, les except larges et les isinstance ne sont pas très python).
Mis à jour par Valentin Deniaud il y a environ 4 ans
- Fichier 0001-dataviz-handle-missing-visualization-40104.patch 0001-dataviz-handle-missing-visualization-40104.patch ajouté
Oui c'est pas beau, mais je n'ai pas l'impression qu'il y ait de bonne manière de partager du code entre le traitement de différentes exceptions. Je trouve des choses à propos de singledispatch mais ça me paraît surtué, donc nouvelle tentative avec un truc plus bateau.
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Oui, bateau is good.
Mis à jour par Valentin Deniaud il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit c11b1b08dd2b4f184a67109c0d7729b910b1956a Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Mon Feb 24 17:10:16 2020 +0100 dataviz: handle missing visualization (#40104)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
dataviz: handle missing visualization (#40104)