ami/tl-vikunja
1
0
Fork 0
Code

fix(task): delete the task after all related attributes to prevent task not found errors

Browse Source
This commit is contained in:
kolaente 2024-01-28 12:23:25 +01:00
parent 1630e4fc08
commit a20f6ac815
No known key found for this signature in database
GPG Key ID: F40E70337AB24C9B
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
pkg/models/tasks.go
View File

@ -1432,10 +1432,6 @@ func (t *Task) Delete(s *xorm.Session, a web.Auth) (err error) {
return err return err
} }
if _, err = s.ID(t.ID).Delete(Task{}); err != nil {
return err
}
// Delete assignees // Delete assignees
if _, err = s.Where("task_id = ?", t.ID).Delete(TaskAssginee{}); err != nil { if _, err = s.Where("task_id = ?", t.ID).Delete(TaskAssginee{}); err != nil {
return err return err
@ -1484,6 +1480,12 @@ func (t *Task) Delete(s *xorm.Session, a web.Auth) (err error) {
return return
} }
// Actually delete the task
_, err = s.ID(t.ID).Delete(Task{})
if err != nil {
return err
}
doer, _ := user.GetFromAuth(a) doer, _ := user.GetFromAuth(a)
err = events.Dispatch(&TaskDeletedEvent{ err = events.Dispatch(&TaskDeletedEvent{
Task: fullTask, Task: fullTask,