I find out that both removeObserver and .removeAllObservers() is not working in GeoFire, hoping another work around to "get out" of this observe (which keeps loading forever)
var newRefHandle: FIRDatabaseHandle?
var gFCircleQuery: GFCircleQuery?
func findFUsersInOnePath(location: CLLocation,
radius: Int,
indexPath: String,
completion: @escaping () -> ()){
var ids = 0
let geofireRef = usersRef.child(indexPath)
if let geoFire = GeoFire(firebaseRef: geofireRef) {
gFCircleQuery = geoFire.query(at: location, withRadius: Double(radius))
newRefHandle = gFCircleQuery?.observe(.keyEntered, with: { (key, location) in
// if key fit some condition then perform some other firebase call to other database(async), afterwards:
ids += 1
if (ids >= 3) {
//RemoveObersver not working below...
self.gFCircleQuery?.removeObserver(withFirebaseHandle: self.newRefHandle!)
completion()
}
})
//I find that observeReady gets call right away despite we haven't complete code above
gFCircleQuery?.observeReady({
completion()
})
}
My guess is that you're still getting some
.keyEnteredevents after the call toremoveObserver, which is actually to be expected.GeoFire performs a single query to get all keys within the range for your query, and then calls your
.keyEnteredcallback for each of these keys. Even if you callremoveObserverfor the first key, GeoFire will continue to call the.keyEnteredcallback with the remainder of the keys that it already got from the database.For this reason, I'd recommend removing the observer inside the ready block for the query. The behavior will not change when you do that, but it'll make more sense as that ready block is called when GeoFire is done telling you about the batch of keys.