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

Einzelbelege: Textkritik auf Testserver oftmals leer #114

Open
mtrojan-ub opened this issue Mar 9, 2023 · 6 comments
Open

Einzelbelege: Textkritik auf Testserver oftmals leer #114

mtrojan-ub opened this issue Mar 9, 2023 · 6 comments
Assignees
Labels
ready for testing Has been solved, should be tested by the issue author & closed when confirmed.

Comments

@mtrojan-ub
Copy link
Contributor

Beispiel Live-Server:
https://neg.ub.uni-tuebingen.de/gast/einzelbeleg.jsp?ID=97877

grafik

Beispiel Test-Server:
https://ub58.v264.uni-tuebingen.de/gast/einzelbeleg?ID=97877

(Leere Anzeige)

Bei der Migration des Testservers (InnoDB / Fremdschlüssel) wurden einige Datensätze gelöscht. Eine erste Vermutung ist, dass einzelbeleg_textkritik.HandschriftID gar nicht auf die Tabelle handschrift zeigen sollte, sondern auf eine andere (z.B. handschrift_ueberlieferung). Das muss aber noch genauer geprüft werden.

@mtrojan-ub mtrojan-ub self-assigned this Mar 9, 2023
@ThoFischer
Copy link

Um wie viele Datensätze handelt es sich denn?
Bei dem vorliegenden Fall "Chronicon Anianense" müsste es eine Verknüpfung zur Edition Kettemann2000 geben und eine Verknüpfung zu handschrift_ueberlieferung bestehen.

@mtrojan-ub
Copy link
Contributor Author

Insgesamt hat einzelbeleg_textkritik 69.970 Einträge. Davon ist die genannte Referenz in insgesamt 38.915 Fällen fehlerhaft. Wenn man die Dubletten nicht berücksichtigt sind es 1.834 IDs. Die letzten beiden Zahlen gelten aber im Vergleich zur Tabelle "Handschrift", die Frage ist ob es eigentlich auf etwas anderes zeigt.

Im eingeloggten Bereich sieht man beim genannten Einzelbeleg unter "Textkritik" folgende Informationen auf dem Live-Server:
grafik

Dort wird allerdings nirgends ein Label für "Handschrift" angezeigt (lediglich "Edition" und "Sigle").
In der Datenbank sieht die Info wie folgt aus:

grafik

Auf dem Testserver ist lediglich die HandschriftID entfernt worden, weil es keine Handschrift mit der ID 6117 gibt.
Wie der Screenshot zeigt ist aber die "EditionID" durchaus nach wie vor noch vorhanden. Lediglich die HandschriftID fehlt auf dem Testserver.

Wie hängt das genau zusammen?

@juro-ub
Copy link
Contributor

juro-ub commented Apr 18, 2023

Aus dem Quellcode ergibt sich, dass die Spalte einzelbeleg_textkritik.HandschriftID
auf handschrift_ueberlieferung.ID und nicht auf handschrift.ID verweist

@juro-ub juro-ub linked a pull request Apr 18, 2023 that will close this issue
@mtrojan-ub
Copy link
Contributor Author

Wie heute besprochen, wollte ich noch weiter prüfen und Beispiele nennen um zu beweisen dass nach der Änderung nur inkonsistente Daten gelöscht werden.

Weiteres Beispiel (aktuell auf dem Live-Server):
https://neg.ub.uni-tuebingen.de/einzelbeleg.jsp?ID=92791

grafik

Der letzte Eintrag mit Bemerkung corr. Chodoveus verweist unter Sigle auf nicht bearbeitet. In der Datenbank sieht das allerdings so aus:

grafik

Die HandschriftID zeigt auf 3890, aber hierzu gibt es keinen Eintrag mehr in handschrift_ueberlieferung. Man sieht auch, dass im Backend die Spalte Sigle auf nicht bearbeitet steht. Es wäre also plausibel, dass diese handschrift_ueberlieferung irgendwann gelöscht wurde. Die Migration würde den Wert also auf den Defaultwert zurücksetzen und für den Endbenutzer würde sich sowieso nichts ändern, aber die DB wäre dann wieder konsistent.

@mtrojan-ub
Copy link
Contributor Author

Dazu noch eine Ergänzung: Im Dropdown gibt es neben nicht bearbeitet (-1) auch noch den Wert unklar (0). Dieser ist in der Datenbank gesetzt, kann aber auch in der alten Liveversion in der Oberfläche nicht richtig aufgelöst werden d.h. es wird auch wenn unklar gesetzt ist auch heute schon immer nicht bearbeitet angezeigt. Das betrifft aber nur 52 Datensätze. Die separate Bezeichnung würde mit bereinigt werden und zukünftig entfallen.

Beispiel:
https://neg.ub.uni-tuebingen.de/einzelbeleg.jsp?ID=88707

grafik
grafik
siehe Zeile 4 (Hilarius), HandschriftID ist laut Datenbank 0 (unklar) aber laut Oberfläche -1 (nicht bearbeitet).

@mtrojan-ub mtrojan-ub added the ready for testing Has been solved, should be tested by the issue author & closed when confirmed. label Apr 25, 2023
@mtrojan-ub
Copy link
Contributor Author

Der Testserver wurde aktualisiert und die verloren gegangenen Werte müssten nun wiederhergestellt worden sein.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for testing Has been solved, should be tested by the issue author & closed when confirmed.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants