FriendsOfREDAXO/search_it

Beispiel-Module: URL Ausgabe bei Nutzung von yrewrite in einer Multidomain Umgebung verbessern

marcohanke opened this issue · 7 comments

rex_getUrl() gibt bei der aktuellen Domain die relative Url aus (deshalb ist in den Beispielen $hit_server vorangestellt. Wenn im Multidomainumfeld auf eine andere Domain verwiesen wird, gibt rex_getUrl() jedoch die absolute Url aus. Daher wird die Domain in dem Fall doppelt ausgegeben.
Die Ausgabe über rex_yrewrite::getFullUrlByArticleId($hit['fid'], $hit['clang'], array('search_highlighter' => $request)) löst das Problem.

die 7.0 kommt bald, dafür werden auch noch mal die Beispiele überarbeitet, da kommt das auf jeden Fall mit rein.

PR welcome

PR war unerwünscht @alxndr-w

Oh man! Du Unglückswurm! ;-)
Wir haben gerade auf slack diskutiert, dass ich keinen PR wollte.

Das konnte ich nicht wissen. 🤷‍♂️

Fände es aber auch geiler, wenn die Beispiele allesamt stimmen würden.

Genau darum geht es ja. Deine Überlegung war ja sogar auch, auf ein instaliertes Modul zu wechslen... gerade dann muss das ja alles gut überlegt sein. Wenn wir den Leuten schon was vorschreiben, dann auch richtig :-)

Das Beispiel ist ja nicht falsch - und bezieht sich auf Installationen ohne Multidomain-Kontext. Das kurz anzupassen ist, finde ich, kein Problem.

Da kein PR möglich und keine 7.0.0 in Sicht, mache ich hier mal zu.