Skip to content

Commit 5bdfa3d

Browse files
a1012112796lafriks
andauthored
Apply suggestions from code review
Co-authored-by: Lauris BH <[email protected]>
1 parent 2cc54a6 commit 5bdfa3d

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

models/migrations/v143.go

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ package migrations
77
import (
88
"code.gitea.io/gitea/models"
99
"code.gitea.io/gitea/modules/log"
10+
1011
"xorm.io/xorm"
1112
)
1213

@@ -30,13 +31,10 @@ func recalculateStars(x *xorm.Engine) (err error) {
3031
var number int64
3132

3233
for _, uid := range userIDs {
33-
if number, err = x.Where("uid = ?", uid).Count(new(models.Star)); err != nil {
34-
return
35-
}
36-
37-
if _, err = x.Exec("UPDATE `user` SET num_stars=? WHERE id = ?", number, uid); err != nil {
34+
if _, err = x.Exec("UPDATE `user` SET num_stars=(SELECT COUNT(*) FROM `star` WHERE uid=?) WHERE id=?", uid, uid); err != nil {
3835
return err
3936
}
37+
4038
}
4139
}
4240

0 commit comments

Comments
 (0)