-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
[Issue] Avoid exception when config.xml nodes exist for not-installed payment methods #29555
Comments
Hi @engcom-Delta. Thank you for working on this issue.
|
✅ Confirmed by @sdzhepa Issue Available: @sdzhepa, You will be automatically unassigned. Contributors/Maintainers can claim this issue to continue. To reclaim and continue work, reassign the ticket to yourself. |
Hi @m2-assistant[bot]. Thank you for your report.
The fix will be available with the upcoming 2.4.2 release. |
This issue is automatically created based on existing pull request: #27940: Avoid exception when config.xml nodes exist for not-installed payment methods
Preconditions (*)
Magento 2.4-develop
When a payment configuration node exists in XML but there is no
<model>
defined,getList()
will throw anUnexpectedValueException
. One use case for creating such a set-up is a module that provides default configuration overrides for payment methods which are not installed on the current website. For example:In this case, the payment method
not_installed_here
does not have a<model>
node defined, as the payment method module is not installed on the website.Steps to reproduce (*)
config.xml
Expected result (*)
Actual result (*)
Additional information
I have not yet written/changed any unit/functional tests for this change. I suspect some existing tests may fail which I will review at a later date.
The text was updated successfully, but these errors were encountered: