Software center reviews can be voted infinite times

Bug #730768 reported by Pablo Quirós on 2011-03-07
This bug affects 1 person
Affects Status Importance Assigned to Milestone
software-center (Ubuntu)
Aaron Peachey

Bug Description

Binary package hint: software-center

When you see a review, the following message is below it:
X/X people have found this review helpful. Did you? Yes / No
You can vote all the times you want by pressing Yes or No, there's no check that you have already voted.

Aaron Peachey (aaronp) wrote :

Confirming this issue as it is part of the spec but we have not yet implemented it as part of the submit usefulness feature.

Changed in software-center (Ubuntu):
status: New → Confirmed
Aaron Peachey (aaronp) wrote :

Linked branch has a fix for this issue, but will only work for usefulness votes submitted AFTER this fix has been released.

This is an interim solution which will work until the server can support keeping track of who has voted (at which point we can also implement the UNDO functionality in the spec)

Changed in software-center (Ubuntu):
assignee: nobody → Aaron Peachey (aaronp)
status: Confirmed → In Progress
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package software-center -

software-center ( natty; urgency=low

  [ Michael Vogt]
  * merged lp:~mmcg069/software-center/little-catview-cleanups
  * merged lp:~mmcg069/software-center/stars-visual-tweaks
  * softwarecenter/db/, test/
    - fix locking and add regression test, thanks to Matthew McGowan
      for the research into this
  * softwarecenter/db/
    - fix support for reviews in not enabled PPAs (like for-pay PPAs)
      LP: #731376
  * softwarecenter/backend/
    - updated to match latest upstream version
  * data/ui/submit_review.ui, data/ui/report_abuse.ui:
    - set softwarecenter icon (thanks to davmor2 for reporting)
  * softwarecenter/db/
    - add "translated" argument to the parser and store untranslated
      name as well and store untranslated value in the DB
  * softwarecenter/db/
    - add get_untranslated_app() call
  * softwarecenter/db/
    - when talking to the rnr-server, always use the untranslated names

  [ Kiwinote ]
  * clean up mimetype handling (allows us to accept apturls once again)
  * softwarecenter/backend/
    - don't return two copies of one channel with different names
    - channel query is based on XOO rather than XOL
  * softwarecenter/backend/
    - import time on demand, this avoids an attribute error
  * softwarecenter/db/
    - avoid using unicode(None)
  * softwarecenter/view/
    - revert to correct sortmode once searchterm has been cleared

  [ Aaron Peachey ]
  * softwarecenter/db/
    - add functionality to keep track of review usefulness votes locally
  * softwarecenter/view/widgets/
    - different behaviour for review display if user has already
      votedon usefulness (LP: #730768)

  [ Gary Lasker ]
  * test/
    - add unit test for Unity launcher integration feature
 -- Michael Vogt <email address hidden> Fri, 11 Mar 2011 18:53:22 +0100

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

Other bug subscribers