odoo-addon-account-loan 16.0.1.0.5__py3-none-any.whl → 17.0.1.0.0.3__py3-none-any.whl

odoo/addons/account_loan/static/description/index.html

@@ -367,23 +367,24 @@ ul.auto-toc {
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-!! source digest: sha256:a97cd9ecd83ceef07255455f44533ac8e6d337004092ed8eb9febf1ee4e626fd
+!! source digest: sha256:fae1f5805841bcbac809ea9548904a5098614ee325624645cec438bb6ec36a25
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/account-financial-tools/tree/16.0/account_loan"><img alt="OCA/account-financial-tools" src="https://img.shields.io/badge/github-OCA%2Faccount--financial--tools-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/account-financial-tools-16-0/account-financial-tools-16-0-account_loan"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/account-financial-tools&amp;target_branch=16.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
-<p>This module extends the functionality of accounting to support loans.
-It will create automatically moves or invoices for loans.
-Moreover, you can check the pending amount to be paid and reduce the debt.</p>
+<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/account-financial-tools/tree/17.0/account_loan"><img alt="OCA/account-financial-tools" src="https://img.shields.io/badge/github-OCA%2Faccount--financial--tools-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/account-financial-tools-17-0/account-financial-tools-17-0-account_loan"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/account-financial-tools&amp;target_branch=17.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
+<p>This module extends the functionality of accounting to support loans. It
+will create automatically moves or invoices for loans. Moreover, you can
+check the pending amount to be paid and reduce the debt.</p>
 <p>It currently supports two kinds of debts:</p>
-<ul class="simple">
-<li><dl class="first docutils">
-<dt>Loans: a standard debt with banks, that only creates account moves.</dt>
-<dd>Loan types info:
+<ul>
+<li><div class="first line-block">
+<div class="line">Loans: a standard debt with banks, that only creates account moves.</div>
+<div class="line">Loan types info:
 <a class="reference external" href="https://en.wikipedia.org/wiki/Annual_percentage_rate">APR</a>,
 <a class="reference external" href="https://en.wikipedia.org/wiki/Effective_interest_rate">EAR</a>,
-<a class="reference external" href="https://en.wikipedia.org/wiki/Real_interest_rate">Real Rate</a>.</dd>
-</dl>
+<a class="reference external" href="https://en.wikipedia.org/wiki/Real_interest_rate">Real Rate</a>.</div>
+</div>
+</li>
+<li><p class="first">Leases: a debt with a bank where purchase invoices are necessary</p>
 </li>
-<li>Leases: a debt with a bank where purchase invoices are necessary</li>
 </ul>
 <p><strong>Table of contents</strong></p>
 <div class="contents local topic" id="contents">
@@ -406,12 +407,13 @@ Moreover, you can check the pending amount to be paid and reduce the debt.</p>
 <h1><a class="toc-backref" href="#toc-entry-1">Usage</a></h1>
 <p>To use this module, you need to:</p>
 <ol class="arabic simple">
-<li>Go to <cite>Invoicing / Accounting &gt; Accounting &gt; Loans</cite></li>
-<li>Configure a loan selecting the company, loan type, amount, rate and accounts</li>
+<li>Go to Invoicing / Accounting &gt; Accounting &gt; Loans</li>
+<li>Configure a loan selecting the company, loan type, amount, rate and
+accounts</li>
 <li>Post the loan, it will automatically create an account move with the
 expected amounts</li>
-<li>Create automatically the account moves / invoices related to loans and
-leases before a selected date</li>
+<li>Create automatically the account moves / invoices related to loans
+and leases before a selected date</li>
 </ol>
 <p>On a posted loan you can:</p>
 <ul class="simple">
@@ -419,15 +421,14 @@ leases before a selected date</li>
 <li>Modify rates when needed (only unposted lines will be modified)</li>
 <li>Reduce or cancel the debt of a loan / lease</li>
 </ul>
-<a class="reference external image-reference" href="https://runbot.odoo-community.org/runbot/92/12.0">
-<img alt="Try me on Runbot" src="https://odoo-community.org/website/image/ir.attachment/5784_f2813bd/datas" />
-</a>
+<p><a class="reference external image-reference" href="https://runbot.odoo-community.org/runbot/92/12.0"><img alt="Try me on Runbot" src="https://odoo-community.org/website/image/ir.attachment/5784_f2813bd/datas" /></a></p>
 </div>
 <div class="section" id="changelog">
 <h1><a class="toc-backref" href="#toc-entry-2">Changelog</a></h1>
 <div class="section" id="section-1">
 <h2><a class="toc-backref" href="#toc-entry-3">16.0.1.0.0</a></h2>
-<p>Due to the changes on 16, we will generate two moves on leasings, one for the invoice, and another one for the change from long to short term.</p>
+<p>Due to the changes on 16, we will generate two moves on leasings, one
+for the invoice, and another one for the change from long to short term.</p>
 </div>
 </div>
 <div class="section" id="bug-tracker">
@@ -435,7 +436,7 @@ leases before a selected date</li>
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/account-financial-tools/issues">GitHub Issues</a>.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us to smash it by providing a detailed and welcomed
-<a class="reference external" href="https://github.com/OCA/account-financial-tools/issues/new?body=module:%20account_loan%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+<a class="reference external" href="https://github.com/OCA/account-financial-tools/issues/new?body=module:%20account_loan%0Aversion:%2017.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
 <p>Do not contact contributors directly about support or help with technical issues.</p>
 </div>
 <div class="section" id="credits">
@@ -465,7 +466,7 @@ mission is to support the collaborative development of Odoo features and
 promote its widespread use.</p>
 <p>Current <a class="reference external" href="https://odoo-community.org/page/maintainer-role">maintainer</a>:</p>
 <p><a class="reference external image-reference" href="https://github.com/etobella"><img alt="etobella" src="https://github.com/etobella.png?size=40px" /></a></p>
-<p>This module is part of the <a class="reference external" href="https://github.com/OCA/account-financial-tools/tree/16.0/account_loan">OCA/account-financial-tools</a> project on GitHub.</p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/account-financial-tools/tree/17.0/account_loan">OCA/account-financial-tools</a> project on GitHub.</p>
 <p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
 </div>
 </div>

odoo/addons/account_loan/tests/test_loan.py

@@ -7,17 +7,20 @@ from dateutil.relativedelta import relativedelta
 
 from odoo import fields
 from odoo.exceptions import UserError
-from odoo.tests import TransactionCase, tagged
+from odoo.tests import Form, tagged
+from odoo.tools import mute_logger
+
+from odoo.addons.base.tests.common import BaseCommon
 
 _logger = logging.getLogger(__name__)
 try:
     import numpy_financial
-except (ImportError, IOError) as err:
+except (OSError, ImportError) as err:
     _logger.error(err)
 
 
 @tagged("post_install", "-at_install")
-class TestLoan(TransactionCase):
+class TestLoan(BaseCommon):
     @classmethod
     def setUpClass(cls):
         super().setUpClass()
@@ -53,7 +56,7 @@ class TestLoan(TransactionCase):
         )
 
     def test_onchange(self):
-        loan = self.env["account.loan"].new(
+        loan = self.env["account.loan"].create(
             {
                 "name": "LOAN",
                 "company_id": self.company.id,
@@ -69,13 +72,11 @@ class TestLoan(TransactionCase):
                 "partner_id": self.partner.id,
             }
         )
-        journal = loan.journal_id.id
-        loan.is_leasing = True
-        loan._onchange_is_leasing()
-        self.assertNotEqual(journal, loan.journal_id.id)
-        loan.company_id = self.company_02
-        loan._onchange_company()
-        self.assertFalse(loan.interest_expenses_account_id)
+        loan_form = Form(loan)
+        loan_form.is_leasing = True
+        self.assertNotEqual(loan.journal_id, loan_form.journal_id)
+        loan_form.company_id = self.company_02
+        self.assertFalse(loan_form.interest_expenses_account_id)
 
     def test_partner_loans(self):
         self.assertFalse(self.partner.lended_loan_count)
@@ -84,6 +85,7 @@ class TestLoan(TransactionCase):
         action = self.partner.action_view_partner_lended_loans()
         self.assertEqual(loan, self.env[action["res_model"]].search(action["domain"]))
 
+    @mute_logger("odoo.models.unlink")
     def test_round_on_end(self):
         loan = self.create_loan("fixed-annuity", 500000, 1, 60)
         loan.round_on_end = True
@@ -103,6 +105,7 @@ class TestLoan(TransactionCase):
         self.assertEqual(line_1.principal_amount, 0)
         self.assertEqual(line_end.principal_amount, 500000)
 
+    @mute_logger("odoo.models.unlink")
     def test_increase_amount_validation(self):
         amount = 10000
         periods = 24
@@ -149,6 +152,7 @@ class TestLoan(TransactionCase):
                 default_loan_id=loan.id
             ).create({"amount": -100, "date": line.date}).run()
 
+    @mute_logger("odoo.models.unlink")
     def test_pay_amount_validation(self):
         amount = 10000
         periods = 24
@@ -200,6 +204,7 @@ class TestLoan(TransactionCase):
                 default_loan_id=loan.id
             ).create({"amount": -100, "fees": 100, "date": line.date}).run()
 
+    @mute_logger("odoo.models.unlink")
     def test_increase_amount_loan(self):
         amount = 10000
         periods = 24
@@ -246,6 +251,7 @@ class TestLoan(TransactionCase):
         self.assertEqual(loan, new_move.loan_id)
         self.assertEqual(loan.pending_principal_amount, pending_principal_amount + 1000)
 
+    @mute_logger("odoo.models.unlink")
     def test_increase_amount_leasing(self):
         amount = 10000
         periods = 24
@@ -298,6 +304,7 @@ class TestLoan(TransactionCase):
         self.assertEqual(loan, new_move.loan_id)
         self.assertEqual(loan.pending_principal_amount, pending_principal_amount + 1000)
 
+    @mute_logger("odoo.models.unlink")
     def test_fixed_annuity_begin_loan(self):
         amount = 10000
         periods = 24
@@ -348,6 +355,7 @@ class TestLoan(TransactionCase):
         with self.assertRaises(UserError):
             line.view_process_values()
 
+    @mute_logger("odoo.models.unlink")
     def test_fixed_annuity_loan(self):
         amount = 10000
         periods = 24
@@ -394,6 +402,7 @@ class TestLoan(TransactionCase):
         with self.assertRaises(UserError):
             line.view_process_values()
 
+    @mute_logger("odoo.models.unlink")
     def test_fixed_principal_loan_leasing(self):
         amount = 24000
         periods = 24
@@ -498,6 +507,7 @@ class TestLoan(TransactionCase):
         with self.assertRaises(UserError):
             line.view_process_values()
 
+    @mute_logger("odoo.models.unlink")
     def test_fixed_principal_loan_auto_post_leasing(self):
         amount = 24000
         periods = 24
@@ -525,6 +535,7 @@ class TestLoan(TransactionCase):
         self.assertTrue(line.has_invoices)
         self.assertTrue(line.has_moves)
 
+    @mute_logger("odoo.models.unlink")
     def test_interests_on_end_loan(self):
         amount = 10000
         periods = 10
@@ -555,6 +566,7 @@ class TestLoan(TransactionCase):
         self.assertEqual(loan.payment_amount - loan.interests_amount, amount)
         self.assertEqual(loan.pending_principal_amount, 0)
 
+    @mute_logger("odoo.models.unlink")
     def test_cancel_loan(self):
         amount = 10000
         periods = 10

odoo/addons/account_loan/views/account_loan_lines_view.xml

@@ -16,31 +16,31 @@
                 <field name="interests_amount" sum="Total interests" />
                 <field
                     name="long_term_pending_principal_amount"
-                    attrs="{'invisible': [('long_term_loan_account_id', '=', False)]}"
+                    column_invisible="not parent.long_term_loan_account_id"
                 />
                 <field
                     name="long_term_principal_amount"
-                    attrs="{'invisible': [('long_term_loan_account_id', '=', False)]}"
+                    column_invisible="not parent.long_term_loan_account_id"
                 />
-                <field name="long_term_loan_account_id" invisible="1" />
-                <field name="loan_state" invisible="1" />
-                <field name="is_leasing" invisible="1" />
-                <field name="has_invoices" invisible="1" />
-                <field name="has_moves" invisible="1" />
-                <field name="currency_id" invisible="1" />
+                <field name="long_term_loan_account_id" column_invisible="True" />
+                <field name="loan_state" column_invisible="True" />
+                <field name="is_leasing" column_invisible="True" />
+                <field name="has_invoices" column_invisible="True" />
+                <field name="has_moves" column_invisible="True" />
+                <field name="currency_id" column_invisible="True" />
                 <button
                     name="view_account_values"
                     string="Values"
                     type="object"
                     icon="fa-eye"
-                    attrs="{'invisible': [('has_moves', '=', False),  ('has_invoices', '=', False)]}"
+                    invisible="not has_moves and not has_invoices"
                 />
                 <button
                     name="view_process_values"
                     string="Process"
                     type="object"
                     icon="fa-cogs"
-                    attrs="{'invisible': ['|', '|', ('has_moves', '=', True),  ('has_invoices', '=', True), ('loan_state', '!=', 'posted')]}"
+                    invisible="has_moves or has_invoices or loan_state != 'posted'"
                 />
             </tree>
         </field>
@@ -86,7 +86,11 @@
                 <field name="interests_amount" />
                 <field name="long_term_pending_principal_amount" />
                 <field name="long_term_principal_amount" />
-                <field name="company_id" optional="hide" />
+                <field
+                    name="company_id"
+                    optional="hide"
+                    groups="base.group_multi_company"
+                />
                 <field name="partner_id" optional="hide" />
                 <field name="journal_id" optional="hide" />
                 <field name="short_term_loan_account_id" optional="hide" />

odoo/addons/account_loan/views/account_loan_view.xml

@@ -21,7 +21,7 @@
             <tree>
                 <field name="name" />
                 <field name="partner_id" optional="hide" />
-                <field name="company_id" />
+                <field name="company_id" groups="base.group_multi_company" />
                 <field name="is_leasing" />
                 <field name="state" />
                 <field name="rate" optional="hide" />
@@ -41,7 +41,7 @@
                     <button name="compute_lines" type="object" string="Compute items" />
                     <button
                         name="%(account_loan_post_action)d"
-                        states="draft"
+                        invisible="state != 'draft'"
                         type="action"
                         string="Post"
                         groups="account.group_account_manager"
@@ -54,7 +54,7 @@
                             name="view_account_moves"
                             class="oe_stat_button"
                             icon="fa-bars"
-                            attrs="{'invisible': [('state', '=', 'draft')]}"
+                            invisible="state == 'draft'"
                             type="object"
                             string="Moves"
                         />
@@ -62,7 +62,7 @@
                             name="view_account_invoices"
                             class="oe_stat_button"
                             icon="fa-pencil-square-o"
-                            attrs="{'invisible': ['|', ('state', '=', 'draft'), ('is_leasing', '=', False)]}"
+                            invisible="state == 'draft' or not is_leasing"
                             type="object"
                             string="Invoices"
                         />
@@ -70,7 +70,7 @@
                             name="%(account_loan_pay_amount_action)d"
                             class="oe_stat_button"
                             icon="fa-arrow-down"
-                            attrs="{'invisible': [('state', '!=', 'posted')]}"
+                            invisible="state != 'posted'"
                             type="action"
                             groups="account.group_account_manager"
                         >
@@ -83,7 +83,7 @@
                             name="%(account_loan_increase_amount_act_window)d"
                             class="oe_stat_button"
                             icon="fa-arrow-up"
-                            attrs="{'invisible': [('state', '!=', 'posted')]}"
+                            invisible="state != 'posted'"
                             type="action"
                             groups="account.group_account_manager"
                         >
@@ -94,34 +94,43 @@
                         </button>
                     </div>
                     <h1>
-                        <field name="name" />
+                        <field name="name" readonly="state != 'draft'" />
                     </h1>
                     <group>
                         <group>
-                            <field name="company_id" options="{'no_create': True}" />
-                            <field name="loan_type" />
-                            <field name="loan_amount" />
+                            <field name="company_id" invisible="1" />
+                            <field
+                                name="company_id"
+                                options="{'no_create': True}"
+                                readonly="state != 'draft'"
+                                groups="base.group_multi_company"
+                            />
+                            <field name="loan_type" readonly="state != 'draft'" />
+                            <field name="loan_amount" readonly="state != 'draft'" />
                         </group>
                         <group>
-                            <field name="rate_type" />
+                            <field name="rate_type" readonly="state != 'draft'" />
                             <field name="rate" />
                             <field name="rate_period" />
                         </group>
                     </group>
                     <group>
                         <group>
-                            <field name="partner_id" />
-                            <field name="start_date" />
-                            <field name="periods" />
-                            <field name="method_period" />
+                            <field name="partner_id" readonly="state != 'draft'" />
+                            <field name="start_date" readonly="state != 'draft'" />
+                            <field name="periods" readonly="state != 'draft'" />
+                            <field name="method_period" readonly="state != 'draft'" />
                         </group>
                         <group>
-                            <field name="is_leasing" />
-                            <field name="round_on_end" />
-                            <field name="payment_on_first_period" />
+                            <field name="is_leasing" readonly="state != 'draft'" />
+                            <field name="round_on_end" readonly="state != 'draft'" />
+                            <field
+                                name="payment_on_first_period"
+                                readonly="state != 'draft'"
+                            />
                         </group>
                     </group>
-                    <group attrs="{'invisible':[('state', '=', 'draft')]}">
+                    <group invisible="state == 'draft'">
                         <group>
                             <field name="pending_principal_amount" />
                             <field name="payment_amount" />
@@ -138,38 +147,47 @@
                         <page string="Accounts" id="accounting">
                             <group>
                                 <group>
-                                    <field name="journal_id" />
-                                    <field name="short_term_loan_account_id" />
+                                    <field
+                                        name="journal_id"
+                                        readonly="state != 'draft'"
+                                    />
+                                    <field
+                                        name="short_term_loan_account_id"
+                                        readonly="state != 'draft'"
+                                    />
                                     <field name="journal_type" invisible="1" />
                                 </group>
                                 <group>
-                                    <field name="long_term_loan_account_id" />
-                                    <field name="interest_expenses_account_id" />
+                                    <field
+                                        name="long_term_loan_account_id"
+                                        readonly="state != 'draft'"
+                                    />
+                                    <field
+                                        name="interest_expenses_account_id"
+                                        readonly="state != 'draft'"
+                                    />
                                     <field name="currency_id" invisible="1" />
                                 </group>
                             </group>
                         </page>
-                        <page
-                            string="Leasing"
-                            id="leasing"
-                            attrs="{'invisible': [('is_leasing', '=', False)]}"
-                        >
+                        <page string="Leasing" id="leasing" invisible="not is_leasing">
                             <group>
                                 <group>
                                     <field
                                         name="leased_asset_account_id"
-                                        attrs="{'required': [('is_leasing', '=', True)]}"
+                                        required="is_leasing"
+                                        readonly="state != 'draft'"
                                     />
-                                    <field name="residual_amount" />
-                                </group>
-                                <group>
                                     <field
-                                        name="product_id"
-                                        attrs="{'required': [('is_leasing', '=', True)]}"
+                                        name="residual_amount"
+                                        readonly="state != 'draft'"
                                     />
+                                </group>
+                                <group>
+                                    <field name="product_id" required="is_leasing" />
                                     <field
                                         name="interests_product_id"
-                                        attrs="{'required': [('is_leasing', '=', True)]}"
+                                        required="is_leasing"
                                     />
                                     <field name="post_invoice" />
                                 </group>

odoo/addons/account_loan/views/account_move_view.xml

@@ -11,7 +11,7 @@
             <field name="date" position="after">
                 <field
                     name="loan_line_id"
-                    attrs="{'invisible': [('loan_line_id', '=', False)]}"
+                    invisible="not loan_line_id"
                     readonly="True"
                 />
             </field>

odoo/addons/account_loan/views/res_partner.xml

@@ -15,8 +15,8 @@
                     icon="fa-money"
                     name="action_view_partner_lended_loans"
                     groups="account.group_account_user"
-                    context="{'default_partner_id': active_id}"
-                    attrs="{'invisible': [('lended_loan_count','=', 0)]}"
+                    context="{'default_partner_id': id}"
+                    invisible="lended_loan_count == 0"
                 >
                     <div class="o_form_field o_stat_info">
                         <span class="o_stat_value">

odoo/addons/account_loan/wizards/account_loan_generate_entries_view.xml

@@ -19,7 +19,6 @@
                         type="object"
                         class="oe_highlight"
                     />
-                    or
                     <button string="Cancel" class="oe_link" special="cancel" />
                 </footer>
             </form>

odoo/addons/account_loan/wizards/account_loan_increase_amount.xml

@@ -21,7 +21,6 @@
                         type="object"
                         class="oe_highlight"
                     />
-                    or
                     <button string="Cancel" class="oe_link" special="cancel" />
                 </footer>
             </form>

odoo/addons/account_loan/wizards/account_loan_pay_amount_view.xml

@@ -23,7 +23,6 @@
                         type="object"
                         class="oe_highlight"
                     />
-                    or
                     <button string="Cancel" class="oe_link" special="cancel" />
                 </footer>
             </form>

odoo/addons/account_loan/wizards/account_loan_post_view.xml

@@ -20,7 +20,6 @@
                         type="object"
                         class="oe_highlight"
                     />
-                    or
                     <button string="Cancel" class="oe_link" special="cancel" />
                 </footer>
             </form>