commit f8d921545b93817101aad618879f85cdb9087bbe
author Robert Chou <robertchou@google.com> Fri Sep 18 03:22:10 2020 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Fri Sep 18 03:22:10 2020 +0000
tree ed72c1c574823746e14a477860f1a6680b29fd56
parent 806bfa56495553f4e1514278a49443adab827c15
parent 7a38d00bc79fe91aba2b813056df97db920c8a30

Merge "Ringtone uri should add type query" am: 96d98a9fad am: caf558ecdc am: 8e9f6c8db1 am: 53aa756a06 am: 7a38d00bc7

Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1426651

Change-Id: Idce0ee0328004bcc5bcf6b0d930d5d52b7d8ce85

media/java/android/media/RingtoneManager.java

diff --git a/media/java/android/media/RingtoneManager.java b/media/java/android/media/RingtoneManager.java
index 8deb0c4..20041a8 100644
--- a/media/java/android/media/RingtoneManager.java
+++ b/media/java/android/media/RingtoneManager.java
@@ -1129,12 +1129,14 @@
             }
 
             // Try finding the scanned ringtone
+            final String whichAudio = getQueryStringForType(type);
             final String filename = getDefaultRingtoneFilename(type);
+            final String where = MediaColumns.DISPLAY_NAME + "=?," + whichAudio + "=?";
             final Uri baseUri = MediaStore.Audio.Media.INTERNAL_CONTENT_URI;
             try (Cursor cursor = context.getContentResolver().query(baseUri,
                     new String[] { MediaColumns._ID },
-                    MediaColumns.DISPLAY_NAME + "=?",
-                    new String[] { filename }, null)) {
+                    where,
+                    new String[] { filename , "1"}, null)) {
                 if (cursor.moveToFirst()) {
                     final Uri ringtoneUri = context.getContentResolver().canonicalizeOrElse(
                             ContentUris.withAppendedId(baseUri, cursor.getLong(0)));
@@ -1162,4 +1164,13 @@
             default: throw new IllegalArgumentException();
         }
     }
+
+    private static String getQueryStringForType(int type) {
+        switch (type) {
+            case TYPE_RINGTONE: return MediaStore.Audio.AudioColumns.IS_RINGTONE;
+            case TYPE_NOTIFICATION: return MediaStore.Audio.AudioColumns.IS_NOTIFICATION;
+            case TYPE_ALARM: return MediaStore.Audio.AudioColumns.IS_ALARM;
+            default: throw new IllegalArgumentException();
+        }
+    }
 }