Allow viewing addons ordered by their rating, popularity etc.

RESOLVED FIXED in Future

Status

--
enhancement
RESOLVED FIXED
12 years ago
3 years ago

People

(Reporter: alex, Unassigned)

Tracking

Future
x86
Windows XP
Dependency tree / graph

Details

Attachments

(3 attachments)

(Reporter)

Description

12 years ago
Currently, remora shows add-ons sorted by name, and there isn't an options for users to view add-ons by most-popular, top rated, newest and so on.

There is a huge number of addons overall and in each category, so viewing them ordered alphabetically isn't very useful. Think of YouTube or Flickr with videos/photos ordered alphabetically.

Proposal:
1. Change default sorting to 'most popular' 
2. Allow users to change the sorting to 'top rated', 'most recent', 'by name'
3. Allow users to see clearly what the current sorting is
(Reporter)

Comment 1

12 years ago
Created attachment 255207 [details]
Suggested implementation for sort changing and display UI element
Severity: normal → enhancement
Status: NEW → ASSIGNED
Target Milestone: --- → Future
Comment on attachment 255209 [details] [diff] [review]
Sort implementation in Addons controller

Please add tests for this before requesting review of the patch.
Attachment #255209 - Flags: first-review?(fwenzel)
(Reporter)

Comment 4

12 years ago
(In reply to comment #3)
> Please add tests for this before requesting review of the patch.
Will do, wasn't sure what the procedure was... 

(Reporter)

Comment 5

12 years ago
Created attachment 255395 [details] [diff] [review]
Unit test for sorting options

Attaching a unit-test for the new sorting code. 
Currently, the test uses the 3 addons in the sample DB.
We can add more sample addons (about 20?) and update the $all_expected_results array in the test accordingly.

Updated

12 years ago
Blocks: 362526

Comment 6

12 years ago
Alright, I added the addon sort order and RSS feeds in SVN r2451. It'll be on stage shortly, and on preview when it is published the next time.

Alex, thanks for your help. While I did not just apply your patches (we are not juggling with as many big arrays anymore, instead we have SQL do the work as it is supposed to be ;)), I for example almost literally used your unit test. Well done.
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED

Updated

12 years ago
Blocks: 372841
Component: Add-ons → Administration
QA Contact: add-ons → administration
(Assignee)

Updated

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