mirror of
https://github.com/OCA/rma.git
synced 2025-02-16 17:11:47 +02:00
[FIX] allows to create a refund on lines without product_id
This commit is contained in:
committed by
Joël Grand-Guillaume
parent
ee191e6966
commit
aaccc7da14
@@ -268,8 +268,10 @@ class claim_line(orm.Model):
|
|||||||
"""Compute and return the destination location ID to take
|
"""Compute and return the destination location ID to take
|
||||||
for a return. Always take 'Supplier' one when return type different
|
for a return. Always take 'Supplier' one when return type different
|
||||||
from company."""
|
from company."""
|
||||||
prod_obj = self.pool.get('product.product')
|
prod = False
|
||||||
prod = prod_obj.browse(cr, uid, product_id, context=context)
|
if product_id:
|
||||||
|
prod_obj = self.pool.get('product.product')
|
||||||
|
prod = prod_obj.browse(cr, uid, product_id, context=context)
|
||||||
wh_obj = self.pool.get('stock.warehouse')
|
wh_obj = self.pool.get('stock.warehouse')
|
||||||
wh = wh_obj.browse(cr, uid, warehouse_id, context=context)
|
wh = wh_obj.browse(cr, uid, warehouse_id, context=context)
|
||||||
location_dest_id = wh.lot_stock_id.id
|
location_dest_id = wh.lot_stock_id.id
|
||||||
@@ -465,14 +467,15 @@ class crm_claim(orm.Model):
|
|||||||
invoice_lines = invoice_line_obj.browse(cr, uid, invoice_line_ids,
|
invoice_lines = invoice_line_obj.browse(cr, uid, invoice_line_ids,
|
||||||
context=context)
|
context=context)
|
||||||
for invoice_line in invoice_lines:
|
for invoice_line in invoice_lines:
|
||||||
|
product_id = invoice_line.product_id and invoice_line.product_id.id or False
|
||||||
location_dest_id = claim_line_obj.get_destination_location(
|
location_dest_id = claim_line_obj.get_destination_location(
|
||||||
cr, uid, invoice_line.product_id.id,
|
cr, uid, product_id,
|
||||||
warehouse_id, context=context)
|
warehouse_id, context=context)
|
||||||
claim_lines.append({
|
claim_lines.append({
|
||||||
'name': invoice_line.name,
|
'name': invoice_line.name,
|
||||||
'claim_origine': "none",
|
'claim_origine': "none",
|
||||||
'invoice_line_id': invoice_line.id,
|
'invoice_line_id': invoice_line.id,
|
||||||
'product_id': invoice_line.product_id.id,
|
'product_id': product_id,
|
||||||
'product_returned_quantity': invoice_line.quantity,
|
'product_returned_quantity': invoice_line.quantity,
|
||||||
'unit_sale_price': invoice_line.price_unit,
|
'unit_sale_price': invoice_line.price_unit,
|
||||||
'location_dest_id': location_dest_id,
|
'location_dest_id': location_dest_id,
|
||||||
|
|||||||
Reference in New Issue
Block a user