From 09de7be22c63e88c44d4aa617318ad6ac9d27282 Mon Sep 17 00:00:00 2001 From: Florent de Labarre Date: Fri, 3 Mar 2017 10:35:43 +0100 Subject: [PATCH] [IMP] hr_expense : prevent removing entries linked to expenses Once it is deleted, the expense is blocked in the "posted" state. Accounting entries should rarely be removed but never when linked to expenses Closes #15704 --- addons/hr_expense/models/hr_expense.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/addons/hr_expense/models/hr_expense.py b/addons/hr_expense/models/hr_expense.py index fa2eadc2572..bffafef46b8 100644 --- a/addons/hr_expense/models/hr_expense.py +++ b/addons/hr_expense/models/hr_expense.py @@ -391,7 +391,7 @@ class HrExpenseSheet(models.Model): help="The journal used when the expense is done.") bank_journal_id = fields.Many2one('account.journal', string='Bank Journal', states={'done': [('readonly', True)], 'post': [('readonly', True)]}, default=lambda self: self.env['account.journal'].search([('type', 'in', ['case', 'bank'])], limit=1), help="The payment method used when the expense is paid by the company.") accounting_date = fields.Date(string="Accounting Date") - account_move_id = fields.Many2one('account.move', string='Journal Entry', copy=False) + account_move_id = fields.Many2one('account.move', string='Journal Entry', ondelete='restrict', copy=False) department_id = fields.Many2one('hr.department', string='Department', states={'post': [('readonly', True)], 'done': [('readonly', True)]}) @api.multi