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] product: Price on pricelist with pricelists inside of it #397

Closed

Conversation

hbto
Copy link

@hbto hbto commented Mar 4, 2021

Based on

#372

Explain

When you have a global pricelist and inside of it, you have an item 'applied on' 'Global' and a pricelist 'A' with product A,
and another item also with 'applied on' 'Global' with a pricelist 'B' with product B.
And you make a sale order with the global pricelist and you add both products, the only product that is going to have the price set with the pricelist B or A it depends which item is first.

With this change, it will check in all items of the global pricelist until it finds the price or it will set the price on the product.

Notice

As this PR is being used in Stable branches not development pushes that have not been tested will done to this one.

When a you have a global pricelist and inside of it, you have an
item 'applied on' 'Global' and a pricelist 'A' with product A,
and another item also with 'applied on' 'Global' with  a pricelist 'B'
with the product B.
And you make a sale order with the global pricelist and you add both
products, the only product that is going to have the price set with the
pricelist B or A it depends which item is first.
With this change it will check in all items of the global pricelist
until it find the price or it will set the price on the product.

Also, adding the field `active` on the model `product.pricelist.item`.
After adding the method `_check_pricelist_rule` we need add the context
and return.
@luisg123v luisg123v closed this Jul 14, 2022
@luisg123v luisg123v deleted the 12.0-Pricelist-Global-Fix-Carmen-hbto branch July 14, 2022 00:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants