ami/tl-vikunja
1
0
Fork 0
Code

fix(filters): do not create a default filter for list view when creating a saved filter

Browse Source 
Because a default filter in a filter is highly unexpected, it can lead to these filters cancelling each other out, resulting in confusing behaviour. See https://kolaente.dev/vikunja/vikunja/issues/2547#issuecomment-65180
This commit is contained in:
kolaente 2024-07-11 12:29:09 +02:00
parent 38c30d1eee
commit 5644130f01
No known key found for this signature in database
GPG Key ID: F40E70337AB24C9B
3 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
pkg/models/project.go
View File

@ -822,7 +822,7 @@ func CreateProject(s *xorm.Session, project *Project, auth web.Auth, createBackl
}
if createDefaultViews {
err = CreateDefaultViewsForProject(s, project, auth, createBacklogBucket)
err = CreateDefaultViewsForProject(s, project, auth, createBacklogBucket, true)
if err != nil {
return
}

6
pkg/models/project_view.go
View File

@ -398,13 +398,15 @@ func GetProjectViewByID(s *xorm.Session, id int64) (view *ProjectView, err error
return
}
func CreateDefaultViewsForProject(s *xorm.Session, project *Project, a web.Auth, createBacklogBucket bool) (err error) {
func CreateDefaultViewsForProject(s *xorm.Session, project *Project, a web.Auth, createBacklogBucket bool, createDefaultListFilter bool) (err error) {
list := &ProjectView{
ProjectID: project.ID,
Title: "List",
ViewKind: ProjectViewKindList,
Position: 100,
Filter: "done = false",
}
if createDefaultListFilter {
list.Filter = "done = false"
}
err = createProjectView(s, list, a, createBacklogBucket)
if err != nil {

2
pkg/models/saved_filters.go
View File

@ -123,7 +123,7 @@ func (sf *SavedFilter) Create(s *xorm.Session, auth web.Auth) (err error) {
return
}
err = CreateDefaultViewsForProject(s, &Project{ID: getProjectIDFromSavedFilterID(sf.ID)}, auth, true)
err = CreateDefaultViewsForProject(s, &Project{ID: getProjectIDFromSavedFilterID(sf.ID)}, auth, true, false)
return err
}