-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
T16308 di dynamic properties 4 #16376
T16308 di dynamic properties 4 #16376
Conversation
@niden does this also fix following deprecation errors?
|
Not yet. I am looking at it and it could be that it comes from Zephir than Phalcon. I will have to talk to @Jeckerson when he comes back |
I tried the patch but to no avail for the following... I'm seeing:
It looks to be from the Phalcon\Mvc\Controller class. Any open ticket that tracks these? Cheers, |
@hakimio thanks, subscribed. though there is no collection of individual dynamic variables causing this. if the problem is wide spread it might make sense to record which ones have been reported and which ones have been addressed (by this patch). the situation is not overly transparent unfortunately. |
Hi @niden ! Do you have any updates on this? |
Nothing changed to my knowledge. We had to disable deprecation warnings because of this so it’s not a quick check. 😉 |
You can follow this issue to get progress updates: zephir-lang/zephir#2405 |
Hello!
In raising this pull request, I confirm the following:
Extended
Phalcon\Di\Injectable
fromstdClass
to remove the deprecation warning (dynamic properties) for PHP 8.2Credit: @escribiendocodigo
Thanks