entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 15 Releases Activity

litteralis: raise API error on bad coordinates (#85310) #445

Merged
nroche merged 1 commits from wip/85310-litteralis-parse-geom into main 2024-01-19 17:10:11 +01:00
Conversation 0 Commits 1 Files Changed 3 +96 -5
nroche commented 2024-01-19 13:32:38 +01:00
Owner
Copy Link

Avoir une erreur explicite quand on ne renseigne pas de géolocalisation, sur le endpoint demandes-recues.

Avoir une erreur explicite quand on ne renseigne pas de géolocalisation, sur le endpoint demandes-recues.
nroche added 1 commit 2024-01-19 13:32:39 +01:00
nroche force-pushed wip/85310-litteralis-parse-geom from ab547c9bd3 to 25588d534d 2024-01-19 13:33:48 +01:00 Compare
tnoel requested changes 2024-01-19 15:54:10 +01:00
tests/test_litteralis.py Outdated
@ -110,0 +125,4 @@
),
({'coordinates': None}, "geom/coordinates: None is not of type 'array'"),
({'coordinates': []}, 'geom/coordinates: [] is too short'),
({'coordinates': ['48.866667', '']}, "'' is not of type 'number'"),
tnoel commented 2024-01-19 15:54:06 +01:00
Owner
Copy Link

Ici on voit que l'erreur ne va pas être claire. On pourrait accepter n'importe quelle string dans le schéma JSON, et lever une exception APIError avec code 400 lors du float(), avec un message préfixé par "geom/coordinates"

Ici on voit que l'erreur ne va pas être claire. On pourrait accepter n'importe quelle string dans le schéma JSON, et lever une exception APIError avec code 400 lors du float(), avec un message préfixé par "geom/coordinates"
nroche commented 2024-01-19 16:09:25 +01:00
Author
Owner
Copy Link

Fait, avec ce message : "geom/coordinates: '' is not a number".

Fait, avec ce message : "geom/coordinates: '' is not a number".
tnoel marked this conversation as resolved
nroche force-pushed wip/85310-litteralis-parse-geom from 25588d534d to 27c88be4fc 2024-01-19 16:08:31 +01:00 Compare
nroche requested review from tnoel 2024-01-19 16:42:43 +01:00
tnoel requested changes 2024-01-19 16:48:43 +01:00
passerelle/apps/litteralis/models.py Outdated
@ -143,0 +144,4 @@
try:
new_coord.append(float(c))
except ValueError:
raise APIError(_("geom/coordinates: '%s' is not a number") % c, http_status=400)
tnoel commented 2024-01-19 16:48:00 +01:00
Owner
Copy Link

Tatasserie : ajouter un /0 ou /1 après le geom/coordinates, pour savoir quel est l'élément planté

Tatasserie : ajouter un /0 ou /1 après le geom/coordinates, pour savoir quel est l'élément planté
tnoel marked this conversation as resolved
nroche force-pushed wip/85310-litteralis-parse-geom from 27c88be4fc to 8459ef11a3 2024-01-19 16:52:02 +01:00 Compare
tnoel approved these changes 2024-01-19 17:08:30 +01:00
nroche merged commit 8459ef11a3 into main 2024-01-19 17:10:11 +01:00
nroche deleted branch wip/85310-litteralis-parse-geom 2024-01-19 17:10:11 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
tnoel
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/passerelle#445
No description provided.
Delete Branch "wip/85310-litteralis-parse-geom"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?