lab/ux
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

url: require 7+ digits to flag a match as a phone number

Browse Source 
AOSP's `Patterns.PHONE` matches any run of 3+ digits, so short codes,
ZIP codes, version strings, sport scores, room numbers — anything
numeric — surfaced as tappable `tel:` links on Android. iOS
`NSDataDetector` is locale-aware and quieter but can still emit short
matches in promotional text.

Filter both shims at the conventional 7-digit minimum: NANP local is 7
digits, all international numbers run 7+. Below that the match is
almost certainly not a dialable number. On Android `canonical_phone`
returns an empty string and `run_pattern` drops the record; on iOS the
detector block bails early before emitting the match.

Fixes user reports of `88773` and `75309` (a famous 7-digit run minus
its area code) being incorrectly flagged.
This commit is contained in:
agra
2026-05-15 00:16:25 +03:00
parent edca5c88f5
commit 5512acd540
3 changed files with 26 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
android/src/main/jni/url_detect.cpp
View File

@@ -186,12 +186,22 @@ std::string canonical_web_url(const std::string& match) {
return std::string("http://") + match;
}
// AOSP's `Patterns.PHONE` matches any run of 3+ digits, which fires on
// short codes, ZIP codes, version strings, sport scores, etc. Require at
// least this many digits to call something a dialable number.
constexpr int kMinPhoneDigits = 7;
std::string canonical_phone(const std::string& match) {
std::string out = "tel:";
std::string digits;
for (char c : match) {
if ((c >= '0' && c <= '9') || c == '+') out.push_back(c);
if ((c >= '0' && c <= '9') || c == '+') digits.push_back(c);
}
return out;
int count = 0;
for (char c : digits) {
if (c >= '0' && c <= '9') ++count;
}
if (count < kMinPhoneDigits) return {};
return std::string("tel:") + digits;
}
void run_pattern(JNIEnv* env, jstring text, jobject pattern, uint32_t kind,
@@ -229,6 +239,7 @@ void run_pattern(JNIEnv* env, jstring text, jobject pattern, uint32_t kind,
} else if (kind == kKindPhone) {
url = canonical_phone(group);
}
if (url.empty()) continue;
out.push_back(RawMatch{(int32_t)start, (int32_t)end, kind, std::move(url)});
}
env->DeleteLocalRef(matcher);