437000 - Search for Foxytunes isn't yielding results

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Public Pages, defect)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

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...

Comment 1

[Chris Pollett]

Attachment: patch that fixes the problem

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.

Comment 2

[Justin Scott [:fligtar]]

Comment on attachment 323512 [details] [diff] [review]
patch that fixes the problem

Looks like it fixes the problem.

Comment 3

[Chris Pollett]

Checked into r14559

Comment 4

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED; also double-checked that themes and dictionaries/language packs were showing up.