[FIX] product_margin_classification : do not loose margin classification when creating new product.template.

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

[FIX] product_margin_classification : do not loose margin classification when creating new product.template. #181

Closed legalsylvain closed 1 year ago

legalsylvain commented 1 year ago

For that purpose, use same algorithm as 'weight' field (https://github.com/odoo/odoo/blob/16.0/addons/product/models/product_template.py#L88)

add _inverse_margin_classification_id function
overload _get_related_fields_variant_template

Add new test.

Fix : #180

CC : @huguesdk, @vdewulf

OCA-git-bot commented 1 year ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

OCA-git-bot commented 1 year ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-181-by-legalsylvain-bump-patch, awaiting test results.

OCA-git-bot commented 1 year ago

Congratulations, your PR was merged at 2756f17e55cc2df20f508159b5dc86eb12008335. Thanks a lot for contributing to OCA. ❤️