@rnmapbox/maps 10.1.13 → 10.1.15

package/android/src/main/java/com/rnmapbox/rnmbx/components/annotation/RNMBXPointAnnotationCoordinator.kt

@@ -29,6 +29,45 @@ class RNMBXPointAnnotationCoordinator(val mapView: MapView) {
             onAnnotationClick(pointAnnotation)
             false
         })
+        manager.addDragListener(object : OnPointAnnotationDragListener {
+            override fun onAnnotationDragStarted(_annotation: Annotation<*>) {
+                annotationDragged = true;
+                var reactAnnotation: RNMBXPointAnnotation? = null
+                for (key in annotations.keys) {
+                    val annotation = annotations[key]
+                    val curMarkerID = annotation?.mapboxID
+                    if (_annotation.id == curMarkerID) {
+                        reactAnnotation = annotation
+                    }
+                }
+                reactAnnotation?.let { it.onDragStart() }
+            }
+
+            override fun onAnnotationDrag(_annotation: Annotation<*>) {
+                var reactAnnotation: RNMBXPointAnnotation? = null
+                for (key in annotations.keys) {
+                    val annotation = annotations[key]
+                    val curMarkerID = annotation?.mapboxID
+                    if (_annotation.id == curMarkerID) {
+                        reactAnnotation = annotation
+                    }
+                }
+                reactAnnotation?.let { it.onDrag() }
+            }
+
+            override fun onAnnotationDragFinished(_annotation: Annotation<*>) {
+                annotationDragged = false;
+                var reactAnnotation: RNMBXPointAnnotation? = null
+                for (key in annotations.keys) {
+                    val annotation = annotations[key]
+                    val curMarkerID = annotation?.mapboxID
+                    if (_annotation.id == curMarkerID) {
+                        reactAnnotation = annotation
+                    }
+                }
+                reactAnnotation?.let { it.onDragEnd() }
+            }
+        })
     }
 
     fun getAndClearAnnotationClicked(): Boolean {
@@ -70,46 +109,6 @@ class RNMBXPointAnnotationCoordinator(val mapView: MapView) {
             newSelected = null
         }
 
-        manager.addDragListener(object : OnPointAnnotationDragListener {
-            override fun onAnnotationDragStarted(_annotation: Annotation<*>) {
-                annotationDragged = true;
-                var reactAnnotation: RNMBXPointAnnotation? = null
-                for (key in annotations.keys) {
-                    val annotation = annotations[key]
-                    val curMarkerID = annotation?.mapboxID
-                    if (_annotation.id == curMarkerID) {
-                        reactAnnotation = annotation
-                    }
-                }
-                reactAnnotation?.let { it.onDragStart() }
-            }
-
-            override fun onAnnotationDrag(_annotation: Annotation<*>) {
-                var reactAnnotation: RNMBXPointAnnotation? = null
-                for (key in annotations.keys) {
-                    val annotation = annotations[key]
-                    val curMarkerID = annotation?.mapboxID
-                    if (_annotation.id == curMarkerID) {
-                        reactAnnotation = annotation
-                    }
-                }
-                reactAnnotation?.let { it.onDrag() }
-            }
-
-            override fun onAnnotationDragFinished(_annotation: Annotation<*>) {
-                annotationDragged = false;
-                var reactAnnotation: RNMBXPointAnnotation? = null
-                for (key in annotations.keys) {
-                    val annotation = annotations[key]
-                    val curMarkerID = annotation?.mapboxID
-                    if (_annotation.id == curMarkerID) {
-                        reactAnnotation = annotation
-                    }
-                }
-                reactAnnotation?.let { it.onDragEnd() }
-            }
-        })
-
         oldSelected?.let { deselectAnnotation(it) }
         newSelected?.let { selectAnnotation(it) }
 

package/android/src/main/java/com/rnmapbox/rnmbx/components/camera/CameraStop.kt

@@ -95,12 +95,10 @@ class CameraStop {
     fun toCameraUpdate(mapView: RNMBXMapView): CameraUpdateItem {
         val map = mapView.getMapboxMap()
         val currentCamera = map.cameraState
+
         val builder = CameraOptions.Builder()
         builder.center(currentCamera.center)
         builder.bearing(currentCamera.bearing)
-
-        val currentPadding = currentCamera.padding
-
         builder.padding(currentCamera.padding)
         builder.zoom(currentCamera.zoom)
         if (mBearing != null) {
@@ -109,7 +107,11 @@ class CameraStop {
         if (mTilt != null) {
             builder.pitch(mTilt)
         }
+        if (mZoom != null) {
+            builder.zoom(mZoom)
+        }
 
+        val currentPadding = currentCamera.padding
         val paddingLeft: Int = mPaddingLeft ?: currentPadding.left.toInt()
         val paddingTop: Int = mPaddingTop ?: currentPadding.top.toInt()
         val paddingRight: Int = mPaddingRight ?: currentPadding.right.toInt()
@@ -117,10 +119,10 @@ class CameraStop {
         val cameraPadding = intArrayOf(paddingLeft, paddingTop, paddingRight, paddingBottom)
         val cameraPaddingClipped = clippedPadding(cameraPadding, mapView)
         val cameraPaddingEdgeInsets = convert(cameraPaddingClipped)
+        builder.padding(cameraPaddingEdgeInsets)
 
         if (mLatLng != null) {
             builder.center(mLatLng!!.point)
-            builder.padding(cameraPaddingEdgeInsets)
         } else if (mBounds != null) {
             val tilt = if (mTilt != null) mTilt!! else currentCamera.pitch
             val bearing = if (mBearing != null) mBearing!! else currentCamera.bearing
@@ -131,17 +133,14 @@ class CameraStop {
                 bearing,
                 tilt
             )
-
             builder.center(boundsCamera.center)
             builder.anchor(boundsCamera.anchor)
             builder.zoom(boundsCamera.zoom)
+            builder.bearing(boundsCamera.bearing)
+            builder.pitch(boundsCamera.pitch)
             builder.padding(boundsCamera.padding)
         }
 
-        if (mZoom != null) {
-            builder.zoom(mZoom)
-        }
-
         return CameraUpdateItem(map, builder.build(), mDuration, mCallback, mMode)
     }
 
@@ -195,6 +194,7 @@ class CameraStop {
                 when (readableMap.getInt("mode")) {
                     CameraMode.FLIGHT -> stop.setMode(CameraMode.FLIGHT)
                     CameraMode.LINEAR -> stop.setMode(CameraMode.LINEAR)
+                    CameraMode.MOVE -> stop.setMode(CameraMode.MOVE)
                     CameraMode.NONE -> stop.setMode(CameraMode.NONE)
                     else -> stop.setMode(CameraMode.EASE)
                 }

package/android/src/main/java/com/rnmapbox/rnmbx/components/camera/CameraUpdateItem.kt

@@ -83,7 +83,7 @@ class CameraUpdateItem(
         val animationOptions = MapAnimationOptions.Builder();
 
         // animateCamera / easeCamera only allows positive duration
-        if (duration == 0 || mCameraMode == CameraMode.NONE) {
+        if (duration == 0 || mCameraMode == CameraMode.MOVE || mCameraMode == CameraMode.NONE) {
             map.flyToV11(mCameraUpdate, animationOptions.apply {
                 duration(0)
             },
@@ -109,7 +109,6 @@ class CameraUpdateItem(
                 callback
             )
         }
-        null
     }
 
     override fun cancel(mayInterruptIfRunning: Boolean): Boolean {

package/android/src/main/java/com/rnmapbox/rnmbx/components/camera/RNMBXCamera.kt

@@ -179,15 +179,12 @@ class RNMBXCamera(private val mContext: Context, private val mManager: RNMBXCame
     private fun updateMaxBounds() {
         withMapView { mapView ->
             val map = mapView.getMapboxMap()
-            val maxBounds = mMaxBounds
             val builder = CameraBoundsOptions.Builder()
-
-            if (maxBounds != null) {
-                builder.bounds(maxBounds.toBounds())
-            }
-            mMinZoomLevel?.let { builder.minZoom(it) }
-            mMaxZoomLevel?.let { builder.maxZoom(it) }
+            builder.bounds(mMaxBounds?.toBounds())
+            builder.minZoom(mMinZoomLevel ?: 0.0) // Passing null does not reset this value.
+            builder.maxZoom(mMaxZoomLevel ?: 25.0) // Passing null does not reset this value.
             map.setBounds(builder.build())
+            mCameraStop?.let { updateCamera(it) }
         }
     }
 

package/android/src/main/java/com/rnmapbox/rnmbx/components/camera/RNMBXCameraManager.kt

@@ -59,12 +59,12 @@ class RNMBXCameraManager(private val mContext: ReactApplicationContext, val view
 
     @ReactProp(name = "minZoomLevel")
     override fun setMinZoomLevel(camera: RNMBXCamera, value: Dynamic) {
-        camera.setMinZoomLevel(value.asDouble())
+        camera.setMinZoomLevel(value.asDoubleOrNull())
     }
 
     @ReactProp(name = "maxZoomLevel")
     override fun setMaxZoomLevel(camera: RNMBXCamera, value: Dynamic) {
-        camera.setMaxZoomLevel(value.asDouble())
+        camera.setMaxZoomLevel(value.asDoubleOrNull())
     }
 
     @ReactProp(name = "followUserLocation")

package/android/src/main/java/com/rnmapbox/rnmbx/components/camera/constants/CameraMode.java

@@ -7,12 +7,13 @@ import java.lang.annotation.RetentionPolicy;
 
 public class CameraMode {
 
-    @IntDef({ FLIGHT, EASE, LINEAR, NONE })
+    @IntDef({ FLIGHT, EASE, LINEAR, MOVE, NONE })
     @Retention(RetentionPolicy.SOURCE)
     public @interface Mode {}
 
     public static final int FLIGHT = 1;
     public static final int EASE = 2;
     public static final int LINEAR = 3;
-    public static final int NONE = 4;
+    public static final int MOVE = 4;
+    public static final int NONE = 5;
 }