search

OCA / margin-analysis

Odoo Financial Controlling (margin, costs, ...)
https://odoo-community.org/project/9
GNU Affero General Public License v3.0
71 stars 161 forks source link

[IMP] sale_report_margin report field purchase_price #200

Closed sistecem closed 3 weeks ago

sistecem commented 6 months ago

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 commented 6 months ago

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

sergio-teruel commented 6 months ago

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.

Selección_2586

sistecem commented 6 months ago

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

image image

sistecem commented 6 months ago

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

sistecem commented 6 months ago

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] commented 2 months ago

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.