search

imperiumlabs / GeoFirestore-Android

GeoFirestore for Android - Realtime location queries with Firestore
MIT License
123 stars 36 forks source link

Error when updating geoQuery.center #4

Closed brendanosborne closed 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 It looks like in GeoQuery.class > setupQueries() the handle is not being returned successfully from the handles HashMap:

44437711-d0e8e400-a5fe-11e8-89da-7bbb8d1e19bf

tanapthetimid commented 6 years ago

I'm having the same problem

cammace commented 6 years ago

I opened a PR to check this, @theonlynick0430 can you look at it when you get a chance?

dhruvshah1214 commented 6 years ago

Hey guys, thanks so much; sorry, that's our bad!

On Tue, Aug 28, 2018 at 12:56 PM Cameron Mace notifications@github.com wrote:

I opened a PR to check this, @theonlynick0430 https://github.com/theonlynick0430 can you look at it when you get a chance?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/imperiumlabs/GeoFirestore-Android/issues/4#issuecomment-416719622, or mute the thread https://github.com/notifications/unsubscribe-auth/ARQ5aOHbVVLPyFzsbbUPavG7ThoYZGb1ks5uVaCKgaJpZM4WG3w- .

brendanosborne commented 6 years ago

@dhruvshah1214 how can I get the fix? The version still seems to be 1.1.1 ?

Marly1995 commented 6 years ago

@dhruvshah1214 Any news on a new version with this fix included? It seems to have been fixed but is not released.

Supercaly commented 5 years ago

Still getting this error settings the center and the radius.

Edit: The library obtained via gradle have the setters for radius and center, but they didn't work; however if you clone this repo and make your own .aar file it works.