'Assign to' field should be labeled 'Set Owner'

[Ryan Ollos <ryano@…>]

I'm opening this as a bug, though I think it is most properly characterized as an ambiguity. Perhaps it should be an enhancement (or perhaps I am just the only one that thinks this is a problem)!

According to the default Trac workflow, all tickets are created with the default Status new. From TracWorkflow#BasicTicketWorkflowCustomization:

There are a couple of hard-coded constraints to the workflow. In particular, tickets are created with status new, and tickets are expected to have a closed state.

The act of Assigning a ticket sets its status to Assigned. However, on ticket creation, there is a box that Sets the Owner which is labeled Assign to:.

I think it would be more technically correct to label this box Set Owner. It is, after all, Setting the Owner, not Assigning the Ticket.

I posted this on the Trac user discussion list (1), but I didn't get any responses. If anyone has recommendations for modifying this behavior as I described in my posting, please let me know.

I've searched through the Trac tickets (2) and have not found that anyone else opened a report for this, which surprises me.

• [1] ​http://groups.google.com/group/trac-users/browse_thread/thread/4a287bcb19cb1fda
• [2] ​http://trac.edgewall.org/query?status=assigned&status=closed&status=new&status=reopened&description=~Assign+to&component=ticket+system&order=priority

[Christian Boos]

I'm ok with that change. Others?

[Remy Blank]

+1

[Emmanuel Blot]

Replying to cboos:

I'm ok with that change. Others?

As long as the internal API does not change (to "assign" a ticket to an owner), I'm ok as well.

[Christian Boos]

I'm not sure what you're referring to by internal API to assign a ticket to an owner, but IIUC the change would be simply:

trac/ticket/web_ui.py

@@ -1133 +1133 @@
                 type_ = field['type']
                 field['skip'] = True
                 if not ticket.exists:
-                    field['label'] = _('Assign to')
+                    field['label'] = _('Owner')
                     if 'TICKET_MODIFY' in req.perm(ticket.resource):
                         field['skip'] = False
                         owner_field = field

(Owner rather than Set owner, for consistency with the label for other fields)

[fr_FR] note: currently translated by "Propriétaire" … I'd rather see "Responsable" here.

[Emmanuel Blot]

Replying to cboos:

I'm not sure what you're referring to by internal API to assign a ticket to an owner,

ticket = Ticket(self.env, ticket_id)
ticket['owner'] = u'john.doe'

[fr_FR] note: currently translated by "Propriétaire" … I'd rather see "Responsable" here.

+1

[Remy Blank]

What about the workflow action "reassign to"? Shouldn't that be changed to "set owner to"?

[Christian Boos]

"reassign" is the workflow transition name, and "to …" gets appended to it. See render_ticket_action_control in source:trunk/trac/ticket/default_workflow.py@8380​.

Not very i18n friendly anyway, but we have already a ticket for that I think (#6234 → #5441, the latter should probably be widened in scope to cover all the places status transition names are displayed in the interface).

[Ryan Ollos <ryano@…>]

Replying to rblank:

What about the workflow action "reassign to"? Shouldn't that be changed to "set owner to"?

If I understand correctly, reassign to changes the owner and puts the ticket in the assigned state; so the term reassign to seems appropriate here.

My issue with using assign to on ticket creation is that the owner is set, but the ticket is not assigned.

[Ryan Ollos <ryano@…>]

Replying to cboos:

(Owner rather than Set owner, for consistency with the label for other fields)

Yeah that sounds good. Owner makes more sense and is more consistent than Set Owner.

[Christian Boos]

Change from comment:4 applied in r8386.