[IMP] stock: Add _prepare_picking_assign method in _picking_assign for easier inheritance
This commit is contained in:
parent
9c30ba7143
commit
404b668370
|
@ -2094,6 +2094,19 @@ class stock_move(osv.osv):
|
||||||
result['location_dest_id'] = loc_dest_id
|
result['location_dest_id'] = loc_dest_id
|
||||||
return {'value': result}
|
return {'value': result}
|
||||||
|
|
||||||
|
def _prepare_picking_assign(self, cr, uid, move, context=None):
|
||||||
|
""" Prepares a new picking for this move as it could not be assigned to
|
||||||
|
another picking. This method is designed to be inherited.
|
||||||
|
"""
|
||||||
|
values = {
|
||||||
|
'origin': move.origin or move.move_dest_id and move.move_dest_id.picking_id.origin,
|
||||||
|
'company_id': move.company_id and move.company_id.id or False,
|
||||||
|
'move_type': move.group_id and move.group_id.move_type or 'direct',
|
||||||
|
'partner_id': move.partner_id.id or False,
|
||||||
|
'picking_type_id': move.picking_type_id and move.picking_type_id.id or False,
|
||||||
|
}
|
||||||
|
return values
|
||||||
|
|
||||||
@api.cr_uid_ids_context
|
@api.cr_uid_ids_context
|
||||||
def _picking_assign(self, cr, uid, move_ids, procurement_group, location_from, location_to, context=None):
|
def _picking_assign(self, cr, uid, move_ids, procurement_group, location_from, location_to, context=None):
|
||||||
"""Assign a picking on the given move_ids, which is a list of move supposed to share the same procurement_group, location_from and location_to
|
"""Assign a picking on the given move_ids, which is a list of move supposed to share the same procurement_group, location_from and location_to
|
||||||
|
@ -2120,13 +2133,7 @@ class stock_move(osv.osv):
|
||||||
[pick] = cr.fetchone() or [None]
|
[pick] = cr.fetchone() or [None]
|
||||||
if not pick:
|
if not pick:
|
||||||
move = self.browse(cr, uid, move_ids, context=context)[0]
|
move = self.browse(cr, uid, move_ids, context=context)[0]
|
||||||
values = {
|
values = self._prepare_picking_assign(cr, uid, move, context=context)
|
||||||
'origin': move.origin or move.move_dest_id and move.move_dest_id.picking_id.origin,
|
|
||||||
'company_id': move.company_id and move.company_id.id or False,
|
|
||||||
'move_type': move.group_id and move.group_id.move_type or 'direct',
|
|
||||||
'partner_id': move.partner_id.id or False,
|
|
||||||
'picking_type_id': move.picking_type_id and move.picking_type_id.id or False,
|
|
||||||
}
|
|
||||||
pick = pick_obj.create(cr, uid, values, context=context)
|
pick = pick_obj.create(cr, uid, values, context=context)
|
||||||
return self.write(cr, uid, move_ids, {'picking_id': pick}, context=context)
|
return self.write(cr, uid, move_ids, {'picking_id': pick}, context=context)
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue