[IMP] fieldservice_sale: black, isort, prettier

Author: brian10048

fieldservice_recurring/static/description/index.html

@@ -1,32 +1,24 @@
 # Copyright (C) 2018 Open Source Integrators
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 {
-    'name': 'Field Service - Sales',
-    'version': '12.0.2.5.1',
-    'summary': 'Sell field services.',
-    'category': 'Field Service',
-    'author': 'Open Source Integrators, Odoo Community Association (OCA)',
-    'website': 'https://github.com/OCA/field-service',
-    'depends': [
-        'fieldservice',
-        'sale_management',
-        'fieldservice_account',
+    "name": "Field Service - Sales",
+    "version": "12.0.2.5.1",
+    "summary": "Sell field services.",
+    "category": "Field Service",
+    "author": "Open Source Integrators, Odoo Community Association (OCA)",
+    "website": "https://github.com/OCA/field-service",
+    "depends": ["fieldservice", "sale_management", "fieldservice_account"],
+    "data": [
+        "security/ir.model.access.csv",
+        "views/fsm_branch.xml",
+        "views/fsm_location.xml",
+        "views/fsm_order.xml",
+        "views/product_template.xml",
+        "views/sale_order.xml",
+        "views/res_config_settings.xml",
     ],
-    'data': [
-        'security/ir.model.access.csv',
-        'views/fsm_branch.xml',
-        'views/fsm_location.xml',
-        'views/fsm_order.xml',
-        'views/product_template.xml',
-        'views/sale_order.xml',
-        'views/res_config_settings.xml',
-    ],
-    'license': 'AGPL-3',
-    'development_status': 'Beta',
-    'maintainers': [
-        'wolfhall',
-        'max3903',
-        'brian10048',
-    ],
-    'installable': True,
+    "license": "AGPL-3",
+    "development_status": "Beta",
+    "maintainers": ["wolfhall", "max3903", "brian10048"],
+    "installable": True,
 }

fieldservice_sale/__manifest__.py

@@ -6,5 +6,7 @@ def migrate(env, version):
     if not version:
         return
 
-    env.execute("UPDATE product_template SET field_service_tracking = 'sale' "
-                "WHERE field_service_tracking = 'order';")
+    env.execute(
+        "UPDATE product_template SET field_service_tracking = 'sale' "
+        "WHERE field_service_tracking = 'order';"
+    )

fieldservice_sale/migrations/12.0.2.0.0/pre-migration.py

@@ -4,8 +4,10 @@
 
 
 class FSMBranch(models.Model):
-    _inherit = 'fsm.branch'
+    _inherit = "fsm.branch"
 
     pricelist_id = fields.Many2one(
-        'product.pricelist', string='Default Pricelist',
-        help="Default pricelist for new customers of this branch.")
+        "product.pricelist",
+        string="Default Pricelist",
+        help="Default pricelist for new customers of this branch.",
+    )

fieldservice_sale/models/fsm_branch.py

@@ -1,11 +1,9 @@
 # Copyright (C) 2018 Open Source Integrators
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
-
 from odoo import fields, models
 
 
 class FSMLocation(models.Model):
-    _inherit = 'fsm.location'
+    _inherit = "fsm.location"
 
-    sales_territory_id = fields.Many2one('fsm.territory',
-                                         string='Sales Territory')
+    sales_territory_id = fields.Many2one("fsm.territory", string="Sales Territory")

fieldservice_sale/models/fsm_location.py

@@ -1,14 +1,14 @@
 # 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 fields, models, _
+from odoo import _, fields, models
 
 
 class FSMOrder(models.Model):
-    _inherit = 'fsm.order'
+    _inherit = "fsm.order"
 
-    sale_id = fields.Many2one('sale.order')
-    sale_line_id = fields.Many2one('sale.order.line')
+    sale_id = fields.Many2one("sale.order")
+    sale_line_id = fields.Many2one("sale.order.line")
 
     def action_view_sales(self):
         self.ensure_one()

fieldservice_sale/models/fsm_order.py

@@ -1,31 +1,34 @@
 # 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
 
 
 class ProductTemplate(models.Model):
-    _inherit = 'product.template'
+    _inherit = "product.template"
 
-    service_type = fields.Selection(selection_add=[
-        ('field', 'Field Service Orders'),
-    ])
-    field_service_tracking = fields.Selection([
-        ('no', 'Don\'t create FSM order'),
-        ('sale', 'Create one FSM order per sale order'),
-        ('line', 'Create one FSM order per sale order line'),
-    ], string='Field Service Tracking', default='no',
+    service_type = fields.Selection(selection_add=[("field", "Field Service Orders")])
+    field_service_tracking = fields.Selection(
+        [
+            ("no", "Don't create FSM order"),
+            ("sale", "Create one FSM order per sale order"),
+            ("line", "Create one FSM order per sale order line"),
+        ],
+        string="Field Service Tracking",
+        default="no",
         help="""Determines what happens upon sale order confirmation:
                 - None: nothing additional, default behavior.
                 - Per Sale Order: One FSM Order will be created for the sale.
                 - Per Sale Order Line: One FSM Order for each sale order line
-                will be created.""")
+                will be created.""",
+    )
     fsm_order_template_id = fields.Many2one(
-        'fsm.template', 'Field Service Order Template',
-        help="Select the field service order template to be created")
+        "fsm.template",
+        "Field Service Order Template",
+        help="Select the field service order template to be created",
+    )
 
-    @api.onchange('field_service_tracking')
+    @api.onchange("field_service_tracking")
     def _onchange_field_service_tracking(self):
-        if self.field_service_tracking == 'no':
+        if self.field_service_tracking == "no":
             self.fsm_order_template_id = False

fieldservice_sale/models/product_template.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 _, api, fields, models
 from odoo.exceptions import ValidationError
 
 

fieldservice_sale/models/sale_order.py

@@ -1,44 +1,46 @@
 # 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 _, api, fields, models
 
 
 class SaleOrderLine(models.Model):
     _inherit = "sale.order.line"
 
     qty_delivered_method = fields.Selection(
-        selection_add=[('field_service', 'Field Service Order')])
+        selection_add=[("field_service", "Field Service Order")]
+    )
     fsm_order_id = fields.Many2one(
-        'fsm.order', 'Order', index=True,
-        help="Field Service Order generated by the sales order item")
+        "fsm.order",
+        "Order",
+        index=True,
+        help="Field Service Order generated by the sales order item",
+    )
 
-    @api.depends('product_id.type')
+    @api.depends("product_id.type")
     def _compute_product_updatable(self):
         for line in self:
-            if line.product_id.type == 'service' and line.state == 'sale':
+            if line.product_id.type == "service" and line.state == "sale":
                 line.product_updatable = False
             else:
                 super(SaleOrderLine, line)._compute_product_updatable()
 
     @api.multi
-    @api.depends('product_id')
+    @api.depends("product_id")
     def _compute_qty_delivered_method(self):
         super(SaleOrderLine, self)._compute_qty_delivered_method()
         for line in self:
-            if not line.is_expense and \
-               line.product_id.field_service_tracking == 'line':
-                line.qty_delivered_method = 'field_service'
+            if not line.is_expense and line.product_id.field_service_tracking == "line":
+                line.qty_delivered_method = "field_service"
 
     @api.multi
-    @api.depends('fsm_order_id.stage_id')
+    @api.depends("fsm_order_id.stage_id")
     def _compute_qty_delivered(self):
         super(SaleOrderLine, self)._compute_qty_delivered()
         lines_by_fsm = self.filtered(
-            lambda sol:
-                sol.qty_delivered_method == 'field_service'
+            lambda sol: sol.qty_delivered_method == "field_service"
         )
-        complete = self.env.ref('fieldservice.fsm_stage_completed')
+        complete = self.env.ref("fieldservice.fsm_stage_completed")
         for line in lines_by_fsm:
             qty = 0
             if line.fsm_order_id.stage_id == complete:
@@ -48,27 +50,27 @@ def _compute_qty_delivered(self):
     @api.model
     def create(self, values):
         line = super(SaleOrderLine, self).create(values)
-        if line.state == 'sale':
+        if line.state == "sale":
             line._field_service_generation()
         return line
 
     def _field_create_fsm_order_prepare_values(self):
         self.ensure_one()
         categories = self.product_id.fsm_order_template_id.category_ids
         return {
-            'customer_id': self.order_id.partner_id.id,
-            'location_id': self.order_id.fsm_location_id.id,
-            'location_directions': self.order_id.fsm_location_id.direction,
-            'request_early': self.order_id.expected_date,
-            'scheduled_date_start': self.order_id.expected_date,
-            'description': self.name,
-            'template_id': self.product_id.fsm_order_template_id.id,
-            'todo': self.product_id.fsm_order_template_id.instructions,
-            'category_ids': [(6, 0, categories.ids)],
-            'scheduled_duration': self.product_id.fsm_order_template_id.hours,
-            'sale_id': self.order_id.id,
-            'sale_line_id': self.id,
-            'company_id': self.company_id.id,
+            "customer_id": self.order_id.partner_id.id,
+            "location_id": self.order_id.fsm_location_id.id,
+            "location_directions": self.order_id.fsm_location_id.direction,
+            "request_early": self.order_id.expected_date,
+            "scheduled_date_start": self.order_id.expected_date,
+            "description": self.name,
+            "template_id": self.product_id.fsm_order_template_id.id,
+            "todo": self.product_id.fsm_order_template_id.instructions,
+            "category_ids": [(6, 0, categories.ids)],
+            "scheduled_duration": self.product_id.fsm_order_template_id.hours,
+            "sale_id": self.order_id.id,
+            "sale_line_id": self.id,
+            "company_id": self.company_id.id,
         }
 
     @api.multi
@@ -81,20 +83,27 @@ def _field_create_fsm_order(self):
         for so_line in self:
             # create fsm_order
             values = so_line._field_create_fsm_order_prepare_values()
-            fsm_order = self.env['fsm.order'].sudo().create(values)
+            fsm_order = self.env["fsm.order"].sudo().create(values)
             so_line.fsm_order_id = fsm_order.id
             # post message on SO
-            msg_body = _(
-                """Field Service Order Created (%s): <a href=
+            msg_body = (
+                _(
+                    """Field Service Order Created (%s): <a href=
                    # data-oe-model=fsm.order data-oe-id=%d>%s</a>
-                """) % (so_line.product_id.name, fsm_order.id, fsm_order.name)
+                """
+                )
+                % (so_line.product_id.name, fsm_order.id, fsm_order.name)
+            )
             so_line.order_id.message_post(body=msg_body)
             # post message on fsm_order
-            fsm_order_msg = _(
-                """This order has been created from: <a href=
+            fsm_order_msg = (
+                _(
+                    """This order has been created from: <a href=
                    # data-oe-model=sale.order data-oe-id=%d>%s</a> (%s)
-                """) % (so_line.order_id.id, so_line.order_id.name,
-                        so_line.product_id.name)
+                """
+                )
+                % (so_line.order_id.id, so_line.order_id.name, so_line.product_id.name)
+            )
             fsm_order.message_post(body=fsm_order_msg)
             result[so_line.id] = fsm_order
         return result
@@ -108,10 +117,10 @@ def _field_find_fsm_order(self):
         """
         for rec in self:
             # one search for all so lines
-            fsm_orders = self.env['fsm.order'].search([
-                ('sale_line_id', 'in', rec.ids)])
+            fsm_orders = self.env["fsm.order"].search([("sale_line_id", "in", rec.ids)])
             fsm_order_sol_mapping = {
-                fsm_order.sale_line_id.id: fsm_order for fsm_order in fsm_orders}
+                fsm_order.sale_line_id.id: fsm_order for fsm_order in fsm_orders
+            }
             res = {}
             for so_line in rec:
                 # If the SO was confirmed, cancelled, set to draft then confirmed,
@@ -129,17 +138,15 @@ def _field_service_generation(self):
             exists, it simply links the existing one to the line.
         """
         for rec in self:
-            if rec.product_id.field_service_tracking == 'sale':
+            if rec.product_id.field_service_tracking == "sale":
                 sale = rec.order_id
                 so_fo_mapping = sale._field_find_fsm_order()
                 rec.fsm_order_id = so_fo_mapping[rec.order_id.id].id
-            if rec.product_id.field_service_tracking == 'line':
+            if rec.product_id.field_service_tracking == "line":
                 rec._field_find_fsm_order()
 
     @api.multi
     def _prepare_invoice_line(self, qty):
         res = super()._prepare_invoice_line(qty)
-        res.update({
-            'fsm_order_id': self.fsm_order_id.id,
-        })
+        res.update({"fsm_order_id": self.fsm_order_id.id})
         return res