Active and passive moderation setting

Bug #686547 reported by Michael Nelson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ratings and Reviews server
Fix Released
Medium
Danny Tamez

Bug Description

Currently when a new review is added it appears and there is no way to alter this behaviour.

According to: https://wiki.ubuntu.com/SoftwareCenter/RatingsAndReviews#Moderating%20reviews there should be two modes active and passive.

Although this could be implemented via a setting, we really need this to be configurable in the admin (rather than by updating a configuration branch), so it might be worth briefly seeing what others do for key/value setting-type info in the admin (although a simple model would work - AppOption just with key/value attributes).

QA Notes:
 0) In the admin, ensure that there is no RNRSettings record,
 1) Create a review via the client,
 2) Access the /admin/ and verify that the review has False for it's 'hide' attribute (ie. it is unchecked)
 3) Create an RNRSettings record in the admin, and set the moderation mode to Active.
 4) Create another review (for a different app),
 5) Verify in the admin that the new review has True for it's 'hide' attribute
 6) Update the settings record, setting the moderation mode to Passive
 7) Create another review (for a different app),
 8) Verify in the admin that the new review has False for it's 'hide' attribute.

Note: the USC client will always show you the new reviews you add, whether or not they are hidden to everyone else.

Related branches

tags: added: kb-task
Changed in rnr-server:
assignee: nobody → Danny Tamez (zematynnad)
Changed in rnr-server:
status: Confirmed → In Progress
Danny Tamez (zematynnad)
Changed in rnr-server:
status: In Progress → Fix Committed
Dave Morley (davmor2)
tags: added: bug-1
Revision history for this message
Danny Tamez (zematynnad) wrote :

For some reason the value for activer mode & passive mode was being saved as a string u'1' instead of as an integer even though the field type was SmallIntegerField. I've fixed this now.

description: updated
Dave Morley (davmor2)
tags: added: bug-2
Changed in rnr-server:
status: Fix Committed → In Progress
Changed in rnr-server:
status: In Progress → Fix Committed
tags: added: sp-1
tags: added: kb-feature
removed: kb-task
Changed in rnr-server:
milestone: none → 11.05
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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