kodev-org
/
vikunja
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

refactor: use nested map for position conflict tracking 

Replace struct key map[viewPos]bool with nested
map[int64]map[float64]bool for cleaner lookups.
This commit is contained in:
kolaente 2026-04-03 19:18:37 +02:00 committed by kolaente
parent 104c8eadae
commit ce3e56f192
1 changed files with 6 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
pkg/models/task_position.go
View File

@ -603,18 +603,16 @@ func resolveTaskPositionConflicts(s *xorm.Session, projectViewID int64, conflict
func resolvePositionConflictsAfterInsert(s *xorm.Session, positions []*TaskPosition) error {
// Track which (viewID, position) pairs we've already checked to avoid
// resolving the same conflict group twice.
type viewPos struct {
viewID int64
position float64
}
checked := make(map[viewPos]bool)
checked := make(map[int64]map[float64]bool)
for _, pos := range positions {
key := viewPos{viewID: pos.ProjectViewID, position: pos.Position}
if checked[key] {
if checked[pos.ProjectViewID] != nil && checked[pos.ProjectViewID][pos.Position] {
continue
}
checked[key] = true
if checked[pos.ProjectViewID] == nil {
checked[pos.ProjectViewID] = make(map[float64]bool)
}
checked[pos.ProjectViewID][pos.Position] = true
conflicts, err := findPositionConflicts(s, pos.ProjectViewID, pos.Position)
if err != nil {