Fix Nullpointer getMap() (#109)

* catch exceptions while parsing and fix poliline on mobile

* add missing return

* initial draft on iOS

* add symbol methods to controller

* parse symbol

* implement useCourseSymbolOnMovement

* implement new type of location display data source

* implement feature

* implement feature

* add example on button click

* add support for local asset android

* add support for local asset android

* fix logic

* implement iOS

* reformat

* review

* update README.md and add button for example only on mobile

* finish iOS location indicator impl

* export

* implement android

* merge indicator

* fix android

* try to fix wrong initial scale

* implement licenseKey and make apiKey optional

* print statement

* try to fix zoom

* fix zoom gets not initialized correctly

* remove print

* fix android error parsing symbol and iOS zoom

* fix bug for manual location source on android

* revert changes

* typo

* fix NaN android getMapScale

* fix

* correct round

* remove web sdk

* format

* fix merge

* fix ios build

* remove duplicate declaration

* remove newline

* Add isAttributionTextVisible to hide map attribution

* Add isAttributionTextVisible for iOS

* Add dependency_overrides

* implement changing attribution text

* implement image-export

* add layer status observation

* remove map status observeration

* implement image export on android

* update readme

* update docs

* fix duplicate code

* specify android namespace

* cleanup

* override setMethodCallHandler

* fix map status callback

* reformat code

* update ArcGis ios sdk

* Revert "update ArcGis ios sdk"

This reverts commit 1e07877.

* fix nullpointer

* revert tollnow specific stuff

---------

Co-authored-by: Julian Bissekkou <julian.bissekkou@tapped.dev>
Co-authored-by: Jean <jean.devezi@tapped.dev>
Co-authored-by: QLF3985 <stefan.bergmair@external.t-systems.com>

Author: 4 people

arcgis_map_sdk_android/android/src/main/kotlin/dev/fluttercommunity/arcgis_map_sdk_android/ArcgisMapView.kt

@@ -72,6 +72,7 @@ internal class ArcgisMapView(
     private val graphicsParser = GraphicsParser(binding)
 
     private val initialZoom: Int
+    private var isDisposed = false
 
     private lateinit var zoomStreamHandler: ZoomStreamHandler
     private lateinit var centerPositionStreamHandler: CenterPositionStreamHandler
@@ -155,6 +156,8 @@ internal class ArcgisMapView(
     }
 
     override fun dispose() {
+        isDisposed = true
+        methodChannel.setMethodCallHandler(null)
         coroutineScope.cancel()
 
         lifecycle.removeObserver(mapView)
@@ -165,6 +168,10 @@ internal class ArcgisMapView(
 
     private fun setupMethodChannel() {
         methodChannel.setMethodCallHandler { call, result ->
+            if (isDisposed) {
+                result.error("disposed", "ArcgisMapView has been disposed", null)
+                return@setMethodCallHandler
+            }
             when (call.method) {
                 "zoom_in" -> onZoomIn(call = call, result = result)
                 "zoom_out" -> onZoomOut(call = call, result = result)
@@ -651,9 +658,13 @@ internal class ArcgisMapView(
 
     private fun onRetryLoad(result: MethodChannel.Result) {
         coroutineScope.launch {
-            mapView.map?.retryLoad()?.onSuccess {
-                result.success(true)
-            }?.onFailure { e ->
+            try {
+                map.retryLoad().onSuccess {
+                    result.success(true)
+                }.onFailure { e ->
+                    result.finishWithError(e)
+                }
+            } catch (e: Throwable) {
                 result.finishWithError(e)
             }
         }