We experienced the same issue when upgrading to Commerce 2.40 (including 2.38) while migrating from Drupal 9 to Drupal 10.
It seems likely that
#3418825
🐛
Exception logging is not working due to a typo in doOrderPreSave()
Fixed
might be the root cause, but I haven't fully investigated it. However, we still need to ensure that logger.channel.commerce_order exists, especially in CI/CD pipelines and production environments where developers do not have access to run drush cache:rebuild.
We have created a simple patch that ensures the logger is always registered before use in OrderStorage::createInstance(), preventing pipeline failures.
Would it be possible to integrate this fix? 🚀
truls1502 → made their first commit to this issue’s fork.
Released!
Released, and thank you for your patience.
truls1502 → made their first commit to this issue’s fork.
Adding a temporary patch to able to use PHP 7.4
We have added the proposed category.
The summary is now updated.
The logo is now fixed.
truls1502 → made their first commit to this issue’s fork.
truls1502 → created an issue.
I apologise, I did not see that you had committed it directly to the branch. I was on the phone and checked only the changes in the MR!12. So just look away from my last question 😅
Maybe I am mistaken, should we not replace the old module swiftmailer with https://www.drupal.org/project/symfony_mailer → ? What are your thoughts about it?
Changing the status.
Added @truls1502 to the ping list
truls1502 → created an issue.
tbsiqueira → credited truls1502 → .
tbsiqueira → credited truls1502 → .
tbsiqueira → credited truls1502 → .
tbsiqueira → credited truls1502 → .
tbsiqueira → credited truls1502 → .
tbsiqueira → credited truls1502 → .
tbsiqueira → credited truls1502 → .
There exists already a PR
Pull request: https://github.com/goalgorilla/open_social/pull/3432
truls1502 → created an issue.
Thanks for the meeting. Looking forward to the next one!