Fix incorrect coroutine scope (#6398)

kmadsen · web-flow · commit 6bc3884dff66 · 2022-09-27T16:49:55.000-07:00
diff --git a/libnavui-androidauto/src/main/java/com/mapbox/androidauto/car/placeslistonmap/PlacesListOnMapManager.kt b/libnavui-androidauto/src/main/java/com/mapbox/androidauto/car/placeslistonmap/PlacesListOnMapManager.kt
@@ -85,6 +85,8 @@ class PlacesListOnMapManager(
         styleLoadedListener = null
         MapboxNavigationApp.unregisterObserver(navigationObserver)
         carMapSurface = null
+        coroutineScope?.cancel()
+        coroutineScope = null
     }
 
     private fun onAttached(mapboxNavigation: MapboxNavigation) {
@@ -99,8 +101,6 @@ class PlacesListOnMapManager(
 
     private fun onDetached() {
         placesListItemMapper = null
-        coroutineScope?.cancel()
-        coroutineScope = null
     }
 
     private fun loadPlaceRecords() {

Original file line number	Diff line number	Diff line change
`@@ -85,6 +85,8 @@ class PlacesListOnMapManager(`
`85`	`85`	`styleLoadedListener = null`
`86`	`86`	`MapboxNavigationApp.unregisterObserver(navigationObserver)`
`87`	`87`	`carMapSurface = null`
	`88`	`+ coroutineScope?.cancel()`
	`89`	`+ coroutineScope = null`
`88`	`90`	`}`
`89`	`91`
`90`	`92`	`private fun onAttached(mapboxNavigation: MapboxNavigation) {`
`@@ -99,8 +101,6 @@ class PlacesListOnMapManager(`
`99`	`101`
`100`	`102`	`private fun onDetached() {`
`101`	`103`	`placesListItemMapper = null`
`102`		`- coroutineScope?.cancel()`
`103`		`- coroutineScope = null`
`104`	`104`	`}`
`105`	`105`
`106`	`106`	`private fun loadPlaceRecords() {`