From 10b555a52b0541fa5e4bc3ddb43001abc2ce0c76 Mon Sep 17 00:00:00 2001 From: Giteabot Date: Sat, 13 Jan 2024 05:53:16 +0800 Subject: [PATCH] Assign pull request to project during creation (#28227) (#28775) Backport #28227 by @denyskon When creating a pull request, allow directly assigning it to a project, as it is already possible for issues. After: ![grafik](https://github.com/go-gitea/gitea/assets/47871822/01dc2b3d-d56a-4053-b2fc-138725d7633a) --------- Co-authored-by: Denys Konovalov Co-authored-by: delvh (cherry picked from commit 80e564087dc1a89b8b74abcdc879c2de444d3278) --- routers/web/repo/compare.go | 1 + routers/web/repo/pull.go | 13 ++++++++++++- 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/routers/web/repo/compare.go b/routers/web/repo/compare.go index fc5f82ec06..b0d93fc7b0 100644 --- a/routers/web/repo/compare.go +++ b/routers/web/repo/compare.go @@ -844,6 +844,7 @@ func CompareDiff(ctx *context.Context) { } } + ctx.Data["IsProjectsEnabled"] = ctx.Repo.CanWrite(unit.TypeProjects) ctx.Data["IsAttachmentEnabled"] = setting.Attachment.Enabled upload.AddUploadContext(ctx, "comment") diff --git a/routers/web/repo/pull.go b/routers/web/repo/pull.go index 852b24c52d..67416927ac 100644 --- a/routers/web/repo/pull.go +++ b/routers/web/repo/pull.go @@ -1371,7 +1371,7 @@ func CompareAndPullRequestPost(ctx *context.Context) { return } - labelIDs, assigneeIDs, milestoneID, _ := ValidateRepoMetas(ctx, *form, true) + labelIDs, assigneeIDs, milestoneID, projectID := ValidateRepoMetas(ctx, *form, true) if ctx.Written() { return } @@ -1453,6 +1453,17 @@ func CompareAndPullRequestPost(ctx *context.Context) { return } + if projectID > 0 { + if !ctx.Repo.CanWrite(unit.TypeProjects) { + ctx.Error(http.StatusBadRequest, "user hasn't the permission to write to projects") + return + } + if err := issues_model.ChangeProjectAssign(pullIssue, ctx.Doer, projectID); err != nil { + ctx.ServerError("ChangeProjectAssign", err) + return + } + } + log.Trace("Pull request created: %d/%d", repo.ID, pullIssue.ID) ctx.JSONRedirect(pullIssue.Link()) }