[6.1][7.0][trunk] price_surcharge must be affiliated with product uom
Bug #1188629 reported by
Yann Papouin
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Odoo Addons (MOVED TO GITHUB) |
Confirmed
|
Low
|
OpenERP R&D Addons Team 2 | ||
| OpenERP Community Backports (Addons) |
Fix Released
|
Low
|
Yann Papouin | ||
Bug Description
The price_surcharge is not linked to product uom. It's a mistake or a design choice ?
eg: ProductA with an UOM = PCE with a price of 10.0
If in the pricelist, you defines a surcharge of 2.0
When the price is computed you will have:
ProductA with Quantity 1.0 x PCE = 10.0 + 2.0 = 12.0
ProductA with Quantity 1.0 x 10PCE = 100.0 + 2.0 = 102.0
ProductA with Quantity 1.0 x 50PCE = 500.0 + 2.0 = 502.0
That's not correct, you should have:
ProductA with Quantity 1.0 x PCE = 10.0 + 2.0 = 12.0
ProductA with Quantity 1.0 x 10PCE = 100.0 + 20.0 = 120.0
ProductA with Quantity 1.0 x 50PCE = 500.0 + 100.0 = 600.0
Related branches
lp:~yann-papouin/openobject-addons/6.1-bug-1188629-price-surchage-uom
- OpenERP Core Team: Pending requested
-
Diff: 22 lines (+8/-1)1 file modifiedproduct/pricelist.py (+8/-1)
lp:~yann-papouin/ocb-addons/6.1-bug-1188629-price-surchage-uom
- Omar (Pexego): Approve (code review)
- Guewen Baconnier @ Camptocamp: Approve
- Holger Brunn (Therp): Approve (code review)
-
Diff: 22 lines (+8/-1)1 file modifiedproduct/pricelist.py (+8/-1)
lp:~yann-papouin/openobject-addons/7.0-bug-1188629-price-surchage-uom
- OpenERP Core Team: Pending requested
-
Diff: 21 lines (+6/-1)1 file modifiedproduct/pricelist.py (+6/-1)
lp:~yann-papouin/ocb-addons/7.0-bug-1188629-price-surchage-uom
- Sandy Carter (http://www.savoirfairelinux.com): Approve (code review, pep8)
- Yannick Vaucher @ Camptocamp: Approve (code review, no test)
- Holger Brunn (Therp): Approve (code review)
-
Diff: 21 lines (+6/-1)1 file modifiedproduct/pricelist.py (+6/-1)
| description: | updated |
| Changed in openobject-addons: | |
| assignee: | nobody → OpenERP R&D Addons Team 2 (openerp-dev-addons2) |
| importance: | Undecided → Low |
| status: | New → Confirmed |
| description: | updated |
| Changed in ocb-addons: | |
| assignee: | nobody → Yann Papouin (yann-papouin) |
| importance: | Undecided → Low |
| status: | New → Fix Committed |
| Changed in ocb-addons: | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c) wrote | #1 |
To post a comment you must log in.
https:/