SD cards weren't listed on some Android devices, attempted to fix it

Author: yasirkula

.github/AAR Source (Android)/java/com/yasirkula/unity/FileBrowser.java

@@ -11,6 +11,8 @@
 import android.net.Uri;
 import android.os.Build;
 import android.os.Environment;
+import android.os.storage.StorageManager;
+import android.os.storage.StorageVolume;
 import android.provider.DocumentsContract;
 import android.util.Log;
 import android.webkit.MimeTypeMap;
@@ -20,8 +22,10 @@
 import java.io.FileOutputStream;
 import java.io.InputStream;
 import java.io.OutputStream;
+import java.lang.reflect.Method;
 import java.util.ArrayList;
 import java.util.Comparator;
+import java.util.HashSet;
 import java.util.List;
 import java.util.Locale;
 
@@ -48,14 +52,24 @@ public int compare( UriPermission a, UriPermission b )
 
 	private static final StringBuilder stringBuilder = new StringBuilder();
 
-	public static String GetExternalDrives()
+	public static String GetExternalDrives( Context context )
 	{
 		File primary = Environment.getExternalStorageDirectory();
 		String primaryPath = primary.getAbsolutePath();
+		String primaryCanonicalPath = primaryPath;
+		try
+		{
+			primaryCanonicalPath = primary.getCanonicalPath();
+		}
+		catch( Exception e )
+		{
+		}
 
 		stringBuilder.setLength( 0 );
 		stringBuilder.append( primaryPath ).append( ":" );
 
+		HashSet<String> potentialDrives = new HashSet<String>( 16 );
+
 		// Try paths saved at system environments
 		// Credit: https://stackoverflow.com/a/32088396/2373034
 		String strSDCardPath = System.getenv( "SECONDARY_STORAGE" );
@@ -69,25 +83,7 @@ public static String GetExternalDrives()
 			{
 				String path = externalPaths[i];
 				if( path != null && path.length() > 0 )
-				{
-					File file = new File( path );
-					if( file.exists() && file.isDirectory() && file.canRead() && !file.getAbsolutePath().equalsIgnoreCase( primaryPath ) )
-					{
-						String absolutePath = file.getAbsolutePath() + File.separator + "Android";
-						if( new File( absolutePath ).exists() )
-						{
-							try
-							{
-								// Check if two paths lead to same storage (aliases)
-								if( !primary.getCanonicalPath().equals( file.getCanonicalPath() ) )
-									stringBuilder.append( file.getAbsolutePath() ).append( ":" );
-							}
-							catch( Exception e )
-							{
-							}
-						}
-					}
-				}
+					potentialDrives.add( path );
 			}
 		}
 
@@ -102,29 +98,51 @@ public static String GetExternalDrives()
 				File[] fileList = new File( root ).listFiles();
 				for( File file : fileList )
 				{
-					if( file.exists() && file.isDirectory() && file.canRead() && !file.getAbsolutePath().equalsIgnoreCase( primaryPath ) )
-					{
-						String absolutePath = file.getAbsolutePath() + File.separator + "Android";
-						if( new File( absolutePath ).exists() )
-						{
-							try
-							{
-								// Check if two paths lead to same storage (aliases)
-								if( !primary.getCanonicalPath().equals( file.getCanonicalPath() ) )
-									stringBuilder.append( file.getAbsolutePath() ).append( ":" );
-							}
-							catch( Exception ex )
-							{
-							}
-						}
-					}
+					if( file.exists() && file.isDirectory() && file.canRead() )
+						potentialDrives.add( file.getAbsolutePath() );
 				}
 			}
 			catch( Exception e )
 			{
 			}
 		}
 
+		// This is the only working method on some Android 11+ devices (when Storage Access Framework isn't used)
+		if( android.os.Build.VERSION.SDK_INT >= android.os.Build.VERSION_CODES.N )
+		{
+			try
+			{
+				Method getPath = StorageVolume.class.getMethod( "getPath" );
+				StorageManager storageManager = (StorageManager) context.getSystemService( Context.STORAGE_SERVICE );
+				for( StorageVolume volume : storageManager.getStorageVolumes() )
+					potentialDrives.add( (String) getPath.invoke( volume ) );
+			}
+			catch( Exception e )
+			{
+			}
+		}
+
+		for( String potentialDrive : potentialDrives )
+		{
+			File file = new File( potentialDrive );
+			if( file.exists() && file.isDirectory() && file.canRead() && !file.getAbsolutePath().equalsIgnoreCase( primaryPath ) )
+			{
+				String absolutePath = file.getAbsolutePath() + File.separator + "Android";
+				if( new File( absolutePath ).exists() )
+				{
+					try
+					{
+						// Check if two paths lead to same storage (aliases)
+						if( !primaryCanonicalPath.equals( file.getCanonicalPath() ) )
+							stringBuilder.append( file.getAbsolutePath() ).append( ":" );
+					}
+					catch( Exception ex )
+					{
+					}
+				}
+			}
+		}
+
 		return stringBuilder.toString();
 	}
 

Plugins/SimpleFileBrowser/Android/SimpleFileBrowser.aar

@@ -939,7 +939,7 @@ private void RefreshDriveQuickLinks()
 		{
 			// Check if drives has changed since the last refresh
 #if !UNITY_EDITOR && UNITY_ANDROID
-			string drivesList = FileBrowserHelpers.AJC.CallStatic<string>( "GetExternalDrives" );
+			string drivesList = FileBrowserHelpers.AJC.CallStatic<string>( "GetExternalDrives", FileBrowserHelpers.Context );
 			if( drivesList == driveQuickLinks || ( string.IsNullOrEmpty( drivesList ) && string.IsNullOrEmpty( driveQuickLinks ) ) )
 				return;
 

Plugins/SimpleFileBrowser/Scripts/FileBrowser.cs

@@ -1,7 +1,7 @@
 {
   "name": "com.yasirkula.simplefilebrowser",
   "displayName": "Simple File Browser",
-  "version": "1.4.5",
+  "version": "1.4.6",
   "documentationUrl": "https://github.com/yasirkula/UnitySimpleFileBrowser",
   "changelogUrl": "https://github.com/yasirkula/UnitySimpleFileBrowser/releases",
   "licensesUrl": "https://github.com/yasirkula/UnitySimpleFileBrowser/blob/master/LICENSE.txt",