[ADD]account_move_line_sale_info

account_move_line_sale_info/README.rst

@@ -0,0 +1,65 @@
+.. image:: https://img.shields.io/badge/license-AGPLv3-blue.svg
+   :target: https://www.gnu.org/licenses/agpl.html
+   :alt: License: AGPL-3
+
+===========================
+Account Move Line Sale Info
+===========================
+
+This module will add the sale order line to journal items.
+
+The ultimate goal is to establish the purchase order line as one of the key
+fields to reconcile the Goods Delivered Not Invoiced accrual account.
+
+
+Usage
+=====
+
+The sale order line will be automatically copied to the journal items.
+
+* When a supplier invoice is created referencing sales orders, the
+  sale order line will be copied to the corresponding journal item.
+
+* When a stock move is validated and generates a journal entry, the sale
+  order line is copied to the account move line.
+
+.. image:: https://odoo-community.org/website/image/ir.attachment/5784_f2813bd/datas
+   :alt: Try me on Runbot
+   :target: https://runbot.odoo-community.org/runbot/92/11.0
+
+Bug Tracker
+===========
+
+Bugs are tracked on `GitHub Issues
+<https://github.com/OCA/account-financial-tools/issues>`_. In case of trouble, please
+check there if your issue has already been reported. If you spotted it first,
+help us smashing it by providing a detailed and welcomed feedback.
+
+
+Credits
+=======
+
+Images
+------
+
+* Odoo Community Association: `Icon <https://github.com/OCA/maintainer-tools/blob/master/template/module/static/description/icon.svg>`_.
+
+Contributors
+------------
+
+* Aaron Henriquez <ahenriquez@forgeflow.com>
+
+Maintainer
+----------
+
+.. image:: https://odoo-community.org/logo.png
+   :alt: Odoo Community Association
+   :target: https://odoo-community.org
+
+This module is maintained by the OCA.
+
+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.
+
+To contribute to this module, please visit https://odoo-community.org.

account_move_line_sale_info/__init__.py

@@ -0,0 +1,4 @@
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
+
+from . import models
+from .hooks import post_init_hook

account_move_line_sale_info/__manifest__.py

@@ -0,0 +1,20 @@
+# Copyright 2020 ForgeFlow S.L.
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
+
+{
+    "name": "Account Move Line Sale Info",
+    "summary": "Introduces the purchase order line to the journal items",
+    "version": "11.0.1.0.0",
+    "author": "ForgeFlow S.L., "
+              "Odoo Community Association (OCA)",
+    "website": "http://www.github.com/OCA/account-financial-tools",
+    "category": "Generic",
+    "depends": ["account", "sale"],
+    "license": "AGPL-3",
+    "data": [
+        "security/account_security.xml",
+        "views/account_move_view.xml",
+    ],
+    'installable': True,
+    'post_init_hook': 'post_init_hook',
+}

account_move_line_sale_info/hooks.py

@@ -0,0 +1,39 @@
+# Copyright 2019 ForgeFlow S.L.
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
+import ast
+from odoo import api, SUPERUSER_ID
+
+
+def post_init_hook(cr, registry):
+
+    """ INIT sale references in acount move line """
+    # FOR stock moves
+    cr.execute("""
+        update account_move_line aml set sale_line_id = sm.sale_line_id
+        FROM account_move_line aml2
+        INNER JOIN stock_move sm ON
+        aml2.stock_move_id = sm.id
+        WHERE aml.id = aml2.id;
+    """)
+    # FOR invoices
+    cr.execute("""
+        update account_move_line aml set sale_line_id = sol.id
+        FROM account_move_line aml2
+        INNER JOIN account_invoice ai ON
+        ai.id = aml2.invoice_id
+        INNER JOIN account_invoice_line ail ON
+        ail.invoice_id = ai.id
+        INNER JOIN sale_order_line_invoice_rel rel ON
+        rel.invoice_line_id = ail.id
+        INNER JOIN sale_order_line sol ON
+        rel.order_line_id = sol.id
+        WHERE aml.id = aml2.id;
+    """)
+
+    # NOW we can fill the SO
+    cr.execute("""
+        UPDATE account_move_line aml
+        SET sale_id = sol.order_id
+        FROM sale_order_line AS sol
+        WHERE aml.sale_line_id = sol.id
+    """)

account_move_line_sale_info/models/__init__.py

@@ -0,0 +1,6 @@
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
+
+from . import account_move
+from . import account_invoice
+from . import sale_order_line
+from . import stock_move

account_move_line_sale_info/models/account_invoice.py

@@ -0,0 +1,28 @@
+# Copyright 2020 ForgeFlow S.L.
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
+
+from odoo import api, models
+
+
+class AccountInvoice(models.Model):
+
+    _inherit = 'account.invoice'
+
+    @api.model
+    def invoice_line_move_line_get(self):
+        res = super(AccountInvoice, self).invoice_line_move_line_get()
+
+        invoice_line_model = self.env['account.invoice.line']
+        for move_line_dict in res:
+            if 'invl_id' in move_line_dict:
+                line = invoice_line_model.browse(move_line_dict['invl_id'])
+                if line.sale_line_ids and len(line.sale_line_ids) == 1:
+                    move_line_dict['sale_line_id'] = line.sale_line_ids[0].id
+        return res
+
+    @api.model
+    def line_get_convert(self, line, part):
+        res = super(AccountInvoice, self).line_get_convert(line, part)
+        if line.get('sale_line_id', False):
+            res['sale_line_id'] = line.get('sale_line_id')
+        return res

account_move_line_sale_info/models/account_move.py

@@ -0,0 +1,23 @@
+# Copyright 2020 ForgeFlow S.L.
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
+
+from odoo import fields, models
+
+
+class AccountMoveLine(models.Model):
+
+    _inherit = 'account.move.line'
+
+    sale_line_id = fields.Many2one(
+        comodel_name='sale.order.line',
+        string='Sale Order Line',
+        ondelete='set null', index=True,
+    )
+
+    sale_id = fields.Many2one(
+        comodel_name='sale.order',
+        related='sale_line_id.order_id',
+        string='Sales Order',
+        ondelete='set null',
+        store=True, index=True,
+    )

account_move_line_sale_info/models/sale_order_line.py

@@ -0,0 +1,20 @@
+# Copyright 2020 ForgeFlow S.L.
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
+
+from odoo import api, models
+
+
+class SaleOrderLine(models.Model):
+    _inherit = "sale.order.line"
+
+    @api.multi
+    def name_get(self):
+        result = []
+        orig_name = dict(super(SaleOrderLine, self).name_get())
+        for line in self:
+            name = orig_name[line.id]
+            if self.env.context.get('so_line_info', False):
+                name = "[%s] %s (%s)" % (line.order_id.name, name,
+                                         line.order_id.state)
+            result.append((line.id, name))
+        return result

account_move_line_sale_info/models/stock_move.py

@@ -0,0 +1,16 @@
+# Copyright 2020 ForgeFlow S.L.
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
+from odoo import api, models
+
+
+class StockMove(models.Model):
+    _inherit = "stock.move"
+
+    @api.model
+    def _prepare_account_move_line(self, qty, cost,
+                                   credit_account_id, debit_account_id):
+        res = super(StockMove, self)._prepare_account_move_line(
+            qty, cost, credit_account_id, debit_account_id)
+        for line in res:
+            line[2]['sale_line_id'] = self.sale_line_id.id
+        return res

account_move_line_sale_info/security/account_security.xml

@@ -0,0 +1,10 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+
+    <record id="group_account_move_sale_info" model="res.groups">
+        <field name="name">Sale info in Journal Items</field>
+        <field name="category_id" ref="base.module_category_hidden"/>
+        <field name="users" eval="[(4, ref('base.user_root'))]"/>
+    </record>
+
+</odoo>

account_move_line_sale_info/tests/__init__.py

@@ -0,0 +1,3 @@
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
+
+from . import test_account_move_line_sale_info

account_move_line_sale_info/tests/test_account_move_line_sale_info.py

@@ -0,0 +1,210 @@
+# Copyright 2020 ForgeFlow S.L.
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
+from odoo.tests import common
+from odoo import fields
+
+
+class TestAccountMoveLineSaleInfo(common.TransactionCase):
+
+    def setUp(self):
+        super(TestAccountMoveLineSaleInfo, self).setUp()
+        self.sale_model = self.env['sale.order']
+        self.sale_line_model = self.env['sale.order.line']
+        self.invoice_model = self.env['account.invoice']
+        self.invoice_line_model = self.env['account.invoice.line']
+        self.product_model = self.env['product.product']
+        self.product_ctg_model = self.env['product.category']
+        self.acc_type_model = self.env['account.account.type']
+        self.account_model = self.env['account.account']
+        self.aml_model = self.env['account.move.line']
+        self.res_users_model = self.env['res.users']
+
+        self.partner1 = self.env.ref('base.res_partner_1')
+        self.location_stock = self.env.ref('stock.stock_location_stock')
+        self.company = self.env.ref('base.main_company')
+        self.group_sale_user = self.env.ref('sales_team.group_sale_salesman')
+        self.group_account_invoice = self.env.ref(
+            'account.group_account_invoice')
+        self.group_account_manager = self.env.ref(
+            'account.group_account_manager')
+
+        # Create account for Goods Received Not Invoiced
+        acc_type = self._create_account_type('equity', 'other')
+        name = 'Goods Received Not Invoiced'
+        code = 'grni'
+        self.account_grni = self._create_account(acc_type, name, code,
+                                                 self.company)
+
+        # Create account for Cost of Goods Sold
+        acc_type = self._create_account_type('expense', 'other')
+        name = 'Cost of Goods Sold'
+        code = 'cogs'
+        self.account_cogs = self._create_account(acc_type, name, code,
+                                                 self.company)
+        # Create account for Inventory
+        acc_type = self._create_account_type('asset', 'other')
+        name = 'Inventory'
+        code = 'inventory'
+        self.account_inventory = self._create_account(acc_type, name, code,
+                                                      self.company)
+        # Create Product
+        self.product = self._create_product()
+
+        # Create users
+        self.sale_user = self._create_user('sale_user',
+                                               [self.group_sale_user,
+                                                self.group_account_invoice],
+                                               self.company)
+        self.account_invoice = self._create_user('account_invoice',
+                                                 [self.group_account_invoice],
+                                                 self.company)
+        self.account_manager = self._create_user('account_manager',
+                                                 [self.group_account_manager],
+                                                 self.company)
+
+    def _create_user(self, login, groups, company):
+        """ Create a user."""
+        group_ids = [group.id for group in groups]
+        user = \
+            self.res_users_model.with_context(
+                {'no_reset_password': True}).create({
+                    'name': 'Test User',
+                    'login': login,
+                    'password': 'demo',
+                    'email': 'test@yourcompany.com',
+                    'company_id': company.id,
+                    'company_ids': [(4, company.id)],
+                    'groups_id': [(6, 0, group_ids)]
+                })
+        return user.id
+
+    def _create_account_type(self, name, type):
+        acc_type = self.acc_type_model.create({
+            'name': name,
+            'type': type
+        })
+        return acc_type
+
+    def _create_account(self, acc_type, name, code, company):
+        """Create an account."""
+        account = self.account_model.create({
+            'name': name,
+            'code': code,
+            'user_type_id': acc_type.id,
+            'company_id': company.id
+        })
+        return account
+
+    def _create_product(self):
+        """Create a Product."""
+        #        group_ids = [group.id for group in groups]
+        product_ctg = self.product_ctg_model.create({
+            'name': 'test_product_ctg',
+            'property_stock_valuation_account_id': self.account_inventory.id,
+            'property_valuation': 'real_time',
+            'property_stock_account_input_categ_id': self.account_grni.id,
+            'property_stock_account_output_categ_id': self.account_cogs.id,
+        })
+        product = self.product_model.create({
+            'name': 'test_product',
+            'categ_id': product_ctg.id,
+            'type': 'product',
+            'standard_price': 1.0,
+            'list_price': 1.0,
+        })
+        return product
+
+    def _create_sale(self, line_products):
+        """ Create a sale order.
+
+        ``line_products`` is a list of tuple [(product, qty)]
+        """
+        lines = []
+        for product, qty in line_products:
+            line_values = {
+                'name': product.name,
+                'product_id': product.id,
+                'product_qty': qty,
+                'product_uom': product.uom_id.id,
+                'price_unit': 500,
+                'date_planned': fields.datetime.now()
+            }
+            lines.append(
+                (0, 0, line_values)
+            )
+        return self.sale_model.create({
+            'partner_id': self.partner1.id,
+            'order_line': lines
+        })
+
+    def _get_balance(self, domain):
+        """
+        Call read_group method and return the balance of particular account.
+        """
+        aml_rec = self.aml_model.read_group(domain,
+                                            ['debit', 'credit', 'account_id'],
+                                            ['account_id'])
+        if aml_rec:
+            return aml_rec[0].get('debit', 0) - aml_rec[0].get('credit', 0)
+        else:
+            return 0.0
+
+    def _check_account_balance(self, account_id, sale_line=None,
+                               expected_balance=0.0):
+        """
+        Check the balance of the account
+        """
+        domain = [('account_id', '=', account_id)]
+        if sale_line:
+            domain.extend([('sale_line_id', '=', sale_line.id)])
+
+        balance = self._get_balance(domain)
+        if sale_line:
+            self.assertEqual(balance, expected_balance,
+                             'Balance is not %s for sale Line %s.'
+                             % (str(expected_balance), sale_line.name))
+
+    def test_sale_invoice(self):
+        """Test that the po line moves from the sale order to the
+        account move line and to the invoice line.
+        """
+        sale = self._create_sale([(self.product, 1)])
+        po_line = False
+        for line in sale.order_line:
+            po_line = line
+            break
+        sale.button_confirm()
+        picking = sale.picking_ids[0]
+        picking.force_assign()
+        picking.move_lines.write({'quantity_done': 1.0})
+        picking.button_validate()
+
+        expected_balance = 1.0
+        self._check_account_balance(self.account_inventory.id,
+                                    sale_line=po_line,
+                                    expected_balance=expected_balance)
+
+        invoice = self.invoice_model.create({
+            'partner_id': self.partner1.id,
+            'sale_id': sale.id,
+            'account_id': sale.partner_id.property_account_payable_id.id,
+        })
+        invoice.sale_order_change()
+        invoice.action_invoice_open()
+
+        for aml in invoice.move_id.line_ids:
+            if aml.product_id == po_line.product_id and aml.invoice_id:
+                self.assertEqual(aml.sale_line_id, po_line,
+                                 'sale Order line has not been copied '
+                                 'from the invoice to the account move line.')
+
+    def test_name_get(self):
+        sale = self._create_sale([(self.product, 1)])
+        po_line = sale.order_line[0]
+        name_get = po_line.with_context({'po_line_info': True}).name_get()
+        self.assertEqual(name_get, [(po_line.id, "[%s] %s (%s)" % (
+            po_line.order_id.name, po_line.name,
+            po_line.order_id.state,
+        ))])
+        name_get_no_ctx = po_line.name_get()
+        self.assertEqual(name_get_no_ctx, [(po_line.id, po_line.name)])

account_move_line_sale_info/views/account_move_view.xml

@@ -0,0 +1,70 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+
+    <record id="view_move_line_form" model="ir.ui.view">
+        <field name="name">account.move.line.form</field>
+        <field name="model">account.move.line</field>
+        <field name="inherit_id" ref="account.view_move_line_form"/>
+        <field name="arch" type="xml">
+            <field name="quantity" position="before">
+                <field name="sale_id"
+                       groups="account_move_line_sale_info.group_account_move_sale_info"/>
+                <field name="sale_line_id"
+                       groups="account_move_line_sale_info.group_account_move_sale_info"/>
+            </field>
+        </field>
+    </record>
+
+    <record id="view_move_line_tree" model="ir.ui.view">
+        <field name="name">account.move.line.tree</field>
+        <field name="model">account.move.line</field>
+        <field name="inherit_id" ref="account.view_move_line_tree"/>
+        <field name="arch" type="xml">
+            <field name="partner_id" position="after">
+                <field name="sale_id"
+                       groups="account_move_line_sale_info.group_account_move_sale_info"/>
+                <field name="sale_line_id"
+                       groups="account_move_line_sale_info.group_account_move_sale_info"/>
+            </field>
+        </field>
+    </record>
+
+
+    <record id="view_account_move_line_filter" model="ir.ui.view">
+        <field name="name">Journal Items</field>
+        <field name="model">account.move.line</field>
+        <field name="inherit_id" ref="account.view_account_move_line_filter"/>
+        <field name="arch" type="xml">
+            <field name="partner_id" position="after">
+                <field name="sale_line_id"
+                groups="account_move_line_sale_info.group_account_move_sale_info"/>
+                <field name="sale_id"
+                groups="account_move_line_sale_info.group_account_move_sale_info"/>
+            </field>
+            <group expand="0" position="inside">
+                 <filter string="sale Order" domain="[]"
+                        context="{'group_by':'sale_id'}"
+                        groups="account_move_line_sale_info.group_account_move_sale_info"/>
+                <filter string="sale Order Line" domain="[]"
+                        context="{'group_by':'sale_line_id','so_line_info':True}"
+                        groups="account_move_line_sale_info.group_account_move_sale_info"/>
+            </group>
+        </field>
+    </record>
+
+
+    <record id="view_move_form" model="ir.ui.view">
+        <field name="name">account.move.form</field>
+        <field name="model">account.move</field>
+        <field name="inherit_id" ref="account.view_move_form"/>
+        <field name="arch" type="xml">
+            <xpath expr="//field[@name='line_ids']/tree//field[@name='partner_id']" position="after">
+                <field name="sale_line_id" context="{'so_line_info': True}"
+                       groups="account_move_line_sale_info.group_account_move_sale_info"/>
+                <field name="sale_id"
+                       groups="account_move_line_sale_info.group_account_move_sale_info"/>
+            </xpath>
+        </field>
+    </record>
+
+</odoo>