search

imperiumlabs / GeoFirestore

GeoFirestore — Realtime location queries with Firestore
MIT License
66 stars 6 forks source link

Error when updating geoQuery.center #1

Open brendanosborne opened 6 years ago

brendanosborne commented 6 years ago

Hi Guys, I can't work this one out...

This works okay if the lat and lng don't change: geoQuery?.center = newLatLng

If the lat and lng are different I am getting this error:

java.lang.NullPointerException: Attempt to read from field 'com.google.firebase.firestore.ListenerRegistration org.imperiumlabs.geofirestore.GeoQuery$GeoHashQueryListener.childAddedListener' on a null object reference

brendanosborne commented 6 years ago

It looks like in GeoQuery.class > setupQueries() the handle is not being returned successfully from the handles HashMap:

image

gustavjon commented 5 years ago

I'm exeriancing this problem on 'com.github.imperiumlabs:GeoFirestore-Android:v1.1.1' Why was this issue closed?

brendanosborne commented 5 years ago

What happened to the replies to this thread? Someone resolved the issue and I asked when it would be added to the next build...

gustavjon commented 5 years ago

This issue is fixed on master so using a snapshot build solves the problem. I'm not really comfortable using snapshot builds though so a proper release with this fix would be very welcome

brendanosborne commented 5 years ago

Yes a proper release would be appreciated.

On Fri., 30 Nov. 2018, 8:59 am gustavjon <notifications@github.com wrote:

This issue is fixed on master so using a snapshot build solves the problem. I'm not really comfortable using snapshot builds though so a proper release with this fix would be very welcome

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/imperiumlabs/GeoFirestore/issues/1#issuecomment-443025826, or mute the thread https://github.com/notifications/unsubscribe-auth/APJdNzGvmZQ8GETik9Tf8q3w3uxARHefks5u0GbOgaJpZM4WG0-w .