MozParams aren't supported as the first entry in a search engine

RESOLVED FIXED in Firefox 64

Status

()

P1
normal
RESOLVED FIXED
7 months ago
6 months ago

People

(Reporter: mkaply, Assigned: mkaply)

Tracking

Trunk
Firefox 64
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox63 wontfix, firefox64 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 months ago
If a MozParam is the first entry in an engine, the searchForm ends up as:

http://foo.com?&foo=bar
(Assignee)

Updated

7 months ago
Blocks: 1484232

Updated

7 months ago
Priority: -- → P1
Comment on attachment 9003491 [details]
Bug 1485508 - Properly construct MozParameterized search URL.

Drew Willcoxon :adw has approved the revision.
Attachment #9003491 - Flags: review+

Comment 3

6 months ago
Pushed by mozilla@kaply.com:
https://hg.mozilla.org/integration/autoland/rev/5740e53a5e2d
Properly construct MozParameterized search URL. r=adw

Comment 4

6 months ago
Pushed by dluca@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0ed6d2eaccfe
Properly construct MozParameterized search URL. r=adw
(Assignee)

Comment 6

6 months ago
> It might be possible to use https://developer.mozilla.org/en-US/docs/Web/API/URLSearchParams for this.

I did think about using that but it seemed like overkill for this case.

Comment 7

6 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/5740e53a5e2d
Status: ASSIGNED → RESOLVED
Last Resolved: 6 months ago
status-firefox64: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
Does this need a Beta uplift or can it ride the trains?
Flags: needinfo?(mozilla)
(Assignee)

Comment 10

6 months ago
Riding the trains.
Flags: needinfo?(mozilla)
status-firefox63: affected → wontfix
You need to log in before you can comment on or make changes to this bug.