2018-08-05 23:43:22 -05:00
// Copyright 2018 The Gitea Authors. All rights reserved.
// Use of this source code is governed by a MIT-style
// license that can be found in the LICENSE file.
package models
import (
2019-11-13 21:57:36 -05:00
"strings"
2018-08-05 23:43:22 -05:00
2019-08-15 09:46:21 -05:00
"code.gitea.io/gitea/modules/timeutil"
2018-08-05 23:43:22 -05:00
2019-06-23 10:22:43 -05:00
"xorm.io/builder"
2018-08-05 23:43:22 -05:00
)
// ReviewType defines the sort of feedback a review gives
type ReviewType int
// ReviewTypeUnknown unknown review type
const ReviewTypeUnknown ReviewType = - 1
const (
// ReviewTypePending is a review which is not published yet
ReviewTypePending ReviewType = iota
// ReviewTypeApprove approves changes
ReviewTypeApprove
// ReviewTypeComment gives general feedback
ReviewTypeComment
// ReviewTypeReject gives feedback blocking merge
ReviewTypeReject
2020-04-06 11:33:34 -05:00
// ReviewTypeRequest request review from others
ReviewTypeRequest
2018-08-05 23:43:22 -05:00
)
// Icon returns the corresponding icon for the review type
func ( rt ReviewType ) Icon ( ) string {
switch rt {
case ReviewTypeApprove :
2020-04-03 00:12:42 -05:00
return "check"
2018-08-05 23:43:22 -05:00
case ReviewTypeReject :
2020-04-03 00:12:42 -05:00
return "request-changes"
case ReviewTypeComment :
2018-10-19 08:36:41 -05:00
return "comment"
2020-04-06 11:33:34 -05:00
case ReviewTypeRequest :
return "primitive-dot"
2018-08-05 23:43:22 -05:00
default :
return "comment"
}
}
// Review represents collection of code comments giving feedback for a PR
type Review struct {
2020-01-23 12:28:15 -05:00
ID int64 ` xorm:"pk autoincr" `
Type ReviewType
Reviewer * User ` xorm:"-" `
ReviewerID int64 ` xorm:"index" `
OriginalAuthor string
OriginalAuthorID int64
Issue * Issue ` xorm:"-" `
IssueID int64 ` xorm:"index" `
Content string ` xorm:"TEXT" `
2019-12-03 20:08:56 -05:00
// Official is a review made by an assigned approver (counts towards approval)
2020-01-08 20:47:45 -05:00
Official bool ` xorm:"NOT NULL DEFAULT false" `
CommitID string ` xorm:"VARCHAR(40)" `
Stale bool ` xorm:"NOT NULL DEFAULT false" `
2018-08-05 23:43:22 -05:00
2019-08-15 09:46:21 -05:00
CreatedUnix timeutil . TimeStamp ` xorm:"INDEX created" `
UpdatedUnix timeutil . TimeStamp ` xorm:"INDEX updated" `
2018-08-05 23:43:22 -05:00
// CodeComments are the initial code comments of the review
CodeComments CodeComments ` xorm:"-" `
2020-01-23 12:28:15 -05:00
Comments [ ] * Comment ` xorm:"-" `
2018-08-05 23:43:22 -05:00
}
func ( r * Review ) loadCodeComments ( e Engine ) ( err error ) {
2019-11-15 07:59:21 -05:00
if r . CodeComments == nil {
r . CodeComments , err = fetchCodeCommentsByReview ( e , r . Issue , nil , r )
}
2018-08-05 23:43:22 -05:00
return
}
// LoadCodeComments loads CodeComments
func ( r * Review ) LoadCodeComments ( ) error {
return r . loadCodeComments ( x )
}
func ( r * Review ) loadIssue ( e Engine ) ( err error ) {
r . Issue , err = getIssueByID ( e , r . IssueID )
return
}
func ( r * Review ) loadReviewer ( e Engine ) ( err error ) {
if r . ReviewerID == 0 {
return nil
}
r . Reviewer , err = getUserByID ( e , r . ReviewerID )
return
}
2019-11-13 21:57:36 -05:00
// LoadReviewer loads reviewer
func ( r * Review ) LoadReviewer ( ) error {
return r . loadReviewer ( x )
}
2018-08-05 23:43:22 -05:00
func ( r * Review ) loadAttributes ( e Engine ) ( err error ) {
if err = r . loadReviewer ( e ) ; err != nil {
return
}
if err = r . loadIssue ( e ) ; err != nil {
return
}
return
}
// LoadAttributes loads all attributes except CodeComments
func ( r * Review ) LoadAttributes ( ) error {
return r . loadAttributes ( x )
}
func getReviewByID ( e Engine , id int64 ) ( * Review , error ) {
review := new ( Review )
if has , err := e . ID ( id ) . Get ( review ) ; err != nil {
return nil , err
} else if ! has {
return nil , ErrReviewNotExist { ID : id }
} else {
return review , nil
}
}
// GetReviewByID returns the review by the given ID
func GetReviewByID ( id int64 ) ( * Review , error ) {
return getReviewByID ( x , id )
}
// FindReviewOptions represent possible filters to find reviews
type FindReviewOptions struct {
2019-12-30 18:34:11 -05:00
Type ReviewType
IssueID int64
ReviewerID int64
OfficialOnly bool
2018-08-05 23:43:22 -05:00
}
func ( opts * FindReviewOptions ) toCond ( ) builder . Cond {
var cond = builder . NewCond ( )
if opts . IssueID > 0 {
cond = cond . And ( builder . Eq { "issue_id" : opts . IssueID } )
}
if opts . ReviewerID > 0 {
cond = cond . And ( builder . Eq { "reviewer_id" : opts . ReviewerID } )
}
if opts . Type != ReviewTypeUnknown {
cond = cond . And ( builder . Eq { "type" : opts . Type } )
}
2019-12-30 18:34:11 -05:00
if opts . OfficialOnly {
cond = cond . And ( builder . Eq { "official" : true } )
}
2018-08-05 23:43:22 -05:00
return cond
}
func findReviews ( e Engine , opts FindReviewOptions ) ( [ ] * Review , error ) {
reviews := make ( [ ] * Review , 0 , 10 )
sess := e . Where ( opts . toCond ( ) )
return reviews , sess .
Asc ( "created_unix" ) .
Asc ( "id" ) .
Find ( & reviews )
}
// FindReviews returns reviews passing FindReviewOptions
func FindReviews ( opts FindReviewOptions ) ( [ ] * Review , error ) {
return findReviews ( x , opts )
}
// CreateReviewOptions represent the options to create a review. Type, Issue and Reviewer are required.
type CreateReviewOptions struct {
Content string
Type ReviewType
Issue * Issue
Reviewer * User
2019-12-03 20:08:56 -05:00
Official bool
2020-01-08 20:47:45 -05:00
CommitID string
Stale bool
2019-12-03 20:08:56 -05:00
}
// IsOfficialReviewer check if reviewer can make official reviews in issue (counts towards required approvals)
func IsOfficialReviewer ( issue * Issue , reviewer * User ) ( bool , error ) {
return isOfficialReviewer ( x , issue , reviewer )
}
func isOfficialReviewer ( e Engine , issue * Issue , reviewer * User ) ( bool , error ) {
pr , err := getPullRequestByIssueID ( e , issue . ID )
if err != nil {
return false , err
}
if err = pr . loadProtectedBranch ( e ) ; err != nil {
return false , err
}
if pr . ProtectedBranch == nil {
return false , nil
}
return pr . ProtectedBranch . isUserOfficialReviewer ( e , reviewer )
2018-08-05 23:43:22 -05:00
}
func createReview ( e Engine , opts CreateReviewOptions ) ( * Review , error ) {
review := & Review {
Type : opts . Type ,
Issue : opts . Issue ,
IssueID : opts . Issue . ID ,
Reviewer : opts . Reviewer ,
ReviewerID : opts . Reviewer . ID ,
Content : opts . Content ,
2019-12-03 20:08:56 -05:00
Official : opts . Official ,
2020-01-08 20:47:45 -05:00
CommitID : opts . CommitID ,
Stale : opts . Stale ,
2018-08-05 23:43:22 -05:00
}
if _ , err := e . Insert ( review ) ; err != nil {
return nil , err
}
2018-12-27 13:04:30 -05:00
2018-08-05 23:43:22 -05:00
return review , nil
}
// CreateReview creates a new review based on opts
func CreateReview ( opts CreateReviewOptions ) ( * Review , error ) {
return createReview ( x , opts )
}
func getCurrentReview ( e Engine , reviewer * User , issue * Issue ) ( * Review , error ) {
if reviewer == nil {
return nil , nil
}
reviews , err := findReviews ( e , FindReviewOptions {
Type : ReviewTypePending ,
IssueID : issue . ID ,
ReviewerID : reviewer . ID ,
} )
if err != nil {
return nil , err
}
if len ( reviews ) == 0 {
return nil , ErrReviewNotExist { }
}
2018-10-18 06:23:05 -05:00
reviews [ 0 ] . Reviewer = reviewer
reviews [ 0 ] . Issue = issue
2018-08-05 23:43:22 -05:00
return reviews [ 0 ] , nil
}
2019-11-24 00:46:16 -05:00
// ReviewExists returns whether a review exists for a particular line of code in the PR
func ReviewExists ( issue * Issue , treePath string , line int64 ) ( bool , error ) {
return x . Cols ( "id" ) . Exist ( & Comment { IssueID : issue . ID , TreePath : treePath , Line : line , Type : CommentTypeCode } )
}
2018-08-05 23:43:22 -05:00
// GetCurrentReview returns the current pending review of reviewer for given issue
func GetCurrentReview ( reviewer * User , issue * Issue ) ( * Review , error ) {
return getCurrentReview ( x , reviewer , issue )
}
2019-11-13 21:57:36 -05:00
// ContentEmptyErr represents an content empty error
type ContentEmptyErr struct {
}
func ( ContentEmptyErr ) Error ( ) string {
return "Review content is empty"
}
// IsContentEmptyErr returns true if err is a ContentEmptyErr
func IsContentEmptyErr ( err error ) bool {
_ , ok := err . ( ContentEmptyErr )
return ok
}
// SubmitReview creates a review out of the existing pending review or creates a new one if no pending review exist
2020-01-08 20:47:45 -05:00
func SubmitReview ( doer * User , issue * Issue , reviewType ReviewType , content , commitID string , stale bool ) ( * Review , * Comment , error ) {
2019-11-13 21:57:36 -05:00
sess := x . NewSession ( )
defer sess . Close ( )
if err := sess . Begin ( ) ; err != nil {
return nil , nil , err
}
2019-12-03 20:08:56 -05:00
var official = false
2019-11-13 21:57:36 -05:00
review , err := getCurrentReview ( sess , doer , issue )
if err != nil {
if ! IsErrReviewNotExist ( err ) {
return nil , nil , err
}
2019-11-14 15:58:01 -05:00
if reviewType != ReviewTypeApprove && len ( strings . TrimSpace ( content ) ) == 0 {
2019-11-13 21:57:36 -05:00
return nil , nil , ContentEmptyErr { }
}
2019-12-03 20:08:56 -05:00
if reviewType == ReviewTypeApprove || reviewType == ReviewTypeReject {
// Only reviewers latest review of type approve and reject shall count as "official", so existing reviews needs to be cleared
if _ , err := sess . Exec ( "UPDATE `review` SET official=? WHERE issue_id=? AND reviewer_id=?" , false , issue . ID , doer . ID ) ; err != nil {
return nil , nil , err
}
official , err = isOfficialReviewer ( sess , issue , doer )
if err != nil {
return nil , nil , err
}
}
2019-11-13 21:57:36 -05:00
// No current review. Create a new one!
review , err = createReview ( sess , CreateReviewOptions {
Type : reviewType ,
Issue : issue ,
Reviewer : doer ,
Content : content ,
2019-12-03 20:08:56 -05:00
Official : official ,
2020-01-08 20:47:45 -05:00
CommitID : commitID ,
Stale : stale ,
2019-11-13 21:57:36 -05:00
} )
if err != nil {
return nil , nil , err
}
} else {
if err := review . loadCodeComments ( sess ) ; err != nil {
return nil , nil , err
}
2019-11-14 15:58:01 -05:00
if reviewType != ReviewTypeApprove && len ( review . CodeComments ) == 0 && len ( strings . TrimSpace ( content ) ) == 0 {
2019-11-13 21:57:36 -05:00
return nil , nil , ContentEmptyErr { }
}
2019-12-03 20:08:56 -05:00
if reviewType == ReviewTypeApprove || reviewType == ReviewTypeReject {
// Only reviewers latest review of type approve and reject shall count as "official", so existing reviews needs to be cleared
if _ , err := sess . Exec ( "UPDATE `review` SET official=? WHERE issue_id=? AND reviewer_id=?" , false , issue . ID , doer . ID ) ; err != nil {
return nil , nil , err
}
official , err = isOfficialReviewer ( sess , issue , doer )
if err != nil {
return nil , nil , err
}
}
review . Official = official
2019-11-13 21:57:36 -05:00
review . Issue = issue
review . Content = content
review . Type = reviewType
2020-01-08 20:47:45 -05:00
review . CommitID = commitID
review . Stale = stale
2019-12-03 20:08:56 -05:00
2020-01-08 20:47:45 -05:00
if _ , err := sess . ID ( review . ID ) . Cols ( "content, type, official, commit_id, stale" ) . Update ( review ) ; err != nil {
2019-11-13 21:57:36 -05:00
return nil , nil , err
}
2018-08-05 23:43:22 -05:00
}
2019-11-13 21:57:36 -05:00
2019-12-15 22:54:24 -05:00
comm , err := createComment ( sess , & CreateCommentOptions {
2019-11-13 21:57:36 -05:00
Type : CommentTypeReview ,
Doer : doer ,
Content : review . Content ,
Issue : issue ,
Repo : issue . Repo ,
ReviewID : review . ID ,
} )
if err != nil || comm == nil {
return nil , nil , err
}
comm . Review = review
return review , comm , sess . Commit ( )
2018-08-05 23:43:22 -05:00
}
2018-11-22 08:17:36 -05:00
2019-12-03 20:08:56 -05:00
// GetReviewersByIssueID gets the latest review of each reviewer for a pull request
func GetReviewersByIssueID ( issueID int64 ) ( reviews [ ] * Review , err error ) {
reviewsUnfiltered := [ ] * Review { }
2018-11-22 08:17:36 -05:00
2019-12-03 20:08:56 -05:00
sess := x . NewSession ( )
defer sess . Close ( )
if err := sess . Begin ( ) ; err != nil {
return nil , err
2018-12-10 20:09:46 -05:00
}
2018-11-22 08:17:36 -05:00
2019-12-03 20:08:56 -05:00
// Get latest review of each reviwer, sorted in order they were made
2020-04-06 11:33:34 -05:00
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" ,
issueID , ReviewTypeApprove , ReviewTypeReject , ReviewTypeRequest ) .
2019-12-03 20:08:56 -05:00
Find ( & reviewsUnfiltered ) ; err != nil {
return nil , err
}
// Load reviewer and skip if user is deleted
for _ , review := range reviewsUnfiltered {
2020-04-06 11:33:34 -05:00
if err = review . loadReviewer ( sess ) ; err != nil {
2019-12-03 20:08:56 -05:00
if ! IsErrUserNotExist ( err ) {
return nil , err
}
} else {
reviews = append ( reviews , review )
2018-11-22 08:17:36 -05:00
}
}
2019-12-03 20:08:56 -05:00
return reviews , nil
2018-11-22 08:17:36 -05:00
}
2020-01-08 20:47:45 -05:00
2020-04-06 11:33:34 -05:00
// GetReviewerByIssueIDAndUserID get the latest review of reviewer for a pull request
func GetReviewerByIssueIDAndUserID ( issueID , userID int64 ) ( review * Review , err error ) {
review = new ( Review )
if _ , err := x . SQL ( "SELECT * FROM review WHERE id IN (SELECT max(id) as id FROM review WHERE issue_id = ? AND reviewer_id = ? AND type in (?, ?, ?))" ,
issueID , userID , ReviewTypeApprove , ReviewTypeReject , ReviewTypeRequest ) .
Get ( review ) ; err != nil {
return nil , err
}
return
}
2020-01-08 20:47:45 -05:00
// MarkReviewsAsStale marks existing reviews as stale
func MarkReviewsAsStale ( issueID int64 ) ( err error ) {
_ , err = x . Exec ( "UPDATE `review` SET stale=? WHERE issue_id=?" , true , issueID )
return
}
// MarkReviewsAsNotStale marks existing reviews as not stale for a giving commit SHA
func MarkReviewsAsNotStale ( issueID int64 , commitID string ) ( err error ) {
_ , err = x . Exec ( "UPDATE `review` SET stale=? WHERE issue_id=? AND commit_id=?" , false , issueID , commitID )
return
}
2020-01-23 12:28:15 -05:00
// InsertReviews inserts review and review comments
func InsertReviews ( reviews [ ] * Review ) error {
sess := x . NewSession ( )
defer sess . Close ( )
if err := sess . Begin ( ) ; err != nil {
return err
}
for _ , review := range reviews {
if _ , err := sess . NoAutoTime ( ) . Insert ( review ) ; err != nil {
return err
}
if _ , err := sess . NoAutoTime ( ) . Insert ( & Comment {
Type : CommentTypeReview ,
Content : review . Content ,
PosterID : review . ReviewerID ,
OriginalAuthor : review . OriginalAuthor ,
OriginalAuthorID : review . OriginalAuthorID ,
IssueID : review . IssueID ,
ReviewID : review . ID ,
CreatedUnix : review . CreatedUnix ,
UpdatedUnix : review . UpdatedUnix ,
} ) ; err != nil {
return err
}
for _ , c := range review . Comments {
c . ReviewID = review . ID
}
if _ , err := sess . NoAutoTime ( ) . Insert ( review . Comments ) ; err != nil {
return err
}
}
return sess . Commit ( )
}
2020-04-06 11:33:34 -05:00
// AddRewiewRequest add a review request from one reviewer
func AddRewiewRequest ( issue * Issue , reviewer * User , doer * User ) ( comment * Comment , err error ) {
review , err := GetReviewerByIssueIDAndUserID ( issue . ID , reviewer . ID )
if err != nil {
return
}
// skip it when reviewer hase been request to review
if review != nil && review . Type == ReviewTypeRequest {
return nil , nil
}
sess := x . NewSession ( )
defer sess . Close ( )
if err := sess . Begin ( ) ; err != nil {
return nil , err
}
var official bool
official , err = isOfficialReviewer ( sess , issue , reviewer )
if err != nil {
return nil , err
}
if ! official {
official , err = isOfficialReviewer ( sess , issue , doer )
if err != nil {
return nil , err
}
}
if official {
if _ , err := sess . Exec ( "UPDATE `review` SET official=? WHERE issue_id=? AND reviewer_id=?" , false , issue . ID , reviewer . ID ) ; err != nil {
return nil , err
}
}
_ , err = createReview ( sess , CreateReviewOptions {
Type : ReviewTypeRequest ,
Issue : issue ,
Reviewer : reviewer ,
Official : official ,
Stale : false ,
} )
if err != nil {
return
}
comment , err = createComment ( sess , & CreateCommentOptions {
Type : CommentTypeReviewRequest ,
Doer : doer ,
Repo : issue . Repo ,
Issue : issue ,
RemovedAssignee : false , // Use RemovedAssignee as !isRequest
AssigneeID : reviewer . ID , // Use AssigneeID as reviewer ID
} )
if err != nil {
return nil , err
}
return comment , sess . Commit ( )
}
//RemoveRewiewRequest remove a review request from one reviewer
func RemoveRewiewRequest ( issue * Issue , reviewer * User , doer * User ) ( comment * Comment , err error ) {
review , err := GetReviewerByIssueIDAndUserID ( issue . ID , reviewer . ID )
if err != nil {
return
}
if review . Type != ReviewTypeRequest {
return nil , nil
}
sess := x . NewSession ( )
defer sess . Close ( )
if err := sess . Begin ( ) ; err != nil {
return nil , err
}
_ , err = sess . Delete ( review )
if err != nil {
return nil , err
}
var official bool
official , err = isOfficialReviewer ( sess , issue , reviewer )
if err != nil {
return
}
if official {
// recalculate which is the latest official review from that user
var review * Review
review , err = GetReviewerByIssueIDAndUserID ( issue . ID , reviewer . ID )
if err != nil {
return nil , err
}
if review != nil {
if _ , err := sess . Exec ( "UPDATE `review` SET official=? WHERE id=?" , true , review . ID ) ; err != nil {
return nil , err
}
}
}
if err != nil {
return nil , err
}
comment , err = CreateComment ( & CreateCommentOptions {
Type : CommentTypeReviewRequest ,
Doer : doer ,
Repo : issue . Repo ,
Issue : issue ,
RemovedAssignee : true , // Use RemovedAssignee as !isRequest
AssigneeID : reviewer . ID , // Use AssigneeID as reviewer ID
} )
if err != nil {
return nil , err
}
return comment , sess . Commit ( )
}