Hotfix: Fixed an inability to open the file browser

Wohlstand · Jul 3, 2023 · ee3834d · ee3834d
1 parent a84367a
commit ee3834d
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/OPNMIDI-Player/build.gradle b/OPNMIDI-Player/build.gradle
@@ -19,8 +19,8 @@ android {
         applicationId "ru.wohlsoft.opnmidiplayer"
         minSdkVersion 14
         targetSdkVersion 33
-        versionCode 1050010
-        versionName '1.5.0.10'
+        versionCode 1050011
+        versionName '1.5.0.11'
         testInstrumentationRunner 'androidx.test.runner.AndroidJUnitRunner'
         externalNativeBuild {
             cmake {

diff --git a/OPNMIDI-Player/src/main/java/ru/wohlsoft/opnmidiplayer/Player.java b/OPNMIDI-Player/src/main/java/ru/wohlsoft/opnmidiplayer/Player.java
@@ -779,8 +779,11 @@ private void checkNotificationsPermissions(int requestCode)
 
     private boolean checkFilePermissions(int requestCode)
     {
+        if(android.os.Build.VERSION.SDK_INT < android.os.Build.VERSION_CODES.JELLY_BEAN)
+            return false;
+
         final int grant = PackageManager.PERMISSION_GRANTED;
-        final String exStorage = Manifest.permission.WRITE_EXTERNAL_STORAGE;
+        final String exStorage = Manifest.permission.READ_EXTERNAL_STORAGE;
 
         if(Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU)
             return false; /* Has no effect, the manage file storage permission is used instead of this */
@@ -849,12 +852,15 @@ public void onClick(DialogInterface dialog, int whichButton)
             b.show();
             return false;
         }
+
         return true;
     }
 
     @Override
-    public void onRequestPermissionsResult(int requestCode, @NonNull String[] permissions,
-                                           @NonNull int[] grantResults) {
+    public void onRequestPermissionsResult(int requestCode,
+                                           @NonNull String[] permissions,
+                                           @NonNull int[] grantResults)
+    {
         super.onRequestPermissionsResult(requestCode, permissions, grantResults);
 
         if(grantResults.length == 0)