From a7f6af28de52d2023fad84a68634c8ee14e3c597 Mon Sep 17 00:00:00 2001
From: ByteHamster <info@bytehamster.com>
Date: Thu, 2 Jul 2020 20:19:14 +0200
Subject: Adding internal storage if no external exists

---
 .../antennapod/adapter/DataFolderAdapter.java      | 26 +++++++++++++---------
 1 file changed, 16 insertions(+), 10 deletions(-)

(limited to 'app/src/main/java/de/danoeh/antennapod/adapter/DataFolderAdapter.java')

diff --git a/app/src/main/java/de/danoeh/antennapod/adapter/DataFolderAdapter.java b/app/src/main/java/de/danoeh/antennapod/adapter/DataFolderAdapter.java
index 64560df56..2c35d5a64 100644
--- a/app/src/main/java/de/danoeh/antennapod/adapter/DataFolderAdapter.java
+++ b/app/src/main/java/de/danoeh/antennapod/adapter/DataFolderAdapter.java
@@ -78,7 +78,9 @@ public class DataFolderAdapter extends RecyclerView.Adapter<DataFolderAdapter.Vi
 
     private String getCurrentPath() {
         File dataFolder = UserPreferences.getDataFolder(null);
-        if (dataFolder != null) return dataFolder.getAbsolutePath();
+        if (dataFolder != null) {
+            return dataFolder.getAbsolutePath();
+        }
         return null;
     }
 
@@ -86,15 +88,19 @@ public class DataFolderAdapter extends RecyclerView.Adapter<DataFolderAdapter.Vi
         File[] mediaDirs = ContextCompat.getExternalFilesDirs(context, null);
         final List<StoragePath> entries = new ArrayList<>(mediaDirs.length);
         for (File dir : mediaDirs) {
-            if (isNotWritable(dir)) continue;
-
+            if (!isWritable(dir)) {
+                continue;
+            }
             entries.add(new StoragePath(dir.getAbsolutePath()));
         }
+        if (entries.isEmpty() && isWritable(context.getFilesDir())) {
+            entries.add(new StoragePath(context.getFilesDir().getAbsolutePath()));
+        }
         return entries;
     }
 
-    private boolean isNotWritable(File dir) {
-        return dir == null || !dir.exists() || !dir.canRead() || !dir.canWrite();
+    private boolean isWritable(File dir) {
+        return dir != null && dir.exists() && dir.canRead() && dir.canWrite();
     }
 
     private void selectAndDismiss(StoragePath storagePath) {
@@ -103,11 +109,11 @@ public class DataFolderAdapter extends RecyclerView.Adapter<DataFolderAdapter.Vi
     }
 
     static class ViewHolder extends RecyclerView.ViewHolder {
-        private View root;
-        private TextView path;
-        private TextView size;
-        private RadioButton radioButton;
-        private ProgressBar progressBar;
+        private final View root;
+        private final TextView path;
+        private final TextView size;
+        private final RadioButton radioButton;
+        private final ProgressBar progressBar;
 
         ViewHolder(View itemView) {
             super(itemView);
-- 
cgit v1.2.3


From 65f844fc8329ef8da38eff04beff7021ba8be989 Mon Sep 17 00:00:00 2001
From: ByteHamster <info@bytehamster.com>
Date: Thu, 2 Jul 2020 22:44:50 +0200
Subject: Allow to select new folder even if current folder is null

---
 .../main/java/de/danoeh/antennapod/adapter/DataFolderAdapter.java   | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

(limited to 'app/src/main/java/de/danoeh/antennapod/adapter/DataFolderAdapter.java')

diff --git a/app/src/main/java/de/danoeh/antennapod/adapter/DataFolderAdapter.java b/app/src/main/java/de/danoeh/antennapod/adapter/DataFolderAdapter.java
index 2c35d5a64..5848d37f7 100644
--- a/app/src/main/java/de/danoeh/antennapod/adapter/DataFolderAdapter.java
+++ b/app/src/main/java/de/danoeh/antennapod/adapter/DataFolderAdapter.java
@@ -65,11 +65,7 @@ public class DataFolderAdapter extends RecyclerView.Adapter<DataFolderAdapter.Vi
 
     @Override
     public int getItemCount() {
-        if (currentPath == null) {
-            return 0;
-        } else {
-            return entries.size();
-        }
+        return entries.size();
     }
 
     public void setDialog(Dialog dialog) {
-- 
cgit v1.2.3


From 1c887a418a9157e807989e8ed3c64a601200cdc2 Mon Sep 17 00:00:00 2001
From: ByteHamster <info@bytehamster.com>
Date: Thu, 2 Jul 2020 23:01:03 +0200
Subject: Removed custom Consumer implementation and decoupled adapter from
 dialog

---
 .../antennapod/adapter/DataFolderAdapter.java      | 32 +++++++---------------
 1 file changed, 10 insertions(+), 22 deletions(-)

(limited to 'app/src/main/java/de/danoeh/antennapod/adapter/DataFolderAdapter.java')

diff --git a/app/src/main/java/de/danoeh/antennapod/adapter/DataFolderAdapter.java b/app/src/main/java/de/danoeh/antennapod/adapter/DataFolderAdapter.java
index 5848d37f7..bcad1b5a4 100644
--- a/app/src/main/java/de/danoeh/antennapod/adapter/DataFolderAdapter.java
+++ b/app/src/main/java/de/danoeh/antennapod/adapter/DataFolderAdapter.java
@@ -1,6 +1,5 @@
 package de.danoeh.antennapod.adapter;
 
-import android.app.Dialog;
 import android.content.Context;
 import android.text.format.Formatter;
 import android.view.LayoutInflater;
@@ -9,29 +8,25 @@ import android.view.ViewGroup;
 import android.widget.ProgressBar;
 import android.widget.RadioButton;
 import android.widget.TextView;
-
 import androidx.annotation.NonNull;
 import androidx.core.content.ContextCompat;
+import androidx.core.util.Consumer;
 import androidx.recyclerview.widget.RecyclerView;
-
-import java.io.File;
-import java.util.ArrayList;
-import java.util.List;
-
 import de.danoeh.antennapod.R;
 import de.danoeh.antennapod.core.preferences.UserPreferences;
 import de.danoeh.antennapod.core.util.StorageUtils;
-import de.danoeh.antennapod.dialog.ChooseDataFolderDialog;
 
+import java.io.File;
+import java.util.ArrayList;
+import java.util.List;
 
 public class DataFolderAdapter extends RecyclerView.Adapter<DataFolderAdapter.ViewHolder> {
-    private final ChooseDataFolderDialog.RunnableWithString selectionHandler;
+    private final Consumer<String> selectionHandler;
     private final String currentPath;
     private final List<StoragePath> entries;
     private final String freeSpaceString;
-    private Dialog dialog;
 
-    public DataFolderAdapter(Context context, ChooseDataFolderDialog.RunnableWithString selectionHandler) {
+    public DataFolderAdapter(Context context, @NonNull Consumer<String> selectionHandler) {
         this.entries = getStorageEntries(context);
         this.currentPath = getCurrentPath();
         this.selectionHandler = selectionHandler;
@@ -56,8 +51,10 @@ public class DataFolderAdapter extends RecyclerView.Adapter<DataFolderAdapter.Vi
         holder.path.setText(storagePath.getShortPath());
         holder.size.setText(String.format(freeSpaceString, freeSpace, totalSpace));
         holder.progressBar.setProgress(storagePath.getUsagePercentage());
-        holder.root.setOnClickListener((View v) -> selectAndDismiss(storagePath));
-        holder.radioButton.setOnClickListener((View v) -> selectAndDismiss(storagePath));
+        View.OnClickListener selectListener = v -> selectionHandler.accept(storagePath.getFullPath());
+        holder.root.setOnClickListener(selectListener);
+        holder.radioButton.setOnClickListener(selectListener);
+
         if (storagePath.getFullPath().equals(currentPath)) {
             holder.radioButton.toggle();
         }
@@ -68,10 +65,6 @@ public class DataFolderAdapter extends RecyclerView.Adapter<DataFolderAdapter.Vi
         return entries.size();
     }
 
-    public void setDialog(Dialog dialog) {
-        this.dialog = dialog;
-    }
-
     private String getCurrentPath() {
         File dataFolder = UserPreferences.getDataFolder(null);
         if (dataFolder != null) {
@@ -99,11 +92,6 @@ public class DataFolderAdapter extends RecyclerView.Adapter<DataFolderAdapter.Vi
         return dir != null && dir.exists() && dir.canRead() && dir.canWrite();
     }
 
-    private void selectAndDismiss(StoragePath storagePath) {
-        selectionHandler.run(storagePath.getFullPath());
-        dialog.dismiss();
-    }
-
     static class ViewHolder extends RecyclerView.ViewHolder {
         private final View root;
         private final TextView path;
-- 
cgit v1.2.3