Add Provenance Filter in FreieSuche #257
Conversation
juro-ub
changed the title
juro-ub
changed the title
updates/julian_1.sql
Outdated
| INSERT INTO datenbank_mapping(Formular, Datenfeld, de_Beschriftung, Feldtyp, Auswahlherkunft, Array, Seite, gb_beschriftung, fr_beschriftung, la_beschriftung) VALUES ("freie_suche", "ProvenanceLemma", "Provenienz (Lemma)", "select", "selektion_provenance", 0, "freie_suche", "Provenance (Lemma)", "Provenance (Lemma)", "Provenance (Lemma)"); | ||
| INSERT INTO datenbank_mapping(Formular, Datenfeld, de_Beschriftung, Feldtyp, Array, Seite, gb_beschriftung, fr_beschriftung, la_beschriftung) VALUES ("freie_suche", "Ausgabe_Provenance_Lemma", "Provenienz (Lemma)", "checkbox", 0, "freie_suche", "Provenance (Lemma)", "Provenance (Lemma)", "Provenance (Lemma)"); | ||
|
|
||
| CREATE TABLE IF NOT EXISTS `neg`.`selektion_provenance` ( |
There was a problem hiding this comment.
Bisher sind fast alle Tabellennamen auf deutsch. Sollen wir dementsprechend auch die Tabelle lieber selektion_provenienz nennen?
There was a problem hiding this comment.
Und bitte auch gleich eine passende Model Class hinzufügen. Es gibt glaube ich auch schon eine SelektionProvenance, die ist aber Abstract und eher als Zwischenebene gedacht für alle Tabellen die entsprechende provenance_id + provenance_source haben um entsprechende Getter / Setter zu vererben. Sollen wir die ggf. umbenennen? (z.B. SelektionAbstractProvenance)
There was a problem hiding this comment.
Ich habe die Tabelle umbenannt und eine Model Klasse erstellt
updates/julian_1.sql
Outdated
| INSERT INTO datenbank_mapping(Formular, Datenfeld, de_Beschriftung, Feldtyp, Auswahlherkunft, Array, Seite, gb_beschriftung, fr_beschriftung, la_beschriftung) VALUES ("freie_suche", "ProvenanceEinzelbeleg", "Provenienz (Einzelbeleg)", "select","selektion_provenance", 0, "freie_suche", "Provenance (Einzelbeleg)", "Provenance (Einzelbeleg)", "Provenance (Einzelbeleg)"); | ||
| INSERT INTO datenbank_mapping(Formular, Datenfeld, de_Beschriftung, Feldtyp, Array, Seite, gb_beschriftung, fr_beschriftung, la_beschriftung) VALUES ("freie_suche", "Ausgabe_Provenance_Einzelbeleg", "Provenienz (Einzelbeleg)", "checkbox", 0, "freie_suche", "Provenance (Einzelbeleg)", "Provenance (Einzelbeleg)", "Provenance (Einzelbeleg)"); | ||
|
|
||
| INSERT INTO datenbank_mapping(Formular, Datenfeld, de_Beschriftung, Feldtyp, Auswahlherkunft, Array, Seite, gb_beschriftung, fr_beschriftung, la_beschriftung) VALUES ("freie_suche", "ProvenanceLemma", "Provenienz (Lemma)", "select", "selektion_provenance", 0, "freie_suche", "Provenance (Lemma)", "Provenance (Lemma)", "Provenance (Lemma)"); |
There was a problem hiding this comment.
Brauchen wir ProvenanceLemma wirklich in der Suche?
There was a problem hiding this comment.
Ich habe es aufgenommen da es ja Teil des Imports ist und es eine Übersicht liefert der importierten Lemmas.
Aber ich kann es auch wieder entfernen. Was meinst du ?
There was a problem hiding this comment.
Ja bitte lieber rausnehmen
There was a problem hiding this comment.
ich habe es entfernt
|
Eigentlich sollten dann auch sämtliche Tabellen wieder umgestellt werden dass provenance_source eine ID ist und einen Fremdschlüssel auf die neue Tabelle hat |
Genau da war ich mir nicht sicher, weil das alignment Tool dann auch nochmal überarbeitet werden muss. Soll ich das dann so umsetzen und die Spalte provenance_source in eine Foreign ID Spalte ändern ? |
|
Das sollten wir der Konsistenz halber auf lange Sicht tun, wir können aber auch erstmal den PR in abgespeckter Form lassen und das mit der Typänderung / Foreign Keys später nachschieben. Die anderen Feedback-Punkte aber bitte trotzdem vorher noch umsetzen. |
Includes the filter for Provenance Information of Einzelbeleg