#298 closed enhancement (fixed)
Add a possibility to remove a ticket
Reported by: | actarus | Owned by: | Christian Boos |
---|---|---|---|
Priority: | normal | Milestone: | 0.10 |
Component: | ticket system | Version: | 0.6.1 |
Severity: | normal | Keywords: | |
Cc: | shishz@… | Branch: | |
Release Notes: | |||
API Changes: | |||
Internal Changes: |
Description
There is no possibility other than going through the db to remove a ticket. Having a web interface to do this would be useful.
Attachments (1)
Change History (34)
comment:1 by , 21 years ago
Component: | general → ticket system |
---|
comment:2 by , 21 years ago
comment:3 by , 21 years ago
Resolution: | → wontfix |
---|---|
Status: | new → closed |
No, tickets shouldn't be easily removable.
Possibly, trac-admin could have a command to do it, but that too seems not worth it.
It's easy to use SQL if you absolutely must delete a ticket, otherwise just close the ticket as invalid.
comment:4 by , 20 years ago
Keywords: | test added |
---|
comment:5 by , 19 years ago
I have a ticket in my Trac that I need to delete. For some reason, any time it's clicked on, tracd crashes. So, in that case, yes deleting a ticket is necessary.
comment:6 by , 19 years ago
It could be interesting to report the error you got, maybe that can be fixed…
comment:7 by , 19 years ago
Resolution: | wontfix |
---|---|
Status: | closed → reopened |
Why don't You let this thing (delete ticket's) to trac-admin. I hope it's easy to implement, and gives further functionality. For example: someone add stupid ticket specially - What shold I do? Waste time learning SQL tables infrastrukture?
Pleas, think about this. It's very unprofesional to decide for user :) (sorry for my English)
comment:8 by , 19 years ago
Keywords: | test removed |
---|
I agree there should be some straightforward administrative route to utterly deleting or at least censoring a ticket. You have to understand that we must deal with people, often superiors, who may obsess over sometimes (but not always) trivial details.
For example: Suppose somebody puts a racist comment in a ticket. Merely marking it "invalid" isn't going to appease HR. They'll want it expunged completely.
Hacking the DB is not straightforward and not documented. There should be an option in trac-admin. If you like, it could leave behind a marker that indicates the ticket was deleted/censored and optionally why.
comment:9 by , 19 years ago
If you like, it could leave behind a marker that indicates the ticket was deleted/censored and optionally why.
The admin can already alter the description, and remove any offending content if needed, and the motivation for closing the ticket as invalid can be explained in a ticket comment associated with the status change.
The only thing that can't be done right now is deleting/modifying the ticket comments (see #454), but once this feature is available, I don't think it's worth to introduce a special case.
comment:10 by , 19 years ago
cboos wrote:
closing the ticket as invalid can be explained in a ticket comment associated with the status change.
You must be joking :)
I'm using Trac for few days, and I understand difference between closing and deleting ticket. I strongly recommend closing ticket, but still in some cases I just HAVE to delet some ticket.
For example: We are testing TRAC and add some stupid tests ticket :) Now I have to reinstall TRAC to delete them. There are meny more examples.
As someone seys. I'm a big boy and have TRAC_ADMIN. Let me do What I want.
I don't believe that it is hard to add this feature :)
comment:11 by , 19 years ago
Agree with adding delete ticket to trac-admin. I just started using trac for a project, and entered an item for another project from a different account. It entered for the same project. After the fact, I now know from reading FAQ that special setup is required to manage more than one project, but I need to delete the ticked since it contains sensitive information inappropriate for the first project. Otherwise, I'm recreating from scratch or learning SQL. Having the delete ticket and reset ticket number (back to one previous) right now would be very convenient as I noticed the problem immediately. I need to eliminate any record of this ticket entry for this project.
I suspect little issues crop up like this all the time when people are trying to get work done, and admins need to fix them.
2c. Thanks for great tracking system.
comment:12 by , 19 years ago
I agree with people that sometimes you just need to delete a ticket.
I'm using a 0.8 version of Trac, recently set-up, and sent test emails into the system when setting up email ticket reporting. I had customized a lot, and did not realize why new ticket were not showing up when I thought the email side was working properly.
Turned out that the report lists I was looking at to check whether the emails were accepted into the system depended on a valid "priority" name to list the tickets. I have duplicated test email tickets in the system, closed as invalid. Now they show up in reports. Just not looking too good there. Nice to have proof that I did my job, but very sloppy.
You don't want users frivolously deleting tickets, but if spam makes it to the system, the ticket database may get pretty messy.
by , 19 years ago
Attachment: | ticket_remove.patch added |
---|
Add a ticket remove <number>
command to trac-admin.
comment:14 by , 19 years ago
Milestone: | → 0.9.1 |
---|---|
Owner: | changed from | to
Status: | reopened → new |
Summary: | can't remove ticket → Add a possibility to remove a ticket |
There's a patch on r2410 for this feature.
comment:15 by , 19 years ago
Status: | new → assigned |
---|
comment:17 by , 19 years ago
Manuzhai: I think that cboos meant that the patch was written when the trunk was at r2410. The file is attached to this ticket.
comment:18 by , 19 years ago
works like a charm! thanks cboos! someone basically filed a ticket under the wrong trac (we have three deployed here)
comment:20 by , 19 years ago
Keywords: | review added |
---|
comment:21 by , 19 years ago
Cc: | added |
---|
comment:22 by , 19 years ago
Milestone: | 0.9.1 → 0.9.2 |
---|
comment:23 by , 19 years ago
Did this not make it into 0.9.2?
Ah, I see the milestone has been move out again to 0.9.3.
comment:24 by , 19 years ago
Keywords: | review removed |
---|---|
Milestone: | 0.9.3 → 1.0 |
Resolution: | → fixed |
Status: | assigned → closed |
Implemented in r2634.
comment:25 by , 19 years ago
Milestone: | 1.0 → 0.10 |
---|
comment:26 by , 19 years ago
Type: | enhancement → defect |
---|
The patch didn't work in the Latest Stable Release - 0.9.4
comment:27 by , 19 years ago
Type: | defect → enhancement |
---|
Well, that patch is now in trunk and the feature will be available for 0.10.
Anyway, the feature itself was an enhancement request, so I'm reverting the type change.
Notice that there's also a plugin TracHacks:TicketDeletePlugin you might find interesting if you want to stick with 0.9.
follow-up: 29 comment:28 by , 18 years ago
For cleaning up after spammer attacks, it should also be possible to revert a change to a ticket.
follow-up: 30 comment:29 by , 18 years ago
Replying to bjorn at linpro dot no:
For cleaning up after spammer attacks, it should also be possible to revert a change to a ticket.
I always just deleted revisions to vandalized wiki pages. Today was my first onslaught of defect reports that I need to buy medicine or something, and I can't delete them. :(
follow-up: 31 comment:30 by , 18 years ago
Replying to anonymous:
Replying to bjorn at linpro dot no:
For cleaning up after spammer attacks, it should also be possible to revert a change to a ticket.
I always just deleted revisions to vandalized wiki pages. Today was my first onslaught of defect reports that I need to buy medicine or something, and I can't delete them. :(
I get about one spam attack per week. Isn't there a way to require a password to create new tickets? But maybe that should be a new ticket….
follow-up: 32 comment:31 by , 18 years ago
Replying to jyoshimi@ucmerced.edu:
I get about one spam attack per week. Isn't there a way to require a password to create new tickets? But maybe that should be a new ticket….
Yes, just remove the TICKET_CREATE permission from 'anonymous', and assign it to 'authenticated' instead. See TracPermissions for more info.
comment:32 by , 18 years ago
Replying to Martijn Pieters <mj@zopatista.com>:
Replying to jyoshimi@ucmerced.edu:
I get about one spam attack per week. Isn't there a way to require a password to create new tickets? But maybe that should be a new ticket….
Yes, just remove the TICKET_CREATE permission from 'anonymous', and assign it to 'authenticated' instead. See TracPermissions for more info.
In 0.10svn, anyways, authenticated users can change the reporter field, though, so far as I can tell.
comment:33 by , 18 years ago
In case someone else needs to delete a ticket in a trac 0.9 instance without installing extra plugins, here's the SQL:
$ sqlite trac.db delete from ticket_change where ticket = 1234; delete from ticket_custom where ticket = 1234; delete from ticket where id = 1234;
Should tickets *be* removable?
The best policy in most enviroments is to have invalid tickets closed as such, rather than deleted, which will allow you to get information on things that get reported regularly.