-
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
Cancel expired orders using OrderManagementInterface #18832
Cancel expired orders using OrderManagementInterface #18832
Conversation
Hi @JeroenVanLeusden. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
61a2748
to
a1fdddc
Compare
@VladimirZaets Updated the PR. |
6ccadab
to
04af36e
Compare
Hi @JeroenVanLeusden. Thank you for your contribution. |
Description (*)
Plugins are most likely be created in the
OrderManagementInterface
to do additional operations instead of thecancel()
method of the order itself.Regarding our use case: We create proforma invoices for an order with pending(_payment) status. When the cron cancels these orders it will call the
cancel()
method on the order instead of using theOrderManagementInterface
. Result is our plugins are nog triggered.Fixed Issues (if relevant)-->
Manual testing scenarios (*)
sales/orders/delete_pending_after
to0
.OrderManagementInterface::cancel()
which cancel any open invoices.sales_clean_orders
to cancel the order.Contribution checklist (*)