-
-
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
[NFR]: Support of PHP-PSR 1.1.0 #15504
Comments
@StudioMaX We rely on PECL's PSR package. As of this moment there is no extension that offers container 1.1.1. The PR has been put on draft until that happens. |
Thanks! As one of the possible improvements to the support of PSR interfaces - do you see the possibility of removing the requirement for the mandatory presence of the PSR extension in runtime?
And just throw an exception if someone try to use the Logger/Cache and any other component if the required interfaces are not found. |
Further to my previous comment. |
Anton and I saw that 1.1.1 is included in the 1.1.0 PSR package. We will work on this once the new Zephir version is released. As to the PSR requirement, yes it is necessary to load PSR before Phalcon, otherwise it will not know where the interfaces are. This is a Zend Engine requirement. |
Related #15610 |
Resolved in #15506 |
Is your feature request related to a problem? Please describe.
Docs says that we are required to use
php-psr 1.0.*
.But this is a problem if other frameworks are used at the same time like Symfony, which in new versions requires
psr/container ^1.1
. The PHP-PSR 1.0.x specifies version1.0.0
forpsr/container
. Because of this, we are in an incomprehensible situation when a project requirespsr/container ^1.1
in the composer.json, successfully installs it, and in fact,psr/container
from PHP-PSR extension is used, since it has priority over the composer's library.Describe the solution you'd like
Allow to install PHP-PSR extension 1.1.0
Additional context
According to the PHP-PSR changelog, the difference between version 1.0.1 and version 1.1.0 is only in fixing support for
psr/container
to version1.1.1
.Also I have installed the PSR extension 1.1.0 locally and don't see any problems with my project. Maybe this only needs to be fixed in the documentation?
The text was updated successfully, but these errors were encountered: