From 6d51bce057ec5d94b26e045cd87e7850405a88e7 Mon Sep 17 00:00:00 2001 From: Nicolas Martinelli Date: Tue, 28 Jun 2016 13:15:31 +0200 Subject: [PATCH] [FIX] account: money in/out in PoS It is not possible to take money out or put money in for a PoS since it leads to a traceback. The decorator `@api.one` returns a list, which is something we don't want when the method returns a dictionary. opw-681417 --- addons/account/wizard/pos_box.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/addons/account/wizard/pos_box.py b/addons/account/wizard/pos_box.py index 1c45feab89e..9d9d7be7799 100644 --- a/addons/account/wizard/pos_box.py +++ b/addons/account/wizard/pos_box.py @@ -35,7 +35,7 @@ class CashBox(models.TransientModel): if record.state == 'confirm': raise UserError(_("You cannot put/take money in/out for a bank statement which is closed.")) values = self._calculate_values_for_statement_line(record) - return record.write({'line_ids': [(0, False, values[0])]}) + return record.write({'line_ids': [(0, False, values)]}) class CashBoxIn(CashBox): @@ -43,7 +43,7 @@ class CashBoxIn(CashBox): ref = fields.Char('Reference') - @api.one + @api.multi def _calculate_values_for_statement_line(self, record): if not record.journal_id.company_id.transfer_account_id: raise UserError(_("You should have defined an 'Internal Transfer Account' in your cash register's journal!")) @@ -61,7 +61,7 @@ class CashBoxIn(CashBox): class CashBoxOut(CashBox): _name = 'cash.box.out' - @api.one + @api.multi def _calculate_values_for_statement_line(self, record): if not record.journal_id.company_id.transfer_account_id: raise UserError(_("You should have defined an 'Internal Transfer Account' in your cash register's journal!"))