Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

omp 1.2 oai plugin problem ? #68

Closed
withanage opened this issue Aug 22, 2016 · 33 comments
Closed

omp 1.2 oai plugin problem ? #68

withanage opened this issue Aug 22, 2016 · 33 comments
Assignees

Comments

@withanage
Copy link

@UmbrellaDish hat getestet.
OMP 1.2 sollte lokal installiert werden und getestet werden, ob der Fehler reproduzierbar ist.

@withanage
Copy link
Author

withanage commented Aug 23, 2016

nils, kannst du bei dir lokal installieren und die schritte dokumentieren?
Florian kriegt eine Fehlermeldung, wir können es zusammen untersuchen.
https://github.com/pkp/omp/tree/omp-stable-1_2_0

@nils-stefan-weiher
Copy link
Member

nils-stefan-weiher commented Aug 23, 2016

Alles klar werde ich machen

@nils-stefan-weiher
Copy link
Member

Hallo,

habe deswegen nochmal ein Issue bei PKP angelegt:
pkp/pkp-lib#1798

@withanage
Copy link
Author

@isgrim Vielen Dank !

@withanage withanage changed the title omp 1.2 oai plugin fehlerhaft ? omp 1.2 oai plugin problem ? Sep 6, 2016
@nils-stefan-weiher
Copy link
Member

Ich würde das auch mal mit OJS 3.0 auf serv21 testen.

@nils-stefan-weiher
Copy link
Member

nils-stefan-weiher commented Sep 12, 2016

OAI scheint unter OJS 3.0 zu funktionieren, aber ich habe es nicht hingekriegt mit "ListRecords" daten anzeigen zu lassen, d.h. ich weiß nicht genau wie der Query aussehen muss, dass auch Daten zurückkommen.

@nils-stefan-weiher
Copy link
Member

Alec Smecher hat geantwortet und die OAI Schnittstelle mit DCMetadataplugin funktioniert jetzt wieder in omp 1.2.0

@withanage
Copy link
Author

withanage commented Sep 13, 2016

@isgrim Danke für dein Einsatz.
Hast Du das patch in serv21 schon getestet ?

@withanage
Copy link
Author

@withanage
Copy link
Author

was denkst du ? können wir es ins serv3 reinnehmen ?

@withanage withanage reopened this Oct 24, 2016
@ankosi
Copy link

ankosi commented Nov 3, 2016

@isgrim
Ist das jetzt schon auf serv3?

@nils-stefan-weiher
Copy link
Member

@ankosi
Zur Zeit funktioniert es nicht mal mehr auf serv21. Wir benutzten dort die stabile OMP Version 1.2.0.
Ich kam diese Woche nicht dazu zu schauen ob es neue Aktualisierungen gibt.

@ankosi ankosi added the high label Nov 10, 2016
@ankosi ankosi self-assigned this Nov 10, 2016
@nils-stefan-weiher
Copy link
Member

nils-stefan-weiher commented Nov 14, 2016

Der Webserver auf dem OMP Testserver macht Probleme. Ich muss morgen schauen was genau da schief läuft.

Inzwischen gibt es eine Übersichtsseite im Github Wiki die den Status der verschiedenen Plugins und deren Abhängikeiten aufschlüsseln soll.

@withanage withanage removed their assignment Nov 15, 2016
@nils-stefan-weiher
Copy link
Member

Ich habe vermutlich rausgefunden was den Fehler in der OAI Ausgabe von heiup auf dem Testserver verursacht hat. Wenn eine Bestimmte Submission (122), als veröffentlicht in den Katalog übernommen wird geht es, sobald man sie wieder rausnimmt geht es nicht mehr.
Es wird ein Eintrag in einer Datenbanktabelle zurückgelassen, der die OAI Schnittstelle durcheinander bringt.

@nils-stefan-weiher
Copy link
Member

Auf dem Produktivserver ist es aber ein anderes Problem. Dort gibt es einen Bug in der installierten Version von OMP, der im aktuellen 1.2.0 stable branch von OMP nicht mehr drin ist.

@withanage
Copy link
Author

Ich habe vermutlich rausgefunden was den Fehler in der OAI Ausgabe von heiup auf dem Testserver verursacht hat. Wenn eine Bestimmte Submission (122), als veröffentlicht in den Katalog übernommen wird geht es, sobald man sie wieder rausnimmt geht es nicht mehr.
Es wird ein Eintrag in einer Datenbanktabelle zurückgelassen, der die OAI Schnittstelle durcheinander bringt.

ja, wenn es reproduzieren lässt, kann man das als Bug definieren

@withanage
Copy link
Author

withanage commented Nov 18, 2016

Auf dem Produktivserver ist es aber ein anderes Problem. Dort gibt es einen Bug in der installierten Version von OMP, der im aktuellen 1.2.0 stable branch von OMP nicht mehr drin ist.

ja, falls Du auch da den genaueren Grund isolieren kann, dass ist sehr hilfreich.

@nils-stefan-weiher
Copy link
Member

Hab ein Issue wegen OAI Bug für pkp aufgemacht: pkp/pkp-lib#2033

@withanage
Copy link
Author

Danke Dir. Können wir den Produktiven Server auf den gleichen Stand wie der Test-Server bringen ?

@nils-stefan-weiher
Copy link
Member

Ich prüfe nochmal wie groß die Änderungen zwischen den Versionen sind

@withanage
Copy link
Author

super!

@nils-stefan-weiher
Copy link
Member

Auf anhieb sehe ich nichts außer Bugfixes:
pkp/omp@omp-1_2_0-0...omp-stable-1_2_0

Ich würde allerdings abwarten ob sie vielleicht noch den OAI bug fixen bevor wir produktiv updaten.

@withanage
Copy link
Author

withanage commented Nov 25, 2016

Das könnte schon etwas dauern, aber wenn keine große Aufwand ist, würde ich das update machen.
Wir können @ankosi fragen, ob sie einen kurzen ein Blick auf den Test-Server werfen kann, ob kritische Probleme da sind.

@ankosi
Copy link

ankosi commented Nov 25, 2016

@withanage Was für kritische Probleme könnten das sein? Änderungen, die schon auf dem Testserver umgesetzt wurden, aber noch nicht auf dem Produktivserver?

@withanage
Copy link
Author

@ankosi
z.B , etwas wird gespeichert und nur ein "Loading button" wird gezeigt. In OMP 1.1 gab ja solche Probleme.
Am besten würde da ein Test-Eintrag machen und schauen, ob alle Schritte ohne Probleme eintragen lässt.

@ankosi
Copy link

ankosi commented Nov 25, 2016

Also einfach mal testen, ob alles auf unsere OMP-Testinstanz läuft? Ist das Instanz, auf der wir schon die ganze Zeit testen konnten, oder gibt es eine neue?

@withanage
Copy link
Author

withanage commented Nov 25, 2016

Also einfach mal testen, ob alles auf unsere OMP-Testinstanz läuft?

ja genau.

@ankosi
Copy link

ankosi commented Nov 25, 2016

Habe jetzt ein Buch auf der Testinstanz hochgeladen. Ich habe nur die Bugs gefunden, die schon vorher da waren. Hat also alles geklappt wie auf der Produktivinstanz.
http://serv21.ub.uni-heidelberg.de/omp/index.php/heiup/catalog/book/177

@nils-stefan-weiher
Copy link
Member

@ankosi
Danke, dann schaue ich mal wie wir am besten das Update machen und gebe Bescheid sobald es soweit ist

@nils-stefan-weiher
Copy link
Member

nils-stefan-weiher commented Dec 12, 2016

Update von OMP auf Produktivserver am 7.12.16 gemacht. Ist jetzt auf dem selben Stand wie der Testserver.
Ich habe jetzt auch das XMetaDissPlus plugin angepasst und Fehler beseitigt.
Die Änderungen sind in einem Branch zu finden: UB-Heidelberg/xmdp22@master...omp-stable-1_2_0
Das XMDP-Plugin läuft jetzt auch auf dem Testserver:
http://serv21/omp/index.php/heiup/oai?verb=ListRecords&metadataPrefix=XMetaDissPlus&set=opentest:pubf_pdf
Als nächtes müssten wir überlegen wie man die Ausgabe von XMetaDissPlus Daten gut testen kann.

@withanage
Copy link
Author

Super. Hast Du schon auch in wiki die Sachen aktualisiert?
Lass uns mal morgen dazu zusammensetzen.

@ankosi
Copy link

ankosi commented Mar 13, 2017

@isgrim Wie ist denn hier der neueste Stand? Die OAI-Schnittstelle ist für uns bei allen Portalen sehr wichtig.

@nils-stefan-weiher
Copy link
Member

@ankosi Die OAI Schnittstelle läuft seit dem 7.12. jetzt auch Produktiv. Entschuldige falls das nicht klar war.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants