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

Neuen Identifier-Typ ISMN in Application integrieren #1134

Closed
j3nsch opened this issue Oct 31, 2023 · 5 comments
Closed

Neuen Identifier-Typ ISMN in Application integrieren #1134

j3nsch opened this issue Oct 31, 2023 · 5 comments
Assignees

Comments

@j3nsch
Copy link
Member

j3nsch commented Oct 31, 2023

Es wurde ein neuer Identifier-Typ ISMN hinzugefügt. Dieser ist für ein neuen Dokumenttyp notwendig (siehe #1131). Für den neuen Typ müssen Übersetzungen usw. hinzugefügt werden.

@alw-bsz
Copy link
Contributor

alw-bsz commented Nov 10, 2023

@j3nsch Ich kann den Identifier ins Publish-Formular einbauen, allerdings wird die Eingabe nicht in der Datenbank gespeichert. Es kommt keine Fehlermeldung, alle anderen Daten werden korrekt gespeichert. Bei der Eingabe der ISMN direkt in der Dokumentenverwaltung wird sie gespeichert. Fehlt da noch was? (S.a. https://github.com/alw-bsz/application/tree/ismn1131)

@j3nsch
Copy link
Member Author

j3nsch commented Nov 10, 2023

Danke, für den Hinweis @alw-bsz, vermutlich spielt und das Publish-Formular wieder Streiche. Ich schau mal, wo die erlaubten Identifier-Typen noch definiert sind. Vermutlich gibt es da keine extra Fehlerbehandlung, weil die Typen bisher fest verdrahtet waren, und es wird einfach verschluckt.

@j3nsch
Copy link
Member Author

j3nsch commented Nov 10, 2023

Ja, in der Publish_Model_Deposit-Klasse, in der Funktion storeIdentifierObject wird jeder "bekannte" Typ einzeln behandelt. Das muss ich ändern. Hatte ich schon mal gesagt, dass wir unbedingt ein neues Publish-Modul brauchen, seit etwa acht Jahren? :-)

Ich habe dafür #1154 angelegt.

@alw-bsz
Copy link
Contributor

alw-bsz commented Nov 13, 2023

@j3nsch Jetzt funktioniert's, danke!

@j3nsch
Copy link
Member Author

j3nsch commented Nov 21, 2023

Ich gehe davon aus, dass das erledigt ist.

@j3nsch j3nsch closed this as completed Nov 21, 2023
@github-project-automation github-project-automation bot moved this from In Progress to Done in OPUS 4.8.1 Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

2 participants