[IMP] products_orderby_invoice: Imp search and catalog based on invoice history

Purpose:
Make it easier to find and show products based on past invoices or bills.

Approach:
Changed name_search to list first the products already invoiced to the customer,
sorted by latest date.
Updated product catalog view to show last invoice date, moved info around, and
added forecasted qty with  color.

Impact:
Improves product search and helps users quickly spot stock levels and recent
purchases.

Author: metu-odoo

products_orderby_invoice/__manifest__.py

@@ -8,4 +8,5 @@
         'views/product_views.xml',
     ],
     'installable': True,
+    'license': 'LGPL-3',
 }

products_orderby_invoice/models/product_product.py

@@ -1,10 +1,23 @@
+# Part of Odoo. See LICENSE file for full copyright and licensing details.
+
 from odoo import api, fields, models
 
+
 class ProductProduct(models.Model):
     _inherit = 'product.product'
 
-
     surplus_qty = fields.Float(string='Surplus Quantity', compute='_compute_surplus_qty')
+    invoice_date = fields.Datetime(string='Last Invoice Date', compute='_compute_invoice_date')
+
+    @api.depends('invoice_date')
+    def _compute_invoice_date(self):
+        for rec in self:
+            lines = self.env['account.move.line'].search([
+                ('product_id', '=', rec.id),
+                ('move_id.move_type', '=', 'out_invoice'),
+                ('move_id.state', '=', 'posted'),
+            ], limit=1, order='date desc')
+            rec.invoice_date = lines.move_id.date if lines else False
 
     @api.depends('virtual_available', 'qty_available')
     def _compute_surplus_qty(self):
@@ -13,7 +26,6 @@ def _compute_surplus_qty(self):
 
     @api.model
     def name_search(self, name, args=None, operator='ilike', limit=100):
-        print("===============This is triggered====================")
         args = list(args) if args else []
         partner_id = self.env.context.get('partner_id')
         results = []
@@ -29,15 +41,12 @@ def name_search(self, name, args=None, operator='ilike', limit=100):
 
             lines = sorted(lines, key=lambda l: l.move_id.invoice_date or l.create_date, reverse=True)
 
-            name_lower = name.lower() if name else ''
-
             for line in lines:
                 product = line.product_id
                 if product.id in matched_ids:
                     continue
-                if not name or (operator == 'ilike' and name_lower in product.name.lower()):
-                    results.append((product.id, product.display_name))
-                    matched_ids.add(product.id)
+                results.append((product.id, product.display_name))
+                matched_ids.add(product.id)
                 if len(results) >= limit:
                     break
 
@@ -47,7 +56,7 @@ def name_search(self, name, args=None, operator='ilike', limit=100):
             if name:
                 domain.append(('name', operator, name))
             if matched_ids:
-                domain.append(('id', 'not in', list(matched_ids)))  # needs to be a list here
+                domain.append(('id', 'not in', list(matched_ids)))
             others = super().name_search(name, domain, operator=operator, limit=remaining)
             results.extend(others)
 

products_orderby_invoice/models/product_template.py

@@ -1,5 +1,6 @@
+# Part of Odoo. See LICENSE file for full copyright and licensing details.
+
 from odoo import models, fields, api
-from datetime import date
 
 
 class ProductTemplate(models.Model):
@@ -10,48 +11,35 @@ def name_search(self, name, args=None, operator="ilike", limit=100):
         args = list(args) if args else []
         partner_id = self.env.context.get("partner_id")
         results = []
-        matched_ids = []
+        seen_ids = set()
 
-        lines = self.env["account.move.line"].search([
+        if partner_id:
+            lines = self.env["account.move.line"].search([
                 ("move_id.move_type", "=", "out_invoice"),
                 ("move_id.partner_id", "=", partner_id),
                 ("move_id.state", "=", "posted"),
                 ("product_id.product_tmpl_id", "!=", False),
             ])
-
-        lines = sorted(
-            lines, key=lambda l: l.move_id.invoice_date or fields.Date.today(),
-            reverse=True)
-
-        tmpl_map = {}
-        for line in lines:
-            tmpl = line.product_id.product_tmpl_id
-            if tmpl.id not in tmpl_map:
-                tmpl_map[tmpl.id] = {"tmpl": tmpl, "date": line.move_id.invoice_date}
-            if len(tmpl_map) >= limit:
-                break
-
-        name_lower = name.lower() if name else ""
-        today = date.today()
-
-        for info in tmpl_map.values():
-            tmpl = info["tmpl"]
-            invoice_date = info["date"]
-            if not name or (operator == "ilike" and name_lower in tmpl.name.lower()):
-                days = (today - invoice_date).days if invoice_date else "?"
-                display = f"{tmpl.display_name} (Last ordered {days} days ago)"
+            lines = sorted(lines, key=lambda l: l.move_id.invoice_date or fields.Date.today(), reverse=True)
+
+            for line in lines:
+                tmpl = line.product_id.product_tmpl_id
+                if tmpl.id in seen_ids:
+                    continue
+                days = line.product_id.product_tmpl_id.sale_delay
+                display = f"{tmpl.display_name} (Order Lead time {days} days)"
                 results.append((tmpl.id, display))
-                matched_ids.append(tmpl.id)
-            if len(results) >= limit:
-                break
+                seen_ids.add(tmpl.id)
+                if len(results) >= limit:
+                    break
 
         remaining = limit - len(results)
         if remaining > 0:
             domain = args[:]
             if name:
                 domain.append(("name", operator, name))
-            if matched_ids:
-                domain.append(("id", "not in", matched_ids))
+            if seen_ids:
+                domain.append(("id", "not in", list(seen_ids)))
             others = super().name_search(name, args=domain, operator=operator, limit=remaining)
             results.extend(others)
 

products_orderby_invoice/views/product_views.xml

@@ -1,15 +1,29 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <odoo>
     <record id="product_view_kanban_catalog_surplus_qty" model="ir.ui.view">
-            <field name="name">product.view.kanban.catalog.inherit.surplus.qty</field>
-            <field name="model">product.product</field>
-            <field name="inherit_id" ref="product.product_view_kanban_catalog"/>
-            <field name="arch" type="xml">
-                <xpath expr="//div[@name='o_kanban_qty_available']/field[@name='uom_id']" position="after">
-                    <span> (</span>
-                    <field name="surplus_qty"></field>
-                    <span>)</span>
-                </xpath>
-            </field>
-        </record>
+        <field name="name">product.view.kanban.catalog.inherit.surplus.qty</field>
+        <field name="model">product.product</field>
+        <field name="inherit_id" ref="product.product_view_kanban_catalog" />
+        <field name="arch" type="xml">
+            <xpath expr="//div[@name='o_kanban_qty_available']/field[@name='uom_id']"
+                position="after">
+                <span> (</span>
+                <t t-if="record.surplus_qty.raw_value &gt; 0">
+                    <span class="text-success">+<field name="surplus_qty" /></span>
+                </t>
+                <t t-elif="record.surplus_qty.raw_value &lt; 0">
+                    <span class="text-danger">
+                        <field name="surplus_qty" />
+                    </span>
+                </t>
+                <t t-else="">
+                    <span class="text-muted">0.00</span>
+                </t>
+                <span>)</span>
+                <div>
+                    Last invoice date :<field name="invoice_date" widget="remaining_days"/>
+                </div>
+            </xpath>
+        </field>
+    </record>
 </odoo>