From a3bdede2ce0701e1ffcfa7510cea404d6d74e2cb Mon Sep 17 00:00:00 2001 From: Martin Hartkorn Date: Thu, 4 Feb 2016 19:15:21 +0100 Subject: [PATCH] Removed unused method GetUnmergedPullRequestByRepoPathAndHeadBranch --- models/pull.go | 39 --------------------------------------- 1 file changed, 39 deletions(-) diff --git a/models/pull.go b/models/pull.go index a6c15c1322..c52c871c3f 100644 --- a/models/pull.go +++ b/models/pull.go @@ -5,7 +5,6 @@ package models import ( - "errors" "fmt" "os" "path" @@ -397,44 +396,6 @@ func GetUnmergedPullRequestsByBaseInfo(repoID int64, branch string) ([]*PullRequ Join("INNER", "issue", "issue.id=pull_request.issue_id").Find(&prs) } -// Gets a Pull Request by the path of the forked repo and the branch from where the PR -// got submitted. -func GetUnmergedPullRequestByRepoPathAndHeadBranch(user, repo, branch string) (*PullRequest, error) { - userLower := strings.ToLower(user) - repoLower := strings.ToLower(repo) - - pr := new(PullRequest) - if x == nil { - return nil, errors.New("Fail") - } - has, err := x. - Where("head_user_name=? AND head_branch=? AND has_merged=? AND issue.is_closed=? AND repository.lower_name=?", userLower, branch, 0, 0, repoLower). - Join("INNER", "repository", "repository.id=pull_request.head_repo_id"). - Join("INNER", "issue", "issue.id=pull_request.issue_id"). - Get(pr) - - if err != nil { - return nil, err - } else if !has { - return nil, ErrPullRequestNotExist{0, 0, 0, 0, branch, ""} - } - - baseRepo := new(Repository) - has, err = x.Where("repository.id=?", pr.BaseRepoID). - Join("LEFT", "user", "user.id=repository.owner_id"). - Get(baseRepo) - - if err != nil { - return nil, err - } else if !has { - return nil, ErrRepoNotExist{pr.BaseRepoID, 0, ""} - } - - pr.BaseRepo = baseRepo - - return pr, nil -} - // GetPullRequestByID returns a pull request by given ID. func GetPullRequestByID(id int64) (*PullRequest, error) { pr := new(PullRequest)