double allocation field 'location_id', file mrp.py

Bug #606319 reported by joan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Status tracked in Trunk
5.0
Invalid
Undecided
Unassigned
Trunk
Fix Released
Low
Unassigned

Bug Description

version 5.0.10
file: mrp.py
class : StockMove
funtion: _action_explode(self, cr, uid, move, context={})

code:

1227 class StockMove(osv.osv):
1228 _inherit = 'stock.move'
            .
            .
1237 def _action_explode(self, cr, uid, move, context={}):
            .
            .
1284 self.pool.get('stock.move').write(cr, uid, [move.id], {
1285 'location_id': move.location_dest_id.id,
1286 'auto_validate': True,
1287 'picking_id': False,
1288 'location_id': dest,
1289 'state': 'waiting'
1290 })
            .
            .
1294 return true
1295 StockMove()

line 1285 and 1288, double allocation to field 'location_id'.

Related branches

joan (joansipa)
summary: - double alocation field 'location_id', file mrp.py
+ double allocation field 'location_id', file mrp.py
joan (joansipa)
description: updated
Revision history for this message
Ravindra Mekhiya(OpenERP) (rme-openerp) wrote :

Hello Joan,

This issue has already been resolved by revision 2783.

Kindly update your code and re-check.

Thanks.

Changed in openobject-addons:
status: New → Invalid
Changed in openobject-addons:
milestone: none → 5.0.13
Revision history for this message
joan (joansipa) wrote :

I run this command bzr, at 19-jul-2010

bzr clone lp:~openerp/openobject-addons/trunk addons

get revision 4196.

no more class StockMove into mrp.py, but new file stock.py within class StockMove and method _action_explode, same double allocation:

revision 4196
module:mrp
file: stock.py
class : StockMove
method: _action_explode(self, cr, uid, move, context={})

code:

099 move_obj.write(cr, uid, [move.id], {
100 'location_id': move.location_dest_id.id,
101 'auto_validate': True,
102 'picking_id': False,
103 'location_id': dest,
104 'state': 'waiting'
105 })

line 100 and 103, same problem.
excuse my poor english.

Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote :

It has been fixed by revision 4200 <email address hidden>.
Thanks Joan.

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.