AndiLeni / statistics

Statistik Addon zur Erfassung von Seitenaufrufen. Für das CMS Redaxo.
MIT License
32 stars 5 forks source link

Im REDAXO log gefunden, PHP 8 #101

Closed tyrant88 closed 1 year ago

tyrant88 commented 1 year ago

stats_visit::__construct(): Argument #4 ($domain) must be of type string, null given, called in /var/www/sssssss/releases/228/src/addons/statistics/boot.php on line 95

src/addons/statistics/lib/stats_visit.php line 73

AndiLeni commented 1 year ago

Komisch, dass rex_yrewrite::getHost() hier offenbar null liefert.

Hast du eine yRewrite Domain angelegt? Bzw könntest vllt ins Project addon ein Log einfügen, damit man sieht, ob Besucher bei dir eine Domain gesetzt haben? rex_logger::logError(2, rex_yrewrite::getHost() ?? 'null', '', 1);

tyrant88 commented 1 year ago

$_SERVER['HTTP_HOST'] ist halt nicht immer gesetzt.

AndiLeni commented 1 year ago

Ja, ich war in der Annahme, dass rex_yrewrite::getHost() immer etwas liefert, aber dem ist nicht so. Ich würde bei fehlendem Host die Domain auf einen leeren String setzen.

tyrant88 commented 1 year ago

Ja, denke ich auch. Es ist einfach nicht auszuschließen. Hatte mich noch mal 3/4 ins Thema eingelesen, konnte aber nichts brauchbares ableiten.