Edgewall Software
Modify

Opened 17 years ago

Closed 17 years ago

#5434 closed defect (fixed)

Mistaken ticket assignment to anonymous

Reported by: asmodai@… Owned by: Christian Boos
Priority: normal Milestone: 0.11
Component: ticket system Version: devel
Severity: normal Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

I filed 3 tickets on a component that has no default assignee.

In the ticket overview page these tickets are shown as having an empty owner/assignee in the Owner column.

When you click a ticket though you see it's assigned to anonymous. With 0.10.4 this is left empty, see for example: http://trac.edgewall.org/ticket/5376 versus http://trac.akumaproject.org/ticket/1

Attachments (1)

t5434-r5616.diff (1.7 KB ) - added by Christian Boos 17 years ago.
'Owner' becomes special as this is the only place where a missing author is left empty instead of being replace by 'anonymous'

Download all attachments as: .zip

Change History (7)

comment:1 by Eli Carter, 17 years ago

It isn't actually assigned to anonymous. authorinfo() translates an empty name to 'anonymous' for presentation. trunk/trac/templates/macros.html@5580#L34 done in [4488]

comment:2 by asmodai@…, 17 years ago

I'd personally argue that that is confusing in the ticket-view though. I'd wager, based on my own confusion, that seeing assigned to anonymous is awkward. If it is empty people will not wonder, anonymous will.

by Christian Boos, 17 years ago

Attachment: t5434-r5616.diff added

'Owner' becomes special as this is the only place where a missing author is left empty instead of being replace by 'anonymous'

comment:3 by Christian Boos, 17 years ago

Owner: changed from Jonas Borgström to Christian Boos
Status: newassigned

Please try the above patch which should also fix the custom query.

comment:4 by Jeffrey Hulten <jhulten@…>, 17 years ago

Did you intentionally drop the (accepted) with this patch?

<py:if test="ticket.status == 'assigned'">(accepted)</py:if> <!--! FIXME -->

comment:5 by Christian Boos, 17 years ago

Yes, for a couple of reasons:

  • first, saying "(accepted)" when the status is 'assigned' was one of the known "warts" of the standard Trac workflow (see discussion in #2045)
  • then, the template should be neutral w.r.t to the chosen workflow, where you can't know if there's an "assigned" state at all
  • finally, is it really that interesting to echo in a different way the information already given by the status?

I'm open to suggestions, however.

comment:6 by Christian Boos, 17 years ago

Resolution: fixed
Status: assignedclosed

Patch applied in r5628.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Christian Boos.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Christian Boos to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.