Canceling query in callback of query.on('key_entered', callback) results in error about geofire-js HOT 5 CLOSED

Thanks for reporting this to us. It does in fact look like a bug. I think your proposed solution is correct. We actually already have that check for undefined logic here, but just don't have it here. I'll put together a fix, try to get a test for it, and deploy a new version of the lib.

from geofire-js.

Just a note - my possible solution is to just break from the loop rather than continue through the loop and check each key. My reasoning was that if the query is cancelled, none of the key_entered callbacks should be fired anyway. Not sure if that's accurate, but I'm using that for now!

Cheers.

from geofire-js.

Both approaches should work. I put together the initial suggestion which works well enough :)

from geofire-js.

Version 3.2.4 of GeoFire JS is out. Should fix your problem. Let me know if you are still running into issues!

from geofire-js.

Awesome, thanks!

from geofire-js.