Closed Bug 907530 Opened 9 years ago Closed 5 years ago

[B2G][Helix][sms][zhaotao]hyperlink with chinese characters in sms can not be opened

Categories

(Firefox OS Graveyard :: Gaia::SMS, defect, P3)

defect

Tracking

(tracking-b2g:backlog)

RESOLVED WONTFIX
tracking-b2g backlog

People

(Reporter: lecky.wanglei, Unassigned)

References

Details

Attachments

(1 file, 1 obsolete file)

44.00 KB, application/octet-stream
Details
User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; .NET4.0C; .NET4.0E; aff-kingsoft-ciba; Zune 4.7)

Steps to reproduce:

1.use an android phone to send a sms contains hyperlink with chinese characters to ffos phone
2.tap the hyperlink in ffos phone


Actual results:

in step 2,the hyperlink can not be opened


Expected results:

the hyperlink should be opened successfully.
Component: General → Gaia::SMS
Priority: -- → P2
blocking-b2g: --- → hd?
Looping Gaia folks.
lecky, it would help a lot if you could provide us a sms db with this, as it's difficult for us european/american to do this.

You just need to "adb pull" the file /data/local/indexedDB/chrome/idb/226660312ssm.sqlite and the directory /data/local/indexedDB/chrome/idb/226660312ssm (the number in the file names might be different). Be careful as when you'll pull the directory it will actually put all the files in your current directory, so better create a new dir locally with that name, cd to that directory, and pull the directory from there.

Thanks !
Flags: needinfo?(lecky.wanglei)
Severity: normal → blocker
Flags: needinfo?(lecky.wanglei)
still need the sms db.
Flags: needinfo?(lecky.wanglei)
hi,julien,sorry for the delay,

please check the attachment file.

the last two format with chinese characters can not be recognized.
Flags: needinfo?(lecky.wanglei)
Attached file 226660312ssm.sqlite (obsolete) —
Attached file 226660312ssm.sqlite
Attachment #798396 - Attachment is obsolete: true
please check the attachment file,

the last three format with chinese characters can not be recognized.
I think the problem is that we use "\w" and this doesn't match extended characters.

I had another algorithm in https://bug846373.bugzilla.mozilla.org/attachment.cgi?id=731827 which did match extended characters. Basically, instead of using "\w" I used "[^\s...]", but I'm afraid this can lead to too many false positives.

Are URL with chinese characters usually prefixed with "http:" so that we can be more strict for those ?
hi,julien

URL with chinese characters may prefixed with "http://", and may not.

It depends on the end user's using habit.

It will be prefered if both format can be supported.
Joe, can we add this to the backlog please ?
Flags: needinfo?(jcheng)
add to backlog bug 891754
Flags: needinfo?(jcheng)
hi,wayne,could you check the issue state for me?

add to backlog means will not fix in V1.1?
blocking-b2g: hd? → ---
Flags: needinfo?
Flags: needinfo?
Flags: needinfo?(wchang)
Lecky,

Correct. Backlogged means for a future version yet to be decided. It will depend on the priority and our available resources.
Severity: blocker → normal
Flags: needinfo?(wchang)
Priority: P2 → P3
blocking-b2g: --- → backlog
blocking-b2g: backlog → ---
Hey Steve, Bryant,

As you know, we support all URLs that start with http://, but we only recognize URLs with ascii characters and some white-listed TLD when http:// is missing. Do you think it's good enough ? What other platforms are doing ? Is it usual that chinese people write URLs in messages without http:// ?

Thanks !
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(schung)
Flags: needinfo?(bmao)
I seldom(almost never) use the Chinese char in the URL actually, not to mention the URL with Chinese char and without http:// prefix. This kind of URL is valid and works on Android, but I'm not sure if it's an important use case for general user.
Flags: needinfo?(schung)
Move to firefox project. Still, let me know if there are any UX emergency.
Flags: needinfo?(bmao)
Mass closing of Gaia::SMS bugs. End of an era :(
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WONTFIX
Mass closing of Gaia::SMS bugs. End of an era :(
You need to log in before you can comment on or make changes to this bug.