Merge PR #3674 into 16.0

Signed-off-by HaraldPanten

Author: OCA-git-bot

sale_quotation_number/__manifest__.py

@@ -6,7 +6,7 @@
 {
     "name": "Sale Quotation Numeration",
     "summary": "Different sequence for sale quotations",
-    "version": "16.0.1.1.1",
+    "version": "16.0.1.1.2",
     "category": "Sales Management",
     "website": "https://github.com/OCA/sale-workflow",
     "author": "Elico Corp, "

sale_quotation_number/migrations/16.0.1.1.2/post-migration.py

@@ -0,0 +1,20 @@
+# Copyright 2023 Manuel Regidor  <manuel.regidor@sygel.es> (Sygel)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl)
+
+import odoo
+
+
+def migrate(cr, version):
+    env = odoo.api.Environment(cr, odoo.SUPERUSER_ID, {})
+    sequence = env["ir.sequence"].search([("code", "=", "sale.quotation")], limit=1)
+    orders = (
+        env["sale.order"]
+        .search(
+            [
+                ("state", "in", ["draft", "sent"]),
+                ("company_id.keep_name_so", "=", False),
+            ]
+        )
+        .filtered(lambda a: a.name[: len(sequence.prefix)] == sequence.prefix)
+    )
+    orders.write({"quotation_seq_used": True})

sale_quotation_number/models/sale_order.py

@@ -4,19 +4,23 @@
 # © 2020 Manuel Regidor  <manuel.regidor@sygel.es>
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl)
 
-from odoo import api, models
+from odoo import api, fields, models
 
 
 class SaleOrder(models.Model):
     _inherit = "sale.order"
 
+    quotation_seq_used = fields.Boolean(
+        string="Quotation Sequence Used", default=False, copy=False, readonly=True
+    )
+
     @api.model_create_multi
     def create(self, vals_list):
         for vals in vals_list:
             if self.is_using_quotation_number(vals):
                 company_id = vals.get("company_id", self.env.company.id)
                 sequence = self.with_company(company_id).get_quotation_seq()
-                vals["name"] = sequence or "/"
+                vals.update({"name": sequence or "/", "quotation_seq_used": True})
         return super().create(vals_list)
 
     @api.model
@@ -42,23 +46,23 @@ def copy(self, default=None):
     def get_quotation_seq(self):
         return self.env["ir.sequence"].next_by_code("sale.quotation")
 
+    def get_sale_order_seq(self):
+        self.ensure_one()
+        return self.env["ir.sequence"].next_by_code("sale.order")
+
     def action_confirm(self):
         sequence = self.env["ir.sequence"].search(
             [("code", "=", "sale.quotation")], limit=1
         )
         for order in self:
-            if sequence and self.name[: len(sequence.prefix)] != sequence.prefix:
+            if not self.quotation_seq_used:
                 continue
             if order.state not in ("draft", "sent") or order.company_id.keep_name_so:
                 continue
             if order.origin and order.origin != "":
                 quo = order.origin + ", " + order.name
             else:
                 quo = order.name
-            sequence = (
-                self.with_company(order.company_id.id)
-                .env["ir.sequence"]
-                .next_by_code("sale.order")
-            )
-            order.write({"origin": quo, "name": sequence})
+            sequence = order.with_company(order.company_id.id).get_sale_order_seq()
+            order.write({"origin": quo, "name": sequence, "quotation_seq_used": False})
         return super().action_confirm()