You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

review.go 9.9KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376
  1. // Copyright 2018 The Gitea Authors. All rights reserved.
  2. // Use of this source code is governed by a MIT-style
  3. // license that can be found in the LICENSE file.
  4. package models
  5. import (
  6. "strings"
  7. "code.gitea.io/gitea/modules/timeutil"
  8. "xorm.io/builder"
  9. )
  10. // ReviewType defines the sort of feedback a review gives
  11. type ReviewType int
  12. // ReviewTypeUnknown unknown review type
  13. const ReviewTypeUnknown ReviewType = -1
  14. const (
  15. // ReviewTypePending is a review which is not published yet
  16. ReviewTypePending ReviewType = iota
  17. // ReviewTypeApprove approves changes
  18. ReviewTypeApprove
  19. // ReviewTypeComment gives general feedback
  20. ReviewTypeComment
  21. // ReviewTypeReject gives feedback blocking merge
  22. ReviewTypeReject
  23. )
  24. // Icon returns the corresponding icon for the review type
  25. func (rt ReviewType) Icon() string {
  26. switch rt {
  27. case ReviewTypeApprove:
  28. return "eye"
  29. case ReviewTypeReject:
  30. return "x"
  31. case ReviewTypeComment, ReviewTypeUnknown:
  32. return "comment"
  33. default:
  34. return "comment"
  35. }
  36. }
  37. // Review represents collection of code comments giving feedback for a PR
  38. type Review struct {
  39. ID int64 `xorm:"pk autoincr"`
  40. Type ReviewType
  41. Reviewer *User `xorm:"-"`
  42. ReviewerID int64 `xorm:"index"`
  43. Issue *Issue `xorm:"-"`
  44. IssueID int64 `xorm:"index"`
  45. Content string `xorm:"TEXT"`
  46. // Official is a review made by an assigned approver (counts towards approval)
  47. Official bool `xorm:"NOT NULL DEFAULT false"`
  48. CreatedUnix timeutil.TimeStamp `xorm:"INDEX created"`
  49. UpdatedUnix timeutil.TimeStamp `xorm:"INDEX updated"`
  50. // CodeComments are the initial code comments of the review
  51. CodeComments CodeComments `xorm:"-"`
  52. }
  53. func (r *Review) loadCodeComments(e Engine) (err error) {
  54. if r.CodeComments == nil {
  55. r.CodeComments, err = fetchCodeCommentsByReview(e, r.Issue, nil, r)
  56. }
  57. return
  58. }
  59. // LoadCodeComments loads CodeComments
  60. func (r *Review) LoadCodeComments() error {
  61. return r.loadCodeComments(x)
  62. }
  63. func (r *Review) loadIssue(e Engine) (err error) {
  64. r.Issue, err = getIssueByID(e, r.IssueID)
  65. return
  66. }
  67. func (r *Review) loadReviewer(e Engine) (err error) {
  68. if r.ReviewerID == 0 {
  69. return nil
  70. }
  71. r.Reviewer, err = getUserByID(e, r.ReviewerID)
  72. return
  73. }
  74. // LoadReviewer loads reviewer
  75. func (r *Review) LoadReviewer() error {
  76. return r.loadReviewer(x)
  77. }
  78. func (r *Review) loadAttributes(e Engine) (err error) {
  79. if err = r.loadReviewer(e); err != nil {
  80. return
  81. }
  82. if err = r.loadIssue(e); err != nil {
  83. return
  84. }
  85. return
  86. }
  87. // LoadAttributes loads all attributes except CodeComments
  88. func (r *Review) LoadAttributes() error {
  89. return r.loadAttributes(x)
  90. }
  91. func getReviewByID(e Engine, id int64) (*Review, error) {
  92. review := new(Review)
  93. if has, err := e.ID(id).Get(review); err != nil {
  94. return nil, err
  95. } else if !has {
  96. return nil, ErrReviewNotExist{ID: id}
  97. } else {
  98. return review, nil
  99. }
  100. }
  101. // GetReviewByID returns the review by the given ID
  102. func GetReviewByID(id int64) (*Review, error) {
  103. return getReviewByID(x, id)
  104. }
  105. // FindReviewOptions represent possible filters to find reviews
  106. type FindReviewOptions struct {
  107. Type ReviewType
  108. IssueID int64
  109. ReviewerID int64
  110. OfficialOnly bool
  111. }
  112. func (opts *FindReviewOptions) toCond() builder.Cond {
  113. var cond = builder.NewCond()
  114. if opts.IssueID > 0 {
  115. cond = cond.And(builder.Eq{"issue_id": opts.IssueID})
  116. }
  117. if opts.ReviewerID > 0 {
  118. cond = cond.And(builder.Eq{"reviewer_id": opts.ReviewerID})
  119. }
  120. if opts.Type != ReviewTypeUnknown {
  121. cond = cond.And(builder.Eq{"type": opts.Type})
  122. }
  123. if opts.OfficialOnly {
  124. cond = cond.And(builder.Eq{"official": true})
  125. }
  126. return cond
  127. }
  128. func findReviews(e Engine, opts FindReviewOptions) ([]*Review, error) {
  129. reviews := make([]*Review, 0, 10)
  130. sess := e.Where(opts.toCond())
  131. return reviews, sess.
  132. Asc("created_unix").
  133. Asc("id").
  134. Find(&reviews)
  135. }
  136. // FindReviews returns reviews passing FindReviewOptions
  137. func FindReviews(opts FindReviewOptions) ([]*Review, error) {
  138. return findReviews(x, opts)
  139. }
  140. // CreateReviewOptions represent the options to create a review. Type, Issue and Reviewer are required.
  141. type CreateReviewOptions struct {
  142. Content string
  143. Type ReviewType
  144. Issue *Issue
  145. Reviewer *User
  146. Official bool
  147. }
  148. // IsOfficialReviewer check if reviewer can make official reviews in issue (counts towards required approvals)
  149. func IsOfficialReviewer(issue *Issue, reviewer *User) (bool, error) {
  150. return isOfficialReviewer(x, issue, reviewer)
  151. }
  152. func isOfficialReviewer(e Engine, issue *Issue, reviewer *User) (bool, error) {
  153. pr, err := getPullRequestByIssueID(e, issue.ID)
  154. if err != nil {
  155. return false, err
  156. }
  157. if err = pr.loadProtectedBranch(e); err != nil {
  158. return false, err
  159. }
  160. if pr.ProtectedBranch == nil {
  161. return false, nil
  162. }
  163. return pr.ProtectedBranch.isUserOfficialReviewer(e, reviewer)
  164. }
  165. func createReview(e Engine, opts CreateReviewOptions) (*Review, error) {
  166. review := &Review{
  167. Type: opts.Type,
  168. Issue: opts.Issue,
  169. IssueID: opts.Issue.ID,
  170. Reviewer: opts.Reviewer,
  171. ReviewerID: opts.Reviewer.ID,
  172. Content: opts.Content,
  173. Official: opts.Official,
  174. }
  175. if _, err := e.Insert(review); err != nil {
  176. return nil, err
  177. }
  178. return review, nil
  179. }
  180. // CreateReview creates a new review based on opts
  181. func CreateReview(opts CreateReviewOptions) (*Review, error) {
  182. return createReview(x, opts)
  183. }
  184. func getCurrentReview(e Engine, reviewer *User, issue *Issue) (*Review, error) {
  185. if reviewer == nil {
  186. return nil, nil
  187. }
  188. reviews, err := findReviews(e, FindReviewOptions{
  189. Type: ReviewTypePending,
  190. IssueID: issue.ID,
  191. ReviewerID: reviewer.ID,
  192. })
  193. if err != nil {
  194. return nil, err
  195. }
  196. if len(reviews) == 0 {
  197. return nil, ErrReviewNotExist{}
  198. }
  199. reviews[0].Reviewer = reviewer
  200. reviews[0].Issue = issue
  201. return reviews[0], nil
  202. }
  203. // ReviewExists returns whether a review exists for a particular line of code in the PR
  204. func ReviewExists(issue *Issue, treePath string, line int64) (bool, error) {
  205. return x.Cols("id").Exist(&Comment{IssueID: issue.ID, TreePath: treePath, Line: line, Type: CommentTypeCode})
  206. }
  207. // GetCurrentReview returns the current pending review of reviewer for given issue
  208. func GetCurrentReview(reviewer *User, issue *Issue) (*Review, error) {
  209. return getCurrentReview(x, reviewer, issue)
  210. }
  211. // ContentEmptyErr represents an content empty error
  212. type ContentEmptyErr struct {
  213. }
  214. func (ContentEmptyErr) Error() string {
  215. return "Review content is empty"
  216. }
  217. // IsContentEmptyErr returns true if err is a ContentEmptyErr
  218. func IsContentEmptyErr(err error) bool {
  219. _, ok := err.(ContentEmptyErr)
  220. return ok
  221. }
  222. // SubmitReview creates a review out of the existing pending review or creates a new one if no pending review exist
  223. func SubmitReview(doer *User, issue *Issue, reviewType ReviewType, content string) (*Review, *Comment, error) {
  224. sess := x.NewSession()
  225. defer sess.Close()
  226. if err := sess.Begin(); err != nil {
  227. return nil, nil, err
  228. }
  229. var official = false
  230. review, err := getCurrentReview(sess, doer, issue)
  231. if err != nil {
  232. if !IsErrReviewNotExist(err) {
  233. return nil, nil, err
  234. }
  235. if reviewType != ReviewTypeApprove && len(strings.TrimSpace(content)) == 0 {
  236. return nil, nil, ContentEmptyErr{}
  237. }
  238. if reviewType == ReviewTypeApprove || reviewType == ReviewTypeReject {
  239. // Only reviewers latest review of type approve and reject shall count as "official", so existing reviews needs to be cleared
  240. if _, err := sess.Exec("UPDATE `review` SET official=? WHERE issue_id=? AND reviewer_id=?", false, issue.ID, doer.ID); err != nil {
  241. return nil, nil, err
  242. }
  243. official, err = isOfficialReviewer(sess, issue, doer)
  244. if err != nil {
  245. return nil, nil, err
  246. }
  247. }
  248. // No current review. Create a new one!
  249. review, err = createReview(sess, CreateReviewOptions{
  250. Type: reviewType,
  251. Issue: issue,
  252. Reviewer: doer,
  253. Content: content,
  254. Official: official,
  255. })
  256. if err != nil {
  257. return nil, nil, err
  258. }
  259. } else {
  260. if err := review.loadCodeComments(sess); err != nil {
  261. return nil, nil, err
  262. }
  263. if reviewType != ReviewTypeApprove && len(review.CodeComments) == 0 && len(strings.TrimSpace(content)) == 0 {
  264. return nil, nil, ContentEmptyErr{}
  265. }
  266. if reviewType == ReviewTypeApprove || reviewType == ReviewTypeReject {
  267. // Only reviewers latest review of type approve and reject shall count as "official", so existing reviews needs to be cleared
  268. if _, err := sess.Exec("UPDATE `review` SET official=? WHERE issue_id=? AND reviewer_id=?", false, issue.ID, doer.ID); err != nil {
  269. return nil, nil, err
  270. }
  271. official, err = isOfficialReviewer(sess, issue, doer)
  272. if err != nil {
  273. return nil, nil, err
  274. }
  275. }
  276. review.Official = official
  277. review.Issue = issue
  278. review.Content = content
  279. review.Type = reviewType
  280. if _, err := sess.ID(review.ID).Cols("content, type, official").Update(review); err != nil {
  281. return nil, nil, err
  282. }
  283. }
  284. comm, err := createComment(sess, &CreateCommentOptions{
  285. Type: CommentTypeReview,
  286. Doer: doer,
  287. Content: review.Content,
  288. Issue: issue,
  289. Repo: issue.Repo,
  290. ReviewID: review.ID,
  291. })
  292. if err != nil || comm == nil {
  293. return nil, nil, err
  294. }
  295. comm.Review = review
  296. return review, comm, sess.Commit()
  297. }
  298. // GetReviewersByIssueID gets the latest review of each reviewer for a pull request
  299. func GetReviewersByIssueID(issueID int64) (reviews []*Review, err error) {
  300. reviewsUnfiltered := []*Review{}
  301. sess := x.NewSession()
  302. defer sess.Close()
  303. if err := sess.Begin(); err != nil {
  304. return nil, err
  305. }
  306. // Get latest review of each reviwer, sorted in order they were made
  307. if err := sess.SQL("SELECT * FROM review WHERE id IN (SELECT max(id) as id FROM review WHERE issue_id = ? AND type in (?, ?) GROUP BY issue_id, reviewer_id) ORDER BY review.updated_unix ASC",
  308. issueID, ReviewTypeApprove, ReviewTypeReject).
  309. Find(&reviewsUnfiltered); err != nil {
  310. return nil, err
  311. }
  312. // Load reviewer and skip if user is deleted
  313. for _, review := range reviewsUnfiltered {
  314. if err := review.loadReviewer(sess); err != nil {
  315. if !IsErrUserNotExist(err) {
  316. return nil, err
  317. }
  318. } else {
  319. reviews = append(reviews, review)
  320. }
  321. }
  322. return reviews, nil
  323. }