- Issue created by @mohammad-fayoumi
- Merge request !247Issue #3482015 by mohammad-fayoumi: In PHP 8.2+, the creation of dynamic properties is deprecated → (Open) created by mohammad-fayoumi
- 🇯🇴Jordan mohammad-fayoumi Amman
Creating a static patch file for the latest stable version 3.0.0-beta1
- Status changed to Needs work
27 days ago 6:31pm 27 January 2025 - 🇨🇦Canada joseph.olstad
See also: 🐛 Use of dynamic properties is deprecated in PHP 8.2+ Active
- First commit to issue fork.
- Merge request !279Issue #3482015: In PHP 8.2+, the creation of dynamic properties is deprecated → (Open) created by herved
- 🇧🇪Belgium herved
Sorry for the noise, I didn't know if I could rebase the existing MR, so I created a new one.
Tests are passing, the branch only needed to be rebased.
All credits to mohammad-fayoumi - 🇨🇦Canada joseph.olstad
ok thanks for that.
There's still many others listed if you're ambitious. Most of the ones excluding dependency injection should be done sooner than later.
https://git.drupalcode.org/issue/facets-3482015/-/jobs/4157965
- 🇨🇦Canada joseph.olstad
With that said, baby steps, suggest making a new issue for the others.
- 🇧🇪Belgium herved
Sure, I can create an issue for phpstan.
Why excluding DI ? as it breaks backwards compatibility ? - 🇨🇦Canada joseph.olstad
@herved , feel free to work on the DI also, it's just that a large amount of code changes will require rerolled patches everywhere, this is fine, I think these phpstan errors are a good thing to fix.
- 🇧🇪Belgium herved
@joseph.olstad see 📌 Fix PHPStan issues reported on CI/gitlab Active and 📌 Fix PHPCS issues reported on CI/gitlab Active , thanks