Fix the nil pointer when assigning issues to projects (#25665)

Fixes #25649 Caused by #25468
2 years ago · ff140d4050
parent 0006169f38
commit ff140d4050
2 changed files with 10 additions and 6 deletions
--- a/routers/web/org/projects.go
+++ b/routers/web/org/projects.go
@ -436,10 +436,12 @@ func UpdateIssueProject(ctx *context.Context) {

 	projectID := ctx.FormInt64("id")
 	for _, issue := range issues {
+		if issue.Project != nil {
 			oldProjectID := issue.Project.ID
 			if oldProjectID == projectID {
 				continue
 			}
+		}

 		if err := issues_model.ChangeProjectAssign(issue, ctx.Doer, projectID); err != nil {
 			ctx.ServerError("ChangeProjectAssign", err)
--- a/routers/web/repo/projects.go
+++ b/routers/web/repo/projects.go
@ -385,10 +385,12 @@ func UpdateIssueProject(ctx *context.Context) {

 	projectID := ctx.FormInt64("id")
 	for _, issue := range issues {
+		if issue.Project != nil {
 			oldProjectID := issue.Project.ID
 			if oldProjectID == projectID {
 				continue
 			}
+		}

 		if err := issues_model.ChangeProjectAssign(issue, ctx.Doer, projectID); err != nil {
 			ctx.ServerError("ChangeProjectAssign", err)