Logo des Projekts wird bei Installation im Unterordner nicht angezeigt
aeberhard opened this issue · comments
in navigation.php sollte rex_url::frontend() vorangestellt werden
echo '<img src="' . rex_url::frontend() . be_branding::checkExtension(rex_addon::get('be_branding')->getConfig('file')) . '" class="img-responsive center-block" style="padding: 20px 10px 5px 10px; width: 100%;"/></a>';
Im Login-Screen muss der Pfad auch noch angepasst werden:
https://github.com/danspringer/be_branding/blob/master/boot.php#L32
$ersetzen = array('<img src="' . be_branding::checkExtension($this->getConfig('file'. be_branding::getCurrentBeDomainId(true) )) . '" class="img-responsive center-block" style="padding: 10px 0px 15px 0px; width: 370px;"/></a><section class="rex-page-main-inner" id="rex-js-page-main">');
Ja korrekt, hatte gar nicht gesehen dass da auch ein Logo ausgegeben wird.
Fixed via 9f1e36b
In beta2 wird das Logo immer noch nicht angezeigt bei Installation im Unterordner.
Auch beim Loginscreen nicht.
Was für ein Pfad wird da dann ausgegeben?
<img src="http://localhost/media/logo.svg" wird ausgegeben
und mit Unterordner rex513 sollte es sein
<img src="http://localhost/rex513/media/logo.svg"
Sorry, wollte da einen Screenshot hochladen aber das klappt hier mit Firmenproxy nicht.
Ah, okay. Danke.
Ich schaue mir das an. Habe es bisher immer nur mit Subdomains getestet, aber noch nicht mit Unterordner-Installationen
Ich arbeite immer lokal mit Unterordnern.
Gleiches Problem auch beim Logo im Login-Screen.
rex_url::frontend() liefert meine ich auch den Unterordner.
Okay, dann muss ich mal schauen, ob rex_url::frontend() auch die richtige Domain liefert für eine Multidomaininstallation.
Theoretisch ist es eigtl. egal, da es ja nur darum geht, dass der Pfad zu einem existenten Bild stimmt. Aber schöne wäre es, wenn da auch die Domain stimmt.
Vielleicht ist bei Medien besser ohne den Host zu arbeiten.
Schau mal bei yrewrite
https://github.com/yakamara/redaxo_yrewrite/blob/9643e644bec340305ef3de64430c24794b280ee9/lib/yrewrite/yrewrite.php#L301-L309
Kannst du nochmal reinsehen @aeberhard :
https://github.com/danspringer/be_branding/releases/tag/1.8.0-beta3
danke!
Jetzt passt es Daniel :)
Okay, cool. Dann kann ich das hier endlich schließen 😅
Ging ja lange genug.
Danke fürs Testen!