Merge branch '172-implement-missing-locationlistener-callbacks' into 'master'

Resolve "Implement missing LocationListener callbacks"

Closes #172

See merge request pace/mobile/android/pace-cloud-sdk!168

Author: Martin Dinh

CHANGELOG.md

@@ -17,6 +17,7 @@ x.y.z Release notes (yyyy-MM-dd)
 * Regenerate communication code to fix problem with unparsable error response
 * Fix overwriting of custom set `utm_params`
 * Fix bug where `AppActivity` was finished when it was started and `AppKit.openApps(...)` was called at the same time
+* Overwrite  `LocationListener` callback functions to avoid problems/crashes for devices with Android version smaller than 11
 
 ### Internal
 

library/src/main/java/cloud/pace/sdk/utils/LocationProvider.kt

@@ -8,13 +8,13 @@ import android.location.Location
 import android.location.LocationListener
 import android.location.LocationManager
 import android.os.Build
+import android.os.Bundle
 import android.os.Looper
 import androidx.core.content.ContextCompat
 import androidx.lifecycle.LiveData
 import androidx.lifecycle.MutableLiveData
 import androidx.lifecycle.distinctUntilChanged
 import cloud.pace.sdk.PACECloudSDK
-import com.google.android.gms.location.LocationAvailability
 import com.google.android.gms.location.LocationCallback
 import com.google.android.gms.location.LocationRequest
 import com.google.android.gms.location.LocationResult
@@ -155,6 +155,18 @@ class LocationProviderImpl(
                                             continuation.resumeIfActive(it)
                                         }
                                     }
+
+                                    override fun onProviderDisabled(provider: String) {
+                                        continuation.resumeWithExceptionIfActive(NoLocationFound)
+                                    }
+
+                                    override fun onProviderEnabled(provider: String) {
+                                        continuation.resumeWithExceptionIfActive(NoLocationFound)
+                                    }
+
+                                    override fun onStatusChanged(provider: String?, status: Int, extras: Bundle?) {
+                                        continuation.resumeWithExceptionIfActive(NoLocationFound)
+                                    }
                                 }
                                 locationManager?.requestLocationUpdates(provider, LOCATION_REQUEST_INTERVAL, LOCATION_REQUEST_SMALLEST_DISPLACEMENT, listener, Looper.getMainLooper())
                                 continuation.invokeOnCancellation {
@@ -234,6 +246,18 @@ class LocationProviderImpl(
                                             locationManager?.removeUpdates(this)
                                             continuation.resumeIfActive(if (validate) getLocationIfValid(location, LOW_ACCURACY, startTime) else location)
                                         }
+
+                                        override fun onProviderDisabled(provider: String) {
+                                            continuation.resumeWithExceptionIfActive(NoLocationFound)
+                                        }
+
+                                        override fun onProviderEnabled(provider: String) {
+                                            continuation.resumeWithExceptionIfActive(NoLocationFound)
+                                        }
+
+                                        override fun onStatusChanged(provider: String?, status: Int, extras: Bundle?) {
+                                            continuation.resumeWithExceptionIfActive(NoLocationFound)
+                                        }
                                     }
                                     locationManager?.requestSingleUpdate(provider, listener, Looper.getMainLooper())
                                     continuation.invokeOnCancellation {