518646 - Search fails to return existing search engines

Status: VERIFIED FIXED

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

Description

[krupa raj[:krupa]]

steps to reproduce:
1.Search for "Merriam-Webster"
2.Search for A9 or BBC News
3.Go to https://preview.addons.mozilla.org/en-US/firefox/addon/4609

observed behavior:
Search fails to return existing search engines

screencast:http://screencast.com/t/ghNflChgk06

Comment 1

[Dave Dash [:davedash, :dd] (assign all bugs to mbrandt)]

blerg, okay these don't get associated with apps and therefore app versions...

so I made a few "hacks" to enable this.  Each search addon is associated with an app value of 999 , and min and max ver of 0..9999999999999.

This seems to work and we additionally filter by APP_ID + 999 to get search dictionaries everywhere.

I need to write some tests before this can get committed.

Comment 2

[Dave Dash [:davedash, :dd] (assign all bugs to mbrandt)]

Attachment: v1

This fixes:

bug 517547
bug 517572
bug 517589
bug 517954
and this bug, bug 518646.

Whence r+'d, I will commit, file a bug for IT to update preview's sphinx.conf, wordlist and the mysql view so QA can QA.

Comment 3

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

Reopening until we figure out why https://preview.addons.mozilla.org/en-US/firefox/search?q=+Merriam-Webster&cat=all doesn't fully work.

Comment 4

[krupa raj[:krupa]]

this looks fixed on preview