Fix wrong hint when status checking is running on pull request view (#9886)

* Fix wrong hint when status checking is running on pull request view

* fix lint

* fix test

* fix test

* fix wrong tmpl

* fix import

* rename function name

1 files changed, 9 insertions, 9 deletions

diff --git a/routers/api/v1/repo/status.go b/routers/api/v1/repo/status.go
index c137b64f5c..b6b3d495ca 100644
--- a/routers/api/v1/repo/status.go
+++ b/routers/api/v1/repo/status.go
@@ -53,7 +53,7 @@ func NewCommitStatus(ctx *context.APIContext, form api.CreateStatusOption) {
 		return
 	}
 	status := &models.CommitStatus{
-		State:       models.CommitStatusState(form.State),
+		State:       api.CommitStatusState(form.State),
 		TargetURL:   form.TargetURL,
 		Description: form.Description,
 		Context:     form.Context,
@@ -220,13 +220,13 @@ func getCommitStatuses(ctx *context.APIContext, sha string) {
 }
 
 type combinedCommitStatus struct {
-	State      models.CommitStatusState `json:"state"`
-	SHA        string                   `json:"sha"`
-	TotalCount int                      `json:"total_count"`
-	Statuses   []*api.Status            `json:"statuses"`
-	Repo       *api.Repository          `json:"repository"`
-	CommitURL  string                   `json:"commit_url"`
-	URL        string                   `json:"url"`
+	State      api.CommitStatusState `json:"state"`
+	SHA        string                `json:"sha"`
+	TotalCount int                   `json:"total_count"`
+	Statuses   []*api.Status         `json:"statuses"`
+	Repo       *api.Repository       `json:"repository"`
+	CommitURL  string                `json:"commit_url"`
+	URL        string                `json:"url"`
 }
 
 // GetCombinedCommitStatusByRef returns the combined status for any given commit hash
@@ -293,7 +293,7 @@ func GetCombinedCommitStatusByRef(ctx *context.APIContext) {
 	retStatus.Statuses = make([]*api.Status, 0, len(statuses))
 	for _, status := range statuses {
 		retStatus.Statuses = append(retStatus.Statuses, status.APIFormat())
-		if status.State.IsWorseThan(retStatus.State) {
+		if status.State.NoBetterThan(retStatus.State) {
 			retStatus.State = status.State
 		}
 	}

path: root/routers/api/v1/repo/status.go

{ "translations": {
    "Sunday" : "Недеља",
    "Monday" : "Понедељак",
    "Tuesday" : "Уторак",
    "Wednesday" : "Среда",
    "Thursday" : "Четвртак",
    "Friday" : "Петак",
    "Saturday" : "Субота",
    "January" : "Јануар",
    "February" : "Фебруар",
    "March" : "Март",
    "April" : "Април",
    "May" : "Мај",
    "June" : "Јун",
    "July" : "Јул",
    "August" : "Август",
    "September" : "Септембар",
    "October" : "Октобар",
    "November" : "Новембар",
    "December" : "Децембар",
    "Settings" : "Поставке",
    "Saving..." : "Уписујем...",
    "No" : "Не",
    "Yes" : "Да",
    "Choose" : "Одабери",
    "Ok" : "У реду",
    "_{count} file conflict_::_{count} file conflicts_" : ["","",""],
    "Cancel" : "Откажи",
    "Your web server is not yet properly setup to allow files synchronization because the WebDAV interface seems to be broken." : "Ваш веб сервер тренутно не подржава синхронизацију датотека јер се чини да је WebDAV сучеље неисправно.",
    "Share" : "Дели",
    "Error" : "Грешка",
    "Error while sharing" : "Грешка у дељењу",
    "Error while unsharing" : "Грешка код искључења дељења",
    "Error while changing permissions" : "Грешка код промене дозвола",
    "Shared with you and the group {group} by {owner}" : "Дељено са вама и са групом {group}. Поделио {owner}.",
    "Shared with you by {owner}" : "Поделио са вама {owner}",
    "Password protect" : "Заштићено лозинком",
    "Password" : "Лозинка",
    "Send" : "Пошаљи",
    "Set expiration date" : "Постави датум истека",
    "Expiration date" : "Датум истека",
    "group" : "група",
    "Resharing is not allowed" : "Поновно дељење није дозвољено",
    "Shared in {item} with {user}" : "Подељено унутар {item} са {user}",
    "Unshare" : "Укини дељење",
    "can share" : "може да дели",
    "can edit" : "може да мења",
    "access control" : "права приступа",
    "create" : "направи",
    "delete" : "обриши",
    "Password protected" : "Заштићено лозинком",
    "Error unsetting expiration date" : "Грешка код поништавања датума истека",
    "Error setting expiration date" : "Грешка код постављања датума истека",
    "Sending ..." : "Шаљем...",
    "Email sent" : "Порука је послата",
    "Warning" : "Упозорење",
    "The object type is not specified." : "Врста објекта није подешена.",
    "Delete" : "Обриши",
    "Add" : "Додај",
    "_download %n file_::_download %n files_" : ["","",""],
    "Use the following link to reset your password: {link}" : "Овом везом ресетујте своју лозинку: {link}",
    "New password" : "Нова лозинка",
    "Reset password" : "Ресетуј лозинку",
    "_{count} search result in other places_::_{count} search results in other places_" : ["","",""],
    "Personal" : "Лично",
    "Users" : "Корисници",
    "Apps" : "Апликације",
    "Admin" : "Администратор",
    "Help" : "Помоћ",
    "Access forbidden" : "Забрањен приступ",
    "Security Warning" : "Сигурносно упозорење",
    "Create an <strong>admin account</strong>" : "Направи <strong>административни налог</strong>",
    "Username" : "Корисничко име",
    "Data folder" : "Фацикла података",
    "Configure the database" : "Подешавање базе",
    "Database user" : "Корисник базе",
    "Database password" : "Лозинка базе",
    "Database name" : "Име базе",
    "Database tablespace" : "Радни простор базе података",
    "Database host" : "Домаћин базе",
    "Finish setup" : "Заврши подешавање",
    "Log out" : "Одјава",
    "Search" : "Претражи",
    "remember" : "упамти",
    "Log in" : "Пријава"
},"pluralForm" :"nplurals=3; plural=(n%10==1 && n%100!=11 ? 0 : n%10>=2 && n%10<=4 && (n%100<10 || n%100>=20) ? 1 : 2);"
}