WooCommerce
WooCommerce copied to clipboard
Wrap psr/log in namespace
This issue is similar as #819.
roots/acorn
uses psr/log:^3.0
because of illuminate/log
.
This raises errors because this package uses ^1.1.4
.
Can psr/log
be prefixed using Mozart?
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.
@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