-
Notifications
You must be signed in to change notification settings - Fork 3
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
FullfillCommand leaves order status as processing_wallee #29
Comments
As expected: Resetting the order state "processing" as default for the order status "processing" instead of "processing_wallee" fixed the issue and the order status is correctly set to "Processing" instead of "Hold Delivery" when wallee calls the webhook to set the order as fullfillable. |
Dear jhruehl-DEN Thanks for your input. We will double-check our product behavior and will give you feedback. Best regards |
Dear jhruehl-DEN The "processing_wallee" is equals the "HOLD DELIVERY" state in Magento. If you want to change the "Processing" Magento state label on the M2 order, you simply can rename the label: Best regards |
@theodorspringwallee It's not at all about the order status processing_wallee, but about your extension making it the default for the order state "processing". That way the order stays on "Hold Delivery" even when the webhook is called to set the order to "The order can be fulfilled now." or send that the payment has been fullfilled. Because of this line: Given you set true in here: $order->addStatusToHistory(true, __('The order can be fulfilled now.')); This leads to the default status to be used for the status "processing", which by your update script is set to be "processing_wallee". In pictures: With my fix to set the order status "processing" back as default for the order state "processing" Click through your code in a Magento2 vanilla project yourself under the premise of your default-value changes. The question is, if it was even your intention to adjust the default-value, because the use in the given upgrade script is wrong, because you serve the string "processing" as the supposedly boolean value for "isDefault" (second argument in assignState function call) here.
|
status? |
@theodorspringwallee, @jhruehl-DEN |
In wallee/magento-2/Model/Webhook/Listener/Transaction/FulfillCommand.php the order status is set to "processing", if the order can be fullfilled, but given that the processing_wallee state is set as default value for the processing statuses (in wallee/magento-2/Setup/Patch/Data/UpdateStatusStateData.php), it leaves the orders in the "Hold delivery" status, because updating the oder status via "$order->addStatusToHistory(true, __('The order can be fulfilled now.'));" sets the default order status value for the last set order state, which your update script ensures to be processing_wallee now.
The basic mistake the developers have made is mingling with the default magento order processes by setting another default state for the processing status. This should be addressed and I guess it already caused a lot of customers quite some headache, because the Magento 2 order flow relies on the order statuses to have certain values like "processing" to continue and not on "processing_wallee".
As the processing_wallee status label "Hold Delivery" suggests, it should not be shown when "The order can be fulfilled now.".
I'm going to check myself now, if I can simply correct that mistake by resetting the order state "processing" as default for the order status "processing".
The text was updated successfully, but these errors were encountered: