ami/tl-vikunja
1
0
Fork 0
Code

fix: don't try to load subscriptions for nonexistent projects

Browse Source
This commit is contained in:
kolaente 2023-07-03 18:18:13 +02:00
parent 32689531ec
commit b5194624e0
No known key found for this signature in database
GPG Key ID: F40E70337AB24C9B
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
pkg/models/subscription.go
View File

@ -262,6 +262,9 @@ func getSubscriptionsForProjects(s *xorm.Session, projectIDs []int64, u *user.Us
var ps = make(map[int64]*Project) var ps = make(map[int64]*Project)
for _, eID := range projectIDs { for _, eID := range projectIDs {
if eID < 1 {
continue
}
ps[eID], err = GetProjectSimpleByID(s, eID) ps[eID], err = GetProjectSimpleByID(s, eID)
if err != nil { if err != nil {
return nil, err return nil, err
@ -309,7 +312,8 @@ func getSubscriptionsForProjects(s *xorm.Session, projectIDs []int64, u *user.Us
// If the current project does not have a subscription, climb up the tree until a project has one, // If the current project does not have a subscription, climb up the tree until a project has one,
// then use that subscription for all child projects // then use that subscription for all child projects
_, has := projectsToSubscriptions[eID] _, has := projectsToSubscriptions[eID]
if !has { _, hasProject := ps[eID]
if !has && hasProject {
var parent = ps[eID].ParentProject var parent = ps[eID].ParentProject
for parent != nil { for parent != nil {
sub, has := projectsToSubscriptions[parent.ID] sub, has := projectsToSubscriptions[parent.ID]