Skip to content
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

[FIX] sale_mrp, sale_stock: No delivered_qty computed with partial delivery #32583

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
4 participants
@simongoffin
Copy link
Contributor

simongoffin commented Apr 10, 2019

Steps to reproduce the bug:

  • Create a consumable or storable final product A with invoicing policy
    in "delivered quantity"
  • Create two storable products B and C
  • Create a BOM kit for A with 1 B and 1 C
  • Create a SO with 5 qty of A and confirm it
  • Deliver 3 B and 2 C and either on "No backorder" or "With backorder"

Bug:

The delivered_qty on the SO line is of course not automatically updated
(as in this case, a right value is not computable) but the delivered_qty
was not updatable manually and then it was possible to invoice the SO.

opw:1966170

@robodoo robodoo added the seen 🙂 label Apr 10, 2019

@C3POdoo C3POdoo added the OE label Apr 10, 2019

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Apr 10, 2019

@simongoffin simongoffin force-pushed the odoo-dev:11.0-opw-1966170-fix-sig branch from d743e05 to f3213c4 Apr 15, 2019

@robodoo robodoo removed the CI 🤖 label Apr 15, 2019

[FIX] sale_mrp, sale_stock: No delivered_qty computed with partial de…
…livery

Steps to reproduce the bug:
- Create a consumable or storable final product A with invoicing policy
in "delivered quantity"
- Create two storable products B and C
- Create a BOM kit for A with 1 B and 1 C
- Create a SO with 5 qty of A and confirm it
- Deliver 3 B and 2 C and either on "No backorder" or "With backorder"

Bug:

The delivered_qty on the SO line is of course not automatically updated
(as in this case, a right value is not computable) but the delivered_qty
was not updatable  manually and then it was possible to invoice the SO.

Co-authored-by: nim-odoo <nim@odoo.com>

DO NOT FORWARD-PORT

opw:1966170

@robodoo robodoo added the CI 🤖 label Apr 15, 2019

@simongoffin simongoffin force-pushed the odoo-dev:11.0-opw-1966170-fix-sig branch from f3213c4 to 7814103 Apr 16, 2019

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Apr 16, 2019

@simongoffin

This comment has been minimized.

Copy link
Contributor Author

simongoffin commented Apr 16, 2019

robodoo r+

pniederlag pushed a commit to pniederlag/odoo that referenced this pull request Apr 16, 2019

[FIX] sale_mrp, sale_stock: No delivered_qty computed with partial de…
…livery

Steps to reproduce the bug:
- Create a consumable or storable final product A with invoicing policy
in "delivered quantity"
- Create two storable products B and C
- Create a BOM kit for A with 1 B and 1 C
- Create a SO with 5 qty of A and confirm it
- Deliver 3 B and 2 C and either on "No backorder" or "With backorder"

Bug:

The delivered_qty on the SO line is of course not automatically updated
(as in this case, a right value is not computable) but the delivered_qty
was not updatable  manually and then it was possible to invoice the SO.

Co-authored-by: nim-odoo <nim@odoo.com>

DO NOT FORWARD-PORT

opw:1966170

closes odoo#32583

Signed-off-by: Simon Goffin (sig) <sig@openerp.com>

@robodoo robodoo added merged 🎉 and removed merging 👷 labels Apr 16, 2019

@robodoo

This comment has been minimized.

Copy link
Contributor

robodoo commented Apr 16, 2019

Merged, thanks!

@robodoo robodoo closed this Apr 16, 2019

@simongoffin

This comment has been minimized.

Copy link
Contributor Author

simongoffin commented Apr 16, 2019

Hello @KangOl
Do not forward-port this commit

@KangOl

This comment has been minimized.

Copy link
Contributor

KangOl commented Apr 16, 2019

forward-port to which version?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.