Account Fiscal Position declared 2 times leads to duplicate key value in sale order lines

Bug #1028747 reported by Guewen Baconnier @ Camptocamp
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Released
Low
OpenERP R&D Addons Team 3

Bug Description

Hello,

Steps to reproduce (on runbot trunk and 6.1):
 - Install modules 'account', 'sale'
 - Install the generic chart of account
 - Create a new fiscal position A
 - Create a line with :
     source: Tax 15.00%
     destination: Purchase Tax 15.00%
 - Create a second line with exact same taxes
     source: Tax 15.00%
     destination: Purchase Tax 15.00%

 - Edit the product Ice cream
 - Set the Customer tax to: Tax 15.00%

 - Create a sale order
 - Change the fiscal position to fiscal position A
 - Select the product Ice Cream (you'll notice that you have to times the same taxes)
 - Save the sale order

Result:
  Integrity Error
  duplicate key value violates unique constraint "sale_order_tax_order_line_id_key"

Related branches

Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

I just pushed 2 branches with corrections for 6.1 and trunk and proposed them for a merge.

Revision history for this message
Ravish(OpenERP) (rmu-openerp) wrote :

Hello,

I have checked patch . It's working fine.

Thanks for post and patch..!!

Changed in openobject-addons:
assignee: nobody → OpenERP R&D Addons Team 3 (openerp-dev-addons3)
assignee: OpenERP R&D Addons Team 3 (openerp-dev-addons3) → nobody
importance: Undecided → Low
status: New → Fix Committed
Revision history for this message
Ravish(OpenERP) (rmu-openerp) wrote :

Hello,

 It has been fixed on lp:~c2c/openobject-addons/trunk-fiscal-position-1028747 branch with Following Revision Number and Revision ID.

Revision Number: 7098
Revision ID: <email address hidden>

Our core team will review it .This will soon merge with trunk

 Thank you!

Changed in openobject-addons:
assignee: nobody → OpenERP R&D Addons Team 3 (openerp-dev-addons3)
Revision history for this message
Fabien (Open ERP) (fp-tinyerp) wrote :

fixed in trunk, thanks for the good contribution.

Changed in openobject-addons:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.