From 28bf6b5071f43c0873ade1bcf3bae9044ff8ff4f Mon Sep 17 00:00:00 2001 From: gsinghpal Date: Sun, 3 May 2026 23:53:34 -0400 Subject: [PATCH] =?UTF-8?q?fix(jobs):=20drop=20display=5Fname=20override?= =?UTF-8?q?=20on=20workflow=20state=20=E2=80=94=20pills=20showed=20'Name?= =?UTF-8?q?=20[code]'?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The compute appended '[code]' so admin pages could disambiguate states at a glance. But display_name is what the status-bar widget uses to render each pill, so every pill came out as 'Received [received]', 'In Progress [in_progress]', etc. Removed the compute. Admin list view already shows code as a separate column. --- fusion_plating/fusion_plating_jobs/__manifest__.py | 2 +- .../fusion_plating_jobs/models/fp_job_workflow_state.py | 9 +++++---- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/fusion_plating/fusion_plating_jobs/__manifest__.py b/fusion_plating/fusion_plating_jobs/__manifest__.py index 5661ed53..2c9c8a18 100644 --- a/fusion_plating/fusion_plating_jobs/__manifest__.py +++ b/fusion_plating/fusion_plating_jobs/__manifest__.py @@ -3,7 +3,7 @@ # License OPL-1 (Odoo Proprietary License v1.0) { 'name': 'Fusion Plating — Native Jobs', - 'version': '19.0.8.18.1', + 'version': '19.0.8.18.2', 'category': 'Manufacturing/Plating', 'summary': 'Native plating job model — replaces mrp.production / mrp.workorder bridge.', 'author': 'Nexa Systems Inc.', diff --git a/fusion_plating/fusion_plating_jobs/models/fp_job_workflow_state.py b/fusion_plating/fusion_plating_jobs/models/fp_job_workflow_state.py index 4523b25e..8e2cb053 100644 --- a/fusion_plating/fusion_plating_jobs/models/fp_job_workflow_state.py +++ b/fusion_plating/fusion_plating_jobs/models/fp_job_workflow_state.py @@ -139,10 +139,11 @@ class FpJobWorkflowState(models.Model): 'Workflow state code must be unique.'), ] - @api.depends('name', 'code') - def _compute_display_name(self): - for s in self: - s.display_name = '%s [%s]' % (s.name or '', s.code or '') + # NOTE: no display_name override on purpose. Earlier draft computed + # "Name [code]" so admin pages could disambiguate at a glance, but + # that string bled into the operator status bar (every pill rendered + # as "Received [received]"). The admin list view shows code as its + # own column, so we don't need it baked into display_name. # ---- Trigger evaluation --------------------------------------------------