Order state does not change after a customer return is received #3198
Comments
3 tasks
spaghetticode
added a commit
to nebulab/solidus
that referenced
this issue
May 10, 2019
…! callback Processing order returns in `ReturnItem#process_inventory_unit!` does not work reliably when there are multiple return items processed at the same time (see issue solidusio#3198). Also, decoupling sensible interactions with other models in ActiveRecord callbacks simplifies the understandability of code and its flow. Order processing in `#process_inventory_unit!` is now only deprecated, but will be completely removed in the following release of Solidus.
spaghetticode
added a commit
to nebulab/solidus
that referenced
this issue
May 15, 2019
…! callback Processing order returns in `ReturnItem#process_inventory_unit!` does not work reliably when there are multiple return items processed at the same time (see issue solidusio#3198). Also, decoupling sensible interactions with other models in ActiveRecord callbacks simplifies the understandability of code and its flow. Order processing in `#process_inventory_unit!` is now only deprecated, but will be completely removed in the following release of Solidus.
|
@spaghetticode Can this issue be closed after #3199? |
|
@jacobherrington sure! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When an order has more than one line item, creating a return for all the items at once and setting the reception status to
Receivedfails to change the order state toReturned, like it does with orders with one single line item. See attached screenshot.Solidus Version:
master
Screenshots
The text was updated successfully, but these errors were encountered: