mirror of
https://github.com/go-gitea/gitea.git
synced 2025-07-04 22:57:34 -04:00
6d0b24064a
Extract from #34531 ## Move Commit status state to a standalone package Move the state from `structs` to `commitstatus` package. It also introduce `CommitStatusStates` so that the combine function could be used from UI and API logic. ## Combined commit status Changed This PR will follow Github's combined commit status. Before this PR, every commit status could be a combined one. According to https://docs.github.com/en/rest/commits/statuses?apiVersion=2022-11-28#get-the-combined-status-for-a-specific-reference > Additionally, a combined state is returned. The state is one of: > failure if any of the contexts report as error or failure > pending if there are no statuses or a context is pending > success if the latest status for all contexts is success This PR will follow that rule and remove the `NoBetterThan` logic. This also fixes the inconsistent between UI and API. In the API convert package, it has implemented this which is different from the UI. It also fixed the missing `URL` and `CommitURL` in the API. ## `CalcCommitStatus` return nil if there is no commit statuses The behavior of `CalcCommitStatus` is changed. If the parameter commit statuses is empty, it will return nil. The reference places should check the returned value themselves.
62 lines
1.8 KiB
Go
62 lines
1.8 KiB
Go
// Copyright 2020 The Gitea Authors. All rights reserved.
|
|
// SPDX-License-Identifier: MIT
|
|
|
|
package convert
|
|
|
|
import (
|
|
"context"
|
|
"net/url"
|
|
|
|
git_model "code.gitea.io/gitea/models/git"
|
|
user_model "code.gitea.io/gitea/models/user"
|
|
api "code.gitea.io/gitea/modules/structs"
|
|
)
|
|
|
|
// ToCommitStatus converts git_model.CommitStatus to api.CommitStatus
|
|
func ToCommitStatus(ctx context.Context, status *git_model.CommitStatus) *api.CommitStatus {
|
|
apiStatus := &api.CommitStatus{
|
|
Created: status.CreatedUnix.AsTime(),
|
|
Updated: status.CreatedUnix.AsTime(),
|
|
State: status.State,
|
|
TargetURL: status.TargetURL,
|
|
Description: status.Description,
|
|
ID: status.Index,
|
|
URL: status.APIURL(ctx),
|
|
Context: status.Context,
|
|
}
|
|
|
|
if status.CreatorID != 0 {
|
|
creator, _ := user_model.GetUserByID(ctx, status.CreatorID)
|
|
apiStatus.Creator = ToUser(ctx, creator, nil)
|
|
}
|
|
|
|
return apiStatus
|
|
}
|
|
|
|
func ToCommitStatuses(ctx context.Context, statuses []*git_model.CommitStatus) []*api.CommitStatus {
|
|
apiStatuses := make([]*api.CommitStatus, len(statuses))
|
|
for i, status := range statuses {
|
|
apiStatuses[i] = ToCommitStatus(ctx, status)
|
|
}
|
|
return apiStatuses
|
|
}
|
|
|
|
// ToCombinedStatus converts List of CommitStatus to a CombinedStatus
|
|
func ToCombinedStatus(ctx context.Context, statuses []*git_model.CommitStatus, repo *api.Repository) *api.CombinedStatus {
|
|
if len(statuses) == 0 {
|
|
return nil
|
|
}
|
|
|
|
combinedStatus := git_model.CalcCommitStatus(statuses)
|
|
|
|
return &api.CombinedStatus{
|
|
State: combinedStatus.State,
|
|
Statuses: ToCommitStatuses(ctx, statuses),
|
|
SHA: combinedStatus.SHA,
|
|
TotalCount: len(statuses),
|
|
Repository: repo,
|
|
CommitURL: repo.URL + "/commits/" + url.PathEscape(combinedStatus.SHA),
|
|
URL: repo.URL + "/commits/" + url.PathEscape(combinedStatus.SHA) + "/status",
|
|
}
|
|
}
|