From: Xavier ALT <xal@openerp.com>
Date: Tue, 6 Mar 2012 15:45:36 +0000 (+0100)
Subject: [FIX] stock: picking invoice: do not invoice scrapped products (simply code)
X-Git-Tag: 6.0.4-server~6^2
X-Git-Url: http://git.inspyration.org/?a=commitdiff_plain;h=9846d9e83cd8821ba49603a529abf76e688019e7;p=odoo%2Fodoo.git

[FIX] stock: picking invoice: do not invoice scrapped products (simply code)

bzr revid: xal@openerp.com-20120306154536-vs5kaxsuyh7l6lqo
---

diff --git a/addons/stock/stock.py b/addons/stock/stock.py
index f1e0940..a9751bc 100644
--- a/addons/stock/stock.py
+++ b/addons/stock/stock.py
@@ -1047,6 +1047,9 @@ class stock_picking(osv.osv):
             for move_line in picking.move_lines:
                 if move_line.state == 'cancel':
                     continue
+                if move_line.scrapped:
+                    # do no invoice scrapped products
+                    continue
                 origin = move_line.picking_id.name or ''
                 if move_line.picking_id.origin:
                     origin += ':' + move_line.picking_id.origin
@@ -1081,22 +1084,20 @@ class stock_picking(osv.osv):
                 account_id = self.pool.get('account.fiscal.position').map_account(cr, uid, partner.property_account_position, account_id)
                 if move_line.price_unit != 0 and price_unit != move_line.price_unit:
                     price_unit = move_line.price_unit
-                
-                if not move_line.location_dest_id.scrap_location:
-                    invoice_line_id = invoice_line_obj.create(cr, uid, {
-                        'name': name,
-                        'origin': origin,
-                        'invoice_id': invoice_id,
-                        'uos_id': uos_id,
-                        'product_id': move_line.product_id.id,
-                        'account_id': account_id,
-                        'price_unit': price_unit,
-                        'discount': discount,
-                        'quantity': move_line.product_uos_qty or move_line.product_qty,
-                        'invoice_line_tax_id': [(6, 0, tax_ids)],
-                        'account_analytic_id': account_analytic_id,
-                    }, context=context)
-                    self._invoice_line_hook(cr, uid, move_line, invoice_line_id)
+                invoice_line_id = invoice_line_obj.create(cr, uid, {
+                    'name': name,
+                    'origin': origin,
+                    'invoice_id': invoice_id,
+                    'uos_id': uos_id,
+                    'product_id': move_line.product_id.id,
+                    'account_id': account_id,
+                    'price_unit': price_unit,
+                    'discount': discount,
+                    'quantity': move_line.product_uos_qty or move_line.product_qty,
+                    'invoice_line_tax_id': [(6, 0, tax_ids)],
+                    'account_analytic_id': account_analytic_id,
+                }, context=context)
+                self._invoice_line_hook(cr, uid, move_line, invoice_line_id)
 
             invoice_obj.button_compute(cr, uid, [invoice_id], context=context,
                     set_total=(inv_type in ('in_invoice', 'in_refund')))