odoo-addon-account-credit-control 17.0.2.2.2__py3-none-any.whl → 18.0.1.0.0.2__py3-none-any.whl

odoo/addons/account_credit_control/models/mail_mail.py

@@ -8,7 +8,10 @@ from odoo import models
 class Mail(models.Model):
     _inherit = "mail.mail"
 
-    def _update_control_line_status(self):
+    def _postprocess_sent_message(
+        self, success_pids, failure_reason=False, failure_type=None
+    ):
+        """Mark credit control lines states."""
         for mail in self:
             msg = mail.mail_message_id
             if msg.model != "credit.control.communication":
@@ -20,34 +23,8 @@ class Mail(models.Model):
                 )
                 new_state = "sent" if mail.state == "sent" else "email_error"
                 lines.write({"state": new_state})
-
-    def _postprocess_sent_message(
-        self, success_pids, failure_reason=False, failure_type=None
-    ):
-        """Mark credit control lines states."""
-        self._update_control_line_status()
         return super()._postprocess_sent_message(
             success_pids=success_pids,
             failure_reason=failure_reason,
             failure_type=failure_type,
         )
-
-    def _send(
-        self,
-        auto_commit=False,
-        raise_exception=False,
-        smtp_session=None,
-        alias_domain_id=False,
-    ):
-        # because of
-        # https://github.com/odoo/odoo/blob/bcba6c0dda4818e67a9023beb26593a7d74ff6a6/
-        # addons/mail/models/mail_mail.py#L606-L607
-        # we don't go through _postprocess_sent_message if the address is blacklisted
-        no_postprocess = self.filtered(lambda m: m.state != "outgoing")
-        no_postprocess._update_control_line_status()
-        return super()._send(
-            auto_commit=auto_commit,
-            raise_exception=raise_exception,
-            smtp_session=smtp_session,
-            alias_domain_id=alias_domain_id,
-        )

odoo/addons/account_credit_control/readme/CONTRIBUTORS.md

@@ -14,7 +14,9 @@
   - João Marques
   - César A. Sánchez
   - Víctor Martínez
+  - Carlos Lopez
 - Enric Tobella \<<etobella@creublanca.es>\>
 - Naglis Jonaitis (Versada UAB) \<<naglis@versada.eu>\>
 - [360ERP](https://www.360erp.com):
   - Andrea Stirpe
+  - Kevin Khao

odoo/addons/account_credit_control/report/account_credit_control_analysis.xml

@@ -8,7 +8,7 @@
         >credit.control.analysis.tree (in account_credit_control_analysis)</field>
         <field name="model">credit.control.analysis</field>
         <field name="arch" type="xml">
-            <tree>
+            <list>
                 <field name="partner_id" />
                 <field name="partner_ref" />
                 <field name="policy_id" />
@@ -16,7 +16,7 @@
                 <field name="level" />
                 <field name="open_balance" widget="monetary" />
                 <field name="currency_id" invisible="1" />
-            </tree>
+            </list>
         </field>
     </record>
     <record id="credit_control_analysis_pivot_view" model="ir.ui.view">
@@ -37,7 +37,7 @@
     <record model="ir.actions.act_window" id="credit_control_analysis_act_window">
         <field name="name">Credit Control Analysis</field>
         <field name="res_model">credit.control.analysis</field>
-        <field name="view_mode">tree,pivot</field>
+        <field name="view_mode">list,pivot</field>
         <field name="domain">[]</field>
         <field name="context">{}</field>
     </record>

odoo/addons/account_credit_control/report/report.xml

@@ -1,16 +1,15 @@
 <?xml version="1.0" encoding="utf-8" ?>
 <odoo>
-     <record id="credit_control_summary" model="ir.actions.report">
-            <field name="name">Credit Control Summary</field>
-            <field name="model">credit.control.communication</field>
-            <field name="report_type">qweb-pdf</field>
-            <field
+    <record id="credit_control_summary" model="ir.actions.report">
+        <field name="name">Credit Control Summary</field>
+        <field name="model">credit.control.communication</field>
+        <field name="report_type">qweb-pdf</field>
+        <field
             name="report_name"
         >account_credit_control.report_credit_control_summary</field>
-            <field
+        <field
             name="report_file"
         >account_credit_control.report_credit_control_summary</field>
-            <field name="binding_type">report</field>
-        </record>
-
+        <field name="binding_type">report</field>
+    </record>
 </odoo>

odoo/addons/account_credit_control/security/account_security.xml

@@ -41,15 +41,13 @@
             <field name="global" eval="True" />
             <field
                 name="domain_force"
-            >['|',('company_id','=',False),('company_id','in',company_ids)]</field>
+            >[('company_id','in',company_ids + [False])]</field>
         </record>
         <record id="credit_control_communication_rule" model="ir.rule">
             <field name="name">Credit Control Communication</field>
             <field name="model_id" ref="model_credit_control_communication" />
             <field name="global" eval="True" />
-            <field
-                name="domain_force"
-            >['|',('company_id','=',False),('company_id','in',company_ids)]</field>
+            <field name="domain_force">[('company_id','in',company_ids)]</field>
         </record>
         <record id="credit_control_line_rule" model="ir.rule">
             <field name="name">Credit Control Line</field>
@@ -57,7 +55,7 @@
             <field name="global" eval="True" />
             <field
                 name="domain_force"
-            >['|',('company_id','=',False),('company_id','in',company_ids)]</field>
+            >[('company_id','in',company_ids + [False])]</field>
         </record>
         <record id="credit_control_policy_rule" model="ir.rule">
             <field name="name">Credit Control Policy</field>
@@ -65,7 +63,7 @@
             <field name="global" eval="True" />
             <field
                 name="domain_force"
-            >['|',('company_id','=',False),('company_id','in',company_ids)]</field>
+            >[('company_id','in',company_ids + [False])]</field>
         </record>
         <record id="credit_control_analysis_rule" model="ir.rule">
             <field name="name">Credit Control Analysis</field>
@@ -73,7 +71,7 @@
             <field name="global" eval="True" />
             <field
                 name="domain_force"
-            >['|',('company_id','=',False),('company_id','in',company_ids)]</field>
+            >[('company_id','in',company_ids + [False])]</field>
         </record>
         <record id="credit_control_partner_payment_action_type_rule" model="ir.rule">
             <field name="name">Credit Control Partner Payment Action Type</field>
@@ -81,7 +79,7 @@
             <field name="global" eval="True" />
             <field
                 name="domain_force"
-            >['|',('company_id','=',False),('company_id','in',company_ids)]</field>
+            >[('company_id','in',company_ids + [False])]</field>
         </record>
         <record id="base.user_admin" model="res.users">
             <field

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

@@ -367,9 +367,9 @@ ul.auto-toc {
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-!! source digest: sha256:2eef36ea14e8ce5d86bf9c00a6b742f1e2c717330ef4d0a718c0f1bf43cf738f
+!! source digest: sha256:62f3291d80cd43c777aa84d4777818c614789496f49c051ac34aaafd73a9bd83
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<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/credit-control/tree/17.0/account_credit_control"><img alt="OCA/credit-control" src="https://img.shields.io/badge/github-OCA%2Fcredit--control-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/credit-control-17-0/credit-control-17-0-account_credit_control"><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/credit-control&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><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/credit-control/tree/18.0/account_credit_control"><img alt="OCA/credit-control" src="https://img.shields.io/badge/github-OCA%2Fcredit--control-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/credit-control-18-0/credit-control-18-0-account_credit_control"><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/credit-control&amp;target_branch=18.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
 <p>Account Credit Control module is a part of Financial Tools used in
 business to ensure that once sales are made they are realised as cash.
 This module helps to identify outstanding debt beyond tolerance level
@@ -430,7 +430,7 @@ automatically added as followers to the credit control lines.</p>
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/credit-control/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/credit-control/issues/new?body=module:%20account_credit_control%0Aversion:%2017.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/credit-control/issues/new?body=module:%20account_credit_control%0Aversion:%2018.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">
@@ -464,12 +464,14 @@ If you spotted it first, help us to smash it by providing a detailed and welcome
 <li>João Marques</li>
 <li>César A. Sánchez</li>
 <li>Víctor Martínez</li>
+<li>Carlos Lopez</li>
 </ul>
 </li>
 <li>Enric Tobella &lt;<a class="reference external" href="mailto:etobella&#64;creublanca.es">etobella&#64;creublanca.es</a>&gt;</li>
 <li>Naglis Jonaitis (Versada UAB) &lt;<a class="reference external" href="mailto:naglis&#64;versada.eu">naglis&#64;versada.eu</a>&gt;</li>
 <li><a class="reference external" href="https://www.360erp.com">360ERP</a>:<ul>
 <li>Andrea Stirpe</li>
+<li>Kevin Khao</li>
 </ul>
 </li>
 </ul>
@@ -483,7 +485,7 @@ If you spotted it first, help us to smash it by providing a detailed and welcome
 <p>OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.</p>
-<p>This module is part of the <a class="reference external" href="https://github.com/OCA/credit-control/tree/17.0/account_credit_control">OCA/credit-control</a> project on GitHub.</p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/credit-control/tree/18.0/account_credit_control">OCA/credit-control</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_credit_control/tests/test_account_move.py

@@ -7,19 +7,16 @@ from dateutil import relativedelta
 
 from odoo import fields
 from odoo.exceptions import UserError
-from odoo.tests import tagged
-from odoo.tests.common import Form
+from odoo.tests import Form, tagged
 
 from odoo.addons.account.tests.common import AccountTestInvoicingCommon
-from odoo.addons.base.tests.common import DISABLED_MAIL_CONTEXT
 
 
 @tagged("post_install", "-at_install")
 class TestAccountInvoice(AccountTestInvoicingCommon):
     @classmethod
-    def setUpClass(cls, chart_template_ref=None):
-        super().setUpClass(chart_template_ref=chart_template_ref)
-        cls.env = cls.env(context=dict(cls.env.context, **DISABLED_MAIL_CONTEXT))
+    def setUpClass(cls):
+        super().setUpClass()
         cls.env.user.groups_id |= cls.env.ref(
             "account_credit_control.group_account_credit_control_manager"
         )

odoo/addons/account_credit_control/tests/test_credit_control_policy.py

@@ -2,18 +2,12 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 from odoo.exceptions import UserError, ValidationError
 from odoo.tests import tagged
-from odoo.tests.common import TransactionCase
 
-from odoo.addons.base.tests.common import DISABLED_MAIL_CONTEXT
+from odoo.addons.base.tests.common import BaseCommon
 
 
 @tagged("post_install", "-at_install")
-class TestCreditControlPolicy(TransactionCase):
-    @classmethod
-    def setUpClass(cls):
-        super().setUpClass()
-        cls.env = cls.env(context=dict(cls.env.context, **DISABLED_MAIL_CONTEXT))
-
+class TestCreditControlPolicy(BaseCommon):
     def test_check_policy_against_account(self):
         """
         Test the model check_policy_against_account with several case

odoo/addons/account_credit_control/tests/test_credit_control_run.py

@@ -9,19 +9,16 @@ from dateutil import relativedelta
 
 from odoo import fields
 from odoo.exceptions import AccessError, UserError
-from odoo.tests import RecordCapturer, tagged
-from odoo.tests.common import Form
+from odoo.tests import Form, RecordCapturer, tagged
 
 from odoo.addons.account.tests.common import AccountTestInvoicingCommon
-from odoo.addons.base.tests.common import DISABLED_MAIL_CONTEXT
 
 
 @tagged("post_install", "-at_install")
 class TestCreditControlRun(AccountTestInvoicingCommon):
     @classmethod
-    def setUpClass(cls, chart_template_ref=None):
-        super().setUpClass(chart_template_ref=chart_template_ref)
-        cls.env = cls.env(context=dict(cls.env.context, **DISABLED_MAIL_CONTEXT))
+    def setUpClass(cls):
+        super().setUpClass()
         cls.env.user.groups_id |= cls.env.ref(
             "account_credit_control.group_account_credit_control_manager"
         )
@@ -130,8 +127,8 @@ class TestCreditControlRun(AccountTestInvoicingCommon):
         )
 
         report_regex = (
-            r'<p>Policy "<b>%s</b>" has generated <b>'
-            r"\d+ Credit Control Lines.</b><br></p>" % self.policy.name
+            rf'<p>Policy "<b>{self.policy.name}</b>" has generated <b>'
+            r"\d+ Credit Control Lines.</b><br></p>"
         )
         regex_result = re.match(report_regex, control_run.report)
         self.assertIsNotNone(regex_result)
@@ -216,8 +213,8 @@ class TestCreditControlRun(AccountTestInvoicingCommon):
         self.assertEqual(control_run.state, "done")
 
         report_regex = (
-            r'<p>Policy "<b>%s</b>" has generated <b>'
-            r"\d+ Credit Control Lines.</b><br></p>" % self.policy.name
+            rf'<p>Policy "<b>{self.policy.name}</b>" has generated <b>'
+            r"\d+ Credit Control Lines.</b><br></p>"
         )
         regex_result = re.match(report_regex, control_run.report)
         self.assertIsNotNone(regex_result)
@@ -251,8 +248,8 @@ class TestCreditControlRun(AccountTestInvoicingCommon):
         self.assertEqual(control_run.state, "done")
 
         report_regex = (
-            r'<p>Policy "<b>%s</b>" has generated <b>'
-            r"\d+ Credit Control Lines.</b><br></p>" % self.policy.name
+            rf'<p>Policy "<b>{self.policy.name}</b>" has generated <b>'
+            r"\d+ Credit Control Lines.</b><br></p>"
         )
         regex_result = re.match(report_regex, control_run.report)
         self.assertIsNotNone(regex_result)
@@ -291,7 +288,9 @@ class TestCreditControlRun(AccountTestInvoicingCommon):
             {"name": "to_be_sent", "line_ids": [(6, 0, control_lines.ids)]}
         )
         marker.mark_lines()
-        emailer_obj = self.env["credit.control.emailer"]
+        emailer_obj = self.env["credit.control.emailer"].with_context(
+            domain_notifications_email="test@example.com"
+        )
         wiz_emailer = emailer_obj.create({})
         wiz_emailer.line_ids = control_lines
         with RecordCapturer(self.env["credit.control.communication"], []) as capture:
@@ -309,7 +308,6 @@ class TestCreditControlRun(AccountTestInvoicingCommon):
             {"name": "to_be_sent", "line_ids": [(6, 0, control_lines.ids)]}
         )
         marker.mark_lines()
-        emailer_obj = self.env["credit.control.emailer"]
         wiz_emailer = emailer_obj.create({})
         wiz_emailer.line_ids = control_lines
         with RecordCapturer(self.env["credit.control.communication"], []) as capture:

odoo/addons/account_credit_control/tests/test_res_partner.py

@@ -3,17 +3,25 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 from odoo.exceptions import ValidationError
 from odoo.tests import tagged
-from odoo.tests.common import TransactionCase
 
-from odoo.addons.base.tests.common import DISABLED_MAIL_CONTEXT
+from odoo.addons.base.tests.common import BaseCommon
 
 
 @tagged("post_install", "-at_install")
-class TestCreditControlPolicyLevel(TransactionCase):
+class TestCreditControlPolicyLevel(BaseCommon):
     @classmethod
     def setUpClass(cls):
         super().setUpClass()
-        cls.env = cls.env(context=dict(cls.env.context, **DISABLED_MAIL_CONTEXT))
+        cls.partner = cls.env["res.partner"].create({"name": "Partner 1"})
+        cls.account = cls.env["account.account"].create(
+            {
+                "code": "400001",
+                "name": "Test",
+                "account_type": "asset_receivable",
+                "reconcile": True,
+            }
+        )
+        cls.partner.property_account_receivable_id = cls.account
 
     def test_check_credit_policy(self):
         """
@@ -24,20 +32,30 @@ class TestCreditControlPolicyLevel(TransactionCase):
         retry to assign this policy and this account on the partner
         """
         policy = self.env.ref("account_credit_control.credit_control_3_time")
-        partner = self.env["res.partner"].create({"name": "Partner 1"})
-        account = self.env["account.account"].create(
-            {
-                "code": "400001",
-                "name": "Test",
-                "account_type": "asset_receivable",
-                "reconcile": True,
-            }
-        )
-        partner.property_account_receivable_id = account
-
         with self.assertRaises(ValidationError):
-            partner.write({"credit_policy_id": policy.id})
+            self.partner.write({"credit_policy_id": policy.id})
 
-        policy.write({"account_ids": [(6, 0, [account.id])]})
-        partner.property_account_receivable_id = account.id
-        partner.credit_policy_id = policy.id
+        policy.write({"account_ids": [(6, 0, [self.account.id])]})
+        self.partner.property_account_receivable_id = self.account.id
+        self.partner.credit_policy_id = policy.id
+
+    def test_search_credit_policy(self):
+        """
+        Test the search of the credit policy
+        First we try to search a policy without account
+        After that we add the account in the policy and
+        retry to search the policy
+        """
+        Policy = self.env["credit.control.policy"].with_context(
+            account_receivable_partner_id=self.partner.id
+        )
+        policy = self.env.ref("account_credit_control.credit_control_3_time")
+        self.assertFalse(policy.account_ids)
+        domain = [("display_name", "=", "3 time policy")]
+        policy_find = Policy.search(domain)
+        self.assertFalse(policy_find)
+        # Add the account to the policy
+        policy.write({"account_ids": [(6, 0, [self.account.id])]})
+        policy_find = Policy.search(domain)
+        self.assertEqual(len(policy_find), 1)
+        self.assertEqual(policy_find, policy)

odoo/addons/account_credit_control/views/account_move.xml

@@ -30,7 +30,7 @@
                     </group>
                     <separator string="Issued Lines" colspan="4" />
                     <field name="credit_control_line_ids" colspan="4" nolabel="1">
-                        <tree>
+                        <list>
                             <field name="date" />
                             <field name="level" />
                             <field name="state" />
@@ -38,7 +38,7 @@
                             <field name="balance_due" />
                             <field name="policy_level_id" />
                             <field name="policy_id" />
-                        </tree>
+                        </list>
                     </field>
                 </page>
             </notebook>

odoo/addons/account_credit_control/views/credit_control_communication.xml

@@ -35,7 +35,7 @@
                         </group>
                     </group>
                     <field name="credit_control_line_ids" nolabel="1">
-                        <tree>
+                        <list>
                             <field name="date" />
                             <field name="level" />
                             <field name="state" />
@@ -43,14 +43,10 @@
                             <field name="balance_due" />
                             <field name="policy_level_id" />
                             <field name="policy_id" />
-                        </tree>
+                        </list>
                     </field>
                 </sheet>
-                <div class="oe_chatter">
-                    <field name="message_follower_ids" />
-                    <field name="activity_ids" />
-                    <field name="message_ids" />
-                </div>
+                <chatter />
             </form>
         </field>
     </record>
@@ -73,21 +69,18 @@
                         name="group_by_report_date"
                         domain='[]'
                         context="{'group_by': 'report_date'}"
-                        icon="terp-go-month"
                         string="Run date"
                     />
                     <filter
                         name="group_by_partner"
                         domain='[]'
                         context="{'group_by': 'partner_id'}"
-                        icon="terp-partner"
                         string="Partner"
                     />
                     <filter
                         name="group_by_company"
                         domain='[]'
                         context="{'group_by': 'company_id'}"
-                        icon="terp-company"
                         string="Company"
                         groups="base.group_multi_company"
                     />
@@ -95,14 +88,12 @@
                         name="group_by_policy"
                         domain='[]'
                         context="{'group_by': 'policy_id'}"
-                        icon="terp-document-new"
                         string="Credit policy"
                     />
                     <filter
                         name="group_by_policy_level"
                         domain='[]'
                         context="{'group_by': 'policy_level_id'}"
-                        icon="terp-document-new"
                         string="Credit policy level"
                     />
                     <filter
@@ -115,33 +106,33 @@
         </field>
     </record>
     <record id="credit_control_communication_tree_view" model="ir.ui.view">
-        <field name="name">credit.control.communication.tree</field>
+        <field name="name">credit.control.communication.list</field>
         <field name="model">credit.control.communication</field>
         <field name="arch" type="xml">
-            <tree>
+            <list>
                 <field name="partner_id" />
                 <field name="user_id" />
                 <field name="company_id" groups="base.group_multi_company" />
                 <field name="policy_level_id" />
                 <field name="policy_id" />
                 <field name="report_date" />
-            </tree>
+            </list>
         </field>
     </record>
     <record model="ir.actions.act_window" id="credit_control_communication_action">
         <field name="name">Credit Control Communications</field>
         <field name="type">ir.actions.act_window</field>
         <field name="res_model">credit.control.communication</field>
-        <field name="view_mode">tree,form</field>
+        <field name="view_mode">list,form</field>
         <!-- <field name="view_id" ref="credit_control_communication_tree_view"/> -->
         <field name="help" type="html">
             <p class="oe_view_nocontent_create">
                 Go to <i
-                >Invoicing > Credit Control > Credit Control Run</i> to create a new credit control communication process.
+            >Invoicing > Credit Control > Credit Control Run</i> to create a new credit control communication process.
             </p>
             <p>
                 That's the most common way of creating these records. However, you can do it from here too if you want, by pressing <i
-                >Create</i>.
+            >Create</i>.
             </p>
         </field>
     </record>