Merge pull request #14405 from nextcloud/dont-ask-ACCESS_MEDIA_LOCATI…

…ON-permission-if-device-have-full-disk-access Don't Ask ACCESS_MEDIA_LOCATION Permission If App Already Has Full Disk Access
nextcloud · Jan 14, 2025 · cb137eb · cb137eb
2 parents 0abff31 + 5c8f4b1
commit cb137eb
Showing 1 changed file with 16 additions and 8 deletions.
diff --git a/app/src/main/java/com/owncloud/android/utils/PermissionUtil.kt b/app/src/main/java/com/owncloud/android/utils/PermissionUtil.kt
@@ -313,14 +313,22 @@ object PermissionUtil {
      */
     @JvmStatic
     fun requestMediaLocationPermission(activity: Activity) {
-        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
-            if (!checkSelfPermission(activity, Manifest.permission.ACCESS_MEDIA_LOCATION)) {
-                ActivityCompat.requestPermissions(
-                    activity,
-                    arrayOf(Manifest.permission.ACCESS_MEDIA_LOCATION),
-                    PERMISSIONS_MEDIA_LOCATION
-                )
-            }
+        if (Build.VERSION.SDK_INT < Build.VERSION_CODES.Q) {
+            return
+        }
+
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.R && Environment.isExternalStorageManager()) {
+            return
         }
+
+        if (checkSelfPermission(activity, Manifest.permission.ACCESS_MEDIA_LOCATION)) {
+            return
+        }
+
+        ActivityCompat.requestPermissions(
+            activity,
+            arrayOf(Manifest.permission.ACCESS_MEDIA_LOCATION),
+            PERMISSIONS_MEDIA_LOCATION
+        )
     }
 }