123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743 |
- // Copyright 2014 The Gogs Authors. All rights reserved.
- // Use of this source code is governed by a MIT-style
- // license that can be found in the LICENSE file.
- package repo
- import (
- "net/http"
- "path"
- "strings"
- "github.com/unknwon/com"
- log "unknwon.dev/clog/v2"
- "github.com/gogs/git-module"
- "gogs.io/gogs/internal/conf"
- "gogs.io/gogs/internal/context"
- "gogs.io/gogs/internal/db"
- "gogs.io/gogs/internal/form"
- "gogs.io/gogs/internal/gitutil"
- )
- const (
- FORK = "repo/pulls/fork"
- COMPARE_PULL = "repo/pulls/compare"
- PULL_COMMITS = "repo/pulls/commits"
- PULL_FILES = "repo/pulls/files"
- PULL_REQUEST_TEMPLATE_KEY = "PullRequestTemplate"
- PULL_REQUEST_TITLE_TEMPLATE_KEY = "PullRequestTitleTemplate"
- )
- var (
- PullRequestTemplateCandidates = []string{
- "PULL_REQUEST.md",
- ".gogs/PULL_REQUEST.md",
- ".github/PULL_REQUEST.md",
- }
- PullRequestTitleTemplateCandidates = []string{
- "PULL_REQUEST_TITLE.md",
- ".gogs/PULL_REQUEST_TITLE.md",
- ".github/PULL_REQUEST_TITLE.md",
- }
- )
- func parseBaseRepository(c *context.Context) *db.Repository {
- baseRepo, err := db.GetRepositoryByID(c.ParamsInt64(":repoid"))
- if err != nil {
- c.NotFoundOrError(err, "get repository by ID")
- return nil
- }
- if !baseRepo.CanBeForked() || !baseRepo.HasAccess(c.User.ID) {
- c.NotFound()
- return nil
- }
- c.Data["repo_name"] = baseRepo.Name
- c.Data["description"] = baseRepo.Description
- c.Data["IsPrivate"] = baseRepo.IsPrivate
- if err = baseRepo.GetOwner(); err != nil {
- c.Error(err, "get owner")
- return nil
- }
- c.Data["ForkFrom"] = baseRepo.Owner.Name + "/" + baseRepo.Name
- if err := c.User.GetOrganizations(true); err != nil {
- c.Error(err, "get organizations")
- return nil
- }
- c.Data["Orgs"] = c.User.Orgs
- return baseRepo
- }
- func Fork(c *context.Context) {
- c.Data["Title"] = c.Tr("new_fork")
- parseBaseRepository(c)
- if c.Written() {
- return
- }
- c.Data["ContextUser"] = c.User
- c.Success(FORK)
- }
- func ForkPost(c *context.Context, f form.CreateRepo) {
- c.Data["Title"] = c.Tr("new_fork")
- baseRepo := parseBaseRepository(c)
- if c.Written() {
- return
- }
- ctxUser := checkContextUser(c, f.UserID)
- if c.Written() {
- return
- }
- c.Data["ContextUser"] = ctxUser
- if c.HasError() {
- c.Success(FORK)
- return
- }
- repo, has, err := db.HasForkedRepo(ctxUser.ID, baseRepo.ID)
- if err != nil {
- c.Error(err, "check forked repository")
- return
- } else if has {
- c.Redirect(repo.Link())
- return
- }
- // Check ownership of organization.
- if ctxUser.IsOrganization() && !ctxUser.IsOwnedBy(c.User.ID) {
- c.Status(http.StatusForbidden)
- return
- }
- // Cannot fork to same owner
- if ctxUser.ID == baseRepo.OwnerID {
- c.RenderWithErr(c.Tr("repo.settings.cannot_fork_to_same_owner"), FORK, &f)
- return
- }
- repo, err = db.ForkRepository(c.User, ctxUser, baseRepo, f.RepoName, f.Description)
- if err != nil {
- c.Data["Err_RepoName"] = true
- switch {
- case db.IsErrReachLimitOfRepo(err):
- c.RenderWithErr(c.Tr("repo.form.reach_limit_of_creation", c.User.RepoCreationNum()), FORK, &f)
- case db.IsErrRepoAlreadyExist(err):
- c.RenderWithErr(c.Tr("repo.settings.new_owner_has_same_repo"), FORK, &f)
- case db.IsErrNameNotAllowed(err):
- c.RenderWithErr(c.Tr("repo.form.name_not_allowed", err.(db.ErrNameNotAllowed).Value()), FORK, &f)
- default:
- c.Error(err, "fork repository")
- }
- return
- }
- log.Trace("Repository forked from '%s' -> '%s'", baseRepo.FullName(), repo.FullName())
- c.Redirect(repo.Link())
- }
- func checkPullInfo(c *context.Context) *db.Issue {
- issue, err := db.GetIssueByIndex(c.Repo.Repository.ID, c.ParamsInt64(":index"))
- if err != nil {
- c.NotFoundOrError(err, "get issue by index")
- return nil
- }
- c.Data["Title"] = issue.Title
- c.Data["Issue"] = issue
- if !issue.IsPull {
- c.NotFound()
- return nil
- }
- if c.IsLogged {
- // Update issue-user.
- if err = issue.ReadBy(c.User.ID); err != nil {
- c.Error(err, "mark read by")
- return nil
- }
- }
- return issue
- }
- func PrepareMergedViewPullInfo(c *context.Context, issue *db.Issue) {
- pull := issue.PullRequest
- c.Data["HasMerged"] = true
- c.Data["HeadTarget"] = issue.PullRequest.HeadUserName + "/" + pull.HeadBranch
- c.Data["BaseTarget"] = c.Repo.Owner.Name + "/" + pull.BaseBranch
- var err error
- c.Data["NumCommits"], err = c.Repo.GitRepo.RevListCount([]string{pull.MergeBase + "..." + pull.MergedCommitID})
- if err != nil {
- c.Error(err, "count commits")
- return
- }
- names, err := c.Repo.GitRepo.DiffNameOnly(pull.MergeBase, pull.MergedCommitID, git.DiffNameOnlyOptions{NeedsMergeBase: true})
- c.Data["NumFiles"] = len(names)
- if err != nil {
- c.Error(err, "get changed files")
- return
- }
- }
- func PrepareViewPullInfo(c *context.Context, issue *db.Issue) *gitutil.PullRequestMeta {
- repo := c.Repo.Repository
- pull := issue.PullRequest
- c.Data["HeadTarget"] = pull.HeadUserName + "/" + pull.HeadBranch
- c.Data["BaseTarget"] = c.Repo.Owner.Name + "/" + pull.BaseBranch
- var (
- headGitRepo *git.Repository
- err error
- )
- if pull.HeadRepo != nil {
- headGitRepo, err = git.Open(pull.HeadRepo.RepoPath())
- if err != nil {
- c.Error(err, "open repository")
- return nil
- }
- }
- if pull.HeadRepo == nil || !headGitRepo.HasBranch(pull.HeadBranch) {
- c.Data["IsPullReuqestBroken"] = true
- c.Data["HeadTarget"] = "deleted"
- c.Data["NumCommits"] = 0
- c.Data["NumFiles"] = 0
- return nil
- }
- baseRepoPath := db.RepoPath(repo.Owner.Name, repo.Name)
- prMeta, err := gitutil.Module.PullRequestMeta(headGitRepo.Path(), baseRepoPath, pull.HeadBranch, pull.BaseBranch)
- if err != nil {
- if strings.Contains(err.Error(), "fatal: Not a valid object name") {
- c.Data["IsPullReuqestBroken"] = true
- c.Data["BaseTarget"] = "deleted"
- c.Data["NumCommits"] = 0
- c.Data["NumFiles"] = 0
- return nil
- }
- c.Error(err, "get pull request meta")
- return nil
- }
- c.Data["NumCommits"] = len(prMeta.Commits)
- c.Data["NumFiles"] = prMeta.NumFiles
- return prMeta
- }
- func ViewPullCommits(c *context.Context) {
- c.Data["PageIsPullList"] = true
- c.Data["PageIsPullCommits"] = true
- issue := checkPullInfo(c)
- if c.Written() {
- return
- }
- pull := issue.PullRequest
- if pull.HeadRepo != nil {
- c.Data["Username"] = pull.HeadUserName
- c.Data["Reponame"] = pull.HeadRepo.Name
- }
- var commits []*git.Commit
- if pull.HasMerged {
- PrepareMergedViewPullInfo(c, issue)
- if c.Written() {
- return
- }
- startCommit, err := c.Repo.GitRepo.CatFileCommit(pull.MergeBase)
- if err != nil {
- c.Error(err, "get commit of merge base")
- return
- }
- endCommit, err := c.Repo.GitRepo.CatFileCommit(pull.MergedCommitID)
- if err != nil {
- c.Error(err, "get merged commit")
- return
- }
- commits, err = c.Repo.GitRepo.RevList([]string{startCommit.ID.String() + "..." + endCommit.ID.String()})
- if err != nil {
- c.Error(err, "list commits")
- return
- }
- } else {
- prInfo := PrepareViewPullInfo(c, issue)
- if c.Written() {
- return
- } else if prInfo == nil {
- c.NotFound()
- return
- }
- commits = prInfo.Commits
- }
- c.Data["Commits"] = db.ValidateCommitsWithEmails(commits)
- c.Data["CommitsCount"] = len(commits)
- c.Success(PULL_COMMITS)
- }
- func ViewPullFiles(c *context.Context) {
- c.Data["PageIsPullList"] = true
- c.Data["PageIsPullFiles"] = true
- issue := checkPullInfo(c)
- if c.Written() {
- return
- }
- pull := issue.PullRequest
- var (
- diffGitRepo *git.Repository
- startCommitID string
- endCommitID string
- gitRepo *git.Repository
- )
- if pull.HasMerged {
- PrepareMergedViewPullInfo(c, issue)
- if c.Written() {
- return
- }
- diffGitRepo = c.Repo.GitRepo
- startCommitID = pull.MergeBase
- endCommitID = pull.MergedCommitID
- gitRepo = c.Repo.GitRepo
- } else {
- prInfo := PrepareViewPullInfo(c, issue)
- if c.Written() {
- return
- } else if prInfo == nil {
- c.NotFound()
- return
- }
- headRepoPath := db.RepoPath(pull.HeadUserName, pull.HeadRepo.Name)
- headGitRepo, err := git.Open(headRepoPath)
- if err != nil {
- c.Error(err, "open repository")
- return
- }
- headCommitID, err := headGitRepo.BranchCommitID(pull.HeadBranch)
- if err != nil {
- c.Error(err, "get head branch commit ID")
- return
- }
- diffGitRepo = headGitRepo
- startCommitID = prInfo.MergeBase
- endCommitID = headCommitID
- gitRepo = headGitRepo
- }
- diff, err := gitutil.RepoDiff(diffGitRepo,
- endCommitID, conf.Git.MaxDiffFiles, conf.Git.MaxDiffLines, conf.Git.MaxDiffLineChars,
- git.DiffOptions{Base: startCommitID},
- )
- if err != nil {
- c.Error(err, "get diff")
- return
- }
- c.Data["Diff"] = diff
- c.Data["DiffNotAvailable"] = diff.NumFiles() == 0
- commit, err := gitRepo.CatFileCommit(endCommitID)
- if err != nil {
- c.Error(err, "get commit")
- return
- }
- setEditorconfigIfExists(c)
- if c.Written() {
- return
- }
- c.Data["IsSplitStyle"] = c.Query("style") == "split"
- c.Data["IsImageFile"] = commit.IsImageFile
- c.Data["IsImageFileByIndex"] = commit.IsImageFileByIndex
- // It is possible head repo has been deleted for merged pull requests
- if pull.HeadRepo != nil {
- c.Data["Username"] = pull.HeadUserName
- c.Data["Reponame"] = pull.HeadRepo.Name
- headTarget := path.Join(pull.HeadUserName, pull.HeadRepo.Name)
- c.Data["SourcePath"] = conf.Server.Subpath + "/" + path.Join(headTarget, "src", endCommitID)
- c.Data["RawPath"] = conf.Server.Subpath + "/" + path.Join(headTarget, "raw", endCommitID)
- c.Data["BeforeSourcePath"] = conf.Server.Subpath + "/" + path.Join(headTarget, "src", startCommitID)
- c.Data["BeforeRawPath"] = conf.Server.Subpath + "/" + path.Join(headTarget, "raw", startCommitID)
- }
- c.Data["RequireHighlightJS"] = true
- c.Success(PULL_FILES)
- }
- func MergePullRequest(c *context.Context) {
- issue := checkPullInfo(c)
- if c.Written() {
- return
- }
- if issue.IsClosed {
- c.NotFound()
- return
- }
- pr, err := db.GetPullRequestByIssueID(issue.ID)
- if err != nil {
- c.NotFoundOrError(err, "get pull request by issue ID")
- return
- }
- if !pr.CanAutoMerge() || pr.HasMerged {
- c.NotFound()
- return
- }
- pr.Issue = issue
- pr.Issue.Repo = c.Repo.Repository
- if err = pr.Merge(c.User, c.Repo.GitRepo, db.MergeStyle(c.Query("merge_style")), c.Query("commit_description")); err != nil {
- c.Error(err, "merge")
- return
- }
- log.Trace("Pull request merged: %d", pr.ID)
- c.Redirect(c.Repo.RepoLink + "/pulls/" + com.ToStr(pr.Index))
- }
- func ParseCompareInfo(c *context.Context) (*db.User, *db.Repository, *git.Repository, *gitutil.PullRequestMeta, string, string) {
- baseRepo := c.Repo.Repository
- // Get compared branches information
- // format: <base branch>...[<head repo>:]<head branch>
- // base<-head: master...head:feature
- // same repo: master...feature
- infos := strings.Split(c.Params("*"), "...")
- if len(infos) != 2 {
- log.Trace("ParseCompareInfo[%d]: not enough compared branches information %s", baseRepo.ID, infos)
- c.NotFound()
- return nil, nil, nil, nil, "", ""
- }
- baseBranch := infos[0]
- c.Data["BaseBranch"] = baseBranch
- var (
- headUser *db.User
- headBranch string
- isSameRepo bool
- err error
- )
- // If there is no head repository, it means pull request between same repository.
- headInfos := strings.Split(infos[1], ":")
- if len(headInfos) == 1 {
- isSameRepo = true
- headUser = c.Repo.Owner
- headBranch = headInfos[0]
- } else if len(headInfos) == 2 {
- headUser, err = db.GetUserByName(headInfos[0])
- if err != nil {
- c.NotFoundOrError(err, "get user by name")
- return nil, nil, nil, nil, "", ""
- }
- headBranch = headInfos[1]
- isSameRepo = headUser.ID == baseRepo.OwnerID
- } else {
- c.NotFound()
- return nil, nil, nil, nil, "", ""
- }
- c.Data["HeadUser"] = headUser
- c.Data["HeadBranch"] = headBranch
- c.Repo.PullRequest.SameRepo = isSameRepo
- // Check if base branch is valid.
- if !c.Repo.GitRepo.HasBranch(baseBranch) {
- c.NotFound()
- return nil, nil, nil, nil, "", ""
- }
- var (
- headRepo *db.Repository
- headGitRepo *git.Repository
- )
- // In case user included redundant head user name for comparison in same repository,
- // no need to check the fork relation.
- if !isSameRepo {
- var has bool
- headRepo, has, err = db.HasForkedRepo(headUser.ID, baseRepo.ID)
- if err != nil {
- c.Error(err, "get forked repository")
- return nil, nil, nil, nil, "", ""
- } else if !has {
- log.Trace("ParseCompareInfo [base_repo_id: %d]: does not have fork or in same repository", baseRepo.ID)
- c.NotFound()
- return nil, nil, nil, nil, "", ""
- }
- headGitRepo, err = git.Open(db.RepoPath(headUser.Name, headRepo.Name))
- if err != nil {
- c.Error(err, "open repository")
- return nil, nil, nil, nil, "", ""
- }
- } else {
- headRepo = c.Repo.Repository
- headGitRepo = c.Repo.GitRepo
- }
- if !c.User.IsWriterOfRepo(headRepo) && !c.User.IsAdmin {
- log.Trace("ParseCompareInfo [base_repo_id: %d]: does not have write access or site admin", baseRepo.ID)
- c.NotFound()
- return nil, nil, nil, nil, "", ""
- }
- // Check if head branch is valid.
- if !headGitRepo.HasBranch(headBranch) {
- c.NotFound()
- return nil, nil, nil, nil, "", ""
- }
- headBranches, err := headGitRepo.Branches()
- if err != nil {
- c.Error(err, "get branches")
- return nil, nil, nil, nil, "", ""
- }
- c.Data["HeadBranches"] = headBranches
- baseRepoPath := db.RepoPath(baseRepo.Owner.Name, baseRepo.Name)
- meta, err := gitutil.Module.PullRequestMeta(headGitRepo.Path(), baseRepoPath, headBranch, baseBranch)
- if err != nil {
- if gitutil.IsErrNoMergeBase(err) {
- c.Data["IsNoMergeBase"] = true
- c.Success(COMPARE_PULL)
- } else {
- c.Error(err, "get pull request meta")
- }
- return nil, nil, nil, nil, "", ""
- }
- c.Data["BeforeCommitID"] = meta.MergeBase
- return headUser, headRepo, headGitRepo, meta, baseBranch, headBranch
- }
- func PrepareCompareDiff(
- c *context.Context,
- headUser *db.User,
- headRepo *db.Repository,
- headGitRepo *git.Repository,
- meta *gitutil.PullRequestMeta,
- headBranch string,
- ) bool {
- var (
- repo = c.Repo.Repository
- err error
- )
- // Get diff information.
- c.Data["CommitRepoLink"] = headRepo.Link()
- headCommitID, err := headGitRepo.BranchCommitID(headBranch)
- if err != nil {
- c.Error(err, "get head branch commit ID")
- return false
- }
- c.Data["AfterCommitID"] = headCommitID
- if headCommitID == meta.MergeBase {
- c.Data["IsNothingToCompare"] = true
- return true
- }
- diff, err := gitutil.RepoDiff(headGitRepo,
- headCommitID, conf.Git.MaxDiffFiles, conf.Git.MaxDiffLines, conf.Git.MaxDiffLineChars,
- git.DiffOptions{Base: meta.MergeBase},
- )
- if err != nil {
- c.Error(err, "get repository diff")
- return false
- }
- c.Data["Diff"] = diff
- c.Data["DiffNotAvailable"] = diff.NumFiles() == 0
- headCommit, err := headGitRepo.CatFileCommit(headCommitID)
- if err != nil {
- c.Error(err, "get head commit")
- return false
- }
- c.Data["Commits"] = db.ValidateCommitsWithEmails(meta.Commits)
- c.Data["CommitCount"] = len(meta.Commits)
- c.Data["Username"] = headUser.Name
- c.Data["Reponame"] = headRepo.Name
- c.Data["IsImageFile"] = headCommit.IsImageFile
- c.Data["IsImageFileByIndex"] = headCommit.IsImageFileByIndex
- headTarget := path.Join(headUser.Name, repo.Name)
- c.Data["SourcePath"] = conf.Server.Subpath + "/" + path.Join(headTarget, "src", headCommitID)
- c.Data["RawPath"] = conf.Server.Subpath + "/" + path.Join(headTarget, "raw", headCommitID)
- c.Data["BeforeSourcePath"] = conf.Server.Subpath + "/" + path.Join(headTarget, "src", meta.MergeBase)
- c.Data["BeforeRawPath"] = conf.Server.Subpath + "/" + path.Join(headTarget, "raw", meta.MergeBase)
- return false
- }
- func CompareAndPullRequest(c *context.Context) {
- c.Data["Title"] = c.Tr("repo.pulls.compare_changes")
- c.Data["PageIsComparePull"] = true
- c.Data["IsDiffCompare"] = true
- c.Data["RequireHighlightJS"] = true
- setTemplateIfExists(c, PULL_REQUEST_TEMPLATE_KEY, PullRequestTemplateCandidates)
- renderAttachmentSettings(c)
- headUser, headRepo, headGitRepo, prInfo, baseBranch, headBranch := ParseCompareInfo(c)
- if c.Written() {
- return
- }
- pr, err := db.GetUnmergedPullRequest(headRepo.ID, c.Repo.Repository.ID, headBranch, baseBranch)
- if err != nil {
- if !db.IsErrPullRequestNotExist(err) {
- c.Error(err, "get unmerged pull request")
- return
- }
- } else {
- c.Data["HasPullRequest"] = true
- c.Data["PullRequest"] = pr
- c.Success(COMPARE_PULL)
- return
- }
- nothingToCompare := PrepareCompareDiff(c, headUser, headRepo, headGitRepo, prInfo, headBranch)
- if c.Written() {
- return
- }
- if !nothingToCompare {
- // Setup information for new form.
- RetrieveRepoMetas(c, c.Repo.Repository)
- if c.Written() {
- return
- }
- }
- setEditorconfigIfExists(c)
- if c.Written() {
- return
- }
- c.Data["IsSplitStyle"] = c.Query("style") == "split"
- setTemplateIfExists(c, PULL_REQUEST_TITLE_TEMPLATE_KEY, PullRequestTitleTemplateCandidates)
- if c.Data[PULL_REQUEST_TITLE_TEMPLATE_KEY] != nil {
- customTitle := c.Data[PULL_REQUEST_TITLE_TEMPLATE_KEY].(string)
- r := strings.NewReplacer("{{headBranch}}", headBranch, "{{baseBranch}}", baseBranch)
- c.Data["title"] = r.Replace(customTitle)
- }
- c.Success(COMPARE_PULL)
- }
- func CompareAndPullRequestPost(c *context.Context, f form.NewIssue) {
- c.Data["Title"] = c.Tr("repo.pulls.compare_changes")
- c.Data["PageIsComparePull"] = true
- c.Data["IsDiffCompare"] = true
- c.Data["RequireHighlightJS"] = true
- renderAttachmentSettings(c)
- var (
- repo = c.Repo.Repository
- attachments []string
- )
- headUser, headRepo, headGitRepo, meta, baseBranch, headBranch := ParseCompareInfo(c)
- if c.Written() {
- return
- }
- labelIDs, milestoneID, assigneeID := ValidateRepoMetas(c, f)
- if c.Written() {
- return
- }
- if conf.Attachment.Enabled {
- attachments = f.Files
- }
- if c.HasError() {
- form.Assign(f, c.Data)
- // This stage is already stop creating new pull request, so it does not matter if it has
- // something to compare or not.
- PrepareCompareDiff(c, headUser, headRepo, headGitRepo, meta, headBranch)
- if c.Written() {
- return
- }
- c.Success(COMPARE_PULL)
- return
- }
- patch, err := headGitRepo.DiffBinary(meta.MergeBase, headBranch)
- if err != nil {
- c.Error(err, "get patch")
- return
- }
- pullIssue := &db.Issue{
- RepoID: repo.ID,
- Index: repo.NextIssueIndex(),
- Title: f.Title,
- PosterID: c.User.ID,
- Poster: c.User,
- MilestoneID: milestoneID,
- AssigneeID: assigneeID,
- IsPull: true,
- Content: f.Content,
- }
- pullRequest := &db.PullRequest{
- HeadRepoID: headRepo.ID,
- BaseRepoID: repo.ID,
- HeadUserName: headUser.Name,
- HeadBranch: headBranch,
- BaseBranch: baseBranch,
- HeadRepo: headRepo,
- BaseRepo: repo,
- MergeBase: meta.MergeBase,
- Type: db.PULL_REQUEST_GOGS,
- }
- // FIXME: check error in the case two people send pull request at almost same time, give nice error prompt
- // instead of 500.
- if err := db.NewPullRequest(repo, pullIssue, labelIDs, attachments, pullRequest, patch); err != nil {
- c.Error(err, "new pull request")
- return
- } else if err := pullRequest.PushToBaseRepo(); err != nil {
- c.Error(err, "push to base repository")
- return
- }
- log.Trace("Pull request created: %d/%d", repo.ID, pullIssue.ID)
- c.Redirect(c.Repo.RepoLink + "/pulls/" + com.ToStr(pullIssue.Index))
- }
|