Fix branch order (#31174)

Fix #31172

The original order or the default order should not be ignored even if we
have an is_deleted order.
This commit is contained in:
Lunny Xiao 2024-05-31 09:58:41 +08:00 committed by GitHub
parent fb7b743bd0
commit 1137a0357e
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -107,17 +107,13 @@ func (opts FindBranchOptions) ToConds() builder.Cond {
func (opts FindBranchOptions) ToOrders() string {
orderBy := opts.OrderBy
if opts.IsDeletedBranch.ValueOrDefault(true) { // if deleted branch included, put them at the end
if orderBy != "" {
orderBy += ", "
}
orderBy += "is_deleted ASC"
}
if orderBy == "" {
// the commit_time might be the same, so add the "name" to make sure the order is stable
return "commit_time DESC, name ASC"
orderBy = "commit_time DESC, name ASC"
}
if opts.IsDeletedBranch.ValueOrDefault(true) { // if deleted branch included, put them at the beginning
orderBy = "is_deleted ASC, " + orderBy
}
return orderBy
}