Opened 12 years ago
Closed 12 years ago
#11092 closed defect (fixed)
Deleting hours when deleting a ticket
| Reported by: | Owned by: | Ryan J Ollos | |
|---|---|---|---|
| Priority: | normal | Component: | TracHoursPlugin | 
| Severity: | major | Keywords: | deleting | 
| Cc: | Trac Release: | 0.12 | 
Description
Hi everybody, 
I'm running TracHoursPlugin 0.6.0dev-r12205 on a TRAC 0.12.3 server and I noticed that the hours allocated to a ticket aren't deleted when I'm deleting this ticket... 
Is there an update that fixes the problem ? 
Thank you in advance for you answer, have a good day
Attachments (0)
Change History (9)
comment:1 follow-up: 2 Changed 12 years ago by
| Status: | new → assigned | 
|---|
comment:2 Changed 12 years ago by
Replying to rjollos:
I haven't tried to reproduce yet, but I think it will be a fairly easy issue to solve. It looks like we just need to delete the hours in ITicketChangeListener.ticket_deleted. In fact, the body of that method reads # TODO delete hours for this ticket.
Thanks you for the answer ! I'll wait for that modification. 
Have a nice day
comment:3 Changed 12 years ago by
comment:4 Changed 12 years ago by
- Implemented 
delete_ticket_hours. - Call 
delete_ticket_hoursinITicketChangeListener.ticket_deletedimplementation. - Fixed error in 
test_add_ticket_hours. Theidcolumn of the database was being confused withticket.id, however the ticket id value is stored in theticketcolumn. 
Note: Plugin version is 0.6.0dev and API is subject to change before the 0.6.0 release.
comment:5 Changed 12 years ago by
comment:6 follow-up: 7 Changed 12 years ago by
| Resolution: | → fixed | 
|---|---|
| Status: | assigned → closed | 
The issue should be resolved now. Please let me know if you encounter any issues.
comment:7 follow-up: 8 Changed 12 years ago by
| Resolution: | fixed | 
|---|---|
| Status: | closed → reopened | 
Replying to rjollos:
The issue should be resolved now. Please let me know if you encounter any issues.
Thank you for resolving this sir ! Hours are deleted when I delete the ticket, works fine!
I tested your modifications by reinstalling the plugin :
easy_install https://trac-hacks.org/svn/trachoursplugin/trunk/
But, I've got another problem now : TicketSidebarProviderPlugin is not showing so I can't add hours to a ticket...
comment:8 follow-up: 9 Changed 12 years ago by
Sorry, bad formatting... Anonymous was me 
Replying to j43l1us@gmail.com:
Replying to rjollos:
The issue should be resolved now. Please let me know if you encounter any issues.
Thank you for resolving this sir ! Hours are deleted when I delete the ticket, works fine ! 
I tested your modifications by reinstalling the plugin :
easy_install https://trac-hacks.org/svn/trachoursplugin/trunk/
But, I've got another problem now : TicketSidebarProviderPlugin is not showing so I can't add hours to a ticket...
comment:9 Changed 12 years ago by
| Resolution: | → fixed | 
|---|---|
| Status: | reopened → closed | 
Replying to j43l1us@gmail.com:
Sorry, bad formatting... Anonymous was me
Replying to j43l1us@gmail.com:Replying to rjollos:
The issue should be resolved now. Please let me know if you encounter any issues.
Thank you for resolving this sir ! Hours are deleted when I delete the ticket, works fine !
I tested your modifications by reinstalling the plugin :easy_install https://trac-hacks.org/svn/trachoursplugin/trunk/
But, I've got another problem now : TicketSidebarProviderPlugin is not showing so I can't add hours to a ticket...
Damn, i'm a noob. My excuses... I tried to reinstall TicketSidebarProviderPlugin with easy_install... And the plugin is now visible !
Thanks a lot for your help sir !



I haven't tried to reproduce yet, but I think it will be a fairly easy issue to solve. It looks like we just need to delete the hours in ITicketChangeListener.ticket_deleted. In fact, the body of that method reads # TODO delete hours for this ticket.