From afd88d3fcdbbd802f25f175a0649bd501e9d03c3 Mon Sep 17 00:00:00 2001 From: Jared Kipe Date: Thu, 20 Jun 2019 10:08:00 -0700 Subject: [PATCH 1/5] Initial commit of `sale_credit_limit` for 11.0 --- sale_credit_limit/__init__.py | 1 + sale_credit_limit/__manifest__.py | 29 ++++++++++++++++++++++ sale_credit_limit/data/sale_exceptions.xml | 21 ++++++++++++++++ sale_credit_limit/models/__init__.py | 1 + sale_credit_limit/models/sale.py | 17 +++++++++++++ sale_credit_limit/views/partner_views.xml | 15 +++++++++++ 6 files changed, 84 insertions(+) create mode 100644 sale_credit_limit/__init__.py create mode 100644 sale_credit_limit/__manifest__.py create mode 100644 sale_credit_limit/data/sale_exceptions.xml create mode 100644 sale_credit_limit/models/__init__.py create mode 100644 sale_credit_limit/models/sale.py create mode 100644 sale_credit_limit/views/partner_views.xml diff --git a/sale_credit_limit/__init__.py b/sale_credit_limit/__init__.py new file mode 100644 index 00000000..0650744f --- /dev/null +++ b/sale_credit_limit/__init__.py @@ -0,0 +1 @@ +from . import models diff --git a/sale_credit_limit/__manifest__.py b/sale_credit_limit/__manifest__.py new file mode 100644 index 00000000..966faa45 --- /dev/null +++ b/sale_credit_limit/__manifest__.py @@ -0,0 +1,29 @@ +{ + 'name': 'Sale Credit Limit', + 'summary': 'Uses credit limit on Partners to warn salespeople if they are over their limit.', + 'version': '11.0.1.0.0', + 'author': "Hibou Corp.", + 'category': 'Sale', + 'license': 'AGPL-3', + 'complexity': 'expert', + 'images': [], + 'website': "https://hibou.io", + 'description': """ +Uses credit limit on Partners to warn salespeople if they are over their limit. + +When confirming a sale order, the current sale order total will be considered and a Sale Order Exception +will be created if the total would put them over their credit limit. +""", + 'depends': [ + 'sale', + 'account', + 'sale_exception', + ], + 'demo': [], + 'data': [ + 'data/sale_exceptions.xml', + 'views/partner_views.xml', + ], + 'auto_install': False, + 'installable': True, +} diff --git a/sale_credit_limit/data/sale_exceptions.xml b/sale_credit_limit/data/sale_exceptions.xml new file mode 100644 index 00000000..ba593a13 --- /dev/null +++ b/sale_credit_limit/data/sale_exceptions.xml @@ -0,0 +1,21 @@ + + + + + Invoice Partner credit limit exceeded. + The Customer or Invoice Address has a credit limit. + This sale order, or the customer has an outstanding balance that, exceeds their credit limit. + 50 + sale.order + sale + +partner = sale.partner_invoice_id.commercial_partner_id +partner_balance = partner.credit + sale.amount_total +if partner.credit_limit and partner.credit_limit <= partner_balance: + failed = True + + + sale + + + \ No newline at end of file diff --git a/sale_credit_limit/models/__init__.py b/sale_credit_limit/models/__init__.py new file mode 100644 index 00000000..8a0dc04e --- /dev/null +++ b/sale_credit_limit/models/__init__.py @@ -0,0 +1 @@ +from . import sale diff --git a/sale_credit_limit/models/sale.py b/sale_credit_limit/models/sale.py new file mode 100644 index 00000000..ff40a466 --- /dev/null +++ b/sale_credit_limit/models/sale.py @@ -0,0 +1,17 @@ +from odoo import api, models + + +class SaleOrder(models.Model): + _inherit = 'sale.order' + + @api.onchange('partner_invoice_id') + def _onchange_partner_invoice_id(self): + for so in self: + partner = so.partner_invoice_id.commercial_partner_id + if partner.credit_limit and partner.credit_limit <= partner.credit: + m = 'Partner outstanding receivables %0.2f is above their credit limit of %0.2f' \ + % (partner.credit, partner.credit_limit) + return { + 'warning': {'title': 'Sale Credit Limit', + 'message': m} + } diff --git a/sale_credit_limit/views/partner_views.xml b/sale_credit_limit/views/partner_views.xml new file mode 100644 index 00000000..009ee4be --- /dev/null +++ b/sale_credit_limit/views/partner_views.xml @@ -0,0 +1,15 @@ + + + + + res.partner.form.inherit + res.partner + + + + + + + + + \ No newline at end of file From 3a13a03230610a4ec6ae880fa24b5845875057a5 Mon Sep 17 00:00:00 2001 From: Jared Kipe Date: Thu, 20 Jun 2019 15:15:54 -0700 Subject: [PATCH 2/5] IMP `sale_credit_limit` Include the Currency and Currency formatting in SO warning. --- sale_credit_limit/models/sale.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/sale_credit_limit/models/sale.py b/sale_credit_limit/models/sale.py index ff40a466..5b1f9b89 100644 --- a/sale_credit_limit/models/sale.py +++ b/sale_credit_limit/models/sale.py @@ -1,4 +1,5 @@ from odoo import api, models +from odoo.addons.mail.models.mail_template import format_amount class SaleOrder(models.Model): @@ -9,8 +10,9 @@ class SaleOrder(models.Model): for so in self: partner = so.partner_invoice_id.commercial_partner_id if partner.credit_limit and partner.credit_limit <= partner.credit: - m = 'Partner outstanding receivables %0.2f is above their credit limit of %0.2f' \ - % (partner.credit, partner.credit_limit) + m = 'Partner outstanding receivables %s is above their credit limit of %s' \ + % (format_amount(self.env, partner.credit, so.currency_id), + format_amount(self.env, partner.credit_limit, so.currency_id)) return { 'warning': {'title': 'Sale Credit Limit', 'message': m} From 8c5555ba17ec82b148f901152b2d1f5e8f36aed9 Mon Sep 17 00:00:00 2001 From: Jared Kipe Date: Tue, 3 Sep 2019 14:03:09 -0700 Subject: [PATCH 3/5] MIG `sale_credit_limit` for 12.0 --- sale_credit_limit/__manifest__.py | 2 +- sale_credit_limit/views/partner_views.xml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/sale_credit_limit/__manifest__.py b/sale_credit_limit/__manifest__.py index 966faa45..f0291499 100644 --- a/sale_credit_limit/__manifest__.py +++ b/sale_credit_limit/__manifest__.py @@ -1,7 +1,7 @@ { 'name': 'Sale Credit Limit', 'summary': 'Uses credit limit on Partners to warn salespeople if they are over their limit.', - 'version': '11.0.1.0.0', + 'version': '12.0.1.0.0', 'author': "Hibou Corp.", 'category': 'Sale', 'license': 'AGPL-3', diff --git a/sale_credit_limit/views/partner_views.xml b/sale_credit_limit/views/partner_views.xml index 009ee4be..e9dbe0ae 100644 --- a/sale_credit_limit/views/partner_views.xml +++ b/sale_credit_limit/views/partner_views.xml @@ -6,7 +6,7 @@ res.partner - + From df48365ca1fb2f936a83fb2f6eafbff0b33a77da Mon Sep 17 00:00:00 2001 From: Bhoomi Vaishnani Date: Tue, 14 Jul 2020 12:30:11 -0400 Subject: [PATCH 4/5] [MIG] sale_credit_limit: For Odoo13.0 --- sale_credit_limit/__manifest__.py | 2 +- sale_credit_limit/data/sale_exceptions.xml | 2 -- sale_credit_limit/models/sale.py | 7 ++-- sale_credit_limit/tests/__init__.py | 1 + .../tests/test_sale_credit_exception.py | 32 +++++++++++++++++++ 5 files changed, 37 insertions(+), 7 deletions(-) create mode 100644 sale_credit_limit/tests/__init__.py create mode 100644 sale_credit_limit/tests/test_sale_credit_exception.py diff --git a/sale_credit_limit/__manifest__.py b/sale_credit_limit/__manifest__.py index f0291499..30ec8fe4 100644 --- a/sale_credit_limit/__manifest__.py +++ b/sale_credit_limit/__manifest__.py @@ -1,7 +1,7 @@ { 'name': 'Sale Credit Limit', 'summary': 'Uses credit limit on Partners to warn salespeople if they are over their limit.', - 'version': '12.0.1.0.0', + 'version': '13.0.1.0.0', 'author': "Hibou Corp.", 'category': 'Sale', 'license': 'AGPL-3', diff --git a/sale_credit_limit/data/sale_exceptions.xml b/sale_credit_limit/data/sale_exceptions.xml index ba593a13..b8f6192d 100644 --- a/sale_credit_limit/data/sale_exceptions.xml +++ b/sale_credit_limit/data/sale_exceptions.xml @@ -7,7 +7,6 @@ This sale order, or the customer has an outstanding balance that, exceeds their credit limit. 50 sale.order - sale partner = sale.partner_invoice_id.commercial_partner_id partner_balance = partner.credit + sale.amount_total @@ -15,7 +14,6 @@ if partner.credit_limit and partner.credit_limit <= partner_balance: failed = True - sale \ No newline at end of file diff --git a/sale_credit_limit/models/sale.py b/sale_credit_limit/models/sale.py index 5b1f9b89..5b90514e 100644 --- a/sale_credit_limit/models/sale.py +++ b/sale_credit_limit/models/sale.py @@ -1,5 +1,4 @@ -from odoo import api, models -from odoo.addons.mail.models.mail_template import format_amount +from odoo import api, models, tools class SaleOrder(models.Model): @@ -11,8 +10,8 @@ class SaleOrder(models.Model): partner = so.partner_invoice_id.commercial_partner_id if partner.credit_limit and partner.credit_limit <= partner.credit: m = 'Partner outstanding receivables %s is above their credit limit of %s' \ - % (format_amount(self.env, partner.credit, so.currency_id), - format_amount(self.env, partner.credit_limit, so.currency_id)) + % (tools.format_amount(self.env, partner.credit, so.currency_id), + tools.format_amount(self.env, partner.credit_limit, so.currency_id)) return { 'warning': {'title': 'Sale Credit Limit', 'message': m} diff --git a/sale_credit_limit/tests/__init__.py b/sale_credit_limit/tests/__init__.py new file mode 100644 index 00000000..a2d422d0 --- /dev/null +++ b/sale_credit_limit/tests/__init__.py @@ -0,0 +1 @@ +from . import test_sale_credit_exception diff --git a/sale_credit_limit/tests/test_sale_credit_exception.py b/sale_credit_limit/tests/test_sale_credit_exception.py new file mode 100644 index 00000000..d679661f --- /dev/null +++ b/sale_credit_limit/tests/test_sale_credit_exception.py @@ -0,0 +1,32 @@ + +from odoo.addons.sale_exception.tests.test_sale_exception import TestSaleException + + +class TestSaleCreditException(TestSaleException): + + def setUp(self): + super(TestSaleCreditException, self).setUp() + + def test_sale_order_credit_limit_exception(self): + self.sale_exception_confirm = self.env['sale.exception.confirm'] + exception = self.env.ref('sale_credit_limit.excep_sale_credit_limit') + exception.active = True + partner = self.env.ref('base.res_partner_12') + partner.credit_limit = 100.00 + p = self.env.ref('product.product_product_25_product_template') + so1 = self.env['sale.order'].create({ + 'partner_id': partner.id, + 'partner_invoice_id': partner.id, + 'partner_shipping_id': partner.id, + 'order_line': [(0, 0, {'name': p.name, + 'product_id': p.id, + 'product_uom_qty': 2, + 'product_uom': p.uom_id.id, + 'price_unit': p.list_price})], + 'pricelist_id': self.env.ref('product.list0').id, + }) + + # confirm quotation + so1.action_confirm() + self.assertTrue(so1.state == 'draft') + self.assertFalse(so1.ignore_exception) From 04eb5ad0a00c51e0f9f1c1aaeb27579cac2c9179 Mon Sep 17 00:00:00 2001 From: Jared Self Date: Wed, 18 Nov 2020 11:51:51 -0700 Subject: [PATCH 5/5] [MIG] Changed User For Test Module --- sale_credit_limit/__manifest__.py | 2 +- sale_credit_limit/tests/test_sale_credit_exception.py | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/sale_credit_limit/__manifest__.py b/sale_credit_limit/__manifest__.py index 30ec8fe4..8b45f97c 100644 --- a/sale_credit_limit/__manifest__.py +++ b/sale_credit_limit/__manifest__.py @@ -1,7 +1,7 @@ { 'name': 'Sale Credit Limit', 'summary': 'Uses credit limit on Partners to warn salespeople if they are over their limit.', - 'version': '13.0.1.0.0', + 'version': '14.0.1.0.0', 'author': "Hibou Corp.", 'category': 'Sale', 'license': 'AGPL-3', diff --git a/sale_credit_limit/tests/test_sale_credit_exception.py b/sale_credit_limit/tests/test_sale_credit_exception.py index d679661f..01a63cfd 100644 --- a/sale_credit_limit/tests/test_sale_credit_exception.py +++ b/sale_credit_limit/tests/test_sale_credit_exception.py @@ -8,8 +8,9 @@ class TestSaleCreditException(TestSaleException): super(TestSaleCreditException, self).setUp() def test_sale_order_credit_limit_exception(self): + admin_user = self.env.ref('base.user_admin') self.sale_exception_confirm = self.env['sale.exception.confirm'] - exception = self.env.ref('sale_credit_limit.excep_sale_credit_limit') + exception = self.env.ref('sale_credit_limit.excep_sale_credit_limit').with_user(admin_user) exception.active = True partner = self.env.ref('base.res_partner_12') partner.credit_limit = 100.00