From 641402c251b2a3a6b4d67ae762952524f69fc0f2 Mon Sep 17 00:00:00 2001 From: artdeell Date: Thu, 2 Mar 2023 15:43:05 +0300 Subject: [PATCH] Replace getFileName implementation in Tools, use it ImportControlActivity --- .../pojavlaunch/ImportControlActivity.java | 13 +--------- .../main/java/net/kdt/pojavlaunch/Tools.java | 24 +++++++------------ 2 files changed, 9 insertions(+), 28 deletions(-) diff --git a/app_pojavlauncher/src/main/java/net/kdt/pojavlaunch/ImportControlActivity.java b/app_pojavlauncher/src/main/java/net/kdt/pojavlaunch/ImportControlActivity.java index 57c7e0e53..59def273b 100644 --- a/app_pojavlauncher/src/main/java/net/kdt/pojavlaunch/ImportControlActivity.java +++ b/app_pojavlauncher/src/main/java/net/kdt/pojavlaunch/ImportControlActivity.java @@ -71,7 +71,7 @@ public class ImportControlActivity extends Activity { finishAndRemoveTask(); return; } - mEditText.setText(getNameFromURI(mUriData)); + mEditText.setText(trimFileName(Tools.getFileName(this, mUriData))); mHasIntentChanged = false; //Import and verify thread @@ -193,15 +193,4 @@ public class ImportControlActivity extends Activity { } } - public String getNameFromURI(Uri uri) { - Cursor c = getContentResolver().query(uri, null, null, null, null); - if(c == null) return uri.getLastPathSegment(); // idk myself but it happens on asus file manager - c.moveToFirst(); - int columnIndex = c.getColumnIndex(OpenableColumns.DISPLAY_NAME); - if(columnIndex == -1) return uri.getLastPathSegment(); - String fileName = c.getString(columnIndex); - c.close(); - return trimFileName(fileName); - } - } diff --git a/app_pojavlauncher/src/main/java/net/kdt/pojavlaunch/Tools.java b/app_pojavlauncher/src/main/java/net/kdt/pojavlaunch/Tools.java index 15973b608..9cf650f7f 100644 --- a/app_pojavlauncher/src/main/java/net/kdt/pojavlaunch/Tools.java +++ b/app_pojavlauncher/src/main/java/net/kdt/pojavlaunch/Tools.java @@ -846,22 +846,14 @@ public final class Tools { } public static String getFileName(Context ctx, Uri uri) { - String result = null; - if (uri.getScheme().equals("content")) { - try (Cursor cursor = ctx.getContentResolver().query(uri, null, null, null, null)) { - if (cursor != null && cursor.moveToFirst()) { - result = cursor.getString(cursor.getColumnIndex(OpenableColumns.DISPLAY_NAME)); - } - } - } - if (result == null) { - result = uri.getPath(); - int cut = result.lastIndexOf('/'); - if (cut != -1) { - result = result.substring(cut + 1); - } - } - return result; + Cursor c = ctx.getContentResolver().query(uri, null, null, null, null); + if(c == null) return uri.getLastPathSegment(); // idk myself but it happens on asus file manager + c.moveToFirst(); + int columnIndex = c.getColumnIndex(OpenableColumns.DISPLAY_NAME); + if(columnIndex == -1) return uri.getLastPathSegment(); + String fileName = c.getString(columnIndex); + c.close(); + return fileName; } /** Swap the main fragment with another */