cannot add review with same reviewer

[Miklos Koren]

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.

[ViktorNagy (viktor-nagy)]

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.
>

[Miklos Koren (korenmiklos)]

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!