Updated PermissionsLauncherFragment to only request permissions that are missing.

Author: tomaszrybakiewicz

libnavui-dropin/src/main/java/com/mapbox/navigation/dropin/permission/LocationPermissionComponent.kt

@@ -66,7 +66,7 @@ internal class LocationPermissionComponent(
         } else {
             val fragActivity = componentActivityRef.get() as? FragmentActivity
             if (fragActivity != null) {
-                PermissionsLauncherFragment.create(fragActivity, LOCATION_PERMISSIONS, callback)
+                PermissionsLauncherFragment.install(fragActivity, LOCATION_PERMISSIONS, callback)
             } else {
                 launcher?.launch(LOCATION_PERMISSIONS)
             }
@@ -99,7 +99,7 @@ internal class LocationPermissionComponent(
         super.onDetached(mapboxNavigation)
 
         (componentActivityRef.get() as? FragmentActivity)?.also {
-            PermissionsLauncherFragment.destroy(it)
+            PermissionsLauncherFragment.uninstall(it)
         }
         launcher?.unregister()
     }

libnavui-dropin/src/main/java/com/mapbox/navigation/dropin/permission/PermissionsLauncherFragment.kt

@@ -22,12 +22,12 @@ internal class PermissionsLauncherFragment(
 
     override fun onAttach(context: Context) {
         super.onAttach(context)
-        val areGranted = permissions.fold(false) { acc, permission ->
-            acc && checkSelfPermission(context, permission) == PERMISSION_GRANTED
+        val missingPermissions = permissions.filter { permission ->
+            checkSelfPermission(context, permission) != PERMISSION_GRANTED
         }
-        if (!areGranted) {
+        if (missingPermissions.isNotEmpty()) {
             launcher = registerForActivityResult(RequestMultiplePermissions(), onResult)
-            launcher?.launch(permissions)
+            launcher?.launch(missingPermissions.toTypedArray())
         }
     }
 
@@ -39,7 +39,7 @@ internal class PermissionsLauncherFragment(
     companion object {
         const val TAG = "MapboxPermissionsLauncherFragment"
 
-        fun create(
+        fun install(
             fragActivity: FragmentActivity,
             permissions: Array<String>,
             onResult: ActivityResultCallback<Map<String, Boolean>>
@@ -52,7 +52,7 @@ internal class PermissionsLauncherFragment(
             }
         }
 
-        fun destroy(fragActivity: FragmentActivity) {
+        fun uninstall(fragActivity: FragmentActivity) {
             if (!fragActivity.isFinishing) {
                 fragActivity.supportFragmentManager.apply {
                     findFragmentByTag(TAG)?.also {