use builder.In

Zettat123 · Zettat123 · commit a34862aadf1d · 2024-09-30T16:45:12.000+08:00
diff --git a/models/issues/review_list.go b/models/issues/review_list.go
@@ -108,11 +108,7 @@ func (opts *FindReviewOptions) toCond() builder.Cond {
 		cond = cond.And(builder.Eq{"reviewer_id": opts.ReviewerID})
 	}
 	if len(opts.Types) > 0 {
-		typeConds := make([]builder.Cond, 0, len(opts.Types))
-		for _, reviewType := range opts.Types {
-			typeConds = append(typeConds, builder.Eq{"type": reviewType})
-		}
-		cond = cond.And(builder.Or(typeConds...))
+		cond = cond.And(builder.In("type", opts.Types))
 	}
 	if opts.OfficialOnly {
 		cond = cond.And(builder.Eq{"official": true})

Original file line number	Diff line number	Diff line change
`@@ -108,11 +108,7 @@ func (opts *FindReviewOptions) toCond() builder.Cond {`
`108`	`108`	`cond = cond.And(builder.Eq{"reviewer_id": opts.ReviewerID})`
`109`	`109`	`}`
`110`	`110`	`if len(opts.Types) > 0 {`
`111`		`- typeConds := make([]builder.Cond, 0, len(opts.Types))`
`112`		`- for _, reviewType := range opts.Types {`
`113`		`- typeConds = append(typeConds, builder.Eq{"type": reviewType})`
`114`		`- }`
`115`		`- cond = cond.And(builder.Or(typeConds...))`
	`111`	`+ cond = cond.And(builder.In("type", opts.Types))`
`116`	`112`	`}`
`117`	`113`	`if opts.OfficialOnly {`
`118`	`114`	`cond = cond.And(builder.Eq{"official": true})`