Propel icon indicating copy to clipboard operation
Propel copied to clipboard

Migration to PHP 7.4

Open tworzenieweb opened this issue 4 years ago • 8 comments

A lot of small changes applied to ensure that code works with PHP7.4, any help and thoughts are highly appreciated.

tworzenieweb avatar Jun 12 '20 11:06 tworzenieweb

The support for PHP 7.4 is very welcome - thank you for doing this!

But from what I can see, this approach breaks the support for older versions of PHP. Perhaps this only makes sense as a separate branch?

mentalstring avatar Sep 16 '20 17:09 mentalstring

Does anyone know of a reliable fork that completely fixes php 7.4 compatibility without breaking compatibility with 5.x (or introducing lots of regressions)?

Right now the most disastrous bc-breaking change in PHP 7.4 that I'm observing that completely breaks Propel 1 is this: https://www.php.net/manual/en/migration74.incompatible.php#migration74.incompatible.spl

teo1978 avatar Nov 30 '20 17:11 teo1978

This does NOT fix compatibility with PHP 7.4.

With 7.3 maybe, but this does not address https://www.php.net/manual/en/migration74.incompatible.php#migration74.incompatible.spl

teo1978 avatar Dec 03 '20 16:12 teo1978

May it be merged as a separate version? For my case its not relevant at all to have php 5.x support . I am not going to get back to it, but project is too big, and upgrading to propel2 is another big change that I am not ready yet.

gevorgmargaryan avatar Dec 29 '20 10:12 gevorgmargaryan

@teo1978 Tests for 7.4 are passing and I must admit this fork is running fine on a commercial project for almost a year without issues. We are doing a slow migration to Doctrine in the meantime.

I am planning to start migrating it towards 8.0 this month.

tworzenieweb avatar Apr 01 '21 15:04 tworzenieweb

Any news with regard to 8.0 support? I might consider helping out as I have a project running on propel as well.

DanielFortuyn avatar Nov 26 '21 08:11 DanielFortuyn

We've been using @tworzenieweb changes for supporting for PHP 7.4 for a long while now and haven't had any issues. 👍

@tworzenieweb Did you do any progress on the support for PHP 8.0 since then? I might give a hand towards that if I find the time, but would be best to avoid duplicate work.

mentalstring avatar Jan 16 '22 15:01 mentalstring

@tworzenieweb Thanks for your work on this! We used this as the basis for our fork with PHP 8.0 support.

@mentalstring @DanielFortuyn We published https://packagist.org/packages/dayspring-tech/propel-bundle#1.8.0 as a drop-in replacement with PHP 8.0 compatibility.

jwong-dayspring avatar Dec 22 '22 01:12 jwong-dayspring