Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PHP 8 Kompatibilität #10

Open
JanWennrichPCSG opened this issue Feb 22, 2022 · 3 comments
Open

PHP 8 Kompatibilität #10

JanWennrichPCSG opened this issue Feb 22, 2022 · 3 comments

Comments

@JanWennrichPCSG
Copy link

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
Copy link
Owner

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
Copy link
Author

JanWennrichPCSG commented Mar 1, 2022

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 d98711d)

@mariolorenz
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants