186819 - Query charset searching on dmoz.org does not match with which dmoz.org expects

Status: RESOLVED FIXED

(SeaMonkey :: Search, defect)

Description

[opmolf]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; es-HN; rv:1.3b) Gecko/20021226
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; es-HN; rv:1.3b) Gecko/20021226

details below

Reproducible: Always

Steps to Reproduce:
1. Set dmoz.org as the search engine: Edit - Preferences... - Navigator -
Internet Search - Search using: dmoz.org
2. Search a string with characters outside the ASCII, for example "mozillá"

Actual Results:  
The query generated by Mozilla is:
http://search.dmoz.org/cgi-bin/search?search=mozill%E1

dmoz.org search for "mozill?"

Expected Results:  
(for example) Generate the query:
http://search.dmoz.org/cgi-bin/search?search=mozill%C3%A1
(which is generated when writing "mozillá" directly on dmoz.org)

It is interesting to see that searching for "mózillá" works properly (two errors
are no error, binary logic ;)
http://search.dmoz.org/cgi-bin/search?search=m%F3zill%E1

But, in general, dmoz.org uses UTF-8 (I think):
http://search.dmoz.org/cgi-bin/search?search=m%C3%B3zill%C3%A1

So, I tried to read those search plugins in "searchplugins" folder of Mozilla
and decided to add a line on "dmoz.src" SEARCH tag: "queryCharset="utf-8""
(changes on dmoz.src added as attachment)

I really don't know the specification of these SRC files but it turns out that
now it works as dmoz.org expects.

Finally, finding similar bugs I have found the 62409 "Local Charsets are not
supported in SearchPlugins", but I think they are not the same although similar.

Comment 1

[opmolf]

Attachment: Changes on /searchplugins/dmoz.src

Comment 2

[opmolf]

See bug "Missing  queryCharset in google.src",
http://bugzilla.mozilla.org/show_bug.cgi?id=180674.

Comment 3

[Asa Dotzler [:asa]]

see also bug 62409

Comment 4

[Phil Ringnalda (:philor)]

Attachment: plugin patch

queryCharset for sending, charset for interpreting, tabs are evil

Comment 5

[Phil Ringnalda (:philor)]

Attachment: -w of above

Comment 6

[Phil Ringnalda (:philor)]

Attachment: website patch

And since it updates from http://www.mozilla.org/projects/search/dmoz.src that
needs to be patched at the same time.

Comment 7

[Dan Mosedale (:dmosedale, :dmose)]

Comment on attachment 197434 [details] [diff] [review]
plugin patch

sr=dmose

Comment 8

[Phil Ringnalda (:philor)]

Fixed, trunk and web.