From 46d244b061ad1d27519f44efa1f89988ea1cf9cd Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?C=C3=A9dric=20Krier?= Date: Fri, 8 Apr 2016 22:54:24 +0200 Subject: [PATCH] Test if there is something to write before calling write ModelStorage.write requires at least a tuple of records, values. issue5456 review21921002 --- statement.py | 3 ++- tests/scenario_account_statement.rst | 6 ++++++ 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/statement.py b/statement.py index 5f1037def0..e3cff25c5e 100644 --- a/statement.py +++ b/statement.py @@ -431,7 +431,8 @@ class Statement(Workflow, ModelSQL, ModelView): to_write.append({ 'move': move.id, }) - Line.write(*to_write) + if to_write: + Line.write(*to_write) move_lines = [] for move, statement, lines in moves: diff --git a/tests/scenario_account_statement.rst b/tests/scenario_account_statement.rst index 82702d30b7..311265dd58 100644 --- a/tests/scenario_account_statement.rst +++ b/tests/scenario_account_statement.rst @@ -430,3 +430,9 @@ Test number of lines validation:: >>> second_line.account = receivable >>> second_line.party = customer >>> statement.click('validate_statement') + +Validate empty statement:: + + >>> statement = Statement(name='empty', journal=statement_journal) + >>> statement.end_balance = statement.start_balance + >>> statement.click('validate_statement')