5f46e583b40d86212e7f9c77492768c071a2aa7c |
|
12-Jul-2017 |
Francisco Pimenta <fpimenta@google.com> |
Fix false positives comparing local and intl numbers Previously when comparing local phone numbers with international it was possible to match the wrong number because the first digit of the area code was being treated as a skippable trunk digit, e.g: "550-123-4567" would be considered equal to "+14501234567". Since there are two different algorithms (loose/old and strict) there are two solutions with the same basic goal: Only ignore mismatches if there's an actual extra digit which could possibly be the trunk digit, i.e: "0550-123-4567" is acceptable as equal to "+15501234567". NB: The US not having a trunk digit to begin with is a different issue entirely - the code is agnostic on which country has which trunk digits! For "loose" matching we achieve this by checking the length of the mismatch. For "strict" matching we keep track of the supposed trunk digit and compare it against the current position in match. So for the above example 5 != 4. Several new unit tests were added (including replicating tests for OldPhoneNumberUtils). I broke down the c++ tests into smaller methods for additional readability. NB: By adding more tests I uncovered that two digit trunk prefixes were never handled correctly for the "strict" matching case. "Loose" matching is much more robust. I commented out the "strict" test cases. Test: All unit tests pass for PhoneNumberUtilsTest, OldPhoneNumberUtilsTest and DatabaseGeneralTest. Bug: 63179537 Bug: 63178542 Bug: 62916088 Change-Id: Idc2a1c2c2f64ed29995208503de4755c521e1c82
/external/sqlite/android/PhoneNumberUtils.cpp
|
37214cd8af79110b7c8bd82855625287237d30ec |
|
06-Jan-2011 |
Dmitri Plotnikov <dplotnikov@google.com> |
Adding a custom function to support legacy API compatibility Bug: 3210604 Change-Id: I6e9a20ec7887375b67c42a28cff54e2e9d399ea6
/external/sqlite/android/PhoneNumberUtils.cpp
|
948a119c94f8b69760437c7d1e784b498cd9b766 |
|
20-Sep-2009 |
Daisuke Miyakawa <dmiyakawa@google.com> |
Revert the default phone number comparation algorithm. Now developers have to explicitly set the third argument of PHONE_NUMBERS_COMPARE() into 1, when they want to use "strict" phone number comparation algorithm, which was used in Donut. In default, PHONE_NUMBER_COMPARE() now uses "loose" phone number comparation algorithm, which had been used in Cupcake. Internal issue id: 1892808
/external/sqlite/android/PhoneNumberUtils.cpp
|
b3b8a9dac508afdbac8de93a31a752c60e34e254 |
|
01-Sep-2009 |
Daisuke Miyakawa <dmiyakawa@google.com> |
Fix the implementation of phone_number_compare in accordance with the tests in PhoneNumberUtils.java. Due to the Thailand's special case, the code became a little nastier than before, but I believe it is inevitable...
/external/sqlite/android/PhoneNumberUtils.cpp
|
0d04c4caa9c0464f555eed458abf2ac7d997ad7d |
|
01-Sep-2009 |
Wei Huang <weih@google.com> |
fix bug 2015059: where the native sqlite comparison method phone_number_compare() treated all alpha addresses as the same phone number. - changed phone_number_compare() to handle alpha chars. Before this change the algorithm skipped over all non-dialable chars for comparison, and thus treating "foo" and "bar" as the same addresses. now we don't skip any char that's alpha. This would treat "foo" and "bar" as separate addresses, as well as treating "1-800-flowers" and "800-flowers" as the same number.
/external/sqlite/android/PhoneNumberUtils.cpp
|
cfdd4a7238fb93f2144f48dd814d786414eb025d |
|
21-Jul-2009 |
Daisuke Miyakawa <dmiyakawa@google.com> |
Fix a bug in which "16\d+6" matches 166 and the number is handled as Thailand one...
/external/sqlite/android/PhoneNumberUtils.cpp
|
f06f5fa351ded0b9e04e15efa7e34f0f085fc9df |
|
07-Jul-2009 |
Daisuke Miyakawa <dmiyakawa@google.com> |
"Rewrite" PhoneNumberUtil so that it compares two phone strings using as many characters as possible, unlike the previous implementation. Unlike the change 5480, this considers the NDD-problem mentioned in the internal issue 1949010. This time, 1-650-555-1234 and 650-555-1234 match each other. Added tests for the cases. related internal issue: 1868702, 1949010
/external/sqlite/android/PhoneNumberUtils.cpp
|
ab3ee7de06d0451d0a34cbd982006f62928e4b48 |
|
07-Jul-2009 |
Daisuke Miyakawa <dmiyakawa@google.com> |
Revert ""Rewrite" PhoneNumberUtil so that it compares two phone strings using as many characters as possible, unlike the previous implementation." This reverts commit de43094477419f8a190a6f97b47d346827310a02. related internal issue: 1949010
/external/sqlite/android/PhoneNumberUtils.cpp
|
de43094477419f8a190a6f97b47d346827310a02 |
|
26-Jun-2009 |
Daisuke Miyakawa <dmiyakawa@google.com> |
"Rewrite" PhoneNumberUtil so that it compares two phone strings using as many characters as possible, unlike the previous implementation. The new implementation considers country code seriously. To confirm this change does not do something wrong, I also created another change (https://android-git.corp.google.com/g/Gerrit#change,4036). All tests related to PhoneNumberUtils pass, including Thailand's ugly buggy behavior around country code handling :-P Also Add tests for PhoneNumberUtils, which can be executed in the usual workstation like this: > g++ -Wall external/sqlite/android/PhoneNumberUtils.cpp external/sqlite/android/PhoneNumberUtilsTest.cpp > ./a.out This change fixes the internal bug 1868702.
/external/sqlite/android/PhoneNumberUtils.cpp
|
7790ef5367fe6731048c3e3a1c067f94b321cb4d |
|
04-Mar-2009 |
The Android Open Source Project <initial-contribution@android.com> |
auto import from //depot/cupcake/@135843
/external/sqlite/android/PhoneNumberUtils.cpp
|
9764acebd9bd23a5e72932accc18cbe31b47ee57 |
|
04-Mar-2009 |
The Android Open Source Project <initial-contribution@android.com> |
auto import from //depot/cupcake/@135843
/external/sqlite/android/PhoneNumberUtils.cpp
|
e9cfd50d8d5633e01dcbd9a58947a25cef1fd792 |
|
11-Feb-2009 |
The Android Open Source Project <initial-contribution@android.com> |
auto import from //branches/cupcake/...@130745
/external/sqlite/android/PhoneNumberUtils.cpp
|
5770aafd1e1040dec51f2ed73cb95ed4dbe2a690 |
|
18-Dec-2008 |
The Android Open Source Project <initial-contribution@android.com> |
Code drop from //branches/cupcake/...@124589
/external/sqlite/android/PhoneNumberUtils.cpp
|
6005ac625aa553fe461b363385a8ed4a3c217a1f |
|
21-Oct-2008 |
The Android Open Source Project <initial-contribution@android.com> |
Initial Contribution
/external/sqlite/android/PhoneNumberUtils.cpp
|