cannot add review with same reviewer

Bug #403108 reported by Miklos Koren
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
MKKE konferencia honlap
Fix Released
Undecided
Unassigned

Bug Description

I am trying to add a Review of Submission B with Reviewer A. Reviewer A has already reviewed Submission C. I get the following error message:

Review with this Reviewer already exists.

UPDATE: In models.Review, reviewer is a OneToOneField, not a ForeignKey. Will fix.

Revision history for this message
ViktorNagy (viktor-nagy) wrote : Re: [Bug 403108] [NEW] cannot add review with same reviewer

you shouldn't it a OneToOneField for a reason

On Wed, Jul 22, 2009 at 6:22 PM, Miklos Koren <email address hidden> wrote:

> Public bug reported:
>
> I am trying to add a Review of Submission B with Reviewer A. Reviewer A
> has already reviewed Submission C. I get the following error message:
>
> Review with this Reviewer already exists.
>
> UPDATE: In models.Review, reviewer is a OneToOneField, not a ForeignKey.
> Will fix.
>
> ** Affects: mkke-konferencia-honlap
> Importance: Undecided
> Status: New
>
> --
> cannot add review with same reviewer
> https://bugs.launchpad.net/bugs/403108
> You received this bug notification because you are the registrant for
> MKKE konferencia honlap.
>

Revision history for this message
Miklos Koren (korenmiklos) wrote :

Azóta már értem, hogy most mit csinál: ha Ádám 4 papírt reviewol, akkor 4 Reviewer object lesz, mindegyik külön értesítési funkcióval.

Ebből viszont az következik, hogy alapesetben nekem három lépés kell ahhoz hogy megkérjek valakit reviewolni: Add Review, Add Reviewer, Add User. És azt biztos el fogom téveszteni, hogy ha már 3 Ádámot látok a Reviewerek között, akkor valamelyiket választom a negyedik papírra is. Error, vissza, plusz két lépés.

Ez azért gond, mert kevés a reviewer (~5-6) és sok a submission (~60-80). Vagyis inkább növelném a per reviewer overheadet (mondjuk írok nekik egy emailt, hogy most már minden submission fönt van, lássanak neki), mint a per submission overheadet (sok oda-vissza klikkelés).

A másik, hogy ha olyan csinálja az admint, aki nem látott még relációs adatbázist, akkor neki a jelenlegi workflow nem fog menni.

Konklúzió: Vegyük ki a Reviewer modellt, sőt, inkább az egész AcceptedRoleAbstractot!

ViktorNagy (viktor-nagy)
Changed in mkke-konferencia-honlap:
status: New → 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.