135

There is a lint warning in AS with regards to android.permission.WRITE_EXTERNAL_STORAGE. The warning says that the permission will no longer provide write access when targeting Android 10 and above. Removal of the said permission can still write in internal storage folder Pictures/MY_APP_NAME to save images, but it only works on Android 10 (SDK 29) and/or above (haven't tested yet on Android R). When I tested it again on lower version such as Android M (SDK 23), saving images stop working so I decided to return the android.permission.WRITE_EXTERNAL_STORAGE thus the warning shows up again. Is it possible that the lint is just false positive that incorrectly diagnosed the problem on different cases? Because currently my support SDK starts with 21 up to the latest which is 30 but the lint only point out that it is no longer needed when targeting Android 10 (SDK 29) and did not consider looking back at the project's minimum SDK support.

11
  • can still write in internal storage folder Pictures/MY_APP_NAME Strange. Please give full path of folder.
    – blackapps
    Oct 6, 2020 at 7:44
  • You can still write to Internal Storage/Pictures/MY_APP_NAME using MediaStore, ContentResolver, and ContentValues in Android 10. Oct 6, 2020 at 7:48
  • 1
    @SiddharthKamaria yeah, I tried adding android:maxSdkVersion="29" but does not remove the warning. Oct 6, 2020 at 8:20
  • 1
    @MihaeKheel Weird enough my AS is not showing me lint warning for WRITE_EXTERNAL_STORAGE even without maxSdkVersion. Maybe try rebuilding or invalidating caches? Oct 6, 2020 at 8:24
  • 1
    @SiddharthKamaria thanks for pointing out but I already tried invalidating, clean build, and rebuild the warning still exist. Maybe it's due to us having a different AS version I am in Canary version of AS anyway. Oct 6, 2020 at 8:31

6 Answers 6

140

A workaround is to actually ignore the warning, as it is just informational and therefore harmless. By setting maxSdkVersion to 28 no need to worry anymore.

<uses-permission
    android:name="android.permission.WRITE_EXTERNAL_STORAGE"
    android:maxSdkVersion="28"
    tools:ignore="ScopedStorage" />

Note that using the android:requestLegacyExternalStorage flag as stated in other answers is not a solution, is just a temporary patch which will no longer work at all in Android 11 (API 30), and future versions

UPDATE, to clarify the doubts and confusions shown by some developers in the comments:

  • If using the requestLegacyExternalStorage flag in Android 10 (API 29) then request the WRITE_EXTERNAL_STORAGE permission as usual.

  • The flag requestLegacyExternalStorage does nothing in Android 11 (API 30), it is completely ignored, and there is not workaround for it.

  • WRITE_EXTERNAL_STORAGE does not give any privileges in Android 11 (API 30), it does nothing at all, therefore in API 11 you need to set the maxSdkVersion to 29.

  • If in Android 10 (API 29) you are also not using requestLegacyExternalStorage then set maxSdkVersion to 28 instead of 29.

  • Starting in Android 11 (API 30), the older File API can again be used but "only" when accessing the public "shared storage" folders (DCIM, Music, etc.), or your app "private" directory. For other locations the DocumentFile API is required.

  • Consider that the File API is now much slower in Android 11 (API 30), because has been refactored becoming essentially a wrapper. This is to enforce its usage just to the allowed locations. So, is no longer a fast system file API, is just a wrapper that internally delegates the work to the MediaStore. When using the File API in Android 11 or above you should consider the performance penalty hit, as according to the Android team it will be 2 to 3 times slower than if accessing directly the MediaStore.

6
  • thank you for your feedback, I am surprised to read that, as the documentation under developer.android.com/guide/topics/manifest/… says that installation will not be possible and in case of revalidation after a system update the app will even be removed
    – Christian
    Jan 28, 2021 at 15:38
  • 8
    @Christian Ok, now I understand your confusion. The link you mention refers to the "<uses-sdk>" element, and not to <uses-permission>. These are 2 different type of elements, and "maxSdkVersion" is actually a common "attribute" which exists for any type of element. What you must "not do" is to declare "maxSdkVersion" in the "<uses-sdk>" element, but this is unrelated to permissions. Check the actual <uses-permision> documentation where it explains how to use "maxSdkVersion" to limit a permission as per API level: developer.android.com/guide/topics/manifest/… Jan 28, 2021 at 16:27
  • 1
    o.k., now I understand your idea. But still you will not get the legacy file access on Android 10 and above, correct? Like folder.listFiles() will not work
    – Christian
    Jan 28, 2021 at 21:30
  • 1
    @Christian For Andorid 10 use requestLegacyExternalStorage. And for Android 11 the File API can be used "only" when accessing the public "shared storage" folders (DCIM, Music, etc.), or your "private" app directory. For other locations use the DocumentFile API. Note that the File API is slower in API 30, because has been refactored as a wrapper to enforce its usage to only the allowed locations. Internally delegates the work to the MediaStore. The flag "requestLegacyExternalStorage" no longer works in Android 11 as it was just temporary to give developers enough time to perform the migration. Jan 28, 2021 at 22:47
  • 1
    This did not mention about the possible problem setting maxSdkVersion to 28 since ContextCompat.checkSelfPermission(this, Manifest.permission.WRITE_EXTERNAL_STORAGE) == PackageManager.PERMISSION_GRANTED will never return true in 29. What a mistake patch update I made in the store. May 1, 2021 at 19:12
30

Try to add this in your Manifest:

<application
android:allowBackup="true"
android:icon="@mipmap/ic_launcher"
android:requestLegacyExternalStorage="true" //Add this Line
android:label="@string/app_name">

 ---------<Activity, Sevices or Recivers>----------

</application>

and remove the READ_EXTERNAL_STORAGE permission again:

<uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" />
4
  • 2
    @blackapps please read developer.android.com/training/data-storage/use-cases Oct 6, 2020 at 8:13
  • 15
    Caution: After you update your app to target Android 11 (API level 30), the system ignores the requestLegacyExternalStorage attribute when your app is running on Android 11 devices, so your app must be ready to support scoped storage and to migrate app data for users on those devices. See also stackoverflow.com/questions/63364476/… Oct 6, 2020 at 8:26
  • 3
    And again: on an Android 11 device your app can just write to /storage/emulated/0/Pictures itself. No need for media store and saf. You do not need MANAGE_EXTERNAL_STORAGE for that as @Siddharth Kamaria is suggesting.
    – blackapps
    Oct 6, 2020 at 8:36
  • 1
    Why is this contribution about READ_EXTERNAL_STORAGE? I don't believe that a read permission helps to get write access... Jan 12 at 12:13
28

Using File-apis ?

Based on @PerracoLabs reply the following changes will make your app work on the devices running

  1. Android-9 and less
  2. Android-10
  3. Android-11 and above

File: "AndroidManifest.xml"

<!-- Without this folders will be inaccessible in Android-11 and above devices -->
<uses-permission android:name="android.permission.MANAGE_EXTERNAL_STORAGE" />

<!-- Without this entry storage-permission entry will not be visible under app-info permissions list Android-10 and below -->
<uses-permission
android:name="android.permission.WRITE_EXTERNAL_STORAGE"
android:maxSdkVersion="29"
tools:ignore="ScopedStorage"/>

<!-- Without this entry the folders will remain in-accessible in Android-10, even if WRITE_EXTERNAL_STORAGE as above is present. -->
<application
    android:requestLegacyExternalStorage="true"/>

Java Source:

  1. Don't forget to give Runtime-permissions, before doing file operation. - Needed for Android-10 and below. DONT request for runtime permissions for Android-11, they are to be requested for Android-10 and below.

  2. Navigate the user to app-permission page to allow him to enable file-permissions using the following code (needed for Android-11 and above support):

     if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.R && false == Environment.isExternalStorageManager()) {
         Uri uri = Uri.parse("package:" + BuildConfig.APPLICATION_ID);
         startActivity(new Intent(Settings.ACTION_MANAGE_APP_ALL_FILES_ACCESS_PERMISSION, uri));
     }
    
2
  • 4
    " Android-9 and less Android-10 Android-11 and above " is the most complicated way to say "all Android versions" I have ever seen. Jan 12 at 12:15
  • "You should request the MANAGE_EXTERNAL_STORAGE permission only when your app cannot effectively make use of the more privacy-friendly APIs, such as Storage Access Framework or the Media Store API. Additionally, the app's usage of the permission must fall within permitted uses, and must be directly tied to the core functionality of the app." developer.android.com/training/data-storage/manage-all-files Jan 23 at 10:50
16

As answered @PerracoLabs the warning is just informational. It will be better if at first you'll read his answer. https://stackoverflow.com/a/65477206/6055194

I use WRITE_EXTERNAL_STORAGE permission only for downloading of .pdf-files into Downloads and app "private" directories by using of DownloadManager (that's important and you can understand why if you will read discussion under @PerracoLabs answer). So I've just added max level of SDK for this permission and ignore tag for scoped storage.

<uses-permission
    android:name="android.permission.WRITE_EXTERNAL_STORAGE"
    android:maxSdkVersion="28"
    tools:ignore="ScopedStorage" />

P.S. if tools tag not recognized, than you must declare it by adding xmlns:tools="http://schemas.android.com/tools" to your AndroidManifest.xml like this.

<manifest xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    package="com.app.my">

But you must note that after it onRequestPermissionsResult function will not be called for Build.VERSION_CODES.Q and higher (API 29+). So you must call request permission WRITE_EXTERNAL_STORAGE only for API lower than 29.

So you can add check write external storage function like this (Kotlin).

   private fun hasWriteStoragePermission(): Boolean {
        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
            return true
        } else if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.M) {
            if (ActivityCompat.checkSelfPermission(activity!!, Manifest.permission.WRITE_EXTERNAL_STORAGE) != PackageManager.PERMISSION_GRANTED) {
                requestPermissions(
                        arrayOf(Manifest.permission.WRITE_EXTERNAL_STORAGE),
                        REQUEST_PERMISSIONS_CODE_WRITE_STORAGE
                )

                return false
            }
        }

        return true
    }
0
1

Add the line:

<uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" tools:remove="android:maxSdkVersion"/>

and add the below code to to AndroidManifest.xml:

 android:requestLegacyExternalStorage="true"
2
  • Where should we add the 2nd line in AndroidManifest.xml? Feb 18 at 15:30
  • 1
    @PrasannjeetSingh In the <application> tag.
    – Sweta Jain
    Aug 19 at 17:05
0

Set maxSdkVersion to 32

<uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE"
    android:maxSdkVersion="32"
    tools:ignore="ScopedStorage" />

Not the answer you're looking for? Browse other questions tagged or ask your own question.