[FEAT] subscription_oca: enable markers to print invoice names

(like in _contract_)

Author: PicchiSeba

Diff for: subscription_oca/models/sale_subscription.py

@@ -277,15 +277,18 @@ def _prepare_account_move(self, line_ids):
         return values
 
     def create_invoice(self):
+        self.ensure_one()
         if not self.env["account.move"].check_access_rights("create", False):
             try:
                 self.check_access_rights("write")
                 self.check_access_rule("write")
             except AccessError:
                 return self.env["account.move"]
         line_ids = []
+        start_date = self.recurring_next_date or self.date_start
+        end_date = self._get_next_period_date_end(start_date, self.date)
         for line in self.sale_subscription_line_ids:
-            line_values = line._prepare_account_move_line()
+            line_values = line._prepare_account_move_line(start_date, end_date)
             line_ids.append((0, 0, line_values))
         invoice_values = self._prepare_account_move(line_ids)
         invoice_id = (
@@ -305,8 +308,10 @@ def create_sale_order(self):
             except AccessError:
                 return self.env["sale.order"]
         line_ids = []
+        start_date = self.recurring_next_date or self.date_start
+        end_date = self._get_next_period_date_end(start_date, self.date)
         for line in self.sale_subscription_line_ids:
-            line_values = line._prepare_sale_order_line()
+            line_values = line._prepare_sale_order_line(start_date, end_date)
             line_ids.append((0, 0, line_values))
         values = self._prepare_sale_order(line_ids)
         order_id = self.env["sale.order"].sudo().create(values)
@@ -431,6 +436,27 @@ def _check_dates(self, start, next_invoice):
                 return True
         return False
 
+    def _get_next_period_date_end(
+        self,
+        next_period_date_start,
+        max_date_end,
+    ):
+        self.ensure_one()
+        if not next_period_date_start:
+            return False
+        if max_date_end and next_period_date_start > max_date_end:
+            return False
+        next_period_date_end = (
+            next_period_date_start
+            + self.template_id.get_relative_delta()
+            - relativedelta(days=1)
+        )
+        if not max_date_end:
+            return next_period_date_end
+        if next_period_date_end > max_date_end:
+            return max_date_end
+        return next_period_date_end
+
     def write(self, values):
         res = super().write(values)
         if "stage_id" in values:

Diff for: subscription_oca/models/sale_subscription_line.py

@@ -1,6 +1,6 @@
 # Copyright 2023 Domatix - Carlos Martínez
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
-from odoo import api, fields, models
+from odoo import _, api, fields, models
 from odoo.tools.misc import get_lang
 
 
@@ -290,11 +290,11 @@ def _get_display_price(self, product):
             )
         return max(base_price, final_price)
 
-    def _prepare_sale_order_line(self):
+    def _prepare_sale_order_line(self, start_date, end_date):
         self.ensure_one()
         return {
             "product_id": self.product_id.id,
-            "name": self.name,
+            "name": self._generate_name(start_date, end_date),
             "product_uom_qty": self.product_uom_qty,
             "price_unit": self.price_unit,
             "discount": self.discount,
@@ -303,15 +303,15 @@ def _prepare_sale_order_line(self):
             "product_uom": self.product_id.uom_id.id,
         }
 
-    def _prepare_account_move_line(self):
+    def _prepare_account_move_line(self, start_date, end_date):
         self.ensure_one()
         account = (
             self.product_id.property_account_income_id
             or self.product_id.categ_id.property_account_income_categ_id
         )
         return {
             "product_id": self.product_id.id,
-            "name": self.name,
+            "name": self._generate_name(start_date, end_date),
             "quantity": self.product_uom_qty,
             "price_unit": self.price_unit,
             "discount": self.discount,
@@ -320,3 +320,43 @@ def _prepare_account_move_line(self):
             "product_uom_id": self.product_id.uom_id.id,
             "account_id": account.id,
         }
+
+    def _generate_name(self, start_date, end_date):
+        return self._insert_markers(start_date, end_date)
+
+    def _insert_markers(self, start_date, end_date):
+        self.ensure_one()
+        lang_obj = self.env["res.lang"]
+        lang = lang_obj.search(
+            [("code", "=", self.sale_subscription_id.partner_id.lang)]
+        )
+        date_format = lang.date_format or "%m/%d/%Y"
+        name = self.name
+        name = name.replace("#START#", start_date.strftime(date_format))
+        name = name.replace("#END#", end_date.strftime(date_format)) if end_date else ""
+        name = name.replace("#INVOICEMONTHNUMBER#", start_date.strftime("%m"))
+        name = name.replace("#INVOICEYEAR#", start_date.strftime("%Y"))
+        name = name.replace(
+            "#INVOICEMONTHNAME#",
+            self.with_context(lang=lang.code)._translate_marker_month_name(
+                start_date.strftime("%m")
+            ),
+        )
+        return name
+
+    def _translate_marker_month_name(self, month_name):
+        months = {
+            "01": _("January"),
+            "02": _("February"),
+            "03": _("March"),
+            "04": _("April"),
+            "05": _("May"),
+            "06": _("June"),
+            "07": _("July"),
+            "08": _("August"),
+            "09": _("September"),
+            "10": _("October"),
+            "11": _("November"),
+            "12": _("December"),
+        }
+        return months[month_name]

Diff for: subscription_oca/models/sale_subscription_template.py

@@ -100,3 +100,16 @@ def action_view_product_ids(self):
             "type": "ir.actions.act_window",
             "domain": [("id", "in", self.product_ids.ids)],
         }
+
+    def get_relative_delta(self):
+        self.ensure_one()
+        rule_type = self.recurring_rule_type
+        interval = self.recurring_interval
+        if rule_type == "days":
+            return relativedelta(days=interval)
+        elif rule_type == "weeks":
+            return relativedelta(weeks=interval)
+        elif rule_type == "months":
+            return relativedelta(months=interval)
+        else:
+            return relativedelta(years=interval)

Diff for: subscription_oca/tests/test_subscription_oca.py

@@ -320,9 +320,11 @@ def test_subscription_oca_sub_lines(self):
         self.assertEqual(self.sub_line.discount, 0)
         res = self.sub_line._get_display_price(self.product_2)
         self.assertEqual(res, 38.25)
-        sol_res = self.sub_line._prepare_sale_order_line()
+        date_start = self.sub_line.sale_subscription_id.date_start
+        date_end = self.sub_line.sale_subscription_id.date
+        sol_res = self.sub_line._prepare_sale_order_line(date_start, date_end)
         self.assertIsInstance(sol_res, dict)
-        move_res = self.sub_line._prepare_account_move_line()
+        move_res = self.sub_line._prepare_account_move_line(date_start, date_end)
         self.assertIsInstance(move_res, dict)
 
     def test_subscription_oca_sub_cron(self):

Diff for: subscription_oca/views/sale_subscription_views.xml

@@ -192,6 +192,40 @@
                                     <field name="user_id" />
                                 </group>
                             </group>
+                            <group
+                                string="Legend (for the markers inside invoice lines description)"
+                                name="group_legend"
+                            >
+                                <p colspan="2">
+                                    <strong>#START#</strong>
+                                    : Start
+                                    date
+                                    of the
+                                    invoiced period
+                                </p>
+                                <p colspan="2">
+                                    <strong>#END#</strong>
+                                    : End date
+                                    of
+                                    the
+                                    invoiced period
+                                </p>
+                                <p colspan="2">
+                                    <strong>#INVOICEMONTHNAME#</strong>
+                                    : Invoice month name
+                                    of
+                                    the
+                                    invoiced period
+                                </p>
+                                <p colspan="2">
+                                    <strong
+                                    >#INVOICEMONTHNUMBER#</strong>: Invoice month number of the period
+                                </p>
+                                <p colspan="2">
+                                    <strong
+                                    >#INVOICEYEAR#</strong>: Invoice year of the period
+                                </p>
+                            </group>
                         </page>
                     </notebook>