search

netgen / ezpublish-legacy

eZ Publish 4.x (legacy) kernel, maintained by Netgen for compatibility with eZ Platform
https://netgen.io
Other
4 stars 17 forks source link

Updated: Further Bugfixes for class properties for php8. Warning: PH… #17

Closed brookinsconsulting closed 1 year ago

brookinsconsulting commented 1 year ago

…P: E_DEPRECATED Creation of dynamic property. Part 3.

Hello @emodric

Here is another wave of tested changes which greatly reduce deprecation warnings logged to eZDebug. This was done by hand while manually re-loading a ezp homepage and fixing the deprecation notices. As this is a very big task sigh much more work needs to be done but I am taking this work in smaller sessions to avoid burnout.

We tried using chat gpt 3.5 to generate a rector ruleset to use as a automated method of solving these deprecations but have had no luck in getting the resulting solution to run without fatal errors. help in this area is welcome.

Cheers, Brookins Consulting

emodric commented 1 year ago

Thanks @brookinsconsulting ! I appreciate all your PRs here, since I am unable to work much on this currently :) Do take care of yourself and take things slow!