Closed Bug 413837 Opened 17 years ago Closed 9 years ago

Bugzilla::Template::quoteUrls should not quote non-ascii as URL

Categories

(Bugzilla :: User Interface, defect)

3.1.2
defect
Not set
minor

Tracking

()

RESOLVED WONTFIX

People

(Reporter: himorin, Assigned: himorin)

Details

Attachments

(2 files, 1 obsolete file)

Using bugzilla tip with utf8 = 1, all strings will be utf-8 flagged. So, all abbrs for regexp will be include > \u{FF}, like \w includes Japanese charactors.

Of cource, we might handle IDNs, but i think that should be an another bug.
Attached patch patch v1 (obsolete) — Splinter Review
Assignee: ui → shimono
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attached patch patch v2Splinter Review
merge the two codes (for non-utf8 and utf8) into one.
Attachment #298937 - Attachment is obsolete: true
Attachment #298944 - Flags: review?(myk)
Um, actually, there was another bug that pointed out that we *didn't* highlight i18n characters, for example for Russian Wikipedia.
Comment on attachment 298944 [details] [diff] [review]
patch v2

Canceling this review request, as I'm no longer able to do Bugzilla reviews.
Attachment #298944 - Flags: review?(myk)
Non ASCII characters are legal in URLs. I don't speak japanese, so I don't know why there is no whitespace after the end of the URL in your screenshot. But assuming this is a mistake of the commenter, there is no reason to catch that.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: