Skipping quants in _create_account_move_line
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenERP new WMS |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
stock_account.py: "return" breaks the for loop and thus only one quant is reported in the accounting... solution: remove the word "return". Or even better would be to sum them up (total+=qty*cost) and then create accounting entry, this way we would avoid multiple valuation entries.
original function:
def _create_
#group quants by cost
for quant in quants:
if quant_cost_
else:
move_obj = self.pool.
for cost, qty in quant_cost_
return move_obj.create(cr, uid, {'journal_id': journal_id,
Related branches
affects: | openobject-addons → openerp-trunk-wms |
description: | updated |
description: | updated |
description: | updated |
description: | updated |
summary: |
- Skipping quants in _account_move_line + Skipping quants in _create_account_move_line |
The goal is to have a separate entry by different price. Maybe we should just remove the 'return' word. (Or maybe in one account move) Thanks for pointing out!