feat(fusion_claims): allow order-less tasks + service-repair SO flag
Relaxes _check_order_link to a no-op (service bookings auto-create their SO; in-shop/walk-in tasks may have none) and adds x_fc_is_service_repair on sale.order. The 'Service Repair' crm.tag from the plan is intentionally omitted: fusion_claims does not depend on crm and sale.order has no tag_ids; the boolean flag is the repair-SO identity. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -338,6 +338,11 @@ class SaleOrder(models.Model):
|
|||||||
help='Type of sale for billing purposes. This field determines the workflow and billing rules.',
|
help='Type of sale for billing purposes. This field determines the workflow and billing rules.',
|
||||||
)
|
)
|
||||||
|
|
||||||
|
x_fc_is_service_repair = fields.Boolean(
|
||||||
|
string='Service Repair', copy=False,
|
||||||
|
help='Auto-created from the technician service booking wizard.',
|
||||||
|
)
|
||||||
|
|
||||||
x_fc_sale_type_locked = fields.Boolean(
|
x_fc_sale_type_locked = fields.Boolean(
|
||||||
string='Sale Type Locked',
|
string='Sale Type Locked',
|
||||||
compute='_compute_sale_type_locked',
|
compute='_compute_sale_type_locked',
|
||||||
|
|||||||
@@ -9,7 +9,7 @@ features to the base fusion.technician.task model.
|
|||||||
"""
|
"""
|
||||||
|
|
||||||
from odoo import models, fields, api, _
|
from odoo import models, fields, api, _
|
||||||
from odoo.exceptions import UserError, ValidationError
|
from odoo.exceptions import UserError
|
||||||
from markupsafe import Markup
|
from markupsafe import Markup
|
||||||
import logging
|
import logging
|
||||||
|
|
||||||
@@ -104,15 +104,9 @@ class FusionTechnicianTaskClaims(models.Model):
|
|||||||
|
|
||||||
@api.constrains('sale_order_id', 'purchase_order_id')
|
@api.constrains('sale_order_id', 'purchase_order_id')
|
||||||
def _check_order_link(self):
|
def _check_order_link(self):
|
||||||
for task in self:
|
# Relaxed 2026-06: service bookings auto-create their SO, and in-shop /
|
||||||
if task.x_fc_sync_source:
|
# walk-in tasks may legitimately have none. No order link is required anymore.
|
||||||
continue
|
return
|
||||||
if task.task_type == 'ltc_visit':
|
|
||||||
continue
|
|
||||||
if not task.sale_order_id and not task.purchase_order_id:
|
|
||||||
raise ValidationError(_(
|
|
||||||
"A task must be linked to either a Sale Order (Case) or a Purchase Order."
|
|
||||||
))
|
|
||||||
|
|
||||||
# ------------------------------------------------------------------
|
# ------------------------------------------------------------------
|
||||||
# HOOK OVERRIDES
|
# HOOK OVERRIDES
|
||||||
|
|||||||
@@ -4,3 +4,4 @@ from . import test_signed_pages_gate
|
|||||||
from . import test_application_received_wizard
|
from . import test_application_received_wizard
|
||||||
from . import test_dashboard
|
from . import test_dashboard
|
||||||
from . import test_service_rate
|
from . import test_service_rate
|
||||||
|
from . import test_service_booking
|
||||||
|
|||||||
31
fusion_claims/tests/test_service_booking.py
Normal file
31
fusion_claims/tests/test_service_booking.py
Normal file
@@ -0,0 +1,31 @@
|
|||||||
|
# -*- coding: utf-8 -*-
|
||||||
|
from datetime import date
|
||||||
|
from odoo.tests.common import TransactionCase, tagged
|
||||||
|
|
||||||
|
|
||||||
|
@tagged('post_install', '-at_install')
|
||||||
|
class TestServiceBooking(TransactionCase):
|
||||||
|
|
||||||
|
@classmethod
|
||||||
|
def setUpClass(cls):
|
||||||
|
super().setUpClass()
|
||||||
|
cls.Task = cls.env['fusion.technician.task']
|
||||||
|
# technician_id is required on a task (domain x_fc_is_field_staff=True).
|
||||||
|
cls.tech = cls.env['res.users'].create({
|
||||||
|
'name': 'Service Booking Tech',
|
||||||
|
'login': 'svcbook_tech',
|
||||||
|
'x_fc_is_field_staff': True,
|
||||||
|
})
|
||||||
|
|
||||||
|
def test_task_without_order_is_allowed(self):
|
||||||
|
# repair for a brand-new client: no SO/PO must NOT raise after the relax
|
||||||
|
t = self.Task.create({
|
||||||
|
'task_type': 'repair',
|
||||||
|
'technician_id': self.tech.id,
|
||||||
|
'scheduled_date': date(2026, 6, 3),
|
||||||
|
})
|
||||||
|
self.assertTrue(t.id)
|
||||||
|
|
||||||
|
def test_sale_order_has_service_repair_flag(self):
|
||||||
|
so = self.env['sale.order'].new({})
|
||||||
|
self.assertIn('x_fc_is_service_repair', so._fields)
|
||||||
Reference in New Issue
Block a user