Opened 17 years ago
Closed 17 years ago
#5434 closed defect (fixed)
Mistaken ticket assignment to anonymous
Reported by: | 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)
Change History (7)
comment:1 by , 17 years ago
comment:2 by , 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 , 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 , 17 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
Please try the above patch which should also fix the custom query.
comment:4 by , 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 , 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.
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]