Du kan inte välja fler än 25 ämnen Ämnen måste starta med en bokstav eller siffra, kan innehålla bindestreck ('-') och vara max 35 tecken långa.

review.go 9.8KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372
  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. }
  111. func (opts *FindReviewOptions) toCond() builder.Cond {
  112. var cond = builder.NewCond()
  113. if opts.IssueID > 0 {
  114. cond = cond.And(builder.Eq{"issue_id": opts.IssueID})
  115. }
  116. if opts.ReviewerID > 0 {
  117. cond = cond.And(builder.Eq{"reviewer_id": opts.ReviewerID})
  118. }
  119. if opts.Type != ReviewTypeUnknown {
  120. cond = cond.And(builder.Eq{"type": opts.Type})
  121. }
  122. return cond
  123. }
  124. func findReviews(e Engine, opts FindReviewOptions) ([]*Review, error) {
  125. reviews := make([]*Review, 0, 10)
  126. sess := e.Where(opts.toCond())
  127. return reviews, sess.
  128. Asc("created_unix").
  129. Asc("id").
  130. Find(&reviews)
  131. }
  132. // FindReviews returns reviews passing FindReviewOptions
  133. func FindReviews(opts FindReviewOptions) ([]*Review, error) {
  134. return findReviews(x, opts)
  135. }
  136. // CreateReviewOptions represent the options to create a review. Type, Issue and Reviewer are required.
  137. type CreateReviewOptions struct {
  138. Content string
  139. Type ReviewType
  140. Issue *Issue
  141. Reviewer *User
  142. Official bool
  143. }
  144. // IsOfficialReviewer check if reviewer can make official reviews in issue (counts towards required approvals)
  145. func IsOfficialReviewer(issue *Issue, reviewer *User) (bool, error) {
  146. return isOfficialReviewer(x, issue, reviewer)
  147. }
  148. func isOfficialReviewer(e Engine, issue *Issue, reviewer *User) (bool, error) {
  149. pr, err := getPullRequestByIssueID(e, issue.ID)
  150. if err != nil {
  151. return false, err
  152. }
  153. if err = pr.loadProtectedBranch(e); err != nil {
  154. return false, err
  155. }
  156. if pr.ProtectedBranch == nil {
  157. return false, nil
  158. }
  159. return pr.ProtectedBranch.isUserOfficialReviewer(e, reviewer)
  160. }
  161. func createReview(e Engine, opts CreateReviewOptions) (*Review, error) {
  162. review := &Review{
  163. Type: opts.Type,
  164. Issue: opts.Issue,
  165. IssueID: opts.Issue.ID,
  166. Reviewer: opts.Reviewer,
  167. ReviewerID: opts.Reviewer.ID,
  168. Content: opts.Content,
  169. Official: opts.Official,
  170. }
  171. if _, err := e.Insert(review); err != nil {
  172. return nil, err
  173. }
  174. return review, nil
  175. }
  176. // CreateReview creates a new review based on opts
  177. func CreateReview(opts CreateReviewOptions) (*Review, error) {
  178. return createReview(x, opts)
  179. }
  180. func getCurrentReview(e Engine, reviewer *User, issue *Issue) (*Review, error) {
  181. if reviewer == nil {
  182. return nil, nil
  183. }
  184. reviews, err := findReviews(e, FindReviewOptions{
  185. Type: ReviewTypePending,
  186. IssueID: issue.ID,
  187. ReviewerID: reviewer.ID,
  188. })
  189. if err != nil {
  190. return nil, err
  191. }
  192. if len(reviews) == 0 {
  193. return nil, ErrReviewNotExist{}
  194. }
  195. reviews[0].Reviewer = reviewer
  196. reviews[0].Issue = issue
  197. return reviews[0], nil
  198. }
  199. // ReviewExists returns whether a review exists for a particular line of code in the PR
  200. func ReviewExists(issue *Issue, treePath string, line int64) (bool, error) {
  201. return x.Cols("id").Exist(&Comment{IssueID: issue.ID, TreePath: treePath, Line: line, Type: CommentTypeCode})
  202. }
  203. // GetCurrentReview returns the current pending review of reviewer for given issue
  204. func GetCurrentReview(reviewer *User, issue *Issue) (*Review, error) {
  205. return getCurrentReview(x, reviewer, issue)
  206. }
  207. // ContentEmptyErr represents an content empty error
  208. type ContentEmptyErr struct {
  209. }
  210. func (ContentEmptyErr) Error() string {
  211. return "Review content is empty"
  212. }
  213. // IsContentEmptyErr returns true if err is a ContentEmptyErr
  214. func IsContentEmptyErr(err error) bool {
  215. _, ok := err.(ContentEmptyErr)
  216. return ok
  217. }
  218. // SubmitReview creates a review out of the existing pending review or creates a new one if no pending review exist
  219. func SubmitReview(doer *User, issue *Issue, reviewType ReviewType, content string) (*Review, *Comment, error) {
  220. sess := x.NewSession()
  221. defer sess.Close()
  222. if err := sess.Begin(); err != nil {
  223. return nil, nil, err
  224. }
  225. var official = false
  226. review, err := getCurrentReview(sess, doer, issue)
  227. if err != nil {
  228. if !IsErrReviewNotExist(err) {
  229. return nil, nil, err
  230. }
  231. if reviewType != ReviewTypeApprove && len(strings.TrimSpace(content)) == 0 {
  232. return nil, nil, ContentEmptyErr{}
  233. }
  234. if reviewType == ReviewTypeApprove || reviewType == ReviewTypeReject {
  235. // Only reviewers latest review of type approve and reject shall count as "official", so existing reviews needs to be cleared
  236. if _, err := sess.Exec("UPDATE `review` SET official=? WHERE issue_id=? AND reviewer_id=?", false, issue.ID, doer.ID); err != nil {
  237. return nil, nil, err
  238. }
  239. official, err = isOfficialReviewer(sess, issue, doer)
  240. if err != nil {
  241. return nil, nil, err
  242. }
  243. }
  244. // No current review. Create a new one!
  245. review, err = createReview(sess, CreateReviewOptions{
  246. Type: reviewType,
  247. Issue: issue,
  248. Reviewer: doer,
  249. Content: content,
  250. Official: official,
  251. })
  252. if err != nil {
  253. return nil, nil, err
  254. }
  255. } else {
  256. if err := review.loadCodeComments(sess); err != nil {
  257. return nil, nil, err
  258. }
  259. if reviewType != ReviewTypeApprove && len(review.CodeComments) == 0 && len(strings.TrimSpace(content)) == 0 {
  260. return nil, nil, ContentEmptyErr{}
  261. }
  262. if reviewType == ReviewTypeApprove || reviewType == ReviewTypeReject {
  263. // Only reviewers latest review of type approve and reject shall count as "official", so existing reviews needs to be cleared
  264. if _, err := sess.Exec("UPDATE `review` SET official=? WHERE issue_id=? AND reviewer_id=?", false, issue.ID, doer.ID); err != nil {
  265. return nil, nil, err
  266. }
  267. official, err = isOfficialReviewer(sess, issue, doer)
  268. if err != nil {
  269. return nil, nil, err
  270. }
  271. }
  272. review.Official = official
  273. review.Issue = issue
  274. review.Content = content
  275. review.Type = reviewType
  276. if _, err := sess.ID(review.ID).Cols("content, type, official").Update(review); err != nil {
  277. return nil, nil, err
  278. }
  279. }
  280. comm, err := createComment(sess, &CreateCommentOptions{
  281. Type: CommentTypeReview,
  282. Doer: doer,
  283. Content: review.Content,
  284. Issue: issue,
  285. Repo: issue.Repo,
  286. ReviewID: review.ID,
  287. })
  288. if err != nil || comm == nil {
  289. return nil, nil, err
  290. }
  291. comm.Review = review
  292. return review, comm, sess.Commit()
  293. }
  294. // GetReviewersByIssueID gets the latest review of each reviewer for a pull request
  295. func GetReviewersByIssueID(issueID int64) (reviews []*Review, err error) {
  296. reviewsUnfiltered := []*Review{}
  297. sess := x.NewSession()
  298. defer sess.Close()
  299. if err := sess.Begin(); err != nil {
  300. return nil, err
  301. }
  302. // Get latest review of each reviwer, sorted in order they were made
  303. 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",
  304. issueID, ReviewTypeApprove, ReviewTypeReject).
  305. Find(&reviewsUnfiltered); err != nil {
  306. return nil, err
  307. }
  308. // Load reviewer and skip if user is deleted
  309. for _, review := range reviewsUnfiltered {
  310. if err := review.loadReviewer(sess); err != nil {
  311. if !IsErrUserNotExist(err) {
  312. return nil, err
  313. }
  314. } else {
  315. reviews = append(reviews, review)
  316. }
  317. }
  318. return reviews, nil
  319. }