c2c-oerpscenario team mailing list archive

["Olivier Dony \(OpenERP\)" <690583@xxxxxxxxxxxxxxxxxx>]

Tristan you're right, this should really be written as
move.write({'state':'done'}), nice catch! :-)

** Changed in: openobject-addons
   Importance: Undecided => Low

** Changed in: openobject-addons
       Status: Invalid => Confirmed

** Changed in: openobject-addons
    Milestone: None => 6.0-rc2

-- 
You received this bug notification because you are a member of C2C
OERPScenario, which is subscribed to the OpenERP Project Group.
https://bugs.launchpad.net/bugs/690583

Title:
  stock.picking.test_finished: write() takes at most 6 arguments (8 given)

Status in OpenObject Addons Modules:
  Confirmed

Bug description:
  While looking at stock/stock.py: stock_picking.test_finished I noticed the move.write(cr, uid, [move.id], {'state': 'done'}) would cause a "TypeError: write() takes at most 6 arguments (8 given)" exception if executed.  As stock_picking.test_finished is usually only executed after stock_picking.action_move by the workflow, which sets the moves to done, this line of code is not usually executed.

move.write({'state': 'done'}) would appear to be the correct line to use.  This is looking at current trunk, revision 4011.