Closed Bug 777721 Opened 13 years ago Closed 13 years ago

remove query_fields-related code

Categories

(support.mozilla.org :: Code Quality, task)

Product:
support.mozilla.org
Component:
Code Quality
Type:
task
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
2012Q3

People

(Reporter: willkg, Assigned: willkg)

Details

ElasticUtils used to have a .query_fields() transform where you could specify the fields you wanted to query and then specify the query later and the query would apply to all the fields. It doesn't have that anymore. Thus I added this code to Sphilastic. However, it's kind of meh and it's probably the case that we don't want exactly the same query in all the places we do searches (search view, aaq suggestions, ...). Thus, we should nix it and build the query explicitly each time.
Grabbing this one and doing it now.
Assignee: nobody → willkg
Landed in master in https://github.com/mozilla/kitsune/commit/6be3f371cf1cd84259f7bbc2f62dd42f2c432061 Pushed to production by Ricky just now. I'll miss you .query_fields()!
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2012Q3
You need to log in before you can comment on or make changes to this bug.