PHP 8.4 compatibility code contributions

💬 General Discussion
1

Hi @suitecrm_team,

We have been working for a few months on adapting our fork (SinergiaCRM) to work with PHP 8.4, starting from SuiteCRM 7.14.6. We have more than 150 modified files.

For the past few weeks, it has been deployed in several production environments and has gone through a bug-fixing process. We are now very confident that the changes are stable and that we’ve done a solid job.

There are still a few minor issues related to third-party libraries, but nothing that currently affects production environments.

We are very pleased that SuiteCRM 7 will continue receiving support for two more years and that compatibility with PHP 8.4 is being addressed.

We would be happy to contribute these changes back to the SuiteCRM project by migrating them to the base repository. We understand that this won’t be a simple task and that it must be done carefully. Therefore, before beginning the migration, we would like confirmation from the SuiteCRM team that there are no conflicting plans and that these changes would be welcome. And if there is any extra consideration that we must know.

Please let us know your thoughts as soon as possible. We’ve just completed our validation process, and this would be a great time to start.

2 Likes
2

That’s an incredible work! :partying_face: :partying_face: :partying_face: :partying_face: :partying_face:

I will try to install and play with it.

3

Hi @sinergiacrm

Thank you for your offer to support the development of PHP 8.4 compatibility and it’s great to hear of the progress you’ve already made with PHP 8.4. So we would very much welcome any support you give for getting this and into core.

Here you should find the working branch for this feature, which already has worked on it to bring compatibility for PHP 8.3, so please use this as a base for any contributions for this specific feature.

3 Likes
4

Hi @Matt ,

We have just opened a PR with our contributions to PHP8.4 compatibility: PHP 8.4 Compatibility contribution by SinergiaCRM · Pull Request #10684 · salesagility/SuiteCRM · GitHub

We decided to not squash the commits for easier validation.

Just let us know if there is anything.

PS: We might be bringing other changes regarding PHP compatibility but those are still not validated internally. However we can consider adding them later in another PR.

2 Likes