Skip to content

Commit

Permalink
Fix wrong user returned in API (go-gitea#15139)
Browse files Browse the repository at this point in the history
The API call: GET /repos/{owner}/{repo}/pulls/{index}/reviews/{id}/comments
returns always the reviewer, but should return the poster.

Co-authored-by: 6543 <6543@obermui.de>
Co-authored-by: zeripath <art27@cantab.net>
  • Loading branch information
3 people committed Mar 25, 2021
1 parent 6dfa92b commit e909d4c
Showing 1 changed file with 6 additions and 7 deletions.
13 changes: 6 additions & 7 deletions modules/convert/pull_review.go
Original file line number Diff line number Diff line change
Expand Up @@ -83,19 +83,18 @@ func ToPullReviewCommentList(review *models.Review, doer *models.User) ([]*api.P

apiComments := make([]*api.PullReviewComment, 0, len(review.CodeComments))

auth := false
if doer != nil {
auth = doer.IsAdmin || doer.ID == review.ReviewerID
}

for _, lines := range review.CodeComments {
for _, comments := range lines {
for _, comment := range comments {
auth := false
if doer != nil {
auth = doer.IsAdmin || doer.ID == comment.Poster.ID
}
apiComment := &api.PullReviewComment{
ID: comment.ID,
Body: comment.Content,
Reviewer: ToUser(review.Reviewer, doer != nil, auth),
ReviewID: review.ID,
Reviewer: ToUser(comment.Poster, doer != nil, auth),
ReviewID: comment.PosterID,
Created: comment.CreatedUnix.AsTime(),
Updated: comment.UpdatedUnix.AsTime(),
Path: comment.TreePath,
Expand Down

0 comments on commit e909d4c

Please sign in to comment.