Suma icon indicating copy to clipboard operation
Suma copied to clipboard

Published 20 hours ago • verified publisher icon suma-project

WIP: PHP 8.1 support

Open cazzerson opened this issue 3 years ago • 1 comments

WIP: this is untested

Attempt to resolve issues that are causing Suma to fail with PHP 8.x. See the mailing list for examples.

  • Update dependencies, particularly zf1-future and Gump.
  • Remove get_magic_quotes_gpc() calls
  • Fix phpcs complaints in our code
  • Rebuild project
  • Document how to use phpcs/PHPCompatibility

cazzerson avatar Jun 13 '22 17:06 cazzerson

Hey @cazzerson, I made a docker version of Suma that's pegged to this branch if that would help you out. I'm still working on the documentation for it, but I think everything's working with PHP 8.1 and MariaDb 10.5.9. Note that the Suma code is running on a Debian based php8.1 image.

If you have any problems starting it up or working with it, please let me know!

jcraitz avatar Aug 16 '22 21:08 jcraitz

@kbeswick we've deployed this to a PHP 8.1 server and everything seems to be working. OK to merge?

cazzerson avatar Jan 24 '23 13:01 cazzerson

I ran this through a PHP scanner and also selected dependencies that still support PHP 7.1+. I think this can be a minor release, but it may be worth testing on one of your servers if you're still running PHP 7.x.

cazzerson avatar Jan 24 '23 14:01 cazzerson

This branch works for me on PHP 7.4. I tested things like administering locations/initiatives, using the collection client, and the analysis tools, and they all work without any issues. I think this is good to merge and call a minor release.

kbeswick avatar Jan 24 '23 17:01 kbeswick