search

mvexel / overpass-api-python-wrapper

Python bindings for the OpenStreetMap Overpass API
Apache License 2.0
367 stars 90 forks source link

Raising exceptions #15

Closed tbolender closed 9 years ago

tbolender commented 9 years ago

Instead of constructing json error messages exceptions should be thrown to show that something went bad.

mvexel commented 9 years ago

Probably a good idea. Do you have time to address this?

tbolender commented 9 years ago

I can address this (and planned so), but it will at least take me week to have time for this... I thought about introducing different types to be able to distinguish between different errors.

tbolender commented 9 years ago

I finally found time to implement it. I am wondering why the returning of no elements raises an exception. If the elements entry is missing I totally agree, but no elements is from my point of view a valid return.