592278 - page not found while loading collection details page

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Search, defect, P2)

Description

[Pratibha]

Steps:
---------
- Selected in Advance Search:
seach term: test, within=all collections
Search URL: https://next.addons.mozilla.org/z/en-US/firefox/search/?q=test&cat=collections&pp=20&advanced=
- there is collection named "test" in results page which do not have an author
-clicked on the collection name to check details page

URL is: https://next.addons.mozilla.org/z/en-US/firefox/collections/anonymous/bf1c7a64-a2fa-7fc5-3379-4994c4/

Result: Page not found

Comment 1

[Jeff Balogh (:jbalogh)]

Let's not index collections that don't have an author_id.

mysql> select * from collections where uuid like 'bf1c7a64-a2fa-7fc5-3379-4994c4%'\G
*************************** 1. row ***************************
                       id: 1094
                     uuid: bf1c7a64-a2fa-7fc5-3379-4994c4649a44
                     name: 397237
            defaultlocale: en-US
          collection_type: 0
                 icontype: 
                 nickname: NULL
              description: 397238
                   access: 0
                   listed: 1
              subscribers: 0
                  created: 2009-06-10 09:15:35
                 modified: 2010-08-06 17:05:26
                downloads: 13
           application_id: 1
       weekly_subscribers: 0
      monthly_subscribers: 0
               addonCount: 0
                  upvotes: 0
                downvotes: 0
                   rating: 0
              addon_index: d41d8cd98f00b204e9800998ecf8427e
recommended_collection_id: NULL
                author_id: NULL
                     slug: bf1c7a64-a2fa-7fc5-3379-4994c4
             all_personas: 0
1 row in set (0.00 sec)

Comment 2

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

Why do we have collections without authors?

We should delete them.

Comment 3

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

http://github.com/jbalogh/zamboni/commit/c525d1a

Comment 4

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

http://github.com/jbalogh/zamboni/commit/8515a45 too

Comment 5

[krupa raj[:krupa]]

collections without authors are not listed in the search results.verified