firebase / geofire-js

GeoFire for JavaScript - Realtime location queries with Firebase
MIT License
1.44k stars 345 forks source link

updateCriteria({radius: 0}) is ignored. #162

Open AndrewKvalheim opened 6 years ago

AndrewKvalheim commented 6 years ago

Noticed that updating the radius of a query to 0 doesn't have any effect on it:

var geoQuery = geoFire.query({center: [1,2], radius: 1000})

expect(geoQuery.radius()).toEqual(1000); /* passes */
geoQuery.updateCriteria({radius: 0});
expect(geoQuery.radius()).toEqual(0); /* fails */