Closed
Bug 489277
Opened 15 years ago
Closed 15 years ago
Search queries can have inconsistent encodings
Categories
(support.mozilla.org :: Knowledge Base Software, task)
support.mozilla.org
Knowledge Base Software
Tracking
(Not tracked)
VERIFIED
FIXED
1.0.2
People
(Reporter: ecooper, Assigned: ecooper)
References
()
Details
(Whiteboard: sumo_only)
Attachments
(2 files)
1.45 KB,
patch
|
laura
:
review+
|
Details | Diff | Splinter Review |
51.52 KB,
image/png
|
Details |
Searches in languages that aren't strictly ASCII seem to run into encoding problems, leading to no results for that search. STR: 1. Visit https://support-stage.mozilla.org/tiki-newsearch.php?locale=ja&q=%E3%81%93%E3%81%AE&where=d&sa=&filter_lang=1&l=ar&en_too=1&lastmodif=0&type=0&author= この should return a lot of Japanese articles, but it doesn't return any. Most likely, it's because of an encoding issue.
Assignee | ||
Comment 1•15 years ago
|
||
This forces UTF-8 for the query without double-encoding it (also a problem with the current code). This problem also affects more popular locales like de: https://support-stage.mozilla.org/tiki-newsearch.php?locale=de&q=enth%C3%A4lt&where=d&sa=&filter_lang=1&l=he&en_too=1&lastmodif=0&type=0&author=
Attachment #373785 -
Flags: review?(laura)
Updated•15 years ago
|
Attachment #373785 -
Flags: review?(laura) → review+
Assignee | ||
Comment 2•15 years ago
|
||
r24691/r24692 URLs like the ones in the description and comment 1 can be used to see if this is fixed. Essentially, any language using characters beyond regular old ASCII. The screenshot is for reference of what the pre-patch results looked like for the url in comment 1.
Assignee | ||
Updated•15 years ago
|
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Comment 3•15 years ago
|
||
Verified FIXED on https://support-stage.mozilla.org/tiki-newsearch.php?locale=ja&q=%E3%81%93%E3%81%AE&where=d&sa=&filter_lang=1&l=ar&en_too=1&lastmodif=0&type=0&author and https://support-stage.mozilla.org/tiki-newsearch.php?locale=de&q=enth%C3%A4lt&where=d&sa=&filter_lang=1&l=he&en_too=1&lastmodif=0&type=0&author
Status: RESOLVED → VERIFIED
Updated•15 years ago
|
Keywords: push-needed
Updated•15 years ago
|
Keywords: push-needed
Updated•15 years ago
|
Whiteboard: sumo_only
You need to log in
before you can comment on or make changes to this bug.
Description
•