search

ubtue / tuefind

Tuebingen University's derivatives of VuFind. Contains former versions like IxTheo, RelBib and KrimDok.
GNU General Public License v2.0
7 stars 4 forks source link

E-Mailversand von Suchergebnissen / Shortlinks im Mailversand #2625

Open NMagin opened 1 year ago

NMagin commented 1 year ago

Momentan scheint es ein Problem mit den Shortlinks beim Versand von Suchergebnissen zu geben. Der Versand der Mail klappt ohne Probleme:

Link in Relbib: https://134.2.67.9/short/u

Beim Klick auf den Link bekomme ich folgende Fehlermeldung: Screenshot 2023-11-17 at 13-03-58 Whoops! There was an error

mtrojan-ub commented 1 year ago

Wir werden diesen Issue möglichst noch vor dem morgigen Upgrade beheben, da PHP-Fehler.

mtrojan-ub commented 1 year ago

Problem ist größer als gedacht: Der Fehler kommt durch Änderungen an der Suchbox, die wir in allen Instanzen abgeleitet haben. Änderungen müssen übernommen werden, ohne dass andere Dinge kaputt gehen.

mtrojan-ub commented 1 year ago

Ready for testing auf ptah+sobek.

Die oben genannte URL lässt sich jetzt wieder fehlerfrei aufrufen. Dadurch waren jedoch Anpassungen in sämtlichen searchbox templates notwendig (obwohl die Suchbox ja in dem Result interessanterweise gar nicht zu sehen ist, ist sie trotzdem versteckt aktiv).

Da das Problem prinzipiell auch in allen anderen Instanzen vorgekommen ist, mussten wir die Suchbox auch dort anpassen. D.h. im Optimalfall sollte man auch die anderen Systeme nochmal kurz anschauen um zu sehen ob sonst noch etwas auffällt (insb. auch den Index Biblicus + Index Canonicus, kann aber im Zweifelsfall auch KrimDok betreffen. Ich habe alle Instanzen nochmal in Kürze angeschaut und keine weiteren Fehler festgestellt).

NMagin commented 1 year ago

Für RelBib, Ixtheo und KrimDok klappt jetzt alles wie es sollte. Vom Index Biblicus und Index Canonicus habe ich die Testsystem-URL nicht. @coeh Kannst du dir das bitte noch anschauen? Danke!

coeh commented 1 year ago

Ich habe die Test-URLs für Index Biblicus und Canonicus leider auch nicht. @mtrojan-ub kannst du sie mir sagen? Dann schaue ich es mir gleich an.

coeh commented 1 year ago

Vielen Dank @thefass für die URLs. Ich habe Index Canonicus und Index Biblicus jetzt getestet; dort funktioniert es auch.

Allerdings ist mir dabei aufgefallen: Der Betreff "Referenz aus" im Betreff dieser Mails ist im Deutschen recht merkwürdig (vermutlich eine nicht ganz passende Übersetzung des im Englischen korrekten reference). Das ist auf dem Produktivsystem genauso, also keine Folge des Upgrades. Hat keine akute Eile. Aber wollen wir das ändern? Z.B. in "Titel aus..." "Bibliographische Angaben aus..." "Datensatz aus..." (was wäre Ihrer/eurer Meinung nach die beste Variante? hat jemand eine bessere Idee?)