agoat / contao-piwikanalytics

Enables tracking with the piwik analytics software
0 stars 0 forks source link

Seiten werden nicht getrackt, wenn der Seitentitel doppelte Anführungszeichen enthält #5

Open arno1979 opened 6 years ago

arno1979 commented 6 years ago

Hallo Arne,

ich schon wieder... :) Über das ganze geteste mit dem Download-Tracking bin ich auf ein separates Problem gestoßen: Wenn der Seitentitel doppelte Anführungszeichen enthält, führt das dazu, dass der von deiner Erweiterung generierte Tracking Code nicht funktioniert. Für den Parameter setDocumentTitle sieht der dann folgendermaßen aus:

 _paq.push(["setDocumentTitle", "Startseite "Test""]);

Das zerlegt einfach den Script-Code, vermute ich. Ich dachte erst an ein Matomo-Problem, konnte dazu aber gar nichts finden. Wenn man in deinem Code in Piwik.php auf Zeile 74 auf das $title z.B. die PHP-Funktion addslashes() anwendet, lässt sich das Problem vermeiden. Den gleichen Effekt haben wohl auch die PHP-Funktionen htmlentities() und htmlspecialchars(), was jetzt in diesem Fall die beste Lösung ist musst du entscheiden. Vielleicht gibt es ja auch noch einen anderen Weg. Andere Sonderzeichen habe ich jetzt noch nicht getestet.

Viele Grüße Arno

agoat commented 6 years ago

Ohne es mir jetzt genau angeschaut zu habe, sollten natürlich Sonderzeichen encodiert bzw. Anführungszeichen freigestellt werden.