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

Merge not working perfectly #2

Open
abhinavvaidya90 opened this Issue Aug 16, 2018 · 8 comments

Comments

Projects
None yet
4 participants
@abhinavvaidya90

abhinavvaidya90 commented Aug 16, 2018

Steps to Reproduce :-

PO 123 –

  • Product A – 100
  • Product B – 100

PO 124 –

  • Product A – 50
  • Product B – 50
  • Product C – 10

When these 2 PO’s are merged with the first option i.e., Create new po and cancel others, new PO is created as below :-

PO 125 –

  • Product A – 150
  • Product B – 160 (Quantity of Product B in both PO + Quantity of Product C)

This merge does not add the Product C in the merged PO but instead adds the product quantity of Product C to the last line of the newly created PO.

@vishmitaaktiv

This comment has been minimized.

vishmitaaktiv commented Aug 17, 2018

@abhinavvaidya90 Thanks for sharing steps towards reproducing issue, we have followed the same in Odoo v11 & v10 however we are not able to generate the issue.
Could you please tell us which odoo version are you using and is it updated version from github ?

@abhinavvaidya90

This comment has been minimized.

abhinavvaidya90 commented Aug 17, 2018

I am using version 10 enterprise and the updated version of your module.

@odooaktiv

This comment has been minimized.

Owner

odooaktiv commented Aug 17, 2018

@abhinavvaidya90 Thanks for your feedback.
We will get back to you ASAP next week.

@abhinavvaidya90

This comment has been minimized.

abhinavvaidya90 commented Aug 17, 2018

Thanks a lot!

@odooaktiv

This comment has been minimized.

Owner

odooaktiv commented Aug 21, 2018

@abhinavvaidya90 We have checked with Odoo 10 Enterprise version and found that its working as per your expected flow.
Please make sure that you are using updated version of Odoo and also version 10 of MergePurchaseOrder module from OCA.

Feel free to contact us if you have future issues.

Thank you !

@jarobase

This comment has been minimized.

jarobase commented Oct 20, 2018

This bug was also observed in Odoo 12.0. I know that This version is not already suported by the module but I tested it. It can be used and it works but it seems to be affected by this bug. The module was obtained here: https://apps.odoo.com/apps/modules/11.0/merge_purchase_order/

@jarobase

This comment has been minimized.

jarobase commented Oct 27, 2018

As I am trying to fix this bug, I observed that it seems only visible when the purchased order are produced by client quotations and orders. If you create the quotations in the "buying" module directly, it will work. I am also refactoring the module as there are a lot a duplicated lines, making it difficult to debug it.

jarobase added a commit to jarobase/MergePurchaseOrder that referenced this issue Oct 27, 2018

@jarobase jarobase referenced this issue Oct 27, 2018

Open

12.0 #4

@jarobase

This comment has been minimized.

jarobase commented Oct 27, 2018

This bug seems fixed in my 12.0 branch:
https://github.com/jarobase/MergePurchaseOrder/tree/12.0
A Pull request has been created in order to fix this issue upstream. See #4.

@jarobase jarobase referenced this issue Oct 27, 2018

Open

Version 12.0 #3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment