Merge PR #1314 into 17.0

Signed-off-by brian10048

Author: OCA-git-bot

.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_account_analytic/demo/fsm_location.xml

@@ -1,16 +1,46 @@
 <odoo>
+
     <!-- fsm.location -->
+    <record id="analytic_account_test_location" model="account.analytic.account">
+        <field name="code">FSM-TEST</field>
+        <field name="name">FSM Test Location</field>
+        <field name="plan_id" ref="analytic.analytic_plan_internal" />
+    </record>
     <record id="fieldservice.test_location" model="fsm.location">
         <field name="customer_id" eval="ref('fieldservice.test_loc_partner')" />
+        <field name="analytic_account_id" ref="analytic_account_test_location" />
     </record>
+
     <!-- Other locations -->
+
+    <record id="analytic_account_location_1" model="account.analytic.account">
+        <field name="code">FSM-LOC1</field>
+        <field name="name">FSM Test Location 1</field>
+        <field name="plan_id" ref="analytic.analytic_plan_internal" />
+    </record>
     <record id="fieldservice.location_1" model="fsm.location">
         <field name="customer_id" eval="ref('fieldservice.location_partner_1')" />
+        <field name="analytic_account_id" ref="analytic_account_location_1" />
+    </record>
+
+    <record id="analytic_account_location_2" model="account.analytic.account">
+        <field name="code">FSM-LOC2</field>
+        <field name="name">FSM Test Location 3</field>
+        <field name="plan_id" ref="analytic.analytic_plan_internal" />
     </record>
     <record id="fieldservice.location_2" model="fsm.location">
         <field name="customer_id" eval="ref('fieldservice.location_partner_2')" />
+        <field name="analytic_account_id" ref="analytic_account_location_2" />
+    </record>
+
+    <record id="analytic_account_location_3" model="account.analytic.account">
+        <field name="code">FSM-LOC3</field>
+        <field name="name">FSM Test Location 3</field>
+        <field name="plan_id" ref="analytic.analytic_plan_internal" />
     </record>
     <record id="fieldservice.location_3" model="fsm.location">
         <field name="customer_id" eval="ref('fieldservice.location_partner_3')" />
+        <field name="analytic_account_id" ref="analytic_account_location_3" />
     </record>
+
 </odoo>

fieldservice_sale/models/sale_order_line.py

@@ -1,7 +1,7 @@
 # Copyright (C) 2019 Brian McMaster
 # Copyright (C) 2019 Open Source Integrators
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
-from odoo import api, fields, models
+from odoo import Command, api, fields, models
 
 
 class SaleOrderLine(models.Model):
@@ -53,8 +53,31 @@ def create(self, vals_list):
                 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": [Command.set(fsm_orders.ids)]})
         return res

fieldservice_sale_recurring/models/sale_order_line.py

@@ -2,7 +2,7 @@
 # Copyright (C) 2019 Open Source Integrators
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
-from odoo import _, fields, models
+from odoo import Command, _, fields, models
 
 
 class SaleOrderLine(models.Model):
@@ -89,5 +89,5 @@ def _prepare_invoice_line(self, **optional_values):
         if self.fsm_recurring_id:
             fsm_orders = self._get_invoiceable_fsm_order()
             if fsm_orders:
-                res.update({"fsm_order_ids": [(6, 0, fsm_orders.ids)]})
+                res.update({"fsm_order_ids": [Command.set(fsm_orders.ids)]})
         return res