Prisjakt search engine does not work

RESOLVED FIXED in Firefox 57

Status

()

defect
P1
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: mkaply, Assigned: flod)

Tracking

55 Branch
Firefox 58
Other
iOS
Points:
---

Firefox Tracking Flags

(firefox57 fixed, firefox58 fixed)

Details

Attachments

(1 attachment)

bug 1408654 was about iOS.

This is also broke on desktop and Android.
Assignee: mozilla → francesco.lodolo
Searchplugins are available for testing here (they're modified with a different shortName, so you can recognize them, and have them both installed at the same time)
https://l10n.mozilla-community.org/~flod/testsp/

Note that it's slightly different from the one that landed in iOS: as far as I know, rel=searchform is not something we use on mobile.
(In reply to Francesco Lodolo [:flod] from comment #1)
> Note that it's slightly different from the one that landed in iOS: as far as
> I know, rel=searchform is not something we use on mobile.

Also: suggestions are http, search is https.
Comment on attachment 8920194 [details]
Bug 1410095 - Fix search URLs in prisjakt.nu searchplugin for Swedish

https://reviewboard.mozilla.org/r/191218/#review196384

::: browser/locales/searchplugins/prisjakt-sv-SE.xml:11
(Diff revision 2)
>  <ShortName>Prisjakt</ShortName>
>  <Description>Prisjakt - jämför priser och produkter</Description>
>  <InputEncoding>UTF-8</InputEncoding>
>  <Image width="16" height="16">data:image/x-icon;base64,AAABAAEAEBAQAAAAAABoBQAAFgAAACgAAAAQAAAAIAAAAAEACAAAAAAAQAEAAAAAAAAAAAAAAAAAAAAAAAAAAAAAKieMACommAAqJaMAKyauAC0prwBMSLsAZF%2FFAH16zQBwa9EAi4nTALu55QCxrvYA7Ov4AP%2F%2F%2FwBwADIAaQBjAG8AAADAHiQAfO8SAEwAAAAoJYAAjAAAAOS%2F9QBxGuYAjAAAAJgFAgAA8BIAGAAAAHDvEgDI7xIA4xq%2BAIwAAACYBQIAAPASABgAAAAAAAAA2T7GABg%2FxgDkCQUAVAAAAGDyEgChUcYA5AkFAFQAAABg8hIAAAAAAMzyEgBghgcAHjvnAPc65wDg8hIAWAcXAKzvEgCw7xIAgP4SAAlI6QBYMOgA%2F%2F%2F%2FAB475wAbrQEAYIYHAODyEgBYBxcABACkAAAApAD%2F%2F%2F8AsAgAAAAAQAAEAKQAZAAAAGIAbQBwADIAaQBjAG8ALgBlAHgAZQAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAHQAAAAAAAAAAAAAAAAAAAACAAAAXPESALgLpADoC8IAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA6AvCAAAAAAAAAAAAAAAAAP%2F%2F%2FwAAAAAAAAAAAAAAAAAAAAAAX6bnAAAAAAAAAAAAAAAAAAAAAAAoLxQAAAAAAAkOAgACDgIADQAAAADw%2FQAA4P0AAg4CAAkOAgAAAAAA9gvCAODyEgBSAAAAAAAAACTVpAAAAAAA%2F%2F%2F%2FAFzxEgBq8RIAXPESAMzx5wAEwPUARPESAAAAFACoRPkARQAAAHgTFAAAABQAoCAUABzxEgAg8RIAZPMSAPCI%2BgBSAAAAAAAAAJDWpAAAAAAAOor1AAAAAAAA7P0AAAAAAAAAAABwADIAaQBjAG8AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAACDO8YApTbGAM0JAQAPAIUAAAAAAJDWpADNCQEAAQAAAAAAAAAAAAAAFTbGAM0JAQAAAAEAwU1BAM0JAQAAAwAABMD1AFik5wB0AAAAAAAAAOjyEgB0AAAAAAAAAAAAAAD%2F%2F%2F8AAAAAAAAAAAAAAAAAAAAAAP%2F%2F%2FwAAAAAAAAAAAAAAAAAAAAAAKC8UAAAAAABkxfUAqfHnAIwAAAAAAAAAAAAAAAAAAAB88hIAAADdAAADAAAAAAAAyfHnAAADAAAAAN0AjAAAAAAAAAAAAwAAAQAYAAAAAABw8hIAAAAAAKqb9QCzm%2FUA4PUSACQAAgAA7P0AEW5AAAUAAAAkAAIAAPD9AJzyEgACAAAAQKP1AJACAgD5m%2FUA4En8ACOj9QAro%2FUAAAAAAAgCAACgIBQAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAVAAAAAAAAAAAAAAAACwgGFQYICwAAAAAAAAANCgUEBAEEBAUKDQAAAAAACgQEBwwCBAQEBAoAAAAACwUEBAQOAwQEBAQFCwAAAAgEBAwODg4OCQQEBAgAAAAGBAQEBAQEBA4EBAQGAAAVFQECAwwODg4MAwIBFRUAAAYEBAQOBAMEBAQEBAYAAAAIBAQECQ4ODg4MBAQIAAAACwUEBAQEAw4EBAQFCwAAAAAKBAQEBAIMBwQECgAAAAAADQoFBAQBBAQFCg0AAAAAAAAACwgGFQYICwAAAAAAAAAAAAAAABUAAAAAAAAAAP%2F%2FRgD%2B%2FwAA8B8AAMAHAADABwAAgAMAFYADAACAAwAAAAEAAIADBhWAAwsAgAMAAMAHDQrABwQB8B8FCv7%2FAAA%3D</Image>
> -<Url type="text/html" method="GET" template="http://www.prisjakt.nu/supersearch.php" resultdomain="prisjakt.nu">
> -  <Param name="s" value="{searchTerms}"/>
> +<Url type="text/html" method="GET" template="https://www.prisjakt.nu/#rparams=ss={searchTerms}" resultdomain="prisjakt.nu" rel="searchform"/>
> +<Url type="application/x-suggestions+json" method="GET" template="https://www.prisjakt.nu/plugins/opensearch/suggestions.php?search={searchTerms}"/>

Should we do a Param for the search?
Attachment #8920194 - Flags: review?(mozilla) → review+
Comment on attachment 8920194 [details]
Bug 1410095 - Fix search URLs in prisjakt.nu searchplugin for Swedish

https://reviewboard.mozilla.org/r/191218/#review197338
(In reply to Mike Kaply [:mkaply] from comment #5)
> Should we do a Param for the search?

Yep, good call.
(tried to comment via MozReview but somehow failed)
Pushed by francesco.lodolo@mozillaitalia.org:
https://hg.mozilla.org/integration/autoland/rev/f9b62ceada2a
Fix search URLs in prisjakt.nu searchplugin for Swedish r=mkaply
https://hg.mozilla.org/mozilla-central/rev/f9b62ceada2a
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 58
Comment on attachment 8920194 [details]
Bug 1410095 - Fix search URLs in prisjakt.nu searchplugin for Swedish

Approval Request Comment
[Feature/Bug causing the regression]: -

[User impact if declined]: searchplugin currently shipping doesn't work

[Is this code covered by automated tests?]: no

[Has the fix been verified in Nightly?]: yes

[Needs manual test from QE? If yes, steps to reproduce]:  no

[List of other uplifts needed for the feature/fix]: -

[Is the change risky?]: no

[Why is the change risky/not risky?]: only affecting a secondary searchplugin for one language (sv-SE)

[String changes made/needed]: -
Attachment #8920194 - Flags: approval-mozilla-beta?
Comment on attachment 8920194 [details]
Bug 1410095 - Fix search URLs in prisjakt.nu searchplugin for Swedish

Must fix, Beta57+
Attachment #8920194 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.