ami/tl-vikunja
1
0
Fork 0
Code

fix(kanban): dispatch task updated event when task position is updated

Browse Source 
This fixes a bug where a task would not be indexed in Typesense when its position changed.
This commit is contained in:
kolaente 2024-07-12 13:30:38 +02:00
parent e5e0db1f6a
commit 51040cf1af
No known key found for this signature in database
GPG Key ID: F40E70337AB24C9B
1 changed files with 12 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
pkg/models/task_position.go
View File

@ -20,6 +20,8 @@ import (
"math" "math"
"code.vikunja.io/api/pkg/events" "code.vikunja.io/api/pkg/events"
"code.vikunja.io/api/pkg/user"
"code.vikunja.io/web" "code.vikunja.io/web"
"xorm.io/xorm" "xorm.io/xorm"
) )
@ -108,9 +110,18 @@ func (tp *TaskPosition) Update(s *xorm.Session, a web.Auth) (err error) {
return RecalculateTaskPositions(s, view, a) return RecalculateTaskPositions(s, view, a)
} }
task, err := GetTaskByIDSimple(s, tp.TaskID)
if err != nil {
return return
} }
doer, _ := user.GetFromAuth(a)
return events.Dispatch(&TaskUpdatedEvent{
Task: &task,
Doer: doer,
})
}
func RecalculateTaskPositions(s *xorm.Session, view *ProjectView, a web.Auth) (err error) { func RecalculateTaskPositions(s *xorm.Session, view *ProjectView, a web.Auth) (err error) {
// Using the collection so that we get all tasks, even in cases where we're dealing with a saved filter underneath // Using the collection so that we get all tasks, even in cases where we're dealing with a saved filter underneath