Cannot pick products with real time stock valuation enabled

[D Verhelst]

I have set up a new nightly 6.2dev-20121106-000101 , and installed bare minimum modules :
- accounting , belgium
- purchase
- warehouse

I define a new product, category: all products , valuation: real-time, fill out required fields :
stock input account (new account)
stock output account (new account)
stock valuation account (new account)

when I create a PO and try to receive goods I get a key error on period id output:
File "/opt/openerp/server/openerp/addons/stock/wizard/stock_partial_picking.py", line 213, in do_partial
    stock_picking.do_partial(cr, uid, [partial.picking_id.id], partial_data, context=context)
  File "/opt/openerp/server/openerp/addons/stock/stock.py", line 1361, in do_partial
    self.action_move(cr, uid, [pick.id])
  File "/opt/openerp/server/openerp/addons/stock/stock.py", line 906, in action_move
    context=context)
  File "/opt/openerp/server/openerp/addons/stock/stock.py", line 2445, in action_done
    self._create_product_valuation_moves(cr, uid, move, context=context)
  File "/opt/openerp/server/openerp/addons/stock/stock.py", line 2407, in _create_product_valuation_moves
    'ref': move.picking_id and move.picking_id.name})
  File "/opt/openerp/server/openerp/addons/account/account.py", line 1434, in create
    c['period_id'] = vals['period_id']
KeyError: 'period_id'

[Amit Parik (amit-parik)]

Hello,

You can try to fix by following code.

 c['period_id'] = vals.get('period_id')

This will help.

Thanks!

[D Verhelst (dimitriverhelst)]

Hi Amit,

tried your update, and also updated other two fields:

        if 'line_id' in vals:
            c = context.copy()
            c['novalidate'] = True
            c['period_id'] = vals.get('period_id')
            c['journal_id'] = vals.get('journal_id')
            c['date'] = vals.get('date')
            result = super(account_move, self).create(cr, uid, vals, c)
            self.validate(cr, uid, [result], context)
        else:
            result = super(account_move, self).create(cr, uid, vals, context)
        return result

now I get a much cleaner warning:

Error!
No period found or more than one period found for the given date.

however, I do have exactly one period open for the given date...

[Amit Parik (amit-parik)]

Hello,

See the better producible steps on lp:1079070

[Michael Telahun Makonnen (mmakonnen)]

I have a fix for this in my branch, which I have submitted a merge proposal for: https://code.launchpad.net/~mmakonnen/openobject-addons/openobject-addons/+merge/135702

Patch is also attached.

[Michael Telahun Makonnen (mmakonnen)]

I think this can be closed now. The fix was merged in rev. 8181

[Serpent Consulting Services (serpent-consulting-services)]

http://bazaar.launchpad.net/~openerp/openobject-addons/trunk/revision/8181 does it all.