Closed
Bug 408680
Opened 17 years ago
Closed 16 years ago
Add support for reviews column in addons table
Categories
(addons.mozilla.org Graveyard :: Administration, defect)
addons.mozilla.org Graveyard
Administration
Tracking
(Not tracked)
RESOLVED
FIXED
3.2
People
(Reporter: morgamic, Assigned: morgamic)
References
Details
Attachments
(2 files)
8.82 KB,
patch
|
Details | Diff | Splinter Review | |
1.40 KB,
patch
|
wenzel
:
review+
|
Details | Diff | Splinter Review |
Review counts are needed in 3.2's redesign, and I don't want to have to do an extra query on the reviews table to get this info. For the same reason we have flattened out average rating, weekly and total downloads, etc. So I'd like an additional field added to addons named 'reviews' that's an unsigned int(11) and also we will need another action in maintenance.php to update these counts (read from slave, update master).
Assignee | ||
Updated•17 years ago
|
Assignee: nobody → morgamic
Assignee | ||
Comment 1•17 years ago
|
||
This depends on fligtar's patch for maintenance.php. Need to add reviews to addon model... will reupload after meeting.
Assignee | ||
Updated•17 years ago
|
Attachment #297373 -
Attachment is patch: true
Attachment #297373 -
Attachment mime type: application/octet-stream → text/plain
Assignee | ||
Comment 2•17 years ago
|
||
When we push this, we will need to have this run on prod: alter table addons add column `totalreviews` int(11) unsigned NOT NULL default '0'
Attachment #299951 -
Flags: review?(fwenzel)
Assignee | ||
Updated•17 years ago
|
Keywords: push-needed
Comment 3•17 years ago
|
||
Comment on attachment 299951 [details] [diff] [review] v1.5, fixes maint script col name and adds column to addon model Yes, this looks good. You want to use it in the stars element too, I assume.
Attachment #299951 -
Flags: review?(fwenzel) → review+
Assignee | ||
Updated•16 years ago
|
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Updated•8 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
•