[6.1][7.0][trunk]Pricelist doesn't take category depth into account

Bug #1169074 reported by Yann Papouin on 2013-04-15
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Confirmed
Medium
OpenERP R&D Addons Team 2
OpenERP Community Backports (Addons)
Status tracked in 7.0
6.1
Medium
Yann Papouin
7.0
Medium
Yann Papouin

Bug Description

Attached is a proposal that fix this issue

eg: If you create two rules:

- Apply a discount of 25% on products in category A with sequence 5
- Apply a discount of 50% on products in category B (where B is a child of A) with sequence 5

Depending on the order of these two lines, your product could have a discount of 25% OR 50% (that's a random order from postgresql)

The sequence number (the only way to make priority) is not usable here as you have to manually link the sequence number with the category depth !!! That's impossible to do when you have 6 and more levels of category.

In my patch, the category depth is first computed then used in the result order

Related branches

Yann Papouin (yann-papouin) wrote :
summary: - Pricelist doesn't take category depth into account
+ [6.1][7.0][trunk]Pricelist doesn't take category depth into account

Hello Yaan,

I have checked your Issue with latest revision. But It's seems fine. That's why I have attached screen cast for your reference. so would you please check It and notify us where you have faced the problem.

Thanks and waiting for your reply!

Changed in openobject-addons:
status: New → Incomplete
Yann Papouin (yann-papouin) wrote :

Please see my demo:
I've done exactly like you, except that in a second part, I'm switching pricelist item content:

Before:
RULE 1 : Sequence 5 - Discount 25% on CatA
RULE 2 : Sequence 5 - Discount 50% on CatB

After (by editing content):
RULE 1 : Sequence 5 - Discount 50% on CatB
RULE 2 : Sequence 5 - Discount 25% on CatA

As you can see the result is not the same at all !!!

Changed in openobject-addons:
status: Incomplete → New
Amit Parik (amit-parik) wrote :

Hello Yann.

Thanks for the video and detailed explanation!

Faced the same problem at my end.

Thank you!

Changed in openobject-addons:
assignee: nobody → OpenERP R&D Addons Team 2 (openerp-dev-addons2)
importance: Undecided → Medium
status: New → Confirmed
Amit Parik (amit-parik) on 2013-08-02
tags: added: product
Changed in ocb-addons:
assignee: nobody → Yann Papouin (yann-papouin)
no longer affects: openobject-addons/6.1
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers