vanilla-thunder / oxid-module-matomo

Piwik Analytics integration for OXID eShop 4.7+
GNU General Public License v3.0
2 stars 10 forks source link

PHP 8 Kompatibilität #10

Open JanWennrichPCSG opened 2 years ago

JanWennrichPCSG commented 2 years ago

Aufgrund der Verwendung einer mit PHP 8 nicht mehr gültigen Syntax an einer einzigen Stelle, ist das Modul leider nicht mit PHP 8 kompatibel.

Für die notwendige Änderung haben wir bereits einen Pull Request erstellt: #9

Wir würden uns freuen wenn die Änderungen in das Modul übernommen werden würden, da OXID in der neuesten Version nun auch PHP 8 kompatibel ist.

vanilla-thunder commented 2 years ago

hey, danke schon mal! Ich werde das definitiv mergen, könnte bei mir nur paar Tage dauern, da ich gerade keinen funktionierenden Shop mit PHP8 habe.

JanWennrichPCSG commented 2 years ago

Hallo @vanilla-thunder,
gibt es hierzu schon etwas Neues?
Wir würden den OXID Shop eines Kunden gerne aktualisieren und aktuell hakt es nur an diesem Modul.

Muss die Änderung durch den Merge Request explizit mit einem PHP 8 Shop getestet werden?
Es handelt sich ja nur um eine kleine Änderung, die nur eine Zeile betrifft (siehe https://github.com/vanilla-thunder/oxid-module-matomo/pull/9/commits/d98711d8ff4e69752ba11431c2c1ad0a7a0ae2c6)

mariolorenz commented 2 years ago

@vanilla-thunder, Kann den Bug bestätigen und den Fix ebenfalls. Der Fix wird in PHP8.0 & PHP8.1 laufen. bitte mergen.