[18.0] [FW] [IMP] fieldservice_sale: Only invoice invoiceable fsm order

.oca/oca-port/blacklist/fieldservice_account.json

@@ -0,0 +1,6 @@
+{
+  "pull_requests": {
+    "OCA/field-service#997": "dotfiles",
+    "OCA/field-service#999": "(auto) Nothing to port from PR #999"
+  }
+}

.oca/oca-port/blacklist/fieldservice_sale.json

@@ -0,0 +1,6 @@
+{
+  "pull_requests": {
+    "OCA/field-service#931": "(auto) Nothing to port from PR #931",
+    "OCA/field-service#1122": "(auto) Nothing to port from PR #1122"
+  }
+}

fieldservice_account/models/fsm_stage.py

@@ -15,4 +15,4 @@ def _get_invoiceable_stage(self):
         by other criteria
         :return:
         """
-        return self.search(["is_invoiceable", "=", "True"])
+        return self.search([("is_invoiceable", "=", "True")])

fieldservice_sale/models/sale_order_line.py

@@ -53,8 +53,31 @@
                 line.order_id._field_service_generation()
         return lines
 
+    def _get_invoiceable_fsm_order_domain(self):
+        """
+        Override this method to define more search criteria for invoiceable
+        fsm order
+        :return:
+        """
+        invoiceable_stage_ids = self.env["fsm.stage"]._get_invoiceable_stage()
+        dom = [
+            "|",
+            ("sale_line_id", "=", self.id),
+            ("sale_id", "=", self.order_id.id),
+            ("invoice_lines", "=", False),
+        ]
+        if invoiceable_stage_ids:
+            dom.append(("stage_id", "in", invoiceable_stage_ids.ids))
+        return dom
+
+    def _get_invoiceable_fsm_order(self):
+        dom = self._get_invoiceable_fsm_order_domain()
+        return self.env["fsm.order"].search(dom)
+
     def _prepare_invoice_line(self, **optional_values):
         res = super()._prepare_invoice_line(**optional_values)
         if self.fsm_order_id:
-            res.update({"fsm_order_ids": [(4, self.fsm_order_id.id)]})
+            fsm_orders = self._get_invoiceable_fsm_order()
+            if fsm_orders:
+                res.update({"fsm_order_ids": [(6, 0, fsm_orders.ids)]})
         return res