Closed Bug 1599742 Opened 5 years ago Closed 8 months ago

Review fallback for purpose parameter to EngineURL.getSubmission

Categories

(Firefox :: Search, task, P3)

task
Points:
2

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: standard8, Unassigned)

References

Details

EngineURL.getSubmission currently takes a purpose parameter. This falls back to searchbar in a couple of instances:

  1. If the parameter is not supplied.
  2. If there are no matching purposes in the associated parameters.

From looking at the callers to getSubmission, then apart from the mobile code, it looks like everyone is supplying a purpose.

As the mobile code may be going away, we should review this and possibly make purpose non-optional and remove the need for the first fallback.

Points: --- → 2
Severity: normal → S3
Depends on: 1924071

We removed the purpose options for parameters a couple of months ago, (bug 1924071) , so this is no longer relevant.

Status: NEW → RESOLVED
Closed: 8 months ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.