Skip to content

[16.0][IMP] rma: Fix calculation for qty_to_receive #623

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

Open
wants to merge 2 commits into
base: 16.0
Choose a base branch
from

Conversation

JasminSForgeFlow
Copy link
Collaborator

Backport from 18.0

Copy link
Contributor

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For me this is ok, as it is merged in v18. In v16 there is this other proposal: #619 but I think we can merge this other because it is already proposed to v17 and merged in v18

@LoisRForgeFlow
Copy link
Contributor

Thanks Jasmin! we'll merge after closing v18 migration

@florian-dacosta
Copy link
Contributor

Thanks.
Has the same been done for qty_to_deliver ?

@JasminSForgeFlow
Copy link
Collaborator Author

Thanks. Has the same been done for qty_to_deliver ?

That's working okay, no need for it

@florian-dacosta
Copy link
Contributor

Thanks. Has the same been done for qty_to_deliver ?

That's working okay, no need for it

Are you sure ?
As far as I know, the problem is exactly the same.
If you have a rma line with product_qty = 1. the qty_to_deliver is also 1 / qty_outgoing is 0 Then you generate the delivery order, the qty_to_deliver is still 1 and the qty_outgoing is also 1 (#623 (comment)) while the qty_to_deliver should probably goes to 0, as it happens with the qty_to_receive in the case of incoming shipments, with this PR, right ?

For me the qty_to_receive and the qty_to_deliver work exactly the same, so the problem is the same, and it would be really weird if the field would some times take into account the incoming/outgoing qty and sometime not !
But I may have misunderstood some part or maybe we do not have the same issue in mind around this field...
Could you elaborate please ?

@JasminSForgeFlow
Copy link
Collaborator Author

Thanks. Has the same been done for qty_to_deliver ?

That's working okay, no need for it

Are you sure ? As far as I know, the problem is exactly the same. If you have a rma line with product_qty = 1. the qty_to_deliver is also 1 / qty_outgoing is 0 Then you generate the delivery order, the qty_to_deliver is still 1 and the qty_outgoing is also 1 (#623 (comment)) while the qty_to_deliver should probably goes to 0, as it happens with the qty_to_receive in the case of incoming shipments, with this PR, right ?

For me the qty_to_receive and the qty_to_deliver work exactly the same, so the problem is the same, and it would be really weird if the field would some times take into account the incoming/outgoing qty and sometime not ! But I may have misunderstood some part or maybe we do not have the same issue in mind around this field... Could you elaborate please ?

Here I have created one RMA order, Qty to receive is 1 and other all are 0

image

I have create shipment, haven't validated yet

image

After validating shipment

image

It show Qty to delivery 1 only after shipment validated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants