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.