Development #39180
geojson: récupération des données sans geolocalisation
0%
Description
Souci réproduit en recette suite à la solution déployée de #38965:
TypeError at /visualization/23/geojson/ 'NoneType' object is not iterable Request Method: GET Request URL: https://stats-nancy.test.entrouvert.org/visualization/23/geojson/ Django Version: 1.11.20 Exception Type: TypeError Exception Value: 'NoneType' object is not iterable Exception Location: /usr/lib/python2.7/dist-packages/bijoe/visualization/views.py in get, line 275
La requete SQL produite:
'SELECT "quartier".id AS quartier_value, "quartier".label AS quartier_label, array_agg("formdata_eo_signalement_voirie_test_1".geolocation_base) FILTER (WHERE "formdata_eo_signalement_voirie_test_1".geolocation_base IS NOT NULL) AS geolocation FROM "formdata_eo_signalement_voirie_test_1" LEFT OUTER JOIN "formdata_eo_signalement_voirie_test_1_field_quartier" AS "quartier" ON "formdata_eo_signalement_voirie_test_1"."field_quartier" = "quartier".id GROUP BY "quartier".id ORDER BY "quartier".id'
retourne des résultats pour lesquels le filtre (WHERE "formdata_eo_signalement_voirie_test_1".geolocation_base IS NOT NULL) ne s'applique pas:
(1, u'Quartier non trouv\xe9', None) (2, u'SAINT NICOLAS / CHARLES III / VIEILLE VILLE / TROIS MAISONS/ LEOPOLD', [Point(x=6.185051, y=48.69018), Point(x=6.185728, y=48.689654)]) (None, None, None)
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Fichier 0001-visualization-handle-empty-geolocation-measure-39180.patch 0001-visualization-handle-empty-geolocation-measure-39180.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
Comme je ne sais pas comment c'est utilisé je ne saurai dire, ça va faire quoi de renvoyer une liste vide ?
Mis à jour par Serghei Mihai il y a plus de 4 ans
C'est utilisé dans les cellules Map de combo qui les passe à Leaflet pour le rendu, qui à son tour doit ignorer les points sans coordonnées (j'ai pas vérifié).
Mis à jour par Serghei Mihai il y a environ 4 ans
Je suis d'avis de zapper les demandes sans geoloc.
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
Serghei Mihai a écrit :
Je suis d'avis de zapper les demandes sans geoloc.
Pour l'instant je ne préfère pas, savoir qu'il n'y aucun point mais que le classement existe c'est déjà une information, que leaflet n'en fasse rien n'est pas pour moi une bonne raison (il/on pourrait afficher en légende, x, y et z : pas de points).
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
Serghei Mihai a écrit :
Si pas d'autres avis sur la question, go.
Faut valider le ticket.
Mis à jour par Serghei Mihai il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Je laissais le temps pour d'autres opinions.
Go.
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 3b5dd98ad644f728bcd6fda77be05a6517d0c98e
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Wed Jan 22 19:52:48 2020 +0100
visualization: handle empty geolocation measure (#39180)Mis à jour par Valentin Deniaud il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
visualization: handle empty geolocation measure (#39180)