-
Notifications
You must be signed in to change notification settings - Fork 9.4k
[Issue] Cannot update Order status in 2.4.7 #38659
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 @Nuranto. Thank you for your report.
Join Magento Community Engineering Slack and ask your questions in #github channel. |
Hi @engcom-Bravo. Thank you for working on this issue.
|
Hi @engcom-Dash. Thank you for working on this issue.
|
@magento give me 2.4-develop instance |
Hi @engcom-Dash. Thank you for your request. I'm working on Magento instance for you. |
Hi @engcom-Dash, here is your Magento Instance: https://fe74e6b08484c0858eb929e57e74a885.instances-prod.magento-community.engineering |
@magento give me 2.4-develop instance |
Hi @engcom-Dash. Thank you for your request. I'm working on Magento instance for you. |
Hi @engcom-Dash, here is your Magento Instance: https://fe74e6b08484c0858eb929e57e74a885.instances-prod.magento-community.engineering |
Hi @Nuranto Thanks for reporting and collaboration. Can you please provide detailed steps to reproduce with screenshots which would help us reproduce the issue accurately Thanks! |
Enregistrement.de.l.ecran.2024-04-29.a.10.31.55.mov |
Hi @Nuranto Verified the issue in 2.4.7 and 2.4 dev instances. But I am seeing only processing and suspend fraud options in order status after generating invoice. Please refer to the screenshots. Let us know if we are missing anything. ![]() |
I added a custom status on your temporary env. As you can see in the code :
We can update status, but only if current status is processing or fraud. |
Hi @Nuranto Thanks for reporting and collaboration. Verified the issue on magento 2.4 dev and 2.4.7 instances and the issue is reproducible. Hence, we are confirming the issue. We are able to update the status only if the current status is in a processing or fraud state. If the status is anything other than processing or fraud state we are not able to update the status. Please refer the attached screen recording. Screen.Recording.2024-04-29.at.7.54.07.PM.mov |
We are also waiting for a patch for this issue. |
I haven't seen it yet, but does 2.4.7-p2 solve this? |
At least the patch was still applied by Compose. As you can see here, |
in a quick solution for an integration here, i just removed this code from that file. vendor/magento/module-sales/Model/Service/OrderService.php /** |
Internal team is working on this issue |
Any update on issue? We are using Magento 2.4.7. |
The internal team already fixed this issue. Currently, in delivery queue. |
So what about tagging this ticket with the milestone/release it is intended for? |
we can determine relese when the code changes will be delivered, since they are in delivery process and not delivered yet we can't specify release which will contain the changes |
Possibly fixed by 18dee3d? |
@hostep Yes |
Hi, As per this comment #38917 (comment) we are closing this issue. Thanks. |
No milestone = release set? |
Source: #38917 (comment) |
same issue here on 2.4.7-p2 |
We are still with the same issue in the 2.4.7-p2 version, via the admin page we can only change from the default processing status (processing state) once to a custom state. After this we can't change to another status in the same state, not even the default one. Also, in the complete state we can't change the status at all. |
Hi @guilhermemap , Thanks for reporting and collaboration. Related commits: https://github.com/search?q=repo%3Amagento%2Fmagento2+ACP2E-3178&type=commits Thanks. |
We can witness that the issue is still there in v2.4.7-p3 |
Hi @vbidou , Thanks for reporting and collaboration. completestatus.mp4Thanks. |
Preconditions and environment
Steps to reproduce
Expected result
Status updated
Actual result
Status not updated
Additional information
I don't understand this change introduced in 2.4.7 :
#36562
Why would we be able to change status only if current status is processing or fraud ? (Also,
Order::STATE_PROCESSING
instead ofOrder::STATUS_PROCESSING
)If I understand correctly the issue #36562, the fix should be something like this instead :
Or am I missing something ?
Release note
No response
Triage and priority
The text was updated successfully, but these errors were encountered: