From 58109a909cf1b798a33b03d8695d3b91a9734a20 Mon Sep 17 00:00:00 2001 From: houssine Date: Fri, 4 Dec 2020 15:28:51 +0100 Subject: [PATCH] [FIX] fix wrong model and misspelling. add user error --- .../wizard/end_of_year_operation.py | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/easy_my_coop_loan_account/wizard/end_of_year_operation.py b/easy_my_coop_loan_account/wizard/end_of_year_operation.py index 0563c1e..bdfc4d7 100644 --- a/easy_my_coop_loan_account/wizard/end_of_year_operation.py +++ b/easy_my_coop_loan_account/wizard/end_of_year_operation.py @@ -3,6 +3,8 @@ # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html). from odoo import api, fields, models +from odoo.exceptions import UserError + class LoanEndOfYearOperation(models.TransientModel): _name = "loan.end.of.year.operation" @@ -22,6 +24,7 @@ class LoanEndOfYearOperation(models.TransientModel): @api.multi def run(self): afy_obj = self.env["account.fiscal.year"] + interest_line_obj = self.env["loan.interest.line"] loan_issues = self.env["loan.issue"].browse( self._context.get("active_ids") @@ -30,15 +33,19 @@ class LoanEndOfYearOperation(models.TransientModel): last_fy_day = self.ongoing_fy_id.date_to next_fy = afy_obj.get_next_fiscal_year(last_fy_day) - if self.operation_type == "debt_eoy": + if self.operation_type == "eoy_debt": if next_fy: - interest_lines = self.search([ + interest_lines = interest_line_obj.search([ ('due_date', '>=', next_fy.date_from), ('due_date', '<=', next_fy.date_to), ('due_loan_amount', '>', 0), ("loan_issue_id", "in", loan_issues.ids) ]) - interest_lines.generate_loan_due_fy(last_fy_day) + if interest_lines: + interest_lines.generate_loan_due_fy(last_fy_day) + else: + raise UserError("There is no account move lines to" + " generate for the selected loan issue") elif self.operation_type == "eoy_interest": print()