-
Notifications
You must be signed in to change notification settings - Fork 3
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
Pimcore 11 compatibility fixes #6
Conversation
Thx a lot - I merge it into #8 to see if PHPStan runs through |
…bugs (#8) * Require ecommerce framework bundle to fix PHPStan * Update composer.json * Pimcore 11 compatibility fixes (#6) * fix(pimcore-11): Fix type declarations * fix(pimcore-11): Fix incorrect Authorization attribute for Guzzle * fix(pimcore-11): PHP 8: Check for keys that may not be present in response * feat(pimcore-11): rename config files * Fix PHPStan autoload * Fix PHPStan autoload * Fix PHPStan autoload --------- Co-authored-by: Lukas B <theill182@gmail.com>
@GNi33 It's now merged into the 2.x branch. Unfortunately it's impossible for me to test if it really works, so please let me know if you had the chance to test the 2.x branch. If it works I can release a 2.0 version. |
@markus-moser works fine so far for us in production as well as testSystem mode. We will continue to monitor it, but the basic use case of payment via the CopyAndPay API is looking good so far. |
@GNi33 Thanks a lot for your feedback, I released v2 now based on your work 👍 |
This PR adds a few compatibility fixes for Pimcore 11.
For example, the PaymentProviderInstaller implementation of Pimcore 11 added typed properties, which this bundle does not have until now, causing exceptions when installing the bundle. Also, the way the bearer authorization is implemented does not work with the version of GuzzleHttp that Pimcore 11 uses.