A single user can write multiple reviews

Bug #710458 reported by jhfhlkjlj
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ratings and Reviews server
Fix Released
Low
Michael Nelson
software-center (Ubuntu)
Invalid
Medium
Unassigned

Bug Description

Binary package hint: software-center

I'm able to post as many reviews as I please in Natty's software center. There should be a limit, even if it's not only one, to how many reviews one can post to prevent spamming.

ProblemType: Bug
DistroRelease: Ubuntu 11.04
Package: software-center 3.1.14
ProcVersionSignature: Ubuntu 2.6.38-1.28-generic 2.6.38-rc2
Uname: Linux 2.6.38-1-generic x86_64
Architecture: amd64
Date: Sun Jan 30 23:14:07 2011
InstallationMedia: Ubuntu 11.04 "Natty Narwhal" - Alpha amd64 (20101128)
PackageArchitecture: all
ProcEnviron:
 LANGUAGE=en_US:en
 LANG=en_US.UTF-8
 LC_MESSAGES=en_US.utf8
 SHELL=/bin/bash
SourcePackage: software-center

QA Notes:
1) Add a review for a package
2) try to add a second review for that package (should fail)
3) Update test server setting (if you want to test it also) ALLOW_MULTIPLE_REVIEWS_FOR_TESTING to true and graceful server
4) Verify that you can now add a second review.

Related branches

Revision history for this message
jhfhlkjlj (fdsuufijjejejejej-deactivatedaccount) wrote :
Revision history for this message
Michael Vogt (mvo) wrote :

Thanks for your bugreport.

This is something that also needs fixing on the server side.

Changed in software-center (Ubuntu):
status: New → Confirmed
importance: Undecided → Medium
Revision history for this message
Matthew Paul Thomas (mpt) wrote :

I don't know what the limit should be. But if it's less than, say, 100, maybe it should be handled on the client side too. Otherwise a genuine reviewer might have carefully written their 51st review for the day, only to get an incomprehensible error when they try to submit it, and no way around it except to leave the computer running until tomorrow. Instead, it would be better to say something like, "Whoa there! We appreciate your enthusiasm, but that’s enough reviews from you today. Try again tomorrow."

If the limit should be more than about 100, though, there's probably no need to handle it on the client side, since it's obviously abuse.

Revision history for this message
jhfhlkjlj (fdsuufijjejejejej-deactivatedaccount) wrote :

To be honest, I submitted this bug concerning the ability to write multiple reviews on the same application. I had accidentally duped a review the day I reported this bug.

tags: added: kb-task
Changed in rnr-server:
assignee: nobody → Michael Nelson (michael.nelson)
importance: Undecided → Low
status: New → In Progress
Changed in rnr-server:
status: In Progress → Fix Committed
tags: added: kb-defect
removed: kb-task
description: updated
Revision history for this message
Michael Nelson (michael.nelson) wrote :

QA'd as in the description above.

The only strangeness is that the USC client always displays 'Write another review' even though the user has written a review already (probably because we were going to allow subsequent reviews to update previous ones initially).

Revision history for this message
jhfhlkjlj (fdsuufijjejejejej-deactivatedaccount) wrote :

@Michael Nelson:

Should I go ahead and report a bug on that?

Revision history for this message
Dave Morley (davmor2) wrote :

There's no easy way to replicate this on a staging production system.

tags: added: qa-untestable
tags: removed: qa-untestable
Revision history for this message
Dave Morley (davmor2) wrote :

My mistake comment on wrong bug

Changed in rnr-server:
status: Fix Committed → Fix Released
Changed in rnr-server:
milestone: none → 11.03
Revision history for this message
dino99 (9d9) wrote :

natty have reached EOL now ; closing that report. Feel free to report against a newer version if needed, with ubuntu-bug

Changed in software-center (Ubuntu):
status: Confirmed → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.