[googlesitemap] Doppleter Slash nur bei "Suchindex neu aufbauen"

[jan.theofel]

Hi,

das Problem mit zwei Slash am Schluss in der XML-Sitemap oder beim Link auf die Startseite ist hier ja schon ein paarmal aufgetreten. Das haben wir hier nicht.

Bei uns tritt das Problem nur beim Erzeugen eines neuen Suchindex über Systemwartung/"Suchindex neu aufbauen" auf.

Die Ursache:
Die Funktion checkInitialPage liefert bei der Startseite ein / als Frontend-URL für die Startseite zurück. In system/modules/backend/Backend.php wird aber in der Funktion findSearchablePages die URL jeweils aus $domain + Frontend-URL zusammengesetzt (Zeile 294):

$arrPages[] = $domain . $this->generateFrontendUrl($objPages->row()); 

Da in $domain an dieser Stelle am Schluss ein Slash steht, wird dieser entsprechend gedoppelt.

Workaround:
Ersetzen der obigen Zeile gegen

$arrPages[] = preg_replace('/\/\/$/', '/', $domain . $this->generateFrontendUrl($objPages->row())); 

Bevor ich genau diesen Workaround als Core-Patch in einem Ticket vorschlagen will, wäre die Frage, ob es andere Lösungen gibt. Ich könnte mir beispielsweise gut vorstellen, dass man generateFrontendUrl als zusätzlichen Parameter mitteilt, ob absolute (mit Domainnamen) oder relative URLs (ohne Domainname) erzeugt werden sollen. Dann könnte man das Problem im Ansatz lösen, statt an jeder Stelle eine solches preg_replace einzubauen. (Sofern es mehr Stellen gibt - wovon ich ausgehe.)

Jan

[xchs]

Bei uns tritt das Problem nur beim Erzeugen eines neuen Suchindex über Systemwartung/"Suchindex neu aufbauen" auf.

Kann das auch bestätigen. Beim Neuindizieren der Seiteninhalte wird für die als "Erste Seite" gekennzeichnete Seite ein doppelter Slash erzeugt.

[Kahmoon]

Jepp...bei uns auch. Danke für den Tipp. Leider aktuell noch nicht updatesicher

[andreas.schempp]

Wäre das nicht Update-sicher machbar, indem der getSearchablePages Hook durchgegangen wird und die doppelten Slashes entfernt werden? Wenn ich mich richtig errinnere, werden dem Hook doch auch alle bereits bekannten URLs übergeben?

[ways2web]

Dieses Problem besteht wohl weiterhin oder ? (2.9.5 + aktuelle googlesitemap ext)
Wobei das bei mir nichts mit dem "neuen suchindex" zu tun hat..
sobald ich "erste seite" option nutze, hab ich für die startseite in der sitemap.xml ne falsche url.

ala :

exmaple.com/subDir//subDir/ statt exmaple.com/subDir/

gruss
ways

[andreas.schempp]

Guck mal ob es auf www.contao-forge.org ein Ticket dafür gibt, wenn nicht bitte ein's anlegen

[ways2web]

ok. zumindest kein offenes Ticket, nur das von leo, dass ich auch schon mal hier im forum angesprochen habe mit dem autom logout...

dann werd ich doch gleich mal eins anlegen...

danke Andreas... dass nicht nur Isotope soll toll von dir supportet wird!

highfive

gruss
Olli

[Supermann]

Hallo,

ich habe gesehen das dass Problem weiterhin auch bei 2.10.0 besteht. Gibt es denn da jetzt schon eine Lösung für oder muss man weiterhin die oben genannte Code Zeile austauschen?

Gruß

[andreas.schempp]

http://www.contao-forge.org/issues/1656

[Supermann]

Hallo,

danke für die Info.

Aber ich wollte auch wissen ob das Workarround erst noch weiter genuzt werden muss.

Weiter habe ich auch das Problem mit dem Doppelslash bei dem Contao Suchindex, dort wird dieser weiterhin angezeigt, bzw im Backend beim Suchindexaufbau wird mir der doppelslash nicht mit angezeigt. Wenn ich im Frontend allerdings die Suche bemühe, wird bei der Startseite ein Slash zuviel mit ausgegegeben.

Bei der Sitemap ist jedoch alles OK.

Hat jemand da schon Erfahrung mit gemacht?

Gruß

[Ben.]

Ich hab das gleiche Problem, kann allerdings die oben genannten Code-Zeilen nicht finden.

Ist der Workaround noch aktuell für die 2.10.3?