Wow, great turnaround time!

Quick question as I'm not sure. By adding the addOnFailureListener, should the addOnCompleteListener above it be changed to addOnSuccessListener so that it doesn't still catch failures, or will only the failure listener fire?

I'm not that familiar with the code, but based on the docs, I was expecting addOnCompleteListener was like a finally block. If not, please ignore me.

Thanks!

Thanks so much for proposed solution, is it possible to finalise it so the pull request can be created?

hi @TimHoogstrate the crash points to LocationSettingsResponse lsr = response.getResult(); on FusedLocationClient line 162. The failure handling is applied on LocationServices.getSettingsClient(context).checkLocationSettings(new LocationSettingsRequest.Builder().build()) which seems to be successful since we are reaching the response.getResult() and encountering the crash from there.

I don't think the solution will fix the issue.

attaching also another similar issue with react native, and their approach resolves the issue. However, still it doesn't fix the root cause since they just added a simple try catch. This probably avoid crashes, while waiting for Google to fix it.
https://github.com/Agontuk/react-native-geolocation-service/pull/147/files

The problem is it still runs the code after the if (!response.isSuccessful()) block on failures, leading to checking the result regardless. It should exit the method after calling listener.onLocationServiceError in that block if it was a failure, or wrap the rest in an else block.

If we add the addOnFailureListener as per the PR, then it also needs to change the addOnCompleteListener to addOnSuccessListener, removing the check as it should be redundant, or leave it, but ensure we don't call listener.onLocationServiceError again if it was a failure. Calling it twice returns a result on the method channel twice and that will also fail.

Not sure we'll use it, but the new error code is nice addition so that we can determine it failed to check, rather than default to disabled.