Publikation about conference-system

[mbasti]

Ja, es fehlt eigentlich dass eine Publikation mehrere Revisionen haben kann oder?

[autarchprinceps]

Stimmt, aber das ist ein eigenes Problem und sollte eigentlich unabhängig vom Format einer einzelnen Revision sein.

[autarchprinceps]

Ich habe den File Dialog mal hinzugefügt, aber ich weiß nicht was ich alles machen muss, sodass in der Publication Entity statt String text; List<byte[]> revisions; stehen kann.
Kann ich das einfach ersetzen, oder was muss ich da noch ändern?

[mbasti]

Der Datentyp ist dann ein Blob? Für ne Liste bräuchte man eigenlich dann noch eine Revision Entity-Klasse damit das auf Tabellen abgebildet wird. Wie bei anderen Collections/List auch.

[mbasti]

Autoren können jetzt sequentiell neue Revisionen hochladen (im PubView, nicht im createView). Es fehlen noch Download-Links für die Reviews.

[autarchprinceps]

Download funktioniert jetzt bei mir.

Ich weiß, dass der Reviewer keine Revisionen seiner Reviews hat, aber die Reviews sollten wahrscheinlich auch Dateien sein. Ich weiß nicht ob du entsprechende Änderungen in der JPA schon gemacht hast. Ich werde später heute oder morgen die UI dafür erstellen/apassen. Ich glaube, wenn du das noch nicht gemacht hast, dass ich die JPA dafür aber auf Basis der Publikationsentityklasse wahrscheinlich selber anpassen kann. Wenn nicht frage ich dich noch mal.

[mbasti]

Ne, hatte ich noch nicht gemacht, aber eigentlich muss man da nur den Datentypen von String auf byte[] überall ändern mit den entsprechenden Annotationen (wie in PublicationRevision). Es kann sein, dass man die Tabelle Review in der DB dann löschen/anpassen muss, wenn man was in der Entity ändert (bin ich mir grad nicht sicher).

Noch was:

1. Du hast filename und contenttype noch in PublicationRevisionPK getan. Also JPA macht für zusammengesetzte Primärschlüssel eine eigene Klasse und die 2 neuen Felder gehören für mich jetzt nicht zum Primärschlüssel, was aber anscheinend praktisch keinen Unterschied macht. Wollte ich nur anmerken.

2. Wie sollen wir diese Anforderung umsetzen?

   FA3: Zuordnung Paper zu Gutachter durch den Organisator der Konferenz

[autarchprinceps]

1. Habe ich mit Commit Review behoben.
2. Hmm ok, im Moment kann jeder Reviewer einer Konferenz alle Paper ohne Einschränkungen reviewen. Glaubst du wir brauchen da noch eine zusätzliche Einschränkung?
   Falls ja müsste der Organisator einer Konferenz wahrscheinlich im PubView aus einer Liste der Reviewer mit Checkbox diejenigen auswählen, die reviewen sollen/können. Und wir müssten den check für das Uploaden eines Reviews von isReviewer auf isReviewerOfPublication ändern.
   Ich bin mir nicht sicher was das für Konsequenzen für das JPA hat. Wahrscheinlich eine neue OneToMany-Beziehung in Publication: Collection authorizedReviewers; oder so.

[autarchprinceps]

zu 2. Ich fände das eigentlich etwas umständlich, ich meine dann müsste der Organizer ja durch jede Publication durchgehen und die Reviewer auswählen. In der Realität wäre das glaub ich übermäßig aufwendig, aber das heißt nicht, dass unser Prof das nicht als Anforderung gesetzt haben könnte.

[mbasti]

Naja, da das offensicht leider eine Anforderung ist, werde ich mal anfangen das umzusetzen.