-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Can't Cancel Order #19230
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
Comments
Hi @friendscottn. Thank you for your report.
Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:
where @friendscottn do you confirm that you was able to reproduce the issue on vanilla Magento instance following steps to reproduce?
|
Hi @maheshWebkul721. Thank you for working on this issue.
|
@magento-engcom-team give me 2.2-develop instance |
Hi @maheshWebkul721. Thank you for your request. I'm working on Magento 2.2-develop instance for you |
Hi @maheshWebkul721, here is your Magento instance. |
@maheshWebkul721 Thank you for verifying the issue. Based on the provided information internal tickets |
Hi there, I'm able to reproduce it, I'm facing the same issue on magento 2.2.6 |
Same for me |
Reproducible on |
Hi @KravetsAndriy. Thank you for working on this issue.
|
Does anyone know if this is an issue in 2.2.3? |
@SamB-GB Hello. It doesn't throw exception, however expected behaviour is not valid, because after making an account, it doesn't increase coupon usage for newly created customer. |
Hi @engcom-backlog-tomash. Thank you for working on this issue.
|
@magento-engcom-team give me 2.3.0 instance |
Hi @engcom-backlog-tomash. Thank you for your request. I'm working on Magento 2.3.0 instance for you |
Hi @engcom-backlog-tomash, here is your Magento instance. |
@friendscottn, thank you for reporting this isuue. The latest released version of Magento (2.3.0) is not affected by this issue. After the order is invoiced you cannot cancel the order. By if the merchant ships the order without invoicing than order gets status processing and also he is able to cancel it until invoice is not created. In such conditions order has no problems with cancelation |
@engcom-backlog-tomash I think, when user makes guest order with coupon code and then registers an account, coupon code usage should be increased and coupon usage per customer should be increased , the same way as customer made account BEFORE checkout with given coupon. |
@Winfle, yes please create new one if it's not already created. |
Hi @friendscottn. Thank you for your report.
The fix will be available with the upcoming 2.2.8 release. |
@sidolov the same issue in Magento 2.2.9 |
Hi @friendscottn. Thank you for your report.
The fix will be available with the upcoming 2.3.4 release. |
The issue still happens with Guest Users. The issue originates from \Magento\SalesRule\Model\Coupon\UpdateCouponUsages::updateCustomerRuleUsages and this line $ruleCustomer->save(); was causing the issue. I am on enterprise version. |
@magento-engcom-team Can you please release a patch for this on 2.2.x for EE customers. Orders can't be cancelled have big effect on stock values within the system |
i am still having this issue in magento 2.3.3 CE: [2020-01-03 10:42:12] main.CRITICAL: SQLSTATE[23000]: Integrity constraint violation: 1452 Cannot add or update a child row: a foreign key constraint fails ( |
This issue still exists on Magento 2.4.2 |
can still confirm this issue in 2.4.3-p1 |
This issue still exist on Magento 2.4.3-p1
|
@simonmaass @igi8819 on 2.4.3-p1 you might have the same issue I did - trying to cancel an order via the REST API immediately after it's placed caused this error. I added a short pause (1 second) between the order place event and the API call to cancel it, and the problem went away. I think it's because the call to place an order is done from javascript, so perhaps the call to cancel the order can happen before the place order code has completed and the database is fully up to date. |
Preconditions (*)
Steps to reproduce (*)
Expected result (*)
Actual result (*)
Related Issues
#16779
The text was updated successfully, but these errors were encountered: