Mirrors/gitea
0
0
Fork 0
mirror of https://github.com/go-gitea/gitea.git synced 2025-11-07 05:40:19 +01:00
Code Issues Activity

hide functions which will not be referenced outside of the package

Browse Source
This commit is contained in:
Lunny Xiao 2025-08-22 19:34:11 -07:00
parent 193c6c05c1
commit 76697604ae
No known key found for this signature in database
GPG Key ID: C3B7C91B632F738A
2 changed files with 22 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
services/pull/pull.go
View File

@ -188,7 +188,7 @@ func NewPullRequest(ctx context.Context, opts *NewPullRequestOptions) error {
} }
for _, reviewer := range opts.Reviewers { for _, reviewer := range opts.Reviewers {
err = IsValidReviewRequest(ctx, reviewer, issue.Poster, true, issue, &permDoer) err = isValidReviewRequest(ctx, reviewer, issue.Poster, true, issue, &permDoer)
if err != nil { if err != nil {
return err return err
} }
@ -197,15 +197,17 @@ func NewPullRequest(ctx context.Context, opts *NewPullRequestOptions) error {
if err != nil { if err != nil {
return err return err
} }
reviewNotifiers = append(reviewNotifiers, &ReviewRequestNotifier{ if comment != nil {
Comment: comment, reviewNotifiers = append(reviewNotifiers, &ReviewRequestNotifier{
IsAdd: true, Comment: comment,
Reviewer: reviewer, IsAdd: true,
}) Reviewer: reviewer,
})
}
} }
for _, teamReviewer := range opts.TeamReviewers { for _, teamReviewer := range opts.TeamReviewers {
err = IsValidTeamReviewRequest(ctx, teamReviewer, issue.Poster, true, issue) err = isValidTeamReviewRequest(ctx, teamReviewer, issue.Poster, true, issue)
if err != nil { if err != nil {
return err return err
} }
@ -214,11 +216,13 @@ func NewPullRequest(ctx context.Context, opts *NewPullRequestOptions) error {
if err != nil { if err != nil {
return err return err
} }
reviewNotifiers = append(reviewNotifiers, &ReviewRequestNotifier{ if comment != nil {
Comment: comment, reviewNotifiers = append(reviewNotifiers, &ReviewRequestNotifier{
IsAdd: true, Comment: comment,
ReviewTeam: teamReviewer, IsAdd: true,
}) ReviewTeam: teamReviewer,
})
}
} }
return nil return nil

12
services/pull/review_request.go
View File

@ -19,7 +19,7 @@ import (
// ReviewRequest add or remove a review request from a user for this PR, and make comment for it. // ReviewRequest add or remove a review request from a user for this PR, and make comment for it.
func ReviewRequest(ctx context.Context, issue *issues_model.Issue, doer *user_model.User, permDoer *access_model.Permission, reviewer *user_model.User, isAdd bool) (comment *issues_model.Comment, err error) { func ReviewRequest(ctx context.Context, issue *issues_model.Issue, doer *user_model.User, permDoer *access_model.Permission, reviewer *user_model.User, isAdd bool) (comment *issues_model.Comment, err error) {
err = IsValidReviewRequest(ctx, reviewer, doer, isAdd, issue, permDoer) err = isValidReviewRequest(ctx, reviewer, doer, isAdd, issue, permDoer)
if err != nil { if err != nil {
return nil, err return nil, err
} }
@ -41,8 +41,8 @@ func ReviewRequest(ctx context.Context, issue *issues_model.Issue, doer *user_mo
return comment, err return comment, err
} }
// IsValidReviewRequest Check permission for ReviewRequest // isValidReviewRequest Check permission for ReviewRequest
func IsValidReviewRequest(ctx context.Context, reviewer, doer *user_model.User, isAdd bool, issue *issues_model.Issue, permDoer *access_model.Permission) error { func isValidReviewRequest(ctx context.Context, reviewer, doer *user_model.User, isAdd bool, issue *issues_model.Issue, permDoer *access_model.Permission) error {
if reviewer.IsOrganization() { if reviewer.IsOrganization() {
return issues_model.ErrNotValidReviewRequest{ return issues_model.ErrNotValidReviewRequest{
Reason: "Organization can't be added as reviewer", Reason: "Organization can't be added as reviewer",
@ -125,8 +125,8 @@ func IsValidReviewRequest(ctx context.Context, reviewer, doer *user_model.User,
} }
} }
// IsValidTeamReviewRequest Check permission for ReviewRequest Team // isValidTeamReviewRequest Check permission for ReviewRequest Team
func IsValidTeamReviewRequest(ctx context.Context, reviewer *org_model.Team, doer *user_model.User, isAdd bool, issue *issues_model.Issue) error { func isValidTeamReviewRequest(ctx context.Context, reviewer *org_model.Team, doer *user_model.User, isAdd bool, issue *issues_model.Issue) error {
if doer.IsOrganization() { if doer.IsOrganization() {
return issues_model.ErrNotValidReviewRequest{ return issues_model.ErrNotValidReviewRequest{
Reason: "Organization can't be doer to add reviewer", Reason: "Organization can't be doer to add reviewer",
@ -174,7 +174,7 @@ func IsValidTeamReviewRequest(ctx context.Context, reviewer *org_model.Team, doe
// TeamReviewRequest add or remove a review request from a team for this PR, and make comment for it. // TeamReviewRequest add or remove a review request from a team for this PR, and make comment for it.
func TeamReviewRequest(ctx context.Context, issue *issues_model.Issue, doer *user_model.User, reviewer *org_model.Team, isAdd bool) (comment *issues_model.Comment, err error) { func TeamReviewRequest(ctx context.Context, issue *issues_model.Issue, doer *user_model.User, reviewer *org_model.Team, isAdd bool) (comment *issues_model.Comment, err error) {
err = IsValidTeamReviewRequest(ctx, reviewer, doer, isAdd, issue) err = isValidTeamReviewRequest(ctx, reviewer, doer, isAdd, issue)
if err != nil { if err != nil {
return nil, err return nil, err
} }