From 2555402ee07f87ac219abb430cd23c668f8f0c9c Mon Sep 17 00:00:00 2001 From: Nancy Chen Date: Wed, 29 Apr 2015 15:03:42 -0700 Subject: Check the voicemail IDs as opposed to the whole URI. When determining whether to notify for a particular voicemail, check the voicemail IDs instead of the whole URI since sometimes the URI will include query parameters and sometimes not so we don't want to do a direct string comparison. Doing a direct comparison was causing a bug where the sound and vibration of the new notification would be suppressed. Bug: 19716109 Change-Id: I7a33b4d3bcd9ca707db9462fcebfec27fd5f8a74 --- src/com/android/dialer/calllog/DefaultVoicemailNotifier.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'src') diff --git a/src/com/android/dialer/calllog/DefaultVoicemailNotifier.java b/src/com/android/dialer/calllog/DefaultVoicemailNotifier.java index 970cad6a6..61f8635be 100644 --- a/src/com/android/dialer/calllog/DefaultVoicemailNotifier.java +++ b/src/com/android/dialer/calllog/DefaultVoicemailNotifier.java @@ -143,7 +143,8 @@ public class DefaultVoicemailNotifier implements VoicemailNotifier { } } // Check if this is the new call we need to notify about. - if (newCallUri != null && newCallUri.equals(newCall.voicemailUri)) { + if (newCallUri != null && + ContentUris.parseId(newCallUri) == ContentUris.parseId(newCall.voicemailUri)) { callToNotify = newCall; } } -- cgit v1.2.3