Hashover 2.0 does not work under PHP 8.2
Torsten-K opened this issue · comments
Hashover 2.0 does not work under PHP 8.2 on Apache. I receive an error message and a white page:
Deprecated: Creation of dynamic property HashOver\SpamCheck::$ip is deprecated in […]/hashover/backend/classes/spamcheck.php on line 36
Furthermore, login fails with several fatal errors.
Thank you for reporting this, I have yet to test HashOver on PHP 8.2. This change to the language is good. Most of my code is already written to avoid this warning, as dynamic class properties is frowned upon in my coding standard. However, it looks like class properties are being set dynamically in a few places by accident. I will try to get my hands on PHP 8.2 and fix these issues, but in any case, this issue should be fixed when 8.2 is officially released later this month.
Ah, that sounds nice, thanks for your efforts!
Can you please give the latest code another try?
Yes, with pleasure, but not until tomorrow, I think.
Uhm, where can I download the latest code? It does not seem to have changed (master branch).
You would get it the normal way, with a git pull. If you think you have the latest code, can you please see if it is giving you a different error?
Sorry for the late reply, I did not find the time to test again, until today. Good news: I can confirm that the latest code works under PHP 8.2.
Only a warning appears in the admin backend, but it is not very annoying and does not seem to affect the functionality:
Warning: file_exists(): open_basedir restriction in effect. File(/docs/de-de) is not within the allowed path(s): (/var/www/[…]:/usr/share/php) in /var/www/[…]/html/[…]/hashover/admin/view-setup.php on line 67