From 706fc6ff73090b78c6d177f967591de3cb9f7f5f Mon Sep 17 00:00:00 2001 From: Jared Kipe Date: Thu, 7 Oct 2021 11:40:04 -0700 Subject: [PATCH] [MIG] account_invoice_margin: to Odoo 15.0 (patch error, direct copy) --- account_invoice_margin/README.rst | 28 ++++++++ account_invoice_margin/__init__.py | 1 + account_invoice_margin/__manifest__.py | 24 +++++++ account_invoice_margin/models/__init__.py | 1 + .../models/account_invoice.py | 66 ++++++++++++++++++ account_invoice_margin/tests/__init__.py | 1 + .../tests/test_invoice_margin.py | 69 +++++++++++++++++++ .../views/account_invoice_views.xml | 18 +++++ 8 files changed, 208 insertions(+) create mode 100644 account_invoice_margin/README.rst create mode 100644 account_invoice_margin/__init__.py create mode 100755 account_invoice_margin/__manifest__.py create mode 100644 account_invoice_margin/models/__init__.py create mode 100644 account_invoice_margin/models/account_invoice.py create mode 100644 account_invoice_margin/tests/__init__.py create mode 100644 account_invoice_margin/tests/test_invoice_margin.py create mode 100644 account_invoice_margin/views/account_invoice_views.xml diff --git a/account_invoice_margin/README.rst b/account_invoice_margin/README.rst new file mode 100644 index 00000000..a148ac23 --- /dev/null +++ b/account_invoice_margin/README.rst @@ -0,0 +1,28 @@ +****************************** +Hibou - Account Invoice Margin +****************************** + +Include a margin calculation on invoices. + +For more information and add-ons, visit `Hibou.io `_. + + +============= +Main Features +============= + +* Adds computed field `margin` to invoices to give the profitability by calculating the difference between the Unit Price and the Cost. + +.. image:: https://user-images.githubusercontent.com/15882954/45578631-880c0000-b837-11e8-9c4d-d2f15c3c0592.png + :alt: 'Customer Invoice' + :width: 988 + :align: left + + +======= +License +======= + +Please see `LICENSE `_. + +Copyright Hibou Corp. 2018 diff --git a/account_invoice_margin/__init__.py b/account_invoice_margin/__init__.py new file mode 100644 index 00000000..0650744f --- /dev/null +++ b/account_invoice_margin/__init__.py @@ -0,0 +1 @@ +from . import models diff --git a/account_invoice_margin/__manifest__.py b/account_invoice_margin/__manifest__.py new file mode 100755 index 00000000..7a7c2d3e --- /dev/null +++ b/account_invoice_margin/__manifest__.py @@ -0,0 +1,24 @@ +{ + 'name': 'Invoice Margin', + 'author': 'Hibou Corp. ', + 'version': '15.0.1.0.0', + 'license': 'LGPL-3', + 'category': 'Accounting', + 'sequence': 95, + 'summary': 'Invoices include margin calculation.', + 'description': """ +Invoices include margin calculation. +If the invoice line comes from a sale order line, the cost will come +from the sale order line. + """, + 'website': 'https://hibou.io/', + 'depends': [ + 'account', + 'sale_margin', + ], + 'data': [ + 'views/account_invoice_views.xml', + ], + 'installable': True, + 'application': False, +} diff --git a/account_invoice_margin/models/__init__.py b/account_invoice_margin/models/__init__.py new file mode 100644 index 00000000..8e072db8 --- /dev/null +++ b/account_invoice_margin/models/__init__.py @@ -0,0 +1 @@ +from . import account_invoice diff --git a/account_invoice_margin/models/account_invoice.py b/account_invoice_margin/models/account_invoice.py new file mode 100644 index 00000000..ff7cd3b8 --- /dev/null +++ b/account_invoice_margin/models/account_invoice.py @@ -0,0 +1,66 @@ +from odoo import api, fields, models + + +class AccountMoveLine(models.Model): + _inherit = "account.move.line" + + margin = fields.Float(compute='_product_margin', digits='Product Price', store=True) + purchase_price = fields.Float(string='Cost', digits='Product Price') + + def _compute_margin(self, move, product, product_uom, sale_lines): + # if sale_line_ids and don't re-browse + for line in sale_lines: + return line.purchase_price + frm_cur = move.company_currency_id + to_cur = move.currency_id + purchase_price = product.standard_price + if product_uom and product_uom != product.uom_id: + purchase_price = product.uom_id._compute_price(purchase_price, product_uom) + ctx = self.env.context.copy() + ctx['date'] = move.date if move.date else fields.Date.context_today(move) + price = frm_cur.with_context(ctx)._convert(purchase_price, to_cur, move.company_id, ctx['date'], round=False) + return price + + @api.onchange('product_id', 'product_uom_id') + def product_id_change_margin(self): + for line in self: + if not line.product_id: + line.purchase_price = 0 + else: + line.purchase_price = line._compute_margin(line.move_id, line.product_id, line.product_uom_id, line.sale_line_ids) + + @api.model_create_multi + def create(self, vals): + lines = super(AccountMoveLine, self).create(vals) + if vals and 'purchase_price' not in vals[0]: + lines.product_id_change_margin() + return lines + + @api.depends('product_id', 'purchase_price', 'quantity', 'price_unit', 'price_subtotal') + def _product_margin(self): + for line in self: + currency = line.move_id.currency_id + price = line.purchase_price + margin = line.price_subtotal - (price * line.quantity) + if line.product_id and not price: + date = line.move_id.date if line.move_id.date else fields.Date.context_today(line.move_id) + from_cur = line.move_id.company_currency_id.with_context(date=date) + price = from_cur._convert(line.product_id.standard_price, currency, line.company_id, date, round=False) + margin = line.price_subtotal - (price * line.quantity) + + line.margin = currency.round(margin) if currency else margin + + +class AccountMove(models.Model): + _inherit = "account.move" + + margin = fields.Monetary(compute='_product_margin', + help="It gives profitability by calculating the difference between the Unit Price and the cost.", + currency_field='currency_id', + digits='Product Price', + store=True) + + @api.depends('invoice_line_ids.margin') + def _product_margin(self): + for invoice in self: + invoice.margin = sum(invoice.invoice_line_ids.mapped('margin')) diff --git a/account_invoice_margin/tests/__init__.py b/account_invoice_margin/tests/__init__.py new file mode 100644 index 00000000..982ec0aa --- /dev/null +++ b/account_invoice_margin/tests/__init__.py @@ -0,0 +1 @@ +from . import test_invoice_margin diff --git a/account_invoice_margin/tests/test_invoice_margin.py b/account_invoice_margin/tests/test_invoice_margin.py new file mode 100644 index 00000000..156c14c2 --- /dev/null +++ b/account_invoice_margin/tests/test_invoice_margin.py @@ -0,0 +1,69 @@ +from odoo.addons.sale_margin.tests.test_sale_margin import TestSaleMargin +from datetime import datetime + + +class TestInvoiceMargin(TestSaleMargin): + + def setUp(self): + super(TestInvoiceMargin, self).setUp() + self.AccountMove = self.env['account.move'] + + def test_invoice_margin(self): + """ Test the sale_margin module in Odoo. """ + # Create a sales order for product Graphics Card. + sale_order_so11 = self.SaleOrder.create({ + 'date_order': datetime.today(), + 'name': 'Test_SO011', + 'order_line': [ + (0, 0, { + 'name': '[CARD] Individual Workplace', + 'purchase_price': 700.0, + 'price_unit': 1000.0, + 'product_uom': self.product_uom_id, + 'product_uom_qty': 10.0, + 'state': 'draft', + 'product_id': self.product_id}), + (0, 0, { + 'name': 'Line without product_uom', + 'price_unit': 1000.0, + 'purchase_price': 700.0, + 'product_uom_qty': 10.0, + 'state': 'draft', + 'product_id': self.product_id})], + 'partner_id': self.partner_id, + 'partner_invoice_id': self.partner_invoice_address_id, + 'partner_shipping_id': self.partner_invoice_address_id, + 'pricelist_id': self.pricelist_id}) + # Confirm the sales order. + sale_order_so11.action_confirm() + # Verify that margin field gets bind with the value. + self.assertEqual(sale_order_so11.margin, 6000.00, "Sales order margin should be 6000.00") + + sale_order_so11.order_line.write({'qty_delivered': 10.0}) + + # Invoice the sales order. + inv = sale_order_so11._create_invoices() + self.assertEqual(inv.margin, sale_order_so11.margin) + + account = self.env['account.account'].search([('internal_type', '=', 'other')], limit=1) + inv = self.AccountMove.create({ + 'move_type': 'in_invoice', + 'partner_id': self.partner_id, + 'invoice_line_ids': [ + (0, 0, { + 'account_id': account.id, + 'name': '[CARD] Graphics Card', + 'purchase_price': 600.0, + 'price_unit': 1000.0, + 'quantity': 10.0, + 'product_id': self.product_id}), + (0, 0, { + 'account_id': account.id, + 'name': 'Line without product_uom', + 'price_unit': 1000.0, + 'purchase_price': 800.0, + 'quantity': 10.0,}) + ], + }) + self.assertEqual(len(inv.invoice_line_ids), 2) + self.assertEqual(inv.margin, 6000.0) diff --git a/account_invoice_margin/views/account_invoice_views.xml b/account_invoice_margin/views/account_invoice_views.xml new file mode 100644 index 00000000..e1495b0b --- /dev/null +++ b/account_invoice_margin/views/account_invoice_views.xml @@ -0,0 +1,18 @@ + + + + + account.invoice.margin.view.form + account.move + + + + + + + + + + + +