fix(jobs): drop display_name override on workflow state — pills showed 'Name [code]'
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.
This commit is contained in:
@@ -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 --------------------------------------------------
|
||||
|
||||
|
||||
Reference in New Issue
Block a user