615177 - [kn] Fix search parameters in yahoo-in.xml plugin

Status: RESOLVED FIXED

(Mozilla Localizations :: kn / Kannada, defect)

Description

[Staś Małolepszy :stas]

The current version of the yahoo-in.xml plugin (https://hg.mozilla.org/l10n-central/kn/file/2e93b2fa06fb/browser/searchplugins/yahoo-in.xml) needs two fixes:

* the following line should be removed:
<Param name="sourceid" value="Mozilla-search"/>

* the following line should be changed:
<MozParam name="fr" condition="pref" pref="fr" />
to:
<MozParam name="fr" condition="pref" pref="yahoo-fr" />

Please attach a patch for a review.

This is blocking the fx40 productization review bug 603984.

Comment 1

[Shankar Prasad]

Attachment: patch for yahoo-in.xml

Stas,

Made changes as per your instructions.

Shankar

Comment 2

[Staś Małolepszy :stas]

Comment on attachment 493655 [details] [diff] [review]
patch for yahoo-in.xml

r=me, thanks Shankar! 

I mentioned l10n-central in the summary of this bug, but actually, the 1.9.2 plugin should also be amended. Please land this patch on both 1.9.2 and central, mentioning this bug and my review in the commit message.

Comment 3

[Shankar Prasad]

Stas,

Landed the patch in both 1.9.2 and l10n-central branches.

Below are the changesets:

l10n central: http://hg.mozilla.org/l10n-central/kn/rev/3fd7ee480972
1.9.2: http://hg.mozilla.org/releases/l10n-mozilla-1.9.2/kn/rev/430743f0bb7d

Shankar

Comment 4

[Staś Małolepszy :stas]

Perfect. Thanks Shankar! Resolving this as fixed.