@rnmapbox/maps 10.0.0-rc.8 → 10.0.0

package/README.md

@@ -84,7 +84,7 @@ _See [iOS](ios/install.md) & [Android](android/install.md) setup guide for more
 
 - [node](https://nodejs.org)
 - [npm](https://www.npmjs.com/)
-- [React Native](https://facebook.github.io/react-native/) (0.60+)
+- [React Native](https://facebook.github.io/react-native/) (0.64+)
 
 
 ## Installation

package/android/rctmgl/src/main/AndroidManifest.xml

@@ -1,4 +1,5 @@
 <manifest xmlns:android="http://schemas.android.com/apk/res/android" package="com.mapbox.rctmgl">
     <uses-permission android:name="android.permission.INTERNET" />
+    <uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION"/>
     <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION"/>
 </manifest>

package/android/rctmgl/src/main/java-v10/com/mapbox/rctmgl/components/AbstractEventEmitter.kt

@@ -46,7 +46,7 @@ abstract class AbstractEventEmitter<T : ViewGroup?>(reactApplicationContext: Rea
         )
     }
 
-    override fun addEventEmitters(context: ThemedReactContext, @Nonnull view: T) {
+    override fun addEventEmitters(context: ThemedReactContext, view: T) {
         mEventDispatcher = context.getNativeModule(UIManagerModule::class.java)!!.eventDispatcher
     }
 
@@ -72,4 +72,4 @@ abstract class AbstractEventEmitter<T : ViewGroup?>(reactApplicationContext: Rea
     companion object {
         private const val BRIDGE_TIMEOUT_MS = 10.0
     }
-}
+}

package/android/rctmgl/src/main/java-v10/com/mapbox/rctmgl/components/images/RCTMGLImagesManager.kt

@@ -61,7 +61,7 @@ class RCTMGLImagesManager(private val mContext: ReactApplicationContext) :
             ReadableType.String -> {
                 val resourceName = dynamic.asString();
                 val drawable =
-                    ResourceUtils.getDrawableByName(mContext, resourceName) as BitmapDrawable
+                    ResourceUtils.getDrawableByName(mContext, resourceName) as BitmapDrawable?
                 if (drawable != null) {
                     return NativeImage(resourceName, drawable)
                 } else {
@@ -97,7 +97,7 @@ class RCTMGLImagesManager(private val mContext: ReactApplicationContext) :
                     scale = map.getDouble("scale")
                 }
                 val drawable =
-                    ResourceUtils.getDrawableByName(mContext, resourceName) as BitmapDrawable
+                    ResourceUtils.getDrawableByName(mContext, resourceName) as BitmapDrawable?
                 if (drawable != null && resourceName != null) {
                     return NativeImage(resourceName, drawable, scale, sdf, stretchX, stretchY)
                 } else {

package/android/rctmgl/src/main/java-v10/com/mapbox/rctmgl/components/mapview/RCTMGLAndroidTextureMapViewManager.kt

@@ -2,27 +2,22 @@ package com.mapbox.rctmgl.components.mapview
 
 import com.facebook.react.bridge.ReactApplicationContext
 import com.facebook.react.uimanager.ThemedReactContext
+import com.mapbox.maps.MapInitOptions
 
-//import com.mapbox.mapboxsdk.maps.MapboxMapOptions;
-/**
- * Created by hernanmateo on 12/11/18.
- */
-class RCTMGLAndroidTextureMapViewManager(context: ReactApplicationContext?) : RCTMGLMapViewManager(
-    context!!
+class RCTMGLAndroidTextureMapViewManager(context: ReactApplicationContext) : RCTMGLMapViewManager(
+    context
 ) {
     override fun getName(): String {
         return REACT_CLASS
     }
 
-    override fun createViewInstance(themedReactContext: ThemedReactContext): RCTMGLAndroidTextureMapView {
-        //MapboxMapOptions options = new MapboxMapOptions();
-        //options.textureMode(true);
-        val context = activity ?: themedReactContext
-        return RCTMGLAndroidTextureMapView(context, this /*, options*/)
+    override fun createViewInstance(themedReactContext: ThemedReactContext): RCTMGLMapView {
+        val context = getMapViewContext(themedReactContext)
+        val options = MapInitOptions(context = context, textureView= true)
+        return RCTMGLMapView(context, this, options)
     }
 
     companion object {
-        const val LOG_TAG = "RCTMGLAndroidTextureMapViewManager"
         const val REACT_CLASS = "RCTMGLAndroidTextureMapView"
     }
 }

package/android/rctmgl/src/main/java-v10/com/mapbox/rctmgl/components/mapview/RCTMGLMapView.kt

@@ -147,7 +147,7 @@ data class FeatureEntry(val feature: AbstractMapFeature?, val view: View?, var a
 
 }
 
-open class RCTMGLMapView(private val mContext: Context, var mManager: RCTMGLMapViewManager /*, MapboxMapOptions options*/) : FrameLayout(mContext), OnMapClickListener, OnMapLongClickListener, OnLayoutChangeListener {
+open class RCTMGLMapView(private val mContext: Context, var mManager: RCTMGLMapViewManager, options: MapInitOptions?) : FrameLayout(mContext), OnMapClickListener, OnMapLongClickListener, OnLayoutChangeListener {
     /**
      * `PointAnnotations` are rendered to a canvas, but the React Native `Image` component is
      * implemented on top of Fresco (https://frescolib.org), which does not load images for
@@ -192,6 +192,13 @@ open class RCTMGLMapView(private val mContext: Context, var mManager: RCTMGLMapV
     val viewAnnotationManager: ViewAnnotationManager
         get() = mapView.viewAnnotationManager
 
+    var requestDisallowInterceptTouchEvent: Boolean = false
+        set(value) {
+            val oldValue = field
+            field = value
+            updateRequestDisallowInterceptTouchEvent(oldValue, value)
+        }
+
     fun getMapboxMap(): MapboxMap {
         return mapView.getMapboxMap()
     }
@@ -1041,7 +1048,7 @@ open class RCTMGLMapView(private val mContext: Context, var mManager: RCTMGLMapV
         offscreenAnnotationViewContainer?.setLayoutParams(p)
         addView(offscreenAnnotationViewContainer)
 
-        mMapView = MapView(mContext)
+        mMapView = if (options != null) MapView(mContext, options) else MapView(mContext)
 
         val matchParent = FrameLayout.LayoutParams(ViewGroup.LayoutParams.MATCH_PARENT, ViewGroup.LayoutParams.MATCH_PARENT)
         mMapView.setLayoutParams(matchParent)
@@ -1356,6 +1363,26 @@ open class RCTMGLMapView(private val mContext: Context, var mManager: RCTMGLMapV
     // endregion
 }
 
+// region requestDisallowInterceptTouchEvent
+fun RCTMGLMapView.updateRequestDisallowInterceptTouchEvent(oldValue: Boolean, value: Boolean) {
+    if (oldValue == value) {
+        return
+    }
+    if (value) {
+        mapView.setOnTouchListener { view, event ->
+            this.requestDisallowInterceptTouchEvent(true)
+            mapView.onTouchEvent(event)
+            true
+        }
+    } else {
+        mapView.setOnTouchListener { view, event ->
+            mapView.onTouchEvent(event)
+        }
+    }
+}
+// endregion
+
+
 fun OrnamentSettings.setPosAndMargins(posAndMargins: ReadableMap?) {
     if (posAndMargins == null) { return }
 

package/android/rctmgl/src/main/java-v10/com/mapbox/rctmgl/components/mapview/RCTMGLMapViewManager.kt

@@ -1,5 +1,6 @@
 package com.mapbox.rctmgl.components.mapview
 
+import android.content.Context
 import android.util.Log
 import android.view.View
 import com.facebook.react.bridge.*
@@ -11,6 +12,7 @@ import com.facebook.react.uimanager.annotations.ReactProp
 import com.mapbox.rctmgl.events.constants.EventKeys
 import com.mapbox.maps.MapboxMap
 import com.facebook.react.common.MapBuilder
+import com.mapbox.maps.MapInitOptions
 import com.mapbox.maps.extension.style.layers.properties.generated.ProjectionName
 import com.mapbox.maps.plugin.compass.compass
 import com.mapbox.maps.plugin.gestures.gestures
@@ -75,9 +77,13 @@ open class RCTMGLMapViewManager(context: ReactApplicationContext) :
         mapView!!.removeFeatureAt(index)
     }
 
+    fun getMapViewContext(themedReactContext: ThemedReactContext): Context {
+        return activity ?: themedReactContext
+    }
+
     override fun createViewInstance(themedReactContext: ThemedReactContext): RCTMGLMapView {
-        val context = activity ?: themedReactContext
-        return RCTMGLMapView(context, this /*, null*/)
+        val context = getMapViewContext(themedReactContext)
+        return RCTMGLMapView(context, this, options=null)
     }
 
     override fun onDropViewInstance(mapView: RCTMGLMapView) {
@@ -227,6 +233,11 @@ open class RCTMGLMapViewManager(context: ReactApplicationContext) :
         //mapView.setTintColor(tintColor);
     }
 
+    @ReactProp(name = "requestDisallowInterceptTouchEvent")
+    fun setRequestDisallowInterceptTouchEvent(mapView: RCTMGLMapView, requestDisallowInterceptTouchEvent: Boolean) {
+        mapView.requestDisallowInterceptTouchEvent = requestDisallowInterceptTouchEvent
+    }
+
     //endregion
     //region Custom Events
     override fun customEvents(): Map<String, String>? {

package/android/rctmgl/src/main/java-v10/com/mapbox/rctmgl/modules/RCTMGLModule.kt

@@ -14,7 +14,9 @@ import com.mapbox.rctmgl.modules.RCTMGLLocationModule
 import com.facebook.react.bridge.ReactMethod
 import com.facebook.react.common.MapBuilder
 import com.mapbox.common.*
+import com.mapbox.maps.MapView
 import com.mapbox.maps.Style
+import com.mapbox.maps.plugin.attribution.attribution
 import com.mapbox.rctmgl.components.camera.constants.CameraMode
 import java.util.HashMap
 
@@ -115,6 +117,15 @@ class RCTMGLModule(private val mReactContext: ReactApplicationContext) : ReactCo
             .build()
     }
 
+    @ReactMethod
+    fun setTelemetryEnabled(telemetryEnabled: Boolean) {
+        mReactContext.runOnUiQueueThread {
+            val dummyView = MapView(mReactContext)
+            val telemetry = dummyView.attribution.getMapAttributionDelegate().telemetry()
+            telemetry.userTelemetryRequestState = telemetryEnabled
+        }
+    }
+
     @ReactMethod
     fun setAccessToken(accessToken: String?, promise: Promise) {
         mReactContext.runOnUiQueueThread(Runnable {

package/android/rctmgl/src/main/java-v10/com/mapbox/rctmgl/modules/RCTMGLOfflineModule.kt

@@ -41,7 +41,7 @@ enum class TileRegionPackState(val rawValue: String) {
     COMPLETE("complete"),
     UNKNOWN("unkown")
 }
-class TileRegionPack(var name: String, var progress: TileRegionLoadProgress?, var state: TileRegionPackState, var metadata: JSONObject) {
+class TileRegionPack(var name: String, var state: TileRegionPackState = TileRegionPackState.UNKNOWN, var progress: TileRegionLoadProgress? = null, var metadata: JSONObject) {
     var cancelable: Cancelable? = null
     var loadOptions: TileRegionLoadOptions? = null
 
@@ -78,7 +78,7 @@ class TileRegionPack(var name: String, var progress: TileRegionLoadProgress?, va
         bounds: Geometry,
         zoomRange: ZoomRange,
         metadata: JSONObject
-    ) : this(name, null, state, metadata) {
+    ) : this(name= name, state= state,progress= null, metadata= metadata) {
         val rnmeta = JSONObject()
         rnmeta.put("styleURI", styleURI)
         this.styleURI = styleURI
@@ -163,12 +163,23 @@ class RCTMGLOfflineModule(private val mReactContext: ReactApplicationContext) :
     @ReactMethod
     fun getPackStatus(name: String, promise: Promise) {
         val pack = tileRegionPacks[name]
-        if (pack != null) {
-            promise.resolve(makeRegionStatus(name, pack.progress, pack))
-        } else {
-            promise.reject(Error("Pack not found"))
-            Logger.w(REACT_CLASS, "getPackStatus - Unknown offline region")
+        if (pack == null) {
+            promise.reject(Error("Pack: $name not found"))
+            return
         }
+        tileStore.getTileRegionMetadata(name) { expected ->
+            expected.value?.also {
+                val pack = TileRegionPack(
+                    name= name,
+                    metadata= it.toJSONObject() ?: JSONObject()
+                )
+                tileRegionPacks[name] = pack
+                promise.resolve(_makeRegionStatusPayload(pack))
+            } ?: run {
+                promise.reject(LOG_TAG, expected.error!!.message)
+            }
+        }
+
     }
 
     @ReactMethod
@@ -246,7 +257,6 @@ class RCTMGLOfflineModule(private val mReactContext: ReactApplicationContext) :
             }
         })
     }
-
     // endregion
 
     fun startLoading(pack: TileRegionPack) {
@@ -277,6 +287,7 @@ class RCTMGLOfflineModule(private val mReactContext: ReactApplicationContext) :
             .minZoom(zoomRange.minZoom)
             .maxZoom(zoomRange.maxZoom)
             .stylePackOptions(stylePackOptions)
+            .pixelRatio(2.0f)
             .build()
         val tilesetDescriptor = offlineManager.createTilesetDescriptor(descriptorOptions)
 
@@ -368,7 +379,7 @@ class RCTMGLOfflineModule(private val mReactContext: ReactApplicationContext) :
                         metadata= metadataJSON ?: JSONObject()
                     )
 
-                    if (region.completedResourceCount == region.requiredResourceCount) {
+                    if (region.hasCompleted()) {
                         pack.state = TileRegionPackState.COMPLETE
                     }
                     tileRegionPacks[region.id] = pack
@@ -404,8 +415,7 @@ class RCTMGLOfflineModule(private val mReactContext: ReactApplicationContext) :
         );
 
         if (region.requiredResourceCount > 0) {
-            val percentage = 100.0 * region.completedResourceCount.toDouble() / region.requiredResourceCount.toDouble()
-            result.putDouble("percentage", percentage)
+            result.putDouble("percentage", region.toPercentage())
         } else {
             result.putNull("percentage")
         }
@@ -429,14 +439,17 @@ class RCTMGLOfflineModule(private val mReactContext: ReactApplicationContext) :
         )
     }
 
-    private fun _makeRegionStatusPayload(name:String, progress: TileRegionLoadProgress?,state: TileRegionPackState, metadata: JSONObject?) : WritableMap {
+    private fun _makeRegionStatusPayload(pack: TileRegionPack): WritableMap {
+        return _makeRegionStatusPayload(pack.name, pack.progress, pack.state, pack.metadata)
+    }
+
+    private fun _makeRegionStatusPayload(name:String, progress: TileRegionLoadProgress?,state: TileRegionPackState, metadata: JSONObject?): WritableMap {
         var result = Arguments.createMap()
         if (progress != null) {
-            val progressPercentage =  progress.completedResourceCount.toDouble() / progress.requiredResourceCount.toDouble()
             result = writableMapOf(
-                "state" to (if (progress.completedResourceCount == progress.requiredResourceCount) TileRegionPackState.COMPLETE.rawValue else state.rawValue),
+                "state" to (if (progress.hasCompleted()) TileRegionPackState.COMPLETE.rawValue else state.rawValue),
                 "name" to name,
-                "perentage" to progressPercentage * 100.0,
+                "percentage" to progress.toPercentage(),
                 "completedResourceCount" to progress.completedResourceCount,
                 "completedResourceSize" to progress.completedResourceSize,
                 "erroredResourceCount" to progress.erroredResourceCount,
@@ -448,7 +461,7 @@ class RCTMGLOfflineModule(private val mReactContext: ReactApplicationContext) :
             result = writableMapOf(
                 "state" to state.rawValue,
                 "name" to name,
-                "perentage" to null,
+                "percentage" to null,
             )
         }
         if (metadata != null) {
@@ -456,16 +469,10 @@ class RCTMGLOfflineModule(private val mReactContext: ReactApplicationContext) :
         }
         return result
     }
-    private fun makeProgresEvent(name: String, progress: TileRegionLoadProgress, state: TileRegionPackState): OfflineEvent {
-        return OfflineEvent(
-            OFFLINE_PROGRESS,
-            EventTypes.OFFLINE_STATUS,
-            _makeRegionStatusPayload(name, progress, state, null)
-        )
-    }
+
     private fun offlinePackProgressDidChange(progress: TileRegionLoadProgress, metadata: JSONObject, state: TileRegionPackState) {
         // TODO throttle
-        sendEvent(this.makeProgresEvent(metadata.getString("name"), progress, state))
+        sendEvent(this.makeProgressEvent(metadata.getString("name"), progress, state))
     }
 
     private fun offlinePackDidReceiveError(name: String, error: TileRegionError) {
@@ -505,36 +512,6 @@ class RCTMGLOfflineModule(private val mReactContext: ReactApplicationContext) :
         )
     }
 
-    fun startPackDownload(pack: TileRegionPack) {
-        val _this = this
-        pack.cancelable = tileStore
-            .loadTileRegion(
-                pack.name,
-                (pack.loadOptions)!!,
-                TileRegionLoadProgressCallback { progress ->
-                    pack.progress = progress
-                    pack.state = TileRegionPackState.ACTIVE
-                    _this.sendEvent(_this.makeStatusEvent(pack.name, progress, pack))
-                },
-                object : TileRegionCallback {
-                    override fun run(region: Expected<TileRegionError, TileRegion>) {
-                        pack.cancelable = null
-                        if (region.isError) {
-                            pack.state = TileRegionPackState.INACTIVE
-                            _this.sendEvent(
-                                _this.makeErrorEvent(
-                                    pack.name, "TileRegionError", region.error!!
-                                        .message
-                                )
-                            )
-                        } else {
-                            pack.state = TileRegionPackState.COMPLETE
-                            _this.sendEvent(_this.makeStatusEvent(pack.name, pack.progress, pack))
-                        }
-                    }
-                })
-    }
-
     @ReactMethod
     fun resetDatabase(promise: Promise) {
         tileStore.getAllTileRegions { expected ->
@@ -622,38 +599,14 @@ class RCTMGLOfflineModule(private val mReactContext: ReactApplicationContext) :
         return OfflineEvent(OFFLINE_ERROR, errorType, payload)
     }
 
-    private fun makeStatusEvent(
-        regionName: String,
-        status: TileRegionLoadProgress?,
-        pack: TileRegionPack
-    ): OfflineEvent {
+    private fun makeProgressEvent(name: String, progress: TileRegionLoadProgress, state: TileRegionPackState): OfflineEvent {
         return OfflineEvent(
             OFFLINE_PROGRESS,
             EventTypes.OFFLINE_STATUS,
-            makeRegionStatus(regionName, status, pack)
+            _makeRegionStatusPayload(name, progress, state, null)
         )
     }
 
-    private fun makeRegionStatus(
-        regionName: String,
-        status: TileRegionLoadProgress?,
-        pack: TileRegionPack
-    ): WritableMap {
-        val map = Arguments.createMap()
-        val progressPercentage =
-            (status!!.completedResourceCount.toDouble() * 100.0) / (status.requiredResourceCount.toDouble())
-        map.putString("name", regionName)
-        map.putString("state", pack.state.rawValue)
-        map.putDouble("percentage", progressPercentage)
-        map.putInt("completedResourceCount", status.completedResourceCount.toInt())
-        map.putInt("completedResourceSize", status.completedResourceSize.toInt())
-        map.putInt("erroredResourceCount", status.erroredResourceCount.toInt())
-        map.putInt("requiredResourceCount", status.requiredResourceCount.toInt())
-        map.putInt("loadedResourceCount", status.loadedResourceCount.toInt())
-        map.putInt("loadedResourceSize", status.loadedResourceSize.toInt())
-        return map
-    }
-
     companion object {
         const val REACT_CLASS = "RCTMGLOfflineModule"
         const val LOG_TAG = REACT_CLASS
@@ -663,6 +616,21 @@ class RCTMGLOfflineModule(private val mReactContext: ReactApplicationContext) :
     }
 }
 
+fun TileRegionLoadProgress.toPercentage(): Double {
+    return (completedResourceCount.toDouble() * 100.0) / (requiredResourceCount.toDouble())
+}
+
+fun TileRegionLoadProgress.hasCompleted(): Boolean {
+    return (completedResourceCount == requiredResourceCount)
+}
+
+fun TileRegion.toPercentage(): Double {
+    return (completedResourceCount.toDouble() * 100.0) / (requiredResourceCount.toDouble())
+}
+
+fun TileRegion.hasCompleted(): Boolean {
+    return (completedResourceCount == requiredResourceCount)
+}
 
 
 

package/ios/RCTMGL-v10/Extensions/DefaultStringInterpolation+optional.swift

@@ -1,5 +1,9 @@
 internal extension DefaultStringInterpolation {
   mutating func appendInterpolation<T>(optional: T?) {
-    appendInterpolation(String(describing: optional))
+    if let optional = optional {
+      appendInterpolation(String(describing: optional))
+    } else {
+      appendInterpolation("nil")
+    }
   }
-}
+}

package/ios/RCTMGL-v10/RCTMGLEvent.swift

@@ -1,15 +1,14 @@
 import Foundation
 
 protocol RCTMGLEventProtocol {
-
-    func toJSON() -> [String: Any];
+    func toJSON() -> [String: Any?];
 }
 
 @objc
 class RCTMGLEvent : NSObject, RCTMGLEventProtocol {
     var type: String = ""
-    var payload: [String:Any]? = nil
-    func toJSON() -> [String: Any]
+    var payload: [String:Any?]? = nil
+    func toJSON() -> [String: Any?]
     {
         if let payload = payload {
             return ["type": type, "payload": payload];
@@ -41,7 +40,7 @@ class RCTMGLEvent : NSObject, RCTMGLEventProtocol {
       case shapeSourceLayerPress
     }
     
-    init(type: EventType, payload: [String:Any]?) {
+    init(type: EventType, payload: [String:Any?]?) {
         self.type = type.rawValue
         self.payload = payload
     }

package/ios/RCTMGL-v10/RCTMGLImageSourceManager.m

@@ -4,6 +4,7 @@
 @interface RCT_EXTERN_MODULE(RCTMGLImageSourceManager, RCTViewManager)
 
 RCT_EXPORT_VIEW_PROPERTY(id, NSString)
+RCT_EXPORT_VIEW_PROPERTY(existing, BOOL)
 RCT_EXPORT_VIEW_PROPERTY(url, NSString)
 RCT_EXPORT_VIEW_PROPERTY(coordinates, NSArray)
 

package/ios/RCTMGL-v10/RCTMGLLogging.swift

@@ -20,7 +20,7 @@ class Logger {
     case verbose = "verbose"
     case debug = "debug"
     case info = "info"
-    case warn = "warn"
+    case warn = "warning"
     case error = "error"
   
     var intValue : Int {

package/ios/RCTMGL-v10/RCTMGLOfflineModule.swift

@@ -98,7 +98,7 @@ class RCTMGLOfflineModule: RCTEventEmitter {
     var cancelable: Cancelable? = nil
     var progress : TileRegionLoadProgress? = nil
     var state : State = .inactive
-    var metadata : [String:Any]? = nil
+    var metadata : [String:Any]
 
     // Stored in metadata for resume functionality:
     var bounds: Geometry? = nil
@@ -190,22 +190,22 @@ class RCTMGLOfflineModule: RCTEventEmitter {
                      resolver: @escaping RCTPromiseResolveBlock,
                      rejecter: @escaping RCTPromiseRejectBlock) {
     guard let pack = tileRegionPacks[name] else {
-      resolver(nil)
+      rejecter("RCTMGLOfflineModule.getPackStatus", "pack \(name) not found", nil)
       return
     }
     
     tileStore.tileRegionMetadata(forId: name) { result in
       switch result {
-      case .failure(let error):
-        Logger.log(level:.error, message: "Unable to fetch metadata for \(name)")
-        rejecter("RCTMGLOfflineModule.getPackStatus", error.localizedDescription, error)
       case .success(let metadata):
-        var pack = self.tileRegionPacks[name] ?? TileRegionPack(
+        var pack = TileRegionPack(
           name: name,
           metadata: logged("RCTMGLOfflineModule.getPackStatus") { metadata as? [String:Any] } ?? [:]
         )
         self.tileRegionPacks[name] = pack
         resolver(self._makeRegionStatusPayload(pack: pack))
+      case .failure(let error):
+        Logger.log(level:.error, message: "Unable to fetch metadata for \(name)")
+        rejecter("RCTMGLOfflineModule.getPackStatus", error.localizedDescription, error)
       }
     }
   }
@@ -325,6 +325,7 @@ class RCTMGLOfflineModule: RCTEventEmitter {
   
   func startLoading(pack: TileRegionPack) {
     let id = pack.name
+    let metadata = pack.metadata
     guard let bounds = pack.bounds else {
       RCTMGLLogError("RCTMGLOfflineModule.startLoading failed as there are no bounds in pack")
       return
@@ -337,10 +338,6 @@ class RCTMGLOfflineModule: RCTEventEmitter {
       RCTMGLLogError("RCTMGLOfflineModule.startLoading failed as there is no styleURI in pack")
       return
     }
-    guard let metadata = pack.metadata else {
-      RCTMGLLogError("RCTMGLOfflineModule.startLoading failed as there is no metadata in pack")
-      return
-    }
     
     let stylePackLoadOptions = StylePackLoadOptions(glyphsRasterizationMode: .ideographsRasterizedLocally, metadata: pack.metadata)
     
@@ -447,7 +444,7 @@ class RCTMGLOfflineModule: RCTEventEmitter {
           metadata: logged("RCTMGLOfflineModule.getPacks metadata is null") { metadata } ?? [:]
         )
         
-        if ((region.completedResourceCount == region.requiredResourceCount)) {
+        if ((region.hasCompleted())) {
           pack.state = .complete
         }
         
@@ -482,8 +479,7 @@ class RCTMGLOfflineModule: RCTEventEmitter {
       ]
 
       if region.requiredResourceCount > 0 {
-        let percentage = Float(100.0) * Float(region.completedResourceCount) / Float(region.requiredResourceCount)
-        result["percentage"] = percentage
+        result["percentage"] = region.toPercentage()
       } else {
         result["percentage"] = nil
       }
@@ -535,12 +531,10 @@ class RCTMGLOfflineModule: RCTEventEmitter {
   func _makeRegionStatusPayload(_ name:String, progress: TileRegionLoadProgress?, state: State, metadata:[String:Any]?) -> [String:Any?] {
     var result : [String:Any?] = [:]
     if let progress = progress {
-      let progressPercentage =  Float(progress.completedResourceCount) / Float(progress.requiredResourceCount)
-      
       result = [
-        "state": (progress.completedResourceCount == progress.requiredResourceCount) ? State.complete.rawValue : state.rawValue,
+        "state": (progress.hasCompleted()) ? State.complete.rawValue : state.rawValue,
         "name": name,
-        "percentage": progressPercentage * 100.0,
+        "percentage": progress.toPercentage(),
         "completedResourceCount": progress.completedResourceCount,
         "completedResourceSize": progress.completedResourceSize,
         "erroredResourceCount": progress.erroredResourceCount,
@@ -615,3 +609,20 @@ extension RCTMGLOfflineModule {
   }
 }
 
+extension TileRegionLoadProgress {
+  func toPercentage() -> Float {
+    return Float(100.0) * Float(completedResourceCount) / Float(requiredResourceCount);
+  }
+  func hasCompleted() -> Bool {
+    return (completedResourceCount == requiredResourceCount)
+  }
+}
+
+extension TileRegion {
+  func toPercentage() -> Float {
+    return Float(100.0) * Float(completedResourceCount) / Float(requiredResourceCount)
+  }
+  func hasCompleted() -> Bool {
+    return (completedResourceCount == requiredResourceCount)
+  }
+}

package/ios/RCTMGL-v10/RCTMGLRasterDemSourceManager.m

@@ -4,6 +4,7 @@
 @interface RCT_EXTERN_MODULE(RCTMGLRasterDemSourceManager, RCTViewManager)
 
 RCT_EXPORT_VIEW_PROPERTY(id, NSString)
+RCT_EXPORT_VIEW_PROPERTY(existing, BOOL)
 RCT_EXPORT_VIEW_PROPERTY(url, NSString)
 RCT_EXPORT_VIEW_PROPERTY(tileUrlTemplates, NSArray)
 RCT_EXPORT_VIEW_PROPERTY(bounds, NSArray)

package/ios/RCTMGL-v10/RCTMGLRasterSourceManager.m

@@ -4,6 +4,7 @@
 @interface RCT_EXTERN_MODULE(RCTMGLRasterSourceManager, RCTViewManager)
 
 RCT_EXPORT_VIEW_PROPERTY(id, NSString)
+RCT_EXPORT_VIEW_PROPERTY(existing, BOOL)
 RCT_EXPORT_VIEW_PROPERTY(url, NSString)
 RCT_EXPORT_VIEW_PROPERTY(tileUrlTemplates, NSArray)
 

package/ios/RCTMGL-v10/RCTMGLShapeSourceManager.m

@@ -5,6 +5,7 @@
 RCT_EXTERN_REMAP_MODULE(RCTMGLShapeSource, RCTMGLShapeSourceManager, RCTViewManager)
 
 RCT_EXPORT_VIEW_PROPERTY(id, NSString)
+RCT_EXPORT_VIEW_PROPERTY(existing, BOOL)
 RCT_EXPORT_VIEW_PROPERTY(url, NSString)
 RCT_EXPORT_VIEW_PROPERTY(shape, NSString)
 

package/ios/RCTMGL-v10/RCTMGLSource.swift

@@ -8,6 +8,8 @@ class RCTMGLSource : RCTMGLInteractiveElement {
   var source : Source? = nil
 
   var ownsSource : Bool = false
+
+  @objc var existing: Bool = false
   
   override func getLayerIDs() -> [String] {
     layers.compactMap {
@@ -69,8 +71,14 @@ class RCTMGLSource : RCTMGLInteractiveElement {
     self.map = map
 
     if style.sourceExists(withId: self.id) {
+      if (!existing) {
+        Logger.log(level: .warn, message: "Warning source with id:\(optional: id) referred to existing source but `existing` attibute was missing. https://github.com/rnmapbox/maps/wiki/Deprecated-ExistingSourceLayer")
+      }
       self.source = try! style.source(withId: self.id)
     } else {
+      if (existing) {
+        Logger.log(level: .warn, message: "Warning source with id:\(optional: id) marked as existing, but could not find in style, source identifiers: \(style.allSourceIdentifiers.map { [$0.id, $0.type.rawValue] })")
+      }
       let source = self.makeSource()
       self.ownsSource = true
       self.source = source