[IMP] sale_report_margin report field purchase_price

[sistecem]

In the report it is more useful the total cost than the unit cost. It also gave wrong data when 2 lines of the same product where used

[OCA-git-bot]

Hi @sergio-teruel, some modules you are maintaining are being modified, check this out!

[sergio-teruel]

Sorry @sistecem The purchase_price field in Odoo core is by unit so to maintain data consistency it should be kept as a unit price.

[sistecem]

Thanks @sergio-teruel , I agree with keeping the consistency , but i think it should be AVG instead of SUM

[sistecem]

I also added the field purchase_price_total , as an option in the report , resultin in something like this

[sistecem]

Sorry for the mess, I am still learning to contribute to the OCA I can make a new pull request if necessary Will do spanish translation if neccesary

[github-actions[bot]]

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.