Opened 18 years ago
Closed 18 years ago
#4069 closed defect (fixed)
Ability to create a ticket on behalf of someone else (population of Reporter field) when logged in
Reported by: | Owned by: | Jonas Borgström | |
---|---|---|---|
Priority: | low | Milestone: | 0.11 |
Component: | ticket system | Version: | 0.10 |
Severity: | normal | Keywords: | |
Cc: | r.s.hatcher@…, ros@… | Branch: | |
Release Notes: | |||
API Changes: | |||
Internal Changes: |
Description
Is there an easy (e.g. trac.ini parameter) way of optionally reverting to the earlier behavior of having to type in the Reporter when creating a new ticket, rather than this field being automatically populated with the username of the logged-in user?
In our setup, we'll often create tickets on behalf of people who will want to recieve email notifications (since they are the real Reporters) but won't be in a position to create tickets themselves. Having to create a ticket, then immediately edit it to change the Reporter (and remembering to do so) is a pain! Similarly, logging in & out of Trac 30 times a day to create tickets on behalf of others and then to actually edit tickets as yourself is not ideal.
Attachments (0)
Change History (11)
comment:1 by , 18 years ago
comment:2 by , 18 years ago
Well, I can imagine that in this special case he really wants to have it as "Reporter" (editing reports, for example).
However, if no one else supports going back to the old behavior, I'd propose a wontfix resolution.
The user's newticket.cs template could easily be modified to restore that behavior for him:
-
templates/newticket.cs
11 11 <?cs include:"site_newticket.cs" ?> 12 12 <form id="newticket" method="post" action="<?cs 13 13 var:trac.href.newticket ?>#preview"> 14 <?cs if:trac.authname == "anonymous" ?>15 14 <div class="field"> 16 15 <label for="reporter">Your email or username:</label><br /> 17 16 <input type="text" id="reporter" name="reporter" size="40" value="<?cs 18 17 var:newticket.reporter ?>" /><br /> 19 18 </div> 20 <?cs /if ?>21 19 <div class="field"> 22 20 <label for="summary">Short summary:</label><br /> 23 21 <input id="summary" type="text" name="summary" size="80" value="<?cs
comment:3 by , 18 years ago
Thanks, if it does end up as a wontfix we'll look at using the template workaround. Cheers, Rich
comment:4 by , 18 years ago
Cc: | added |
---|
I need this too for the same reasons as Richard. It is a real pain having to change the reporter field separately.
And more importantly I need this because we have a generic username to access support of different systems which our users log in with - they need to be able to specify their email rather than it being a meaningless generic user. Getting them to remember to add their name to the cc: field is risky and also means we will lose the information about who created the ticket initially as other people may come along and want to be kept informed of progress to a ticket. Have a separate account for each user it is unmanageable.
I hope this doesn't end up as a wontfix.
I've tried the solution above, but removing those two lines from my projectEnv/templates/ticket.cs doesn't help. The entry box is visible, but when the ticket is submitted it's contents are ignored. Any ideas?
Thanks.
follow-up: 6 comment:5 by , 18 years ago
Could it be done from a plugin ? I think this could be a good plugin candidate.
follow-up: 7 comment:6 by , 18 years ago
Replying to eblot:
Could it be done from a plugin ? I think this could be a good plugin candidate.
It could be done. If there is sufficient interest I can code it up later tonight.
comment:7 by , 18 years ago
Replying to Noah Kantrowitz (coderanger) <coderanger@yahoo.com>:
Replying to eblot:
Could it be done from a plugin ? I think this could be a good plugin candidate.
It could be done.
IMHO this kind of special features are better implemented as plugins, in order to keep the Trac source code and UI simple, clean and lean.
follow-up: 9 comment:8 by , 18 years ago
Okay, created a plugin for this. Just run easy_install TracFakeUsername
and then enable it in trac.ini.
Propose a worksforme for this ticket.
comment:9 by , 18 years ago
Resolution: | → wontfix |
---|---|
Status: | new → closed |
Replying to Noah Kantrowitz (coderanger) <coderanger@yahoo.com>:
Okay, created a plugin for this. Just run
easy_install TracFakeUsername
and then enable it in trac.ini.Propose a worksforme for this ticket.
Based on our recent discussions for the triage guidelines, this is a wontfix since we're not implementing this in the Trac core.
comment:10 by , 18 years ago
Resolution: | wontfix |
---|---|
Status: | closed → reopened |
This request has been raised again on the Trac-users mailing list, and I suspect it will annoy more than one people making the switch from 0.9 to 0.10.
When the ticket-preview branch was merged, I was not surprised to see the Reporter: field on the new ticket page, it felt natural to have it there, just after the summary (when logged in).
I think that we shouldn't end up in a situation where logged in users have less privileges that anonymous users, which would require users to log in and out repeatedly, as described above.
The problem of users masquerading as others is a real one, but was not addressed by this restriction anyway (as anonymous users could still do it). Real solutions to this issue are discussed in #1890 (display if the author of the change was authenticated or not when he did the change).
When #1890 gets implemented, one should take care of making the distinction between the authenticated actual creator of the ticket and the non-authenticated reporter.
comment:11 by , 18 years ago
Milestone: | → 0.11 |
---|---|
Resolution: | → fixed |
Status: | reopened → closed |
In r4989, the changes made in the Reporter: field are taken into account.
Just put their name in the CC field.