From c9e48a7b35eee5266f60f56ee75d2f17a679f4ab Mon Sep 17 00:00:00 2001
From: dkanada <dkanada@users.noreply.github.com>
Date: Sun, 30 May 2021 16:21:44 +0900
Subject: [PATCH] only request storage permission when external directory is
 chosen

---
 .../activities/base/AbsMusicServiceActivity.java          | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/app/src/main/java/com/dkanada/gramophone/activities/base/AbsMusicServiceActivity.java b/app/src/main/java/com/dkanada/gramophone/activities/base/AbsMusicServiceActivity.java
index dd49accb..fd2807f9 100644
--- a/app/src/main/java/com/dkanada/gramophone/activities/base/AbsMusicServiceActivity.java
+++ b/app/src/main/java/com/dkanada/gramophone/activities/base/AbsMusicServiceActivity.java
@@ -17,6 +17,7 @@ import com.dkanada.gramophone.R;
 import com.dkanada.gramophone.helper.MusicPlayerRemote;
 import com.dkanada.gramophone.interfaces.MusicServiceEventListener;
 import com.dkanada.gramophone.service.MusicService;
+import com.dkanada.gramophone.util.PreferenceUtil;
 
 import java.lang.ref.WeakReference;
 import java.util.ArrayList;
@@ -198,6 +199,13 @@ public abstract class AbsMusicServiceActivity extends AbsBaseActivity implements
 
     @Override
     protected List<String> getPermissionRequest() {
+        Context context = getApplicationContext();
+        String downloadLocation = PreferenceUtil.getInstance(context).getLocationDownload();
+
+        if (downloadLocation.equals(context.getCacheDir().toString())) {
+            return new ArrayList<>();
+        }
+
         return Arrays.asList(Manifest.permission.READ_EXTERNAL_STORAGE, Manifest.permission.WRITE_EXTERNAL_STORAGE);
     }
 }