Query MediaStore for media file URI for setting ringtone

TranceLove · TranceLove · commit 17221be5b68b · 2026-04-05T10:30:06.000+08:00
Fixes #4367
diff --git a/app/src/main/java/com/amaze/filemanager/asynchronous/asynctasks/DeleteTask.java b/app/src/main/java/com/amaze/filemanager/asynchronous/asynctasks/DeleteTask.java
@@ -115,7 +115,7 @@ protected final AsyncTaskResult<Boolean> doInBackground(
 
       // delete file from media database
       if (!file.isSmb() && !file.isSftp()) {
-        MediaConnectionUtils.scanFile(
+        MediaConnectionUtils.scanFiles(
             applicationContext, files.toArray(new HybridFile[files.size()]));
 
         if (FileUtils.NOMEDIA_FILE.equals(file.getName()))
diff --git a/app/src/main/java/com/amaze/filemanager/asynchronous/asynctasks/movecopy/MoveFilesTask.kt b/app/src/main/java/com/amaze/filemanager/asynchronous/asynctasks/movecopy/MoveFilesTask.kt
@@ -107,8 +107,8 @@ class MoveFilesTask(
             for (hybridFileParcelables in files) {
                 sourcesFiles.addAll(hybridFileParcelables)
             }
-            MediaConnectionUtils.scanFile(applicationContext, sourcesFiles.toTypedArray())
-            MediaConnectionUtils.scanFile(applicationContext, targetFiles.toTypedArray())
+            MediaConnectionUtils.scanFiles(applicationContext, sourcesFiles.toTypedArray())
+            MediaConnectionUtils.scanFiles(applicationContext, targetFiles.toTypedArray())
         }
 
         // updating encrypted db entry if any encrypted file was moved
diff --git a/app/src/main/java/com/amaze/filemanager/asynchronous/asynctasks/movecopy/PreparePasteTask.kt b/app/src/main/java/com/amaze/filemanager/asynchronous/asynctasks/movecopy/PreparePasteTask.kt
@@ -30,7 +30,6 @@ import com.afollestad.materialdialogs.DialogAction
 import com.afollestad.materialdialogs.MaterialDialog
 import com.amaze.filemanager.R
 import com.amaze.filemanager.asynchronous.asynctasks.fromTask
-import com.amaze.filemanager.asynchronous.asynctasks.movecopy.PreparePasteTask.CopyNode
 import com.amaze.filemanager.asynchronous.management.ServiceWatcherUtil
 import com.amaze.filemanager.asynchronous.services.CopyService
 import com.amaze.filemanager.databinding.CopyDialogBinding
@@ -142,7 +141,7 @@ class PreparePasteTask(strongRefMain: MainActivity) {
             filesToCopy[0].getParent(context.get()) == targetPath
         ) {
             Toast.makeText(context.get(), R.string.same_dir_move_error, Toast.LENGTH_SHORT).show()
-            MediaConnectionUtils.scanFile(context.get() as Context, filesToCopy.toTypedArray())
+            MediaConnectionUtils.scanFiles(context.get() as Context, filesToCopy.toTypedArray())
             return
         }
 
diff --git a/app/src/main/java/com/amaze/filemanager/asynchronous/services/CopyService.java b/app/src/main/java/com/amaze/filemanager/asynchronous/services/CopyService.java
@@ -469,7 +469,7 @@ void copyRoot(HybridFileParcelable sourceFile, HybridFile targetFile, boolean mo
               e);
           failedFOps.add(sourceFile);
         }
-        MediaConnectionUtils.scanFile(c, new HybridFile[] {targetFile});
+        MediaConnectionUtils.scanFiles(c, new HybridFile[] {targetFile});
       }
 
       private void copyFiles(
diff --git a/app/src/main/java/com/amaze/filemanager/filesystem/HybridFile.java b/app/src/main/java/com/amaze/filemanager/filesystem/HybridFile.java
@@ -1581,7 +1581,7 @@ public void restoreFromBin(Context context) {
             if (!source.renameTo(dest)) {
               return false;
             }
-            MediaConnectionUtils.scanFile(context, new HybridFile[] {this});
+            MediaConnectionUtils.scanFiles(context, new HybridFile[] {this});
             return true;
           });
     }
diff --git a/app/src/main/java/com/amaze/filemanager/filesystem/MediaStoreHack.java b/app/src/main/java/com/amaze/filemanager/filesystem/MediaStoreHack.java
@@ -40,6 +40,7 @@
 import android.provider.MediaStore;
 import android.util.Log;
 
+import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 
 /**
@@ -164,6 +165,43 @@ public static OutputStream getOutputStream(Context context, String str) {
     }
   }
 
+  public static @Nullable Uri getUriForMusicMediaFrom(
+      @NonNull String path, @NonNull Context context) {
+    Uri retval = getUriForMusicMediaInternal(path, context);
+    return retval;
+  }
+
+  private static @Nullable Uri getUriForMusicMediaInternal(
+      @NonNull String path, @NonNull Context context) {
+    String[] projection = {MediaStore.Audio.Media._ID, MediaStore.Audio.Media.DATA};
+
+    String selection = MediaStore.Audio.Media.DATA + "=?";
+    String[] selectionArgs = new String[] {path};
+
+    try (Cursor cursor =
+        context
+            .getContentResolver()
+            .query(
+                MediaStore.Audio.Media.EXTERNAL_CONTENT_URI,
+                projection,
+                selection,
+                selectionArgs,
+                null)) {
+      if (cursor != null && cursor.moveToFirst()) {
+        int id = cursor.getInt(cursor.getColumnIndexOrThrow(MediaStore.Audio.Media._ID));
+        return MediaStore.Audio.Media.EXTERNAL_CONTENT_URI
+            .buildUpon()
+            .appendPath(String.valueOf(id))
+            .build();
+      } else {
+        return null;
+      }
+    } catch (Exception e) {
+      Log.e(TAG, "Error querying MediaStore", e);
+      return null;
+    }
+  }
+
   /** Returns an OutputStream to write to the file. The file will be truncated immediately. */
   private static int getTemporaryAlbumId(final Context context) {
     final File temporaryTrack;
diff --git a/app/src/main/java/com/amaze/filemanager/filesystem/Operations.java b/app/src/main/java/com/amaze/filemanager/filesystem/Operations.java
@@ -728,7 +728,7 @@ protected void onPostExecute(Void aVoid) {
         super.onPostExecute(aVoid);
         if (newFile != null && oldFile != null) {
           HybridFile[] hybridFiles = {newFile, oldFile};
-          MediaConnectionUtils.scanFile(context, hybridFiles);
+          MediaConnectionUtils.scanFiles(context, hybridFiles);
         }
       }
     }.executeOnExecutor(executor);
diff --git a/app/src/main/java/com/amaze/filemanager/filesystem/files/GenericCopyUtil.java b/app/src/main/java/com/amaze/filemanager/filesystem/files/GenericCopyUtil.java
@@ -269,7 +269,7 @@ private void startCopy(
       // If target file is copied onto the device and copy was successful, trigger media store
       // rescan
       if (mTargetFile != null) {
-        MediaConnectionUtils.scanFile(mContext, new HybridFile[] {mTargetFile});
+        MediaConnectionUtils.scanFiles(mContext, new HybridFile[] {mTargetFile});
       }
     }
   }
diff --git a/app/src/main/java/com/amaze/filemanager/filesystem/files/MediaConnectionUtils.kt b/app/src/main/java/com/amaze/filemanager/filesystem/files/MediaConnectionUtils.kt
@@ -36,13 +36,14 @@ object MediaConnectionUtils {
      * @param hybridFiles files to be scanned
      */
     @JvmStatic
-    fun scanFile(
+    fun scanFiles(
         context: Context,
         hybridFiles: Array<HybridFile>,
     ) {
-        val paths = arrayOfNulls<String>(hybridFiles.size)
-
-        for (i in hybridFiles.indices) paths[i] = hybridFiles[i].path
+        val paths: Array<String> =
+            hybridFiles.map {
+                it.path
+            }.toTypedArray()
 
         MediaScannerConnection.scanFile(
             context,
@@ -72,4 +73,31 @@ object MediaConnectionUtils {
             LOG.info("MediaConnectionUtils#scanFile finished scanning path$path")
         }
     }
+
+    /**
+     * Invokes MediaScannerConnection#scanFile for the given file.
+     *
+     * @param context the context
+     * @param path the file path to be scanned
+     * @param mimeType the mime type of the file. Optional.
+     *
+     */
+    @JvmStatic
+    fun scanFileByFileSystemPathAndMimeType(
+        context: Context,
+        path: String,
+        mimeType: String? = null,
+        callback: MediaScannerConnection.OnScanCompletedListener? = null,
+    ) {
+        MediaScannerConnection.scanFile(
+            context,
+            arrayOf(path),
+            mimeType?.let {
+                arrayOf(it)
+            },
+        ) { scannedPath: String, uri: Uri? ->
+            LOG.info("Finished scanning path $scannedPath")
+            callback?.onScanCompleted(scannedPath, uri)
+        }
+    }
 }
diff --git a/app/src/main/java/com/amaze/filemanager/ui/activities/MainActivityViewModel.kt b/app/src/main/java/com/amaze/filemanager/ui/activities/MainActivityViewModel.kt
@@ -40,7 +40,7 @@ import com.amaze.filemanager.asynchronous.asynctasks.searchfilesystem.SearchResu
 import com.amaze.filemanager.asynchronous.asynctasks.searchfilesystem.searchParametersFromBoolean
 import com.amaze.filemanager.fileoperations.filesystem.OpenMode
 import com.amaze.filemanager.filesystem.HybridFile
-import com.amaze.filemanager.filesystem.files.MediaConnectionUtils.scanFile
+import com.amaze.filemanager.filesystem.files.MediaConnectionUtils.scanFiles
 import com.amaze.filemanager.ui.fragments.preferencefragments.PreferencesConstants.PREFERENCE_REGEX
 import com.amaze.filemanager.ui.fragments.preferencefragments.PreferencesConstants.PREFERENCE_REGEX_MATCHES
 import com.amaze.filemanager.ui.fragments.preferencefragments.PreferencesConstants.PREFERENCE_SHOW_HIDDENFILES
@@ -235,7 +235,7 @@ class MainActivityViewModel(val applicationContext: Application) :
                                 OpenMode.TRASH_BIN,
                                 originalFilePath,
                             )
-                        scanFile(applicationContext, arrayOf(hybridFile))
+                        scanFiles(applicationContext, arrayOf(hybridFile))
                         val intent = Intent(MainActivity.KEY_INTENT_LOAD_LIST)
                         hybridFile.getParent(applicationContext)?.let {
                             intent.putExtra(MainActivity.KEY_INTENT_LOAD_LIST_FILE, it)
@@ -292,7 +292,7 @@ class MainActivityViewModel(val applicationContext: Application) :
                                 OpenMode.TRASH_BIN,
                                 source,
                             )
-                        scanFile(applicationContext, arrayOf(hybridFile))
+                        scanFiles(applicationContext, arrayOf(hybridFile))
                         val intent = Intent(MainActivity.KEY_INTENT_LOAD_LIST)
                         hybridFile.getParent(applicationContext)?.let {
                             intent.putExtra(MainActivity.KEY_INTENT_LOAD_LIST_FILE, it)
diff --git a/app/src/main/java/com/amaze/filemanager/ui/fragments/MainFragment.java b/app/src/main/java/com/amaze/filemanager/ui/fragments/MainFragment.java
@@ -58,6 +58,7 @@
 import com.amaze.filemanager.filesystem.FileProperties;
 import com.amaze.filemanager.filesystem.HybridFile;
 import com.amaze.filemanager.filesystem.HybridFileParcelable;
+import com.amaze.filemanager.filesystem.MediaStoreHack;
 import com.amaze.filemanager.filesystem.SafRootHolder;
 import com.amaze.filemanager.filesystem.files.CryptUtil;
 import com.amaze.filemanager.filesystem.files.EncryptDecryptUtils;
@@ -136,6 +137,7 @@
 import jcifs.smb.SmbFile;
 import kotlin.collections.ArraysKt;
 import kotlin.collections.CollectionsKt;
+import kotlin.text.StringsKt;
 
 public class MainFragment extends Fragment
     implements BottomBarButtonPath,
@@ -576,10 +578,27 @@ public void returnIntentResults(HybridFileParcelable[] baseFiles) {
         HybridFileParcelable resultBaseFile = result.getKey();
 
         if (requireMainActivity().mRingtonePickerIntent) {
-          intent.setDataAndType(
-              resultUri,
-              MimeTypes.getMimeType(resultBaseFile.getPath(), resultBaseFile.isDirectory()));
-          intent.putExtra(RingtoneManager.EXTRA_RINGTONE_PICKED_URI, resultUri);
+          // Query MediaStore to get the proper content URI for the selected audio file
+          Uri mediaFileUri =
+              MediaStoreHack.getUriForMusicMediaFrom(resultBaseFile.getPath(), requireContext());
+          if (mediaFileUri != null) {
+            String filename = resultBaseFile.getName();
+            Uri properUri =
+                mediaFileUri
+                    .buildUpon()
+                    .appendQueryParameter("canonical", "1")
+                    .appendQueryParameter("title", StringsKt.substringBeforeLast(filename, ".", ""))
+                    .build();
+            // Set the proper content URI as result
+            String mimeType = MimeTypes.getMimeType(resultBaseFile.getPath(), false);
+            intent.setDataAndType(properUri, mimeType != null ? mimeType : "audio/*");
+            intent.putExtra(RingtoneManager.EXTRA_RINGTONE_PICKED_URI, properUri);
+          } else {
+            Toast.makeText(requireContext(), R.string.error_mediastore_query_uri, Toast.LENGTH_LONG)
+                .show();
+            requireActivity().setResult(FragmentActivity.RESULT_CANCELED);
+            return;
+          }
         } else {
           LOG.debug("pickup file");
           intent.setDataAndType(resultUri, MimeTypes.getExtension(resultBaseFile.getPath()));
@@ -1374,7 +1393,7 @@ public void hide(String path) {
           LOG.warn("failure when hiding file", e);
         }
       }
-      MediaConnectionUtils.scanFile(
+      MediaConnectionUtils.scanFiles(
           requireMainActivity(), new HybridFile[] {new HybridFile(OpenMode.FILE, path)});
     }
   }
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
@@ -875,5 +875,6 @@ You only need to do this once, until the next time you select a new location for
     <string name="trash_bin_cleanup_interval_title">Cleanup interval</string>
     <string name="trash_bin_cleanup_interval_summary">Trigger auto-cleanup interval (hours)</string>
     <string name="file_deletion">File Deletion</string>
+    <string name="error_mediastore_query_uri">Error querying MediaStore for content URI of selected media file. Refresh the MediaStore and try again.</string>
 </resources>
 

Original file line number	Diff line number	Diff line change
`@@ -107,8 +107,8 @@ class MoveFilesTask(`
`107`	`107`	`for (hybridFileParcelables in files) {`
`108`	`108`	`sourcesFiles.addAll(hybridFileParcelables)`
`109`	`109`	`}`
`110`		`- MediaConnectionUtils.scanFile(applicationContext, sourcesFiles.toTypedArray())`
`111`		`- MediaConnectionUtils.scanFile(applicationContext, targetFiles.toTypedArray())`
	`110`	`+ MediaConnectionUtils.scanFiles(applicationContext, sourcesFiles.toTypedArray())`
	`111`	`+ MediaConnectionUtils.scanFiles(applicationContext, targetFiles.toTypedArray())`
`112`	`112`	`}`
`113`	`113`
`114`	`114`	`// updating encrypted db entry if any encrypted file was moved`
Original file line number	Diff line number	Diff line change
`@@ -469,7 +469,7 @@ void copyRoot(HybridFileParcelable sourceFile, HybridFile targetFile, boolean mo`
`469`	`469`	`e);`
`470`	`470`	`failedFOps.add(sourceFile);`
`471`	`471`	`}`
`472`		`- MediaConnectionUtils.scanFile(c, new HybridFile[] {targetFile});`
	`472`	`+ MediaConnectionUtils.scanFiles(c, new HybridFile[] {targetFile});`
`473`	`473`	`}`
`474`	`474`
`475`	`475`	`private void copyFiles(`
Original file line number	Diff line number	Diff line change
`@@ -1581,7 +1581,7 @@ public void restoreFromBin(Context context) {`
`1581`	`1581`	`if (!source.renameTo(dest)) {`
`1582`	`1582`	`return false;`
`1583`	`1583`	`}`
`1584`		`- MediaConnectionUtils.scanFile(context, new HybridFile[] {this});`
	`1584`	`+ MediaConnectionUtils.scanFiles(context, new HybridFile[] {this});`
`1585`	`1585`	`return true;`
`1586`	`1586`	`});`
`1587`	`1587`	`}`
Original file line number	Diff line number	Diff line change
`@@ -728,7 +728,7 @@ protected void onPostExecute(Void aVoid) {`
`728`	`728`	`super.onPostExecute(aVoid);`
`729`	`729`	`if (newFile != null && oldFile != null) {`
`730`	`730`	`HybridFile[] hybridFiles = {newFile, oldFile};`
`731`		`- MediaConnectionUtils.scanFile(context, hybridFiles);`
	`731`	`+ MediaConnectionUtils.scanFiles(context, hybridFiles);`
`732`	`732`	`}`
`733`	`733`	`}`
`734`	`734`	`}.executeOnExecutor(executor);`
Original file line number	Diff line number	Diff line change
`@@ -269,7 +269,7 @@ private void startCopy(`
`269`	`269`	`// If target file is copied onto the device and copy was successful, trigger media store`
`270`	`270`	`// rescan`
`271`	`271`	`if (mTargetFile != null) {`
`272`		`- MediaConnectionUtils.scanFile(mContext, new HybridFile[] {mTargetFile});`
	`272`	`+ MediaConnectionUtils.scanFiles(mContext, new HybridFile[] {mTargetFile});`
`273`	`273`	`}`
`274`	`274`	`}`
`275`	`275`	`}`