Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Geojson refactor #24

Closed
wants to merge 3 commits into from
Closed

Geojson refactor #24

wants to merge 3 commits into from

Conversation

dlutz2
Copy link

@dlutz2 dlutz2 commented Jul 27, 2020 •
edited
Loading

Adjusted the logic in the GeoJSONReader to avoid NPEs and to collect all properties from all features. Resolves #22

@dlutz2
Copy link
Author

dlutz2 commented Jul 27, 2020

I'll drop this PR. I'll rework so there is not so much formatting change.
Je vais laisser tomber ce PR. Je vais retravailler pour qu'il n'y ait pas tellement de changement de formatage. (Google Translate)

@dlutz2 dlutz2 closed this Jul 27, 2020
@dlutz2 dlutz2 deleted the geojson_refactor branch July 27, 2020 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Null pointer exception when property value is null or empty object in GeoJSON file
1 participant
@dlutz2