From 900731de0c9fc52ea0b5eed0a3479f3b6e4f4f74 Mon Sep 17 00:00:00 2001 From: Daisuke Miyakawa Date: Wed, 3 Feb 2010 12:42:26 -0800 Subject: Honor '+' in phone export in vCard. Currently no test because of temporal reason. I'll do in the near future... Bug: 2417138 --- core/java/android/pim/vcard/VCardBuilder.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/core/java/android/pim/vcard/VCardBuilder.java b/core/java/android/pim/vcard/VCardBuilder.java index 09ac1fd6262f..0a6415dd200c 100644 --- a/core/java/android/pim/vcard/VCardBuilder.java +++ b/core/java/android/pim/vcard/VCardBuilder.java @@ -689,7 +689,9 @@ public class VCardBuilder { final int length = phoneNumber.length(); for (int i = 0; i < length; i++) { final char ch = phoneNumber.charAt(i); - if (Character.isDigit(ch)) { + // TODO: add a test case for string with '+', and care the other possible issues + // which may happen by ignoring non-digits other than '+'. + if (Character.isDigit(ch) || ch == '+') { builder.append(ch); } else if ((ch == ';' || ch == '\n') && builder.length() > 0) { phoneList.add(builder.toString()); -- cgit v1.2.3-59-g8ed1b