Fixes for PHP 7/8

If someone is interested in the work of Phorum with PHP 7/8 you can try the version with my fixes which you can download at [github.com]



Edited 1 time(s). Last edit at 08/13/2021 03:22PM by Sergei K.

In local it's good with php 7,4 . Modules for 7,2 no

How long did it take you to fix the code ? hours ? days ? 1 week ?

is there some generic changes to do, or has it be a step by step (after seing errors).

I'm asking because i need to move, but i have a specific release of 5.2.19 (with 20 mods) and i guess, i would be quicker to update the code directly, without even going 5.2.23 before (as there won't be any official future release; otherwise, i would move to the last release).

Thanks

I tried to run on the new version of PHP, collected the errors that occurred. Then I fixed them consistently. Both where errors occur and sometimes other places that are searched through files. After the changes, I tested a little creating a new installation and publishing messages. It took me a few hours throughout the day.

Thanks Sergei, it helps

Sergei,
Thank you for your work. I manage a phorum installation that's used a fair bit, and am nervous about upgrading, but also hate the errors that a slightly modified Phorum 5.2.23 generates with PHP7. Would you be willing to help me benefit from your experience? mpotts@mcn.org

Hello Michael,
I replied to you in email.

@Sergei K

I'm so grateful for the work you did. I was able to get this version installed. However, it did not fix my issue. It did move it, though. ;)

I'm still getting no results if I search for someting. And I get errors like this on the top of the screen (I've taken out my home directory):

Warning: count(): Parameter must be an array or an object that implements Countable in /include/forum_functions.php on line 71

Warning: count(): Parameter must be an array or an object that implements Countable in /include/forum_functions.php on line 71

Warning: Cannot modify header information - headers already sent by (output started at /include/forum_functions.php:71) in /.cache/tpl-lightweight_blue-header-4078174761e651367bdf29ce021b700b.php-stage2 on line 4

Warning: Cannot modify header information - headers already sent by (output started at /include/forum_functions.php:71) in .cache/tpl-lightweight_blue-header-4078174761e651367bdf29ce021b700b.php-stage2 on line 6

Hello Preston

Glad to hear that my efforts aren't completely lost. It's not an easy time for me (and for all of us) and I'll need some time to dive into php specifics, but I'll fix the problem. There are signs of activity in the github repository and may even be able to implement a maintenance release. Personally, I want that.