diff options
author | Tyler Gunn <tgunn@google.com> | 2015-04-03 16:40:05 -0700 |
---|---|---|
committer | Tyler Gunn <tgunn@google.com> | 2015-04-03 16:40:05 -0700 |
commit | d3ffa4914baebe668d540fc966efddd0c89a92a2 (patch) | |
tree | 31b0043fb5090b0463459c1fe783b152558286a5 | |
parent | b1dff5e869abe3fa558f639ca24c1a856d515306 (diff) |
Cannot direct dial or video call from Dialer search results.
- Change made in partner CL introduced a regression in dialing from
the search results:
https://partner-android-review.googlesource.com/#/c/206297/
The problem with that change is that it was intended to ensure that the +
was not stripped off when dialing from the dial pad. It had the
unintential impact of rendering direct dial from the search box inoperative.
since mAddToContactNumber was null in this case.
Bug: 20068746
Change-Id: Ic3a75ccf00164e7c865cc633a6a16d1699227ed2
-rw-r--r-- | src/com/android/dialer/list/SearchFragment.java | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/src/com/android/dialer/list/SearchFragment.java b/src/com/android/dialer/list/SearchFragment.java index 9abd7b048..1c88c4e81 100644 --- a/src/com/android/dialer/list/SearchFragment.java +++ b/src/com/android/dialer/list/SearchFragment.java @@ -180,9 +180,10 @@ public class SearchFragment extends PhoneNumberPickerFragment { final DialerPhoneNumberListAdapter adapter = (DialerPhoneNumberListAdapter) getAdapter(); final int shortcutType = adapter.getShortcutTypeFromPosition(position); final OnPhoneNumberPickerActionListener listener; + final String phoneNumber = TextUtils.isEmpty(mAddToContactNumber) ? + adapter.getQueryString() : mAddToContactNumber; - boolean ret = checkForProhibitedPhoneNumber(mAddToContactNumber); - + boolean ret = checkForProhibitedPhoneNumber(phoneNumber); switch (shortcutType) { case DialerPhoneNumberListAdapter.SHORTCUT_INVALID: super.onItemClick(position, id); @@ -190,7 +191,7 @@ public class SearchFragment extends PhoneNumberPickerFragment { case DialerPhoneNumberListAdapter.SHORTCUT_DIRECT_CALL: listener = getOnPhoneNumberPickerListener(); if (listener != null && !ret) { - listener.onCallNumberDirectly(mAddToContactNumber); + listener.onCallNumberDirectly(phoneNumber); } break; case DialerPhoneNumberListAdapter.SHORTCUT_ADD_NUMBER_TO_CONTACTS: @@ -203,7 +204,7 @@ public class SearchFragment extends PhoneNumberPickerFragment { case DialerPhoneNumberListAdapter.SHORTCUT_MAKE_VIDEO_CALL: listener = getOnPhoneNumberPickerListener(); if (listener != null && !ret) { - listener.onCallNumberDirectly(mAddToContactNumber, true /* isVideoCall */); + listener.onCallNumberDirectly(phoneNumber, true /* isVideoCall */); } break; } |