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

Status: RESOLVED WONTFIX

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

Description

[lecky]

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.

Comment 1

[Gene Lian (I already quit Mozilla)]

Looping Gaia folks.

Comment 2

[Julien Wajsberg [:julienw]]

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 !

Comment 3

[Julien Wajsberg [:julienw]]

still need the sms db.

Comment 4

[lecky]

hi,julien,sorry for the delay,

please check the attachment file.

the last two format with chinese characters can not be recognized.

Comment 5

[lecky]

Attachment: 226660312ssm.sqlite

Comment 6

[lecky]

Attachment: 226660312ssm.sqlite

Comment 7

[lecky]

please check the attachment file,

the last three format with chinese characters can not be recognized.

Comment 8

[Julien Wajsberg [:julienw]]

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 ?

Comment 9

[lecky]

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.

Comment 10

[Julien Wajsberg [:julienw]]

Joe, can we add this to the backlog please ?

Comment 11

[Joe Cheng [:jcheng] (please needinfo)]

add to backlog bug 891754

Comment 12

[lecky]

hi,wayne,could you check the issue state for me?

add to backlog means will not fix in V1.1?

Comment 13

[Wayne Chang [:wchang]]

Lecky,

Correct. Backlogged means for a future version yet to be decided. It will depend on the priority and our available resources.

Comment 14

[Julien Wajsberg [:julienw]]

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 !

Comment 15

[Steve Chung [:steveck]]

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.

Comment 16

[Bryant Mao [:bryantmao]]

Move to firefox project. Still, let me know if there are any UX emergency.

Comment 17

[Julien Wajsberg [:julienw]]

Mass closing of Gaia::SMS bugs. End of an era :(

Comment 18

[Julien Wajsberg [:julienw]]

Mass closing of Gaia::SMS bugs. End of an era :(