Closed
Bug 437000
Opened 17 years ago
Closed 17 years ago
Search for Foxytunes isn't yielding results
Categories
(addons.mozilla.org Graveyard :: Public Pages, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
3.4.3
People
(Reporter: stephend, Assigned: cpollett)
References
()
Details
(Keywords: regression)
Attachments
(1 file)
|
797 bytes,
patch
|
fligtar
:
review+
|
Details | Diff | Splinter Review |
URL: https://preview.addons.mozilla.org/en-US/firefox/search?q=foxytunes&cat=all
A regression; that search is failing to find Foxytunes on preview -- this is post-advanced search backout, BTW...
|
Assignee | |
Comment 1•17 years ago
|
||
This problem was caused by my patch to 436937l. Basicaully, I changed the default parameter for $aversions in the model but not the search_controller. This had the effect that if no version range was provided then it acted as if the compatible versions was set to the empty array causing nothing to be returned.
Attachment #323512 -
Flags: review?(fligtar)
|
||
Comment 2•17 years ago
|
||
Comment on attachment 323512 [details] [diff] [review]
patch that fixes the problem
Looks like it fixes the problem.
Attachment #323512 -
Flags: review?(fligtar) → review+
|
|
Assignee | |
Comment 3•17 years ago
|
||
Checked into r14559
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
|
Reporter | |
Comment 4•17 years ago
|
||
Verified FIXED; also double-checked that themes and dictionaries/language packs were showing up.
Status: RESOLVED → VERIFIED
|
|
Assignee | |
Updated•17 years ago
|
Keywords: push-needed
|
||
Updated•17 years ago
|
Keywords: push-needed
|
||
Updated•9 years ago
|
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•