tryton-do/tryton
2
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Test account before using it

Browse Source 
Since AR, there are no more BrowseRecordNull. So Many2One must be tested before
usage.

issue2825
This commit is contained in:
Cédric Krier
2012-10-10 17:54:13 +02:00
parent cdba4e10dc
commit df8e21f6af
1 changed files with 10 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
invoice.py
+10 -8
View File
@@ -1043,6 +1043,11 @@ class Invoice(Workflow, ModelSQL, ModelView):
lines = []
if self.type in ('out_invoice', 'in_credit_note'):
if self.account == journal.debit_account:
self.raise_user_error('same_debit_account')
if not journal.debit_account:
self.raise_user_error('missing_debit_account')
lines.append({
'description': description,
'account': self.account.id,
@@ -1061,11 +1066,12 @@ class Invoice(Workflow, ModelSQL, ModelView):
'amount_second_currency': amount_second_currency,
'second_currency': second_currency,
})
if self.account == journal.debit_account:
self.raise_user_error('same_debit_account')
if not journal.debit_account:
self.raise_user_error('missing_debit_account')
else:
if self.account == journal.credit_account:
self.raise_user_error('same_credit_account')
if not journal.credit_account:
self.raise_user_error('missing_credit_account')
lines.append({
'description': description,
'account': self.account.id,
@@ -1084,10 +1090,6 @@ class Invoice(Workflow, ModelSQL, ModelView):
'amount_second_currency': amount_second_currency,
'second_currency': second_currency,
})
if self.account == journal.credit_account:
self.raise_user_error('same_credit_account')
if not journal.credit_account:
self.raise_user_error('missing_credit_account')
period_id = Period.find(self.company.id, date=date)