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.

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

Box.ofWSEN silently accepts latitude values outside the valid range #86

Open
tordanik opened this issue Mar 6, 2023 · 0 comments
Open
Assignees
Labels
todo A feature that has been specified, but not yet implemented

Comments

@tordanik
Copy link

tordanik commented Mar 6, 2023

Geodesk silently accepts Box.ofWSEN calls with nonsensical latitude values. Example:

Box.ofWSEN(-10, -100, 10, 100);

I would prefer to receive an IllegalArgumentException. Instead, the invalid parameter is just set to 0. In this example, this results in an empty bbox. In general, this can make the error harder to find (in my case, it was swapped latitude and longitude values).

@clarisma clarisma added the todo A feature that has been specified, but not yet implemented label Mar 16, 2023
@clarisma clarisma self-assigned this Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
todo A feature that has been specified, but not yet implemented
Projects
None yet
Development

No branches or pull requests

2 participants