-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Creation of dynamic property Tio\Laravel\GettextTranslationSaver::$config is deprecated #34
Comments
To fix the warning, need to change the declaration of the $config property from a dynamic property to a class property.
|
Thanks @Outsidaz ! We'll try to fix that soon, it seems quite easy to do. Just so you know, we've got some weird issues between PHP 8.2 and one dependency of our test suite (cf. php-vcr/php-vcr#373) but once it's fixed, we'll update our CI to include PHP versions > 8.1 |
Thanks! That was my first impression too. |
We fixed the deprecation warning in the 1.23 release. |
Hi there,
I want you to notice, that on PHP 8.2 I've got a warning:
Creation of dynamic property Tio\Laravel\GettextTranslationSaver::$config is deprecated in <...>/vendor/tio/laravel/src/GettextTranslationSaver.php on line 26
PHP: 8.2.4
Laravel: 9.52.5
The text was updated successfully, but these errors were encountered: