Merge lp:~openerp-dev/openobject-addons/6.0-opw-582438-ksa into lp:openobject-addons/6.0

Proposed by Kirti Savalia(OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openobject-addons/6.0-opw-582438-ksa
Merge into: lp:openobject-addons/6.0
Diff against target: 12 lines (+0/-2)
1 file modified
stock/stock.py (+0/-2)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/6.0-opw-582438-ksa
Reviewer Review Type Date Requested Status
Naresh(OpenERP) Pending
Review via email: mp+138083@code.launchpad.net

Description of the change

Hello,

Steps to reproduce the bug:

-> Make a sales order with at least one product that is configured to have an "average price" configured.
-> Confirm the sales order and make the outgoing delivery (invoicing based on delivery).
-> From the outgoing delivery, create the invoice. - Return the outgoing delivery with the corresponding button.
-> From the return-delivery, also create an invoice, using the button "Create invoice" on the return delivery.
-> Compare both invoices. As you will see, the goods on the refund invoice are invoiced at a different value compared to the original invoice.

Thanks
KSA

To post a comment you must log in.

Unmerged revisions

5386. By Kirti Savalia(OpenERP) <email address hidden>

[FIX]:the goods on the refund invoice are invoiced at a different value compared to the original invoice

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'stock/stock.py'
2--- stock/stock.py 2012-08-17 07:46:56 +0000
3+++ stock/stock.py 2012-12-05 10:21:27 +0000
4@@ -1078,8 +1078,6 @@
5 if not uos_id and inv_type in ('out_invoice', 'out_refund'):
6 uos_id = move_line.product_uom.id
7 account_id = self.pool.get('account.fiscal.position').map_account(cr, uid, partner.property_account_position, account_id)
8- if move_line.price_unit != 0 and price_unit != move_line.price_unit:
9- price_unit = move_line.price_unit
10 invoice_line_id = invoice_line_obj.create(cr, uid, {
11 'name': name,
12 'origin': origin,