-
-
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
[BUG]: Container is not passed to the autoloader registration method. #14787
Comments
Bug for sure. |
Thank you, @niden. Maybe in this situation, make the container argument mandatory in the registerAutoloaders () method? |
@TimurFlush Unfortunately we cannot do that because it will break the interface and we are only allowed to do that during major versions i.e. v5. We can correct this however by just injecting the container in the place you mentioned. If it not too much trouble can you send a PR for this? If not I will try and get that sorted out |
Resolved in #14790 Thank you! |
Hello there!
I was wondering if it is a bug or not? (Please see the link)
cphalcon/phalcon/Cli/Console.zep
Line 118 in a48b319
Why is the current container not passed to the module's autoloader registration method?
I noticed that this is only relevant for the application Phalcon\Cli\Console.
Details
The text was updated successfully, but these errors were encountered: