femsci/forgejo
femsci/forgejo
1
0
Fork 0
forked from NYANDEV/forgejo
Code Pull requests Activity

Fix ghost user bug (#1913)

Browse source 
Fix bug where unassigned issues would appear to be assigned to a ghost user
This commit is contained in:
Ethan Koenig 2017-06-08 21:51:31 -04:00 committed by Lunny Xiao
parent b1b8c5e9f4
commit 73836ce8c4
1 changed files with 6 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
models/issue_list.go
View file

@ -69,6 +69,9 @@ func (issues IssueList) loadPosters(e Engine) error {
} }
for _, issue := range issues { for _, issue := range issues {
if issue.PosterID <= 0 {
continue
}
var ok bool var ok bool
if issue.Poster, ok = posterMaps[issue.PosterID]; !ok { if issue.Poster, ok = posterMaps[issue.PosterID]; !ok {
issue.Poster = NewGhostUser() issue.Poster = NewGhostUser()
@ -176,6 +179,9 @@ func (issues IssueList) loadAssignees(e Engine) error {
} }
for _, issue := range issues { for _, issue := range issues {
if issue.AssigneeID <= 0 {
continue
}
var ok bool var ok bool
if issue.Assignee, ok = assigneeMaps[issue.AssigneeID]; !ok { if issue.Assignee, ok = assigneeMaps[issue.AssigneeID]; !ok {
issue.Assignee = NewGhostUser() issue.Assignee = NewGhostUser()