react-native-google-maps-plus 1.4.1 → 1.5.0

package/android/src/main/java/com/rngooglemapsplus/GoogleMapsViewImpl.kt

@@ -2,7 +2,6 @@ package com.rngooglemapsplus
 
 import android.annotation.SuppressLint
 import android.graphics.Bitmap
-import android.location.Location
 import android.util.Base64
 import android.util.Size
 import android.widget.FrameLayout
@@ -11,7 +10,6 @@ import com.facebook.react.bridge.LifecycleEventListener
 import com.facebook.react.bridge.UiThreadUtil
 import com.facebook.react.uimanager.PixelUtil.dpToPx
 import com.facebook.react.uimanager.ThemedReactContext
-import com.google.android.gms.common.ConnectionResult
 import com.google.android.gms.maps.CameraUpdateFactory
 import com.google.android.gms.maps.GoogleMap
 import com.google.android.gms.maps.GoogleMapOptions
@@ -39,7 +37,11 @@ import com.rngooglemapsplus.extensions.toLatLng
 import com.rngooglemapsplus.extensions.toLocationErrorCode
 import com.rngooglemapsplus.extensions.toRNIndoorBuilding
 import com.rngooglemapsplus.extensions.toRNIndoorLevel
+import com.rngooglemapsplus.extensions.toRNMapErrorCodeOrNull
+import com.rngooglemapsplus.extensions.toRnCamera
 import com.rngooglemapsplus.extensions.toRnLatLng
+import com.rngooglemapsplus.extensions.toRnLocation
+import com.rngooglemapsplus.extensions.toRnRegion
 import java.io.ByteArrayInputStream
 import java.io.ByteArrayOutputStream
 import java.io.File
@@ -65,6 +67,7 @@ class GoogleMapsViewImpl(
   LifecycleEventListener {
   private var initialized = false
   private var mapReady = false
+  private var destroyed = false
   private var googleMap: GoogleMap? = null
   private var mapView: MapView? = null
 
@@ -83,7 +86,6 @@ class GoogleMapsViewImpl(
   private val kmlLayersById = mutableMapOf<String, KmlLayer>()
 
   private var cameraMoveReason = -1
-  private var lastSubmittedLocation: Location? = null
   private var lastSubmittedCameraPosition: CameraPosition? = null
 
   init {
@@ -94,31 +96,16 @@ class GoogleMapsViewImpl(
     if (initialized) return
     initialized = true
     val result = playServiceHandler.playServicesAvailability()
+    val errorCode = result.toRNMapErrorCodeOrNull()
 
-    when (result) {
-      ConnectionResult.SERVICE_MISSING -> {
-        onMapError?.invoke(RNMapErrorCode.PLAY_SERVICES_MISSING)
-        return
-      }
+    if (errorCode != null) {
+      onMapError?.invoke(errorCode)
 
-      ConnectionResult.SERVICE_INVALID -> {
-        onMapError?.invoke(RNMapErrorCode.PLAY_SERVICES_INVALID)
+      if (errorCode == RNMapErrorCode.PLAY_SERVICES_MISSING ||
+        errorCode == RNMapErrorCode.PLAY_SERVICES_INVALID
+      ) {
         return
       }
-
-      ConnectionResult.SERVICE_VERSION_UPDATE_REQUIRED ->
-        onMapError?.invoke(RNMapErrorCode.PLAY_SERVICES_OUTDATED)
-
-      ConnectionResult.SERVICE_UPDATING ->
-        onMapError?.invoke(RNMapErrorCode.PLAY_SERVICE_UPDATING)
-
-      ConnectionResult.SERVICE_DISABLED ->
-        onMapError?.invoke(RNMapErrorCode.PLAY_SERVICES_DISABLED)
-
-      ConnectionResult.SUCCESS -> {}
-
-      else ->
-        onMapError?.invoke(RNMapErrorCode.UNKNOWN)
     }
 
     mapView =
@@ -143,8 +130,8 @@ class GoogleMapsViewImpl(
         googleMap?.setOnMapClickListener(this@GoogleMapsViewImpl)
         googleMap?.setOnMarkerDragListener(this@GoogleMapsViewImpl)
       }
+      applyProps()
       initLocationCallbacks()
-      applyPending()
       mapReady = true
       onMapReady?.invoke(true)
     }
@@ -160,21 +147,9 @@ class GoogleMapsViewImpl(
     }
     val isGesture = GoogleMap.OnCameraMoveStartedListener.REASON_GESTURE == reason
 
-    val latDelta = bounds.northeast.latitude - bounds.southwest.latitude
-    val lngDelta = bounds.northeast.longitude - bounds.southwest.longitude
-
     onCameraChangeStart?.invoke(
-      RNRegion(
-        center = bounds.center.toRnLatLng(),
-        latitudeDelta = latDelta,
-        longitudeDelta = lngDelta,
-      ),
-      RNCamera(
-        center = cameraPosition.target.toRnLatLng(),
-        zoom = cameraPosition.zoom.toDouble(),
-        bearing = cameraPosition.bearing.toDouble(),
-        tilt = cameraPosition.tilt.toDouble(),
-      ),
+      bounds.toRnRegion(),
+      cameraPosition.toRnCamera(),
       isGesture,
     )
   }
@@ -192,21 +167,9 @@ class GoogleMapsViewImpl(
 
     val isGesture = GoogleMap.OnCameraMoveStartedListener.REASON_GESTURE == cameraMoveReason
 
-    val latDelta = bounds.northeast.latitude - bounds.southwest.latitude
-    val lngDelta = bounds.northeast.longitude - bounds.southwest.longitude
-
-    onCameraChange?.invoke(
-      RNRegion(
-        center = bounds.center.toRnLatLng(),
-        latitudeDelta = latDelta,
-        longitudeDelta = lngDelta,
-      ),
-      RNCamera(
-        center = cameraPosition.target.toRnLatLng(),
-        zoom = cameraPosition.zoom.toDouble(),
-        bearing = cameraPosition.bearing.toDouble(),
-        tilt = cameraPosition.tilt.toDouble(),
-      ),
+    onCameraChangeStart?.invoke(
+      bounds.toRnRegion(),
+      cameraPosition.toRnCamera(),
       isGesture,
     )
   }
@@ -220,39 +183,16 @@ class GoogleMapsViewImpl(
     }
     val isGesture = GoogleMap.OnCameraMoveStartedListener.REASON_GESTURE == cameraMoveReason
 
-    val latDelta = bounds.northeast.latitude - bounds.southwest.latitude
-    val lngDelta = bounds.northeast.longitude - bounds.southwest.longitude
-
-    onCameraChangeComplete?.invoke(
-      RNRegion(
-        center = bounds.center.toRnLatLng(),
-        latitudeDelta = latDelta,
-        longitudeDelta = lngDelta,
-      ),
-      RNCamera(
-        center = cameraPosition.target.toRnLatLng(),
-        zoom = cameraPosition.zoom.toDouble(),
-        bearing = cameraPosition.bearing.toDouble(),
-        tilt = cameraPosition.tilt.toDouble(),
-      ),
+    onCameraChangeStart?.invoke(
+      bounds.toRnRegion(),
+      cameraPosition.toRnCamera(),
       isGesture,
     )
   }
 
   fun initLocationCallbacks() {
     locationHandler.onUpdate = { location ->
-      // / only the coordinated are relevant right now
-      if (lastSubmittedLocation?.latitude != location.latitude || lastSubmittedLocation?.longitude != location.longitude ||
-        lastSubmittedLocation?.bearing != location.bearing
-      ) {
-        onLocationUpdate?.invoke(
-          RNLocation(
-            RNLatLng(location.latitude, location.longitude),
-            location.bearing.toDouble(),
-          ),
-        )
-      }
-      lastSubmittedLocation = location
+      onLocationUpdate?.invoke(location.toRnLocation())
     }
 
     locationHandler.onError = { error ->
@@ -261,65 +201,18 @@ class GoogleMapsViewImpl(
     locationHandler.start()
   }
 
-  fun applyPending() {
-    onUi {
-      mapPadding?.let {
-        googleMap?.setPadding(
-          it.left.dpToPx().toInt(),
-          it.top.dpToPx().toInt(),
-          it.right.dpToPx().toInt(),
-          it.bottom.dpToPx().toInt(),
-        )
-      }
-
-      uiSettings?.let { v ->
-        googleMap?.uiSettings?.apply {
-          v.allGesturesEnabled?.let { setAllGesturesEnabled(it) }
-          v.compassEnabled?.let { isCompassEnabled = it }
-          v.indoorLevelPickerEnabled?.let { isIndoorLevelPickerEnabled = it }
-          v.mapToolbarEnabled?.let { isMapToolbarEnabled = it }
-          v.myLocationButtonEnabled?.let {
-            googleMap?.setLocationSource(locationHandler)
-            isMyLocationButtonEnabled = it
-          }
-          v.rotateEnabled?.let { isRotateGesturesEnabled = it }
-          v.scrollEnabled?.let { isScrollGesturesEnabled = it }
-          v.scrollDuringRotateOrZoomEnabled?.let {
-            isScrollGesturesEnabledDuringRotateOrZoom = it
-          }
-          v.tiltEnabled?.let { isTiltGesturesEnabled = it }
-          v.zoomControlsEnabled?.let { isZoomControlsEnabled = it }
-          v.zoomGesturesEnabled?.let { isZoomGesturesEnabled = it }
-        }
-      }
-
-      buildingEnabled?.let {
-        googleMap?.isBuildingsEnabled = it
-      }
-      trafficEnabled?.let {
-        googleMap?.isTrafficEnabled = it
-      }
-      indoorEnabled?.let {
-        googleMap?.isIndoorEnabled = it
-      }
-      googleMap?.setMapStyle(customMapStyle)
-      mapType?.let {
-        googleMap?.mapType = it
-      }
-      userInterfaceStyle?.let {
-        googleMap?.mapColorScheme = it
-      }
-      mapZoomConfig?.let {
-        googleMap?.setMinZoomPreference(it.min?.toFloat() ?: 2.0f)
-        googleMap?.setMaxZoomPreference(it.max?.toFloat() ?: 21.0f)
-      }
-    }
-
-    locationConfig?.let {
-      locationHandler.priority = it.android?.priority?.toGooglePriority()
-      locationHandler.interval = it.android?.interval?.toLong()
-      locationHandler.minUpdateInterval = it.android?.minUpdateInterval?.toLong()
-    }
+  fun applyProps() {
+    mapPadding = mapPadding
+    uiSettings = uiSettings
+    myLocationEnabled = myLocationEnabled
+    buildingEnabled = buildingEnabled
+    trafficEnabled = trafficEnabled
+    indoorEnabled = indoorEnabled
+    customMapStyle = customMapStyle
+    mapType = mapType
+    userInterfaceStyle = userInterfaceStyle
+    mapZoomConfig = mapZoomConfig
+    locationConfig = locationConfig
 
     if (pendingMarkers.isNotEmpty()) {
       pendingMarkers.forEach { (id, opts) ->
@@ -480,9 +373,11 @@ class GoogleMapsViewImpl(
   var locationConfig: RNLocationConfig? = null
     set(value) {
       field = value
-      locationHandler.priority = value?.android?.priority?.toGooglePriority()
-      locationHandler.interval = value?.android?.interval?.toLong()
-      locationHandler.minUpdateInterval = value?.android?.minUpdateInterval?.toLong()
+      locationHandler.updateConfig(
+        value?.android?.priority?.toGooglePriority(),
+        value?.android?.interval?.toLong(),
+        value?.android?.minUpdateInterval?.toLong(),
+      )
     }
 
   var onMapError: ((RNMapErrorCode) -> Unit)? = null
@@ -919,7 +814,7 @@ class GoogleMapsViewImpl(
     onUi {
       heatmapsById.values.forEach { it.remove() }
     }
-    circlesById.clear()
+    heatmapsById.clear()
     pendingHeatmaps.clear()
   }
 
@@ -968,6 +863,8 @@ class GoogleMapsViewImpl(
   }
 
   fun destroyInternal() {
+    if (destroyed) return
+    destroyed = true
     onUi {
       locationHandler.stop()
       markerBuilder.cancelAllJobs()

package/android/src/main/java/com/rngooglemapsplus/LocationHandler.kt

@@ -35,30 +35,28 @@ class LocationHandler(
   private var listener: LocationSource.OnLocationChangedListener? = null
   private var locationRequest: LocationRequest? = null
   private var locationCallback: LocationCallback? = null
-
-  var priority: Int? = PRIORITY_DEFAULT
-    set(value) {
-      field = value ?: PRIORITY_DEFAULT
-      start()
-    }
-
-  var interval: Long? = INTERVAL_DEFAULT
-    set(value) {
-      field = value ?: INTERVAL_DEFAULT
-      buildLocationRequest()
-    }
-
-  var minUpdateInterval: Long? = MIN_UPDATE_INTERVAL
-    set(value) {
-      field = value ?: MIN_UPDATE_INTERVAL
-      buildLocationRequest()
-    }
+  private var priority: Int = PRIORITY_DEFAULT
+  private var interval: Long = INTERVAL_DEFAULT
+  private var minUpdateInterval: Long = MIN_UPDATE_INTERVAL
+  private var lastSubmittedLocation: Location? = null
+  private var isActive = false
 
   var onUpdate: ((Location) -> Unit)? = null
   var onError: ((RNLocationErrorCode) -> Unit)? = null
 
   init {
-    buildLocationRequest()
+    buildLocationRequest(priority, interval, minUpdateInterval)
+  }
+
+  fun updateConfig(
+    priority: Int? = null,
+    interval: Long? = null,
+    minUpdateInterval: Long? = null,
+  ) {
+    this.priority = priority ?: PRIORITY_DEFAULT
+    this.interval = interval ?: INTERVAL_DEFAULT
+    this.minUpdateInterval = minUpdateInterval ?: MIN_UPDATE_INTERVAL
+    buildLocationRequest(this.priority, this.interval, this.minUpdateInterval)
   }
 
   fun showLocationDialog() {
@@ -108,11 +106,11 @@ class LocationHandler(
   }
 
   @Suppress("deprecation")
-  private fun buildLocationRequest() {
-    val priority = priority ?: Priority.PRIORITY_BALANCED_POWER_ACCURACY
-    val interval = interval ?: INTERVAL_DEFAULT
-    val minUpdateInterval = minUpdateInterval ?: MIN_UPDATE_INTERVAL
-
+  private fun buildLocationRequest(
+    priority: Int,
+    interval: Long,
+    minUpdateInterval: Long,
+  ) {
     locationRequest =
       if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.S) {
         LocationRequest
@@ -126,11 +124,13 @@ class LocationHandler(
           .setInterval(interval)
           .setFastestInterval(minUpdateInterval)
       }
-    restartLocationUpdates()
   }
 
-  private fun restartLocationUpdates() {
-    stop()
+  @SuppressLint("MissingPermission")
+  fun start() {
+    if (isActive) return
+    isActive = true
+
     val playServicesStatus =
       GoogleApiAvailability
         .getInstance()
@@ -139,17 +139,13 @@ class LocationHandler(
       onError?.invoke(RNLocationErrorCode.PLAY_SERVICE_NOT_AVAILABLE)
       return
     }
-    start()
-  }
-
-  @SuppressLint("MissingPermission")
-  fun start() {
     try {
       fusedLocationClientProviderClient.lastLocation
         .addOnSuccessListener(
           OnSuccessListener { location ->
-            if (location != null) {
+            if (location != null && location != lastSubmittedLocation) {
               onUpdate?.invoke(location)
+              lastSubmittedLocation = location
             }
           },
         ).addOnFailureListener { e ->
@@ -161,8 +157,11 @@ class LocationHandler(
           override fun onLocationResult(locationResult: LocationResult) {
             val location = locationResult.lastLocation
             if (location != null) {
-              listener?.onLocationChanged(location)
-              onUpdate?.invoke(location)
+              if (location != lastSubmittedLocation) {
+                lastSubmittedLocation = location
+                listener?.onLocationChanged(location)
+                onUpdate?.invoke(location)
+              }
             } else {
               onError?.invoke(RNLocationErrorCode.POSITION_UNAVAILABLE)
             }
@@ -186,9 +185,11 @@ class LocationHandler(
   }
 
   fun stop() {
-    listener = null
+    if (!isActive) return
+    isActive = false
     if (locationCallback != null) {
       fusedLocationClientProviderClient.removeLocationUpdates(locationCallback!!)
+      fusedLocationClientProviderClient.flushLocations()
       locationCallback = null
     }
   }
@@ -199,6 +200,7 @@ class LocationHandler(
   }
 
   override fun deactivate() {
+    listener = null
     stop()
   }
 }

package/android/src/main/java/com/rngooglemapsplus/extensions/CameraPositionExtension.kt

@@ -0,0 +1,12 @@
+package com.rngooglemapsplus.extensions
+
+import com.google.android.gms.maps.model.CameraPosition
+import com.rngooglemapsplus.RNCamera
+
+fun CameraPosition.toRnCamera(): RNCamera =
+  RNCamera(
+    center = target.toRnLatLng(),
+    zoom = zoom.toDouble(),
+    bearing = bearing.toDouble(),
+    tilt = tilt.toDouble(),
+  )

package/android/src/main/java/com/rngooglemapsplus/extensions/IntExtension.kt

@@ -0,0 +1,28 @@
+package com.rngooglemapsplus.extensions
+
+import com.google.android.gms.common.ConnectionResult
+import com.rngooglemapsplus.RNMapErrorCode
+
+fun Int.toRNMapErrorCodeOrNull(): RNMapErrorCode? =
+  when (this) {
+    ConnectionResult.SERVICE_MISSING ->
+      RNMapErrorCode.PLAY_SERVICES_MISSING
+
+    ConnectionResult.SERVICE_INVALID ->
+      RNMapErrorCode.PLAY_SERVICES_INVALID
+
+    ConnectionResult.SERVICE_VERSION_UPDATE_REQUIRED ->
+      RNMapErrorCode.PLAY_SERVICES_OUTDATED
+
+    ConnectionResult.SERVICE_UPDATING ->
+      RNMapErrorCode.PLAY_SERVICE_UPDATING
+
+    ConnectionResult.SERVICE_DISABLED ->
+      RNMapErrorCode.PLAY_SERVICES_DISABLED
+
+    ConnectionResult.SUCCESS ->
+      null
+
+    else ->
+      RNMapErrorCode.UNKNOWN
+  }

package/android/src/main/java/com/rngooglemapsplus/extensions/LatLngBounds.kt

@@ -0,0 +1,15 @@
+package com.rngooglemapsplus.extensions
+
+import com.google.android.gms.maps.model.LatLngBounds
+import com.rngooglemapsplus.RNRegion
+
+fun LatLngBounds.toRnRegion(): RNRegion {
+  val latDelta = northeast.latitude - southwest.latitude
+  val lngDelta = northeast.longitude - southwest.longitude
+
+  return RNRegion(
+    center = center.toRnLatLng(),
+    latitudeDelta = latDelta,
+    longitudeDelta = lngDelta,
+  )
+}

package/android/src/main/java/com/rngooglemapsplus/extensions/LocationExtension.kt

@@ -0,0 +1,59 @@
+package com.rngooglemapsplus.extensions
+
+import android.location.Location
+import android.os.Build
+import com.rngooglemapsplus.RNLatLng
+import com.rngooglemapsplus.RNLocation
+import com.rngooglemapsplus.RNLocationAndroid
+
+fun Location.toRnLocation(): RNLocation =
+  RNLocation(
+    center = RNLatLng(latitude, longitude),
+    altitude = altitude,
+    accuracy = accuracy.toDouble(),
+    bearing = bearing.toDouble(),
+    speed = speed.toDouble(),
+    time = time.toDouble(),
+    android =
+      RNLocationAndroid(
+        provider = provider,
+        elapsedRealtimeNanos = elapsedRealtimeNanos.toDouble(),
+        bearingAccuracyDegrees =
+          if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+            bearingAccuracyDegrees.toDouble()
+          } else {
+            null
+          },
+        speedAccuracyMetersPerSecond =
+          if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+            speedAccuracyMetersPerSecond.toDouble()
+          } else {
+            null
+          },
+        verticalAccuracyMeters =
+          if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
+            verticalAccuracyMeters.toDouble()
+          } else {
+            null
+          },
+        mslAltitudeMeters =
+          if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.UPSIDE_DOWN_CAKE) {
+            mslAltitudeMeters
+          } else {
+            null
+          },
+        mslAltitudeAccuracyMeters =
+          if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.UPSIDE_DOWN_CAKE) {
+            mslAltitudeAccuracyMeters.toDouble()
+          } else {
+            null
+          },
+        isMock =
+          if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.S) {
+            isMock
+          } else {
+            isFromMockProvider
+          },
+      ),
+    ios = null,
+  )