diff --git a/src/org/kde/kdeconnect/Plugins/SftpPlugin/SftpPlugin.java b/src/org/kde/kdeconnect/Plugins/SftpPlugin/SftpPlugin.java index 6842c00a..59d33f63 100644 --- a/src/org/kde/kdeconnect/Plugins/SftpPlugin/SftpPlugin.java +++ b/src/org/kde/kdeconnect/Plugins/SftpPlugin/SftpPlugin.java @@ -75,50 +75,48 @@ public class SftpPlugin extends Plugin { //Kept for compatibility, in case "multiPaths" is not possible or the other end does not support it np2.set("path", Environment.getExternalStorageDirectory().getAbsolutePath()); - File root = new File("/"); - if (root.canExecute() && root.canRead()) { - List storageList = StorageHelper.getStorageList(); - ArrayList paths = new ArrayList<>(); - ArrayList pathNames = new ArrayList<>(); + List storageList = StorageHelper.getStorageList(); + ArrayList paths = new ArrayList<>(); + ArrayList pathNames = new ArrayList<>(); - for (StorageHelper.StorageInfo storage : storageList) { - paths.add(storage.path); - StringBuilder res = new StringBuilder(); + for (StorageHelper.StorageInfo storage : storageList) { + paths.add(storage.path); + StringBuilder res = new StringBuilder(); - if (storageList.size() > 1) { - if (!storage.removable) { - res.append(context.getString(R.string.sftp_internal_storage)); - } else if (storage.number > 1) { - res.append(context.getString(R.string.sftp_sdcard_num, storage.number)); - } else { - res.append(context.getString(R.string.sftp_sdcard)); - } + if (storageList.size() > 1) { + if (!storage.removable) { + res.append(context.getString(R.string.sftp_internal_storage)); + } else if (storage.number > 1) { + res.append(context.getString(R.string.sftp_sdcard_num, storage.number)); } else { - res.append(context.getString(R.string.sftp_all_files)); + res.append(context.getString(R.string.sftp_sdcard)); } - String pathName = res.toString(); - if (storage.readonly) { - res.append(" "); - res.append(context.getString(R.string.sftp_readonly)); - } - pathNames.add(res.toString()); + } else { + res.append(context.getString(R.string.sftp_all_files)); + } + String pathName = res.toString(); + if (storage.readonly) { + res.append(" "); + res.append(context.getString(R.string.sftp_readonly)); + } + pathNames.add(res.toString()); - //Shortcut for users that only want to browse camera pictures - String dcim = storage.path + "/DCIM/Camera"; - if (new File(dcim).exists()) { - paths.add(dcim); - if (storageList.size() > 1) { - pathNames.add(context.getString(R.string.sftp_camera) + "(" + pathName + ")"); - } else { - pathNames.add(context.getString(R.string.sftp_camera)); - } + //Shortcut for users that only want to browse camera pictures + String dcim = storage.path + "/DCIM/Camera"; + if (new File(dcim).exists()) { + paths.add(dcim); + if (storageList.size() > 1) { + pathNames.add(context.getString(R.string.sftp_camera) + "(" + pathName + ")"); + } else { + pathNames.add(context.getString(R.string.sftp_camera)); } } + } + + if (paths.size() > 0) { + np2.set("multiPaths", paths); + np2.set("pathNames", pathNames); - if (paths.size() > 0) { - np2.set("multiPaths", paths); - np2.set("pathNames", pathNames); - } } device.sendPackage(np2);