Closed Bug 570617 Opened 14 years ago Closed 14 years ago

Review count does not update when all of an add-on's reviews are removed

Categories

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

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
5.11.4

People

(Reporter: krupa.mozbugs, Assigned: jbalogh)

References

()

Details

(Whiteboard: [z])

user agent:Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:1.9.2.3) Gecko/20100401 Firefox/3.6.3

steps to reproduce:
1.go to https://preview.addons.mozilla.org/en-US/firefox/addon/6028/
2.notice that this add-on has 106 reviews
3.click to view all the reviews.

expected result:
-There is a Reviews section in the add-on detail page,where at least three reviews are featured.
-There is a link to all review page

actual result:
The reviews section is missing
There is no link to the all reviews page.
This add-on has no reviews.  I suspect that the author deleted old versions.

The review count hasn't been reset because the query in maintenance.php does an INNER JOIN on reviews.  This add-on has 0 reviews, so there won't be a row for it and it never gets updated.
Depends on: 557879
Summary: Reviews missing for theme-"Abstract Zune" → Review count does not update when all of an add-on's reviews are removed
Target Milestone: 5.11.1 → Future
Blocks: 573773
http://github.com/jbalogh/zamboni/commit/2c4525a7b gets this right.

QA: this cronjob runs at 2am, so check back tomorrow.
Assignee: nobody → jbalogh
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: Future → 5.11.3
Fixed real good in http://github.com/jbalogh/zamboni/commits/de9ec8e.

Check it tomorrow.
Target Milestone: 5.11.3 → 5.11.4
not fixed yet.Reopening https://preview.addons.mozilla.org/en-US/firefox/addon/6028/#reviews
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Status: REOPENED → RESOLVED
Closed: 14 years ago14 years ago
Resolution: --- → FIXED
Jeff, the whole host of these bugs was fixed by the re-updating of the non-updating cron, right?
yessir, the fancy new cron wasn't getting run (bug 574837).
Depends on: 577332
No longer depends on: 577332
Will verify this bug once bug 557879 is fixed
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.