Adapt to write access at Android/data/sushi.hardcore.droidfs on SD cards

app/src/main/java/sushi/hardcore/droidfs/ChangePasswordActivity.kt

@@ -1,6 +1,5 @@
 package sushi.hardcore.droidfs
 
-import android.net.Uri
 import android.os.Build
 import android.os.Bundle
 import android.text.Editable
@@ -84,27 +83,6 @@ class ChangePasswordActivity : VolumeActionActivity() {
         }
     }
 
-    override fun onDirectoryPicked(uri: Uri) {
-        if (PathUtils.isTreeUriOnPrimaryStorage(uri)){
-            val path = PathUtils.getFullPathFromTreeUri(uri, this)
-            if (path != null){
-                editVolumePath.setText(path)
-            } else {
-                ColoredAlertDialogBuilder(this)
-                    .setTitle(R.string.error)
-                    .setMessage(R.string.path_from_uri_null_error_msg)
-                    .setPositiveButton(R.string.ok, null)
-                    .show()
-            }
-        } else {
-            ColoredAlertDialogBuilder(this)
-                .setTitle(R.string.warning)
-                .setMessage(R.string.change_pwd_on_sdcard_error_msg)
-                .setPositiveButton(R.string.ok, null)
-                .show()
-        }
-    }
-
     fun checkVolumePathThenChangePassword() {
         loadVolumePath {
             val volumeFile = File(currentVolumePath)
@@ -115,11 +93,7 @@ class ChangePasswordActivity : VolumeActionActivity() {
                     .setPositiveButton(R.string.ok, null)
                     .show()
             } else if (!volumeFile.canWrite()){
-                ColoredAlertDialogBuilder(this)
+                errorDirectoryNotWritable(R.string.change_pwd_cant_write_error_msg)
-                    .setTitle(R.string.warning)
-                    .setMessage(R.string.change_pwd_cant_write_error_msg)
-                    .setPositiveButton(R.string.ok, null)
-                    .show()
             } else {
                 changePassword()
             }

app/src/main/java/sushi/hardcore/droidfs/CreateActivity.kt

@@ -1,7 +1,6 @@
 package sushi.hardcore.droidfs
 
 import android.content.Intent
-import android.net.Uri
 import android.os.Build
 import android.os.Bundle
 import android.view.View
@@ -9,7 +8,6 @@ import android.widget.*
 import androidx.appcompat.app.AppCompatActivity
 import sushi.hardcore.droidfs.databinding.ActivityCreateBinding
 import sushi.hardcore.droidfs.explorers.ExplorerActivity
-import sushi.hardcore.droidfs.util.PathUtils
 import sushi.hardcore.droidfs.util.Wiper
 import sushi.hardcore.droidfs.widgets.ColoredAlertDialogBuilder
 import java.io.File
@@ -65,27 +63,6 @@ class CreateActivity : VolumeActionActivity() {
         }
     }
 
-    override fun onDirectoryPicked(uri: Uri) {
-        if (PathUtils.isTreeUriOnPrimaryStorage(uri)){
-            val path = PathUtils.getFullPathFromTreeUri(uri, this)
-            if (path != null){
-                editVolumePath.setText(path)
-            } else {
-                ColoredAlertDialogBuilder(this)
-                    .setTitle(R.string.error)
-                    .setMessage(R.string.path_from_uri_null_error_msg)
-                    .setPositiveButton(R.string.ok, null)
-                    .show()
-            }
-        } else {
-            ColoredAlertDialogBuilder(this)
-                .setTitle(R.string.warning)
-                .setMessage(R.string.create_on_sdcard_error_msg)
-                .setPositiveButton(R.string.ok, null)
-                .show()
-        }
-    }
-
     fun createVolume() {
         loadVolumePath {
             val password = binding.editPassword.text.toString().toCharArray()
@@ -93,45 +70,33 @@ class CreateActivity : VolumeActionActivity() {
             if (!password.contentEquals(passwordConfirm)) {
                 Toast.makeText(this, R.string.passwords_mismatch, Toast.LENGTH_SHORT).show()
             } else {
-                object: LoadingTask(this, R.string.loading_msg_create){
+                val volumeFile = File(currentVolumePath)
-                    override fun doTask(activity: AppCompatActivity) {
+                var goodDirectory = false
-                        val volumeFile = File(currentVolumePath)
+                if (!volumeFile.isDirectory) {
-                        var goodDirectory = false
+                    if (volumeFile.mkdirs()) {
-                        if (!volumeFile.isDirectory) {
+                        goodDirectory = true
-                            if (volumeFile.mkdirs()) {
+                    } else {
+                        errorDirectoryNotWritable(R.string.create_cant_write_error_msg)
+                    }
+                } else {
+                    val dirContent = volumeFile.list()
+                    if (dirContent != null) {
+                        if (dirContent.isEmpty()) {
+                            if (volumeFile.canWrite()) {
                                 goodDirectory = true
                             } else {
-                                stopTask {
+                                errorDirectoryNotWritable(R.string.create_cant_write_error_msg)
-                                    ColoredAlertDialogBuilder(activity)
-                                        .setTitle(R.string.warning)
-                                        .setMessage(R.string.create_cant_write_error_msg)
-                                        .setPositiveButton(R.string.ok, null)
-                                        .show()
-                                }
                             }
                         } else {
-                            val dirContent = volumeFile.list()
+                            Toast.makeText(this, R.string.dir_not_empty, Toast.LENGTH_SHORT).show()
-                            if (dirContent != null){
-                                if (dirContent.isEmpty()) {
-                                    if (volumeFile.canWrite()){
-                                        goodDirectory = true
-                                    } else {
-                                        stopTask {
-                                            ColoredAlertDialogBuilder(activity)
-                                                .setTitle(R.string.warning)
-                                                .setMessage(R.string.create_cant_write_error_msg)
-                                                .setPositiveButton(R.string.ok, null)
-                                                .show()
-                                        }
-                                    }
-                                } else {
-                                    stopTaskWithToast(R.string.dir_not_empty)
-                                }
-                            } else {
-                                stopTaskWithToast(R.string.listdir_null_error_msg)
-                            }
                         }
-                        if (goodDirectory) {
+                    } else {
+                        Toast.makeText(this, R.string.listdir_null_error_msg, Toast.LENGTH_SHORT).show()
+                    }
+                }
+                if (goodDirectory) {
+                    object: LoadingTask(this, R.string.loading_msg_create){
+                        override fun doTask(activity: AppCompatActivity) {
                             val xchacha = when (binding.spinnerXchacha.selectedItemPosition) {
                                 0 -> 0
                                 1 -> 1
@@ -172,10 +137,10 @@ class CreateActivity : VolumeActionActivity() {
                                 }
                             }
                         }
-                    }
+                        override fun doFinally(activity: AppCompatActivity) {
-                    override fun doFinally(activity: AppCompatActivity) {
+                            Arrays.fill(password, 0.toChar())
-                        Arrays.fill(password, 0.toChar())
+                            Arrays.fill(passwordConfirm, 0.toChar())
-                        Arrays.fill(passwordConfirm, 0.toChar())
+                        }
                     }
                 }
             }

app/src/main/java/sushi/hardcore/droidfs/OpenActivity.kt

@@ -1,13 +1,13 @@
 package sushi.hardcore.droidfs
 
 import android.content.Intent
-import android.net.Uri
 import android.os.Build
 import android.os.Bundle
 import android.text.Editable
 import android.text.TextWatcher
 import android.view.MenuItem
 import android.widget.AdapterView.OnItemClickListener
+import android.widget.TextView
 import androidx.appcompat.app.AppCompatActivity
 import sushi.hardcore.droidfs.adapters.SavedVolumesAdapter
 import sushi.hardcore.droidfs.content_providers.RestrictedFileProvider
@@ -112,19 +112,6 @@ class OpenActivity : VolumeActionActivity() {
         isStartingActivity = true
     }
 
-    override fun onDirectoryPicked(uri: Uri) {
-        val path = PathUtils.getFullPathFromTreeUri(uri, this)
-        if (path != null){
-            editVolumePath.setText(path)
-        } else {
-            ColoredAlertDialogBuilder(this)
-                .setTitle(R.string.error)
-                .setMessage(R.string.path_from_uri_null_error_msg)
-                .setPositiveButton(R.string.ok, null)
-                .show()
-        }
-    }
-
     fun checkVolumePathThenOpen() {
         loadVolumePath {
             val volumeFile = File(currentVolumePath)
@@ -147,7 +134,12 @@ class OpenActivity : VolumeActionActivity() {
                         .setCancelable(false)
                         .setPositiveButton(R.string.ok) { _, _ -> openVolume() }
                     if (PathUtils.isPathOnExternalStorage(currentVolumeName, this)){
-                        dialog.setMessage(R.string.open_on_sdcard_warning)
+                        dialog.setView(
+                            layoutInflater.inflate(R.layout.dialog_sdcard_error, null).apply {
+                                findViewById<TextView>(R.id.path).text = PathUtils.getPackageDataFolder(this@OpenActivity)
+                                findViewById<TextView>(R.id.footer).text = getString(R.string.open_read_only)
+                            }
+                        )
                     } else {
                         dialog.setMessage(R.string.open_cant_write_warning)
                     }

app/src/main/java/sushi/hardcore/droidfs/VolumeActionActivity.kt

@@ -118,7 +118,18 @@ abstract class VolumeActionActivity : BaseActivity() {
     }
 
     protected open fun onPickingDirectory() {}
-    protected abstract fun onDirectoryPicked(uri: Uri)
+    protected fun onDirectoryPicked(uri: Uri) {
+        val path = PathUtils.getFullPathFromTreeUri(uri, this)
+        if (path != null) {
+            editVolumePath.setText(path)
+        } else {
+            ColoredAlertDialogBuilder(this)
+                .setTitle(R.string.error)
+                .setMessage(R.string.path_from_uri_null_error_msg)
+                .setPositiveButton(R.string.ok, null)
+                .show()
+        }
+    }
 
     private fun safePickDirectory() {
         try {
@@ -368,4 +379,20 @@ abstract class VolumeActionActivity : BaseActivity() {
             callback()
         }
     }
+
+    fun errorDirectoryNotWritable(errorMsg: Int) {
+        val dialog = ColoredAlertDialogBuilder(this)
+            .setTitle(R.string.error)
+            .setPositiveButton(R.string.ok, null)
+        if (PathUtils.isPathOnExternalStorage(currentVolumePath, this)) {
+            dialog.setView(
+                layoutInflater.inflate(R.layout.dialog_sdcard_error, null).apply {
+                    findViewById<TextView>(R.id.path).text = PathUtils.getPackageDataFolder(this@VolumeActionActivity)
+                }
+            )
+        } else {
+            dialog.setMessage(errorMsg)
+        }
+        dialog.show()
+    }
 }

app/src/main/java/sushi/hardcore/droidfs/util/PathUtils.kt

@@ -94,19 +94,14 @@ object PathUtils {
         ) + " " + units[digitGroups]
     }
 
-    fun isTreeUriOnPrimaryStorage(treeUri: Uri): Boolean {
+    fun getPackageDataFolder(context: Context): String {
-        val volumeId = getVolumeIdFromTreeUri(treeUri)
+        return "Android/data/${context.packageName}/"
-        return if (volumeId != null) {
-            volumeId == PRIMARY_VOLUME_NAME || volumeId == "home" || volumeId == "downloads"
-        } else {
-            false
-        }
     }
 
     private fun getExternalStoragePath(context: Context): List<String> {
         val externalPaths: MutableList<String> = ArrayList()
         ContextCompat.getExternalFilesDirs(context, null).forEach {
-            val rootPath = it.path.substring(0, it.path.indexOf(pathJoin("Android/data/", context.packageName, "files")))
+            val rootPath = it.path.substring(0, it.path.indexOf(getPackageDataFolder(context)+"files"))
             if (!rootPath.endsWith("/0/")){ //not primary storage
                 externalPaths.add(rootPath)
             }

app/src/main/res/layout/dialog_sdcard_error.xml

@@ -0,0 +1,30 @@
+<?xml version="1.0" encoding="utf-8"?>
+<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    android:orientation="vertical"
+    android:layout_width="match_parent"
+    android:layout_height="match_parent"
+    android:paddingHorizontal="25dp"
+    android:paddingTop="10dp">
+
+    <TextView
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:text="@string/sdcard_error_header"
+        android:textSize="16sp"/>
+
+    <TextView
+        android:id="@+id/path"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:textSize="14sp"
+        android:textStyle="bold"
+        android:layout_marginVertical="10dp"/>
+
+    <TextView
+        android:id="@+id/footer"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:text="@string/sdcard_error_footer"
+        android:textSize="16sp"/>
+
+</LinearLayout>

app/src/main/res/values-pt-rBR/strings.xml

@@ -159,12 +159,9 @@
     <string name="timer_empty_error_msg">Por favor, digite um valor numérico</string>
     <string name="path_from_uri_null_error_msg">Falha ao carregar o local selecionado.</string>
     <string name="create_cant_write_error_msg">DroidFS não pode salvar neste local. Por favor, tente algum outro.</string>
-    <string name="create_on_sdcard_error_msg">DroidFS não pode salvar em cartões SD externos, por favor selecione um local no armazenamento interno.</string>
-    <string name="open_on_sdcard_warning">DroidFS não pode salvar em cartões SD exteros. Abrindo o volume apenas com acesso de leitura.</string>
     <string name="open_cant_write_warning">DroidFS não tem permissão para salvar neste local. Abrindo o volume apenas com acesso de leitura.</string>
     <string name="open_cant_write_error_msg">DroidFS não tem permissão para salvar neste local. Por favor, tente outro volume.</string>
     <string name="change_pwd_cant_write_error_msg">DroidFS não tem permissão para salvar neste local. Pode tentar mover o volume para um local com permissão de escrita.</string>
-    <string name="change_pwd_on_sdcard_error_msg">DroidFS não pode salvar em cartões SD externos, por favor transfira o volume ao armazenamento interno.</string>
     <string name="slideshow_stopped">Apresentação parou</string>
     <string name="slideshow_started">Apresentação começou</string>
     <string name="ask_save_img_rotated">A imagem foi alternada. Você deseja salvar estas mudanças e substituir a imagem original?</string>

app/src/main/res/values-ru/strings.xml

@@ -156,12 +156,9 @@
     <string name="timer_empty_error_msg">Введите числовое значение.</string>
     <string name="path_from_uri_null_error_msg">Невозможно получить выбранный путь.</string>
     <string name="create_cant_write_error_msg">DroidFS не имеет доступа на запись по этому пути. Попробуйте найти другое место.</string>
-    <string name="create_on_sdcard_error_msg">DroidFS не может записывать на съёмные SD-карты, выберите путь во внутренней памяти.</string>
-    <string name="open_on_sdcard_warning">DroidFS не может писать на съёмные SD-карты. Том открывается только для чтения.</string>
     <string name="open_cant_write_warning">DroidFS не имеет доступа на запись по этому пути. Том открывается только для чтения.</string>
     <string name="open_cant_write_error_msg">DroidFS не имеет доступа на запись по этому пути. Попробуйте другой том.</string>
     <string name="change_pwd_cant_write_error_msg">DroidFS не имеет доступа на запись по этому пути. Попробуйте переместить том в место, доступное для записи.</string>
-    <string name="change_pwd_on_sdcard_error_msg">DroidFS не может писать на съёмные SD-карты, переместите том во внутреннюю память.</string>
     <string name="slideshow_stopped">Слайдшоу остановлено</string>
     <string name="slideshow_started">Слайдшоу началось</string>
     <string name="ask_save_img_rotated">Изображение было повёрнуто. Сохранить изменения и перезаписать исходное изображение?</string>

app/src/main/res/values/strings.xml

@@ -162,12 +162,12 @@
     <string name="timer_empty_error_msg">Please enter a numeric value</string>
     <string name="path_from_uri_null_error_msg">Failed to retrieve the selected path.</string>
     <string name="create_cant_write_error_msg">DroidFS doesn\'t have write access to this path. Please try another location.</string>
-    <string name="create_on_sdcard_error_msg">DroidFS can\'t write on removable SD cards, please select a path on internal storage.</string>
+    <string name="open_read_only">Opening volume with read-only access.</string>
-    <string name="open_on_sdcard_warning">DroidFS can\'t write on removable SD cards. Opening volume with read-only access.</string>
     <string name="open_cant_write_warning">DroidFS doesn\'t have write access to this path. Opening volume with read-only access.</string>
     <string name="open_cant_write_error_msg">DroidFS doesn\'t have write access to this path. Please try another volume.</string>
+    <string name="sdcard_error_header">DroidFS can only write to removable SD cards under:</string>
+    <string name="sdcard_error_footer">Please use a subdirectory of this path or internal storage.</string>
     <string name="change_pwd_cant_write_error_msg">DroidFS doesn\'t have write access to this path. You can try to move the volume to a writable location.</string>
-    <string name="change_pwd_on_sdcard_error_msg">DroidFS can\'t write on removable SD cards, please move the volume to internal storage.</string>
     <string name="slideshow_stopped">Slideshow stopped</string>
     <string name="slideshow_started">Slideshow started</string>
     <string name="ask_save_img_rotated">The image has been rotated. Do you want to save these changes and overwrite the original image ?</string>