Skip to content
This repository has been archived by the owner on Aug 27, 2021. It is now read-only.

enabling "Display as Cluster" of uploaded geojson causes Save of site to fail #224

Open
NicoledeGreef opened this issue Aug 2, 2018 · 4 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@NicoledeGreef
Copy link
Collaborator

NicoledeGreef commented Aug 2, 2018

How to reproduce

-Access: http://smk.pathfinder.gov.bc.ca/smk-ui/
-Create a site
-Upload geojson file (ZIP attached below)
-Turn on "Display as Cluster" for the uploaded layer
-Click Done Edits
-Click Save

What behaviour I'm expecting and which behaviour I'm seeing

Expecting the site to Save and the version to increase.

Seeing an error:
Error saving application alrc-draft-4. Error:
HTTP Status 400 -
type Status report

message
description The request sent by the client was syntactically incorrect.
Apache Tomcat/8.5.3

Minimal example reproducing the issue

The site called "ALRC draft 1" previously worked (July 18) but then my Save attempts started not working (July 30) .

"ALRC draft 4" is a new site with the same uploaded data that was used to track down the issue.

@NicoledeGreef
Copy link
Collaborator Author

NicoledeGreef commented Aug 2, 2018

The geojson is in the attached ZIP file.

assisted-living-and-residential-care-facilities.zip

@NicoledeGreef NicoledeGreef added the bug Something isn't working label Aug 2, 2018
@NicoledeGreef NicoledeGreef added this to the MVP milestone Aug 2, 2018
@dhlevi
Copy link
Collaborator

dhlevi commented Aug 5, 2018

@NicoledeGreef I just gave the json file a test in our local as well as the deployed version in dev, and it worked fine in both places. Just to check... you're not uploading the zip file, just the geojson file within, correct? zip files are reserved for shapefile only, and attempting to load a zip that contains geojson (or anything else) will fail.

@dhlevi
Copy link
Collaborator

dhlevi commented Aug 5, 2018

Nope, scratch that, there is a problem occuring from the jackson parsing in java from the main mapconfiguration class that looks like it can cause problems with geojson files. I'm fixing it now and will check in a patch asap

@NicoledeGreef
Copy link
Collaborator Author

Fix verified. Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants