-
Notifications
You must be signed in to change notification settings - Fork 55
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
Wrap psr/log in namespace #892
Comments
Also related #883 |
@mmaymo Is this on your roadmap? We have multiple websites waiting to be upgraded, but this issue is blocking the upgrade. |
I just tried to install the Sentry plugin and bumped into this issue as well as it's giving me inheritance errors: Fatal error: Declaration of Mollie\WooCommerce\Log\WcPsrLoggerAdapter::log($level, $message, array $context = []) must be compatible with Psr\Log\AbstractLogger::log($level, Stringable|string $message, array $context = []): void in /var/www/html/public_html/wp-content/plugins/mollie-payments-for-woocommerce/src/Log/WcPsrLoggerAdapter.php on line 75 |
Hi folks, we are aware of this dependency conflict and are working on a proper solution for the resulting error. That said, this has a significant impact on the development pipeline, so we can't commit to a timeframe just yet, but we'll let you know when there are any news. |
…ency conflicts
@InpsydeNiklas I've opened a PR aiming to fix this issue, could you have a look when time permits please? |
We have also been running into this issue for a while now, it would be huge timesaver if this PR could get merged |
Hello folks, thanks for your patience! A solution for this issue will be finally included in the upcoming update next week and can already be tested with the 7.8.1-beta version from here. |
This issue is similar as #819.
roots/acorn
usespsr/log:^3.0
because ofilluminate/log
.This raises errors because this package uses
^1.1.4
.Can
psr/log
be prefixed using Mozart?The text was updated successfully, but these errors were encountered: