Vauxoo / odoo

Fork of Odoo (formerly OpenERP). [This project is not publically mantained just born for internal usage with some little patches] go to official repository on github.com/odoo/odoo
https://www.odoo.com
Other
9 stars 9 forks source link

[WIP][FIX] product: Price on pricelist with pricelists inside of it #372

Closed CarmenMiranda closed 2 years ago

CarmenMiranda commented 3 years ago

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.

luisg123v commented 3 years ago

@CarmenMiranda maybe you should remove commits intended to be patches.

CarmenMiranda commented 3 years ago

@CarmenMiranda maybe you should remove commits intended to be patches.

@luisg123v This is intended to be a patch also, the other ones are also patches? I guess I should remove them

luisg123v commented 3 years ago

Yes, because patches will fail to be applied otherwise, as they will be already applied.

nhomar commented 3 years ago

@CarmenMiranda @JulioSerna

Why this change adding fields is not part of an external module?

cc @moylop260

hbto commented 3 years ago

ping @hbto

hbto commented 3 years ago

@CarmenMiranda PR https://github.com/Vauxoo/odoo/pull/397 will be used from now on in stable,

This will be kept to allow you to keep developing the features being tackled here.

Regards.