Skip to content

Commit 031b305

Browse files
committed
fix(spp_drims): fix 3 critical audit issues
- Add missing data/approval_definitions.xml to manifest - Fix alert search filters: alert_type → alert_type_id.code - Add @api.depends("sla_due_datetime") to _compute_sla_hours_remaining
1 parent b17a746 commit 031b305

3 files changed

Lines changed: 5 additions & 3 deletions

File tree

spp_drims/__manifest__.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@
4343
"data/audit_rules.xml",
4444
"data/config_defaults.xml",
4545
"data/gis_report_templates.xml",
46+
"data/approval_definitions.xml",
4647
# Reports (before views, as views may reference report actions)
4748
"report/waybill_template.xml",
4849
"views/dashboard_views.xml",

spp_drims/models/request.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -348,6 +348,7 @@ def _compute_sla_status(self):
348348
rec.sla_due_datetime = False
349349
rec.sla_status = False
350350

351+
@api.depends("sla_due_datetime")
351352
def _compute_sla_hours_remaining(self):
352353
"""Compute hours remaining until SLA breach."""
353354
now = fields.Datetime.now()

spp_drims/views/alert_views.xml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -289,17 +289,17 @@
289289
<filter
290290
name="filter_low_stock"
291291
string="Low Stock"
292-
domain="[('alert_type', '=', 'low_stock')]"
292+
domain="[('alert_type_id.code', '=', 'low_stock')]"
293293
/>
294294
<filter
295295
name="filter_expiry"
296296
string="Expiry"
297-
domain="[('alert_type', '=', 'expiry')]"
297+
domain="[('alert_type_id.code', '=', 'expiry')]"
298298
/>
299299
<filter
300300
name="filter_sla"
301301
string="SLA"
302-
domain="[('alert_type', 'in', ['sla_warning', 'sla_breach'])]"
302+
domain="[('alert_type_id.code', 'in', ['sla_warning', 'sla_breach'])]"
303303
/>
304304
<separator />
305305

0 commit comments

Comments
 (0)