mirror of
https://codeberg.org/forgejo/forgejo.git
synced 2025-02-06 22:22:29 +00:00
user, topic, project, label, milestone, repository, pull_request, release, asset, comment, reaction, review providers Signed-off-by: Earl Warren <contact@earl-warren.org> Preserve file size when creating attachments Introduced inc6f5029708
repoList.LoadAttributes has a ctx argument now Rename `repo.GetOwner` to `repo.LoadOwner`bd66fa586a
upgrade to the latest gof3 (cherry picked from commitc770713656
) [F3] ID remapping logic is in place, remove workaround (cherry picked from commitd0fee30167
) [F3] it is experimental, do not enable by default (cherry picked from commitde325b21d0
) (cherry picked from commit547e7b3c40
) (cherry picked from commit820df3a56b
) (cherry picked from commiteaba87689b
) (cherry picked from commit1b86896b3b
) (cherry picked from commit0046aac1c6
) (cherry picked from commitf14220df8f
) (cherry picked from commit559b731001
) (cherry picked from commit801f7d600d
) (cherry picked from commit6aa76e9bcf
) (cherry picked from commita8757dcb07
) [F3] promote F3 users to matching OAuth2 users on first sign-in (cherry picked from commitbd7fef7496
) (cherry picked from commit07412698e8
) (cherry picked from commitd143e5b2a3
) [F3] upgrade to gof3 50a6e740ac04 Add new methods GetIDString() & SetIDString() & ToFormatInterface() Change the prototype of the fixture function (cherry picked from commitd7b263ff8b
) (cherry picked from commitb3eaf2249d
) (cherry picked from commitd492ddd9bb
) [F3] add GetLocalMatchingRemote with a default implementation (cherry picked from commit0a22015039
) (cherry picked from commitf1310c38fb
) (cherry picked from commitdeb68552f2
) [F3] GetLocalMatchingRemote for user (cherry picked from commite73cb837f5
) (cherry picked from commita24bc0b85e
) (cherry picked from commit846a522ecc
) [F3] GetAdminUser now has a ctx argument (cherry picked from commit37357a92af
) (cherry picked from commit660bc1673c
) (cherry picked from commit72d692a767
) [F3] introduce UserTypeF3 To avoid conflicts should UserTypeRemoteUser be used differently by Gitea (cherry picked from commit6de2701bb3
) [F3] user.Put: idempotency (cherry picked from commit821e38573c
) (cherry picked from commitf7638f5414
) [F3] upgrade to urfave v2 (cherry picked from commitcc3dbdfd1d
) [F3] update gof3 (cherry picked from commit2eee960751
) [F3] move f3 under forgejo-cli * simplify the tests by re-using the forgejo-cli helpers to capture the output * unify CmdF3 to be structured in the same way CmdActions is (cherry picked from commit4c9fe58b74
) [F3] replace f3 with forgejo-cli f3 (cherry picked from commit7ba7ceef1b
) [F3] s/ListOptions/Paginator/ [F3] user: add unit tests [F3] user comparison of F3 managed users is on content [F3] issue: add unit tests [F3] gof3 now has one more argument to Put() [F3] re-use gof3 unit tests for the driver (cherry picked from commitaf7ee6200c
) Conflicts: tests/integration/integration_test.go because of some code removed in forgejo-development, trivial context conflict resolution [F3] more idempotent tests (#1275) Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/1275 Co-authored-by: Loïc Dachary <loic@dachary.org> Co-committed-by: Loïc Dachary <loic@dachary.org> [F3] tests: do SQL update if nothing changes [F3] tests comment idempotence [F3] tests milestone idempotence [F3] tests pull_request idempotence [F3] tests release idempotence [F3] tests asset idempotence [F3] tests project idempotence [F3] tests review idempotence (cherry picked from commit91038bb4e8
) (cherry picked from commita7d2a65214
) (cherry picked from commit59a17e5a34
) [F3] sub command of forgejo-cli (cherry picked from commit4d098e9b83
) [F3] implement --quiet, --debug, --verbose (cherry picked from commit82e2e17b45
) [F3] fix off by one error when importing repositories (cherry picked from commit31689b1397
) [F3] upgrade gof3 (cherry picked from commit87b8cfe5a1
) [F3] set the logger for all drivers The logger is set for the local Forgejo driver only. Even when --debug is specified, the other drivers do not display debug information. Use the gof3 context to set the logger for all of them at once. (cherry picked from commit8aa7de8ba0
) [F3] the closed date of an issue may be nil (cherry picked from commit93d3eaf0b5
) [F3] update gof3 to support system users there now is a workaround to hardcode system users when they are not supported by the API (cherry picked from commit915484daa7
)
243 lines
6.5 KiB
Go
243 lines
6.5 KiB
Go
// SPDX-License-Identifier: MIT
|
|
|
|
package driver
|
|
|
|
import (
|
|
"context"
|
|
"fmt"
|
|
|
|
"code.gitea.io/gitea/models/db"
|
|
issues_model "code.gitea.io/gitea/models/issues"
|
|
user_model "code.gitea.io/gitea/models/user"
|
|
"code.gitea.io/gitea/modules/timeutil"
|
|
"code.gitea.io/gitea/services/convert"
|
|
|
|
"lab.forgefriends.org/friendlyforgeformat/gof3/format"
|
|
"lab.forgefriends.org/friendlyforgeformat/gof3/util"
|
|
)
|
|
|
|
type Review struct {
|
|
issues_model.Review
|
|
}
|
|
|
|
func ReviewConverter(f *issues_model.Review) *Review {
|
|
return &Review{
|
|
Review: *f,
|
|
}
|
|
}
|
|
|
|
func (o Review) GetID() int64 {
|
|
return o.ID
|
|
}
|
|
|
|
func (o Review) GetIDString() string {
|
|
return fmt.Sprintf("%d", o.GetID())
|
|
}
|
|
|
|
func (o *Review) SetID(id int64) {
|
|
o.ID = id
|
|
}
|
|
|
|
func (o *Review) SetIDString(id string) {
|
|
o.SetID(util.ParseInt(id))
|
|
}
|
|
|
|
func (o *Review) IsNil() bool {
|
|
return o.ID == 0
|
|
}
|
|
|
|
func (o *Review) Equals(other *Review) bool {
|
|
return o.Content == other.Content
|
|
}
|
|
|
|
func (o *Review) ToFormatInterface() format.Interface {
|
|
return o.ToFormat()
|
|
}
|
|
|
|
func (o *Review) ToFormat() *format.Review {
|
|
comments := make([]*format.ReviewComment, 0, len(o.Comments))
|
|
for _, comment := range o.Comments {
|
|
comments = append(comments, &format.ReviewComment{
|
|
Common: format.NewCommon(comment.ID),
|
|
// InReplyTo
|
|
Content: comment.Content,
|
|
TreePath: comment.TreePath,
|
|
DiffHunk: convert.Patch2diff(comment.Patch),
|
|
Patch: comment.Patch,
|
|
Line: int(comment.Line),
|
|
CommitID: comment.CommitSHA,
|
|
PosterID: format.NewUserReference(comment.PosterID),
|
|
CreatedAt: comment.CreatedUnix.AsTime(),
|
|
UpdatedAt: comment.UpdatedUnix.AsTime(),
|
|
})
|
|
}
|
|
|
|
review := format.Review{
|
|
Common: format.NewCommon(o.Review.ID),
|
|
IssueIndex: o.IssueID,
|
|
Official: o.Review.Official,
|
|
CommitID: o.Review.CommitID,
|
|
Content: o.Review.Content,
|
|
CreatedAt: o.Review.CreatedUnix.AsTime(),
|
|
State: format.ReviewStateUnknown,
|
|
Comments: comments,
|
|
}
|
|
|
|
if o.Review.Reviewer != nil {
|
|
review.ReviewerID = format.NewUserReference(o.Review.Reviewer.ID)
|
|
}
|
|
|
|
switch o.Type {
|
|
case issues_model.ReviewTypeApprove:
|
|
review.State = format.ReviewStateApproved
|
|
case issues_model.ReviewTypeReject:
|
|
review.State = format.ReviewStateChangesRequested
|
|
case issues_model.ReviewTypeComment:
|
|
review.State = format.ReviewStateCommented
|
|
case issues_model.ReviewTypePending:
|
|
review.State = format.ReviewStatePending
|
|
case issues_model.ReviewTypeRequest:
|
|
review.State = format.ReviewStateRequestReview
|
|
}
|
|
|
|
return &review
|
|
}
|
|
|
|
func (o *Review) FromFormat(review *format.Review) {
|
|
comments := make([]*issues_model.Comment, 0, len(review.Comments))
|
|
for _, comment := range review.Comments {
|
|
comments = append(comments, &issues_model.Comment{
|
|
ID: comment.GetID(),
|
|
Type: issues_model.CommentTypeReview,
|
|
// InReplyTo
|
|
CommitSHA: comment.CommitID,
|
|
Line: int64(comment.Line),
|
|
TreePath: comment.TreePath,
|
|
Content: comment.Content,
|
|
Patch: comment.Patch,
|
|
PosterID: comment.PosterID.GetID(),
|
|
CreatedUnix: timeutil.TimeStamp(comment.CreatedAt.Unix()),
|
|
UpdatedUnix: timeutil.TimeStamp(comment.UpdatedAt.Unix()),
|
|
})
|
|
}
|
|
*o = Review{
|
|
Review: issues_model.Review{
|
|
ID: review.GetID(),
|
|
ReviewerID: review.ReviewerID.GetID(),
|
|
Reviewer: &user_model.User{
|
|
ID: review.ReviewerID.GetID(),
|
|
},
|
|
IssueID: review.IssueIndex,
|
|
Official: review.Official,
|
|
CommitID: review.CommitID,
|
|
Content: review.Content,
|
|
CreatedUnix: timeutil.TimeStamp(review.CreatedAt.Unix()),
|
|
Comments: comments,
|
|
},
|
|
}
|
|
|
|
switch review.State {
|
|
case format.ReviewStateApproved:
|
|
o.Type = issues_model.ReviewTypeApprove
|
|
case format.ReviewStateChangesRequested:
|
|
o.Type = issues_model.ReviewTypeReject
|
|
case format.ReviewStateCommented:
|
|
o.Type = issues_model.ReviewTypeComment
|
|
case format.ReviewStatePending:
|
|
o.Type = issues_model.ReviewTypePending
|
|
case format.ReviewStateRequestReview:
|
|
o.Type = issues_model.ReviewTypeRequest
|
|
}
|
|
}
|
|
|
|
type ReviewProvider struct {
|
|
BaseProvider
|
|
}
|
|
|
|
func (o *ReviewProvider) ToFormat(ctx context.Context, review *Review) *format.Review {
|
|
return review.ToFormat()
|
|
}
|
|
|
|
func (o *ReviewProvider) FromFormat(ctx context.Context, r *format.Review) *Review {
|
|
var review Review
|
|
review.FromFormat(r)
|
|
return &review
|
|
}
|
|
|
|
func (o *ReviewProvider) GetObjects(ctx context.Context, user *User, project *Project, pullRequest *PullRequest, page int) []*Review {
|
|
reviews, err := issues_model.FindReviews(ctx, issues_model.FindReviewOptions{
|
|
ListOptions: db.ListOptions{Page: page, PageSize: o.g.perPage},
|
|
IssueID: pullRequest.IssueID,
|
|
Type: issues_model.ReviewTypeUnknown,
|
|
})
|
|
if err != nil {
|
|
panic(fmt.Errorf("error while listing reviews: %v", err))
|
|
}
|
|
|
|
return util.ConvertMap[*issues_model.Review, *Review](reviews, ReviewConverter)
|
|
}
|
|
|
|
func (o *ReviewProvider) ProcessObject(ctx context.Context, user *User, project *Project, pullRequest *PullRequest, review *Review) {
|
|
if err := (&review.Review).LoadAttributes(ctx); err != nil {
|
|
panic(err)
|
|
}
|
|
}
|
|
|
|
func (o *ReviewProvider) Get(ctx context.Context, user *User, project *Project, pullRequest *PullRequest, exemplar *Review) *Review {
|
|
id := exemplar.GetID()
|
|
review, err := issues_model.GetReviewByID(ctx, id)
|
|
if issues_model.IsErrReviewNotExist(err) {
|
|
return &Review{}
|
|
}
|
|
if err != nil {
|
|
panic(err)
|
|
}
|
|
if err := review.LoadAttributes(ctx); err != nil {
|
|
panic(err)
|
|
}
|
|
return ReviewConverter(review)
|
|
}
|
|
|
|
func (o *ReviewProvider) Put(ctx context.Context, user *User, project *Project, pullRequest *PullRequest, review, existing *Review) *Review {
|
|
var result *Review
|
|
|
|
if existing == nil || existing.IsNil() {
|
|
r := &review.Review
|
|
r.ID = 0
|
|
for _, comment := range r.Comments {
|
|
comment.ID = 0
|
|
}
|
|
r.IssueID = pullRequest.IssueID
|
|
if err := issues_model.InsertReviews([]*issues_model.Review{r}); err != nil {
|
|
panic(err)
|
|
}
|
|
result = ReviewConverter(r)
|
|
} else {
|
|
var u issues_model.Review
|
|
u.ID = existing.GetID()
|
|
cols := make([]string, 0, 10)
|
|
|
|
if review.Content != existing.Content {
|
|
u.Content = review.Content
|
|
cols = append(cols, "content")
|
|
}
|
|
if len(cols) > 0 {
|
|
if _, err := db.GetEngine(ctx).ID(existing.ID).Cols(cols...).Update(u); err != nil {
|
|
panic(err)
|
|
}
|
|
}
|
|
result = existing
|
|
}
|
|
|
|
return o.Get(ctx, user, project, pullRequest, result)
|
|
}
|
|
|
|
func (o *ReviewProvider) Delete(ctx context.Context, user *User, project *Project, pullRequest *PullRequest, review *Review) *Review {
|
|
r := o.Get(ctx, user, project, pullRequest, review)
|
|
if !r.IsNil() {
|
|
if err := issues_model.DeleteReview(&r.Review); err != nil {
|
|
panic(err)
|
|
}
|
|
}
|
|
return r
|
|
}
|