[IMP] last_ordered_products: Bridge modules created

- Created 3 other bridge modules to ensure that it will work if any individual
  module is installed.
- Updated main module, Now it has only helper methods and compute method.

Author: ppch-odoo

last_ordered_products/__manifest__.py

@@ -1,17 +1,13 @@
 {
     'name': "Last Ordered Products",
     'version': '1.0',
-    'depends': ['sale_management', 'purchase', 'stock'],
+    'depends': ['product'],
     'author': "Parthav Chodvadiya (PPCH)",
     'category': '',
     'description': """
     Show last ordered products for customers in sale order and for vendors in purchase order
     """,
     'data': [
-        'data/last_ordered_products_tour.xml',
-        'views/account_move_form.xml',
-        'views/sale_order_form.xml',
-        'views/purchase_order_form.xml',
         'views/product_views.xml',
     ],
     'assets': {

last_ordered_products/data/last_ordered_products_tour.xml

@@ -1,4 +1,2 @@
 from . import product_product
 from . import product_template
-from . import sale_order
-from . import sale_order_line

last_ordered_products/models/__init__.py

@@ -1,6 +1,4 @@
-from datetime import datetime
 from odoo import api, fields, models
-from odoo.osv import expression
 
 
 class ProductProduct(models.Model):
@@ -11,7 +9,7 @@ class ProductProduct(models.Model):
 
     @api.depends_context('order_id')
     def _compute_last_order_time(self):
-        """Compute the last order time for each product based on the latest sale or purchase."""
+        '''Compute the last order time for each product based on the latest sale or purchase.'''
 
         order_type = False
         if self.env.context.get('active_model') == 'sale.order.line':
@@ -24,10 +22,7 @@ def _compute_last_order_time(self):
             active_id = self.env.context.get('active_id')
             if active_id:
                 order_type = self.env['account.journal'].browse(active_id).type
-            partner_id = self.env.context.get('partner_id') or self.env.context.get('default_partner_id')
-        else:
-            partner_id = self.env.context.get('partner_id')
-            order_type = self.env.context.get('order_type')
+            partner_id = self.env.context.get('default_partner_id')
 
         if not partner_id:
             for record in self:
@@ -99,34 +94,3 @@ def _get_last_purchased_products(self, partner_id):
                     last_purchased_order_products[product_id] = last_date
 
         return last_purchased_order_products
-
-    @api.model
-    def name_search(self, name='', args=None, operator='ilike', limit=100):
-        '''Modify product dropdown in sale order line to show last sold date'''
-
-        domain = args or []
-        partner_id = self.env.context.get('partner_id')
-        order_type = self.env.context.get('order_type')
-        active_id = self.env.context.get('active_id')
-        if not order_type and active_id:
-            order_type = self.env['account.journal'].browse(active_id).type
-
-        if partner_id:
-            last_ordered_products = {}
-            if order_type == 'sale':
-                last_ordered_products = self._get_last_sold_products(partner_id)
-            elif order_type == 'purchase':
-                last_ordered_products = self._get_last_purchased_products(partner_id)
-
-            product_ids = list(last_ordered_products.keys())
-
-            products = self.search_fetch(expression.AND([domain, [('id', 'in', product_ids)], [("name", operator, name)]]), ['display_name'], limit=limit)
-            limit_rest = limit and limit - len(products)
-            if limit_rest is None or limit_rest > 0:
-                products |= self.search_fetch(expression.AND([domain, [('id', 'not in', product_ids)], [("name", operator, name)]]), ['display_name'], limit=limit_rest)
-
-            products = sorted(products, key=lambda p: last_ordered_products.get(p.id, datetime.min), reverse=True)
-
-            return [(product.id, product.display_name, self.env['product.template']._get_time_ago_string(last_ordered_products.get(product.id, False))) for product in products]
-
-        return super().name_search(name, args, operator, limit)

last_ordered_products/models/product_product.py

@@ -1,6 +1,4 @@
-from datetime import datetime
 from odoo import api, fields, models
-from odoo.osv import expression
 
 
 class ProductTemplate(models.Model):
@@ -11,7 +9,7 @@ class ProductTemplate(models.Model):
 
     @api.depends_context('order_id')
     def _compute_last_order_time(self):
-        """Compute the last order time for each product based on the latest sale or purchase."""
+        '''Compute the last order time for each product based on the latest sale or purchase.'''
 
         partner_id = self.env.context.get('partner_id')
         order_type = self.env.context.get('order_type')
@@ -80,32 +78,3 @@ def _get_time_ago_string(self, last_date):
             return f"{diff.seconds // 60}m"
         else:
             return f"{diff.seconds}s"
-
-    @api.model
-    def name_search(self, name='', args=None, operator='ilike', limit=100):
-        '''Modify product dropdown in sale order line to show last sold date'''
-
-        domain = args or []
-        partner_id = self.env.context.get('partner_id')
-        order_type = self.env.context.get('order_type')
-        active_id = self.env.context.get('active_id')
-        if not order_type and active_id:
-            order_type = self.env['account.journal'].browse(active_id).type
-
-        if partner_id:
-            last_ordered_products = {}
-            if order_type == 'sale':
-                last_ordered_products = self._get_last_sold_products(partner_id)
-
-            product_ids = list(last_ordered_products.keys())
-
-            products = self.search_fetch(expression.AND([domain, [('id', 'in', product_ids)], [("name", operator, name)]]), ['display_name'], limit=limit)
-            limit_rest = limit and limit - len(products)
-            if limit_rest is None or limit_rest > 0:
-                products |= self.search_fetch(expression.AND([domain, [('id', 'not in', product_ids)], [("name", operator, name)]]), ['display_name'], limit=limit_rest)
-
-            products = sorted(products, key=lambda p: p.last_order_time if p.last_order_time else datetime.min, reverse=True)
-
-            return [(product.id, product.display_name, product.last_date_str if product.last_date_str else False) for product in products]
-
-        return super().name_search(name, args, operator, limit)

last_ordered_products/models/product_template.py

@@ -3,9 +3,9 @@
     <t t-inherit="web.AutoComplete" t-inherit-mode="extension">
         <xpath expr="//t[@t-esc='option.label']" position="replace">
             <t t-if="option.time_str">
-                <div class="d-flex justify-content-between">
+                <div class="d-flex justify-content-between text-success">
                     <span><t t-esc="option.label" /></span>
-                    <span><t t-esc="option.time_str" /></span>
+                    <span class="px-2"><t t-esc="option.time_str" /></span>
                 </div>
             </t>
             <t t-else="">

last_ordered_products/static/src/core/autocomplete/autocomplete.xml

@@ -5,6 +5,9 @@ patch(ProductLabelSectionAndNoteFieldAutocomplete.prototype, {
     mapRecordToOption(result) {
         let res = super.mapRecordToOption(result)
         let time_str = result[2] ? result[2] : ""
+        if (time_str === ""){
+            return res;
+        }
         res['time_str'] = time_str
         return res
     },

last_ordered_products/static/src/js/tours/last_ordered_products.js

@@ -146,7 +146,7 @@ def test_product_variant_in_sale_order(self):
         res_ids = [r[0] for r in res]
         self.assertEqual(self.test_product_1.id, res_ids[0])
         self.assertEqual(self.test_product_2.id, res_ids[1])
-        
+
     def test_product_variant_in_purchase_order(self):
         po_context = {
             'partner_id': self.test_partner_1.id,

last_ordered_products/static/src/views/fields/product_label_section_and_note_field.js

@@ -10,23 +10,8 @@
                     <field name="default_code"/>
                 </div>
             </xpath>
-
-            <xpath expr="//div[@name='o_kanban_qty_available']" position="replace">
-                <div t-if="record.is_storable.raw_value"
-                    name="o_kanban_qty_available">
-                    <field name="qty_available"/> On Hand
-                    <field name="virtual_available" invisible="1"/>
-                    <span t-if="record.virtual_available.raw_value - record.qty_available.raw_value >= 0" class="text-success">
-                        <span>(+</span><t t-out="record.virtual_available.raw_value - record.qty_available.raw_value"/><span>)</span>
-                    </span>
-                    <span t-else="" class="text-danger">
-                        <span>(</span><t t-out="record.virtual_available.raw_value - record.qty_available.raw_value"/><span>)</span>
-                    </span>
-                </div>
-            </xpath>
-            
             <field name="product_template_attribute_value_ids" position="after">
-                <span invisible="not last_date_str"><field name="last_date_str"/> ago</span>
+                <span invisible="not last_date_str" class="text-success"><field name="last_date_str"/> ago</span>
                 <span invisible="true"><field name="last_order_time"/></span>
             </field>
         </field>