309379 - update answers.com plug-in

Status: RESOLVED FIXED

(Firefox :: Search, defect)

Description

[Rafael Ebron (:rebron)]

From the Answers people, need to update the search url.

Assuming you're looking up the word topic
 
Method #1:
    http://www.answers.com/main/ntquery?s=topic&gwp=13 or
    http://www.answers.com/main/ntquery?gwp=13&s=topic
 
Method #2:
    http://www.answers.com/topic?gwp=13

Comment 1

[Phil Ringnalda (:philor)]

*** Bug 308122 has been marked as a duplicate of this bug. ***

Comment 2

[Phil Ringnalda (:philor)]

rebron: did you really mean to assign this to Ben's account with the realname
"Ben Goodger (Do not assign bugs to this!)"?

Comment 3

[Ben Goodger (use ben at mozilla dot org for email)]

Attachment: update the answers.com plugin to include tracking code.

Comment 4

[Ben Goodger (use ben at mozilla dot org for email)]

Yes, I told him to. Sorry for the confusion. 

Comment 5

[Axel Hecht [:Pike]]

Do we need this for the copies of this plugin in l10n, too? Probably the change
of the action, but the tracking code as well?

Comment 6

[Rafael Ebron (:rebron)]

Yes for locales as well.

Comment 7

[Axel Hecht [:Pike]]

Setting late-l10n as per rebron's comment, I may be tempted to just land the
appropriate changes on the branch myself, as this is just technical stuff.
I'll assemble a list of affected locales and CC their owners once this has 
approval.

Comment 8

[Axel Hecht [:Pike]]

Comment on attachment 196840 [details] [diff] [review]
update the answers.com plugin to include tracking code. 

I wonder if
+	 action="http://www.answers.com/main/ntquery
and
+   <input name="s" user>
+  <input name="gwp" value="13">
wouldn't be the more common way to fix this.

Comment 9

[Ben Goodger (use ben at mozilla dot org for email)]

Axel: probably. 

Last time we did one of these updates to the answers plugin Benjamin said this
wasn't late-l10n since only en-US used answers.com...

Comment 10

[Phil Ringnalda (:philor)]

That... turns out not to be the case. mn has been using it long enough that they
didn't get the ISO-8859-1 to UTF-8 queryCharset change; es-AR, el, and da didn't
get the Dictionary to Answers.com name change.

Comment 11

[Ben Goodger (use ben at mozilla dot org for email)]

OK. 

Comment 12

[Ben Goodger (use ben at mozilla dot org for email)]

consolidating under one account

Comment 13

[Ben Goodger (use ben at mozilla dot org for email)]

Comment on attachment 196840 [details] [diff] [review]
update the answers.com plugin to include tracking code. 

requesting approval for checkin

Comment 14

[Ben Goodger (use ben at mozilla dot org for email)]

fixed branch and trunk

Comment 15

[Uri Bernstein]

fixed1.8, per comment #14