realaravinth/bench-forgejo
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Make searching issues by keyword case insensitive on DB (#14848)

Browse source 
Most DBs apart from SQLite will use a default Collation that is not case insensitive.
This means that SearchIssuesByKeyword becomes case sensitive for db indexing - in
contrast to the bleve and elastic indexers.

This PR simply uses UPPER(...) to do the LIKE - and although it may be more efficient
to change collations this would be a non-trivial task.

Fix #13663

Signed-off-by: Andrew Thornton <art27@cantab.net>
This commit is contained in:
zeripath 2021-03-02 13:07:01 +00:00 committed by GitHub
parent f878c8231f
commit def964e57f
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
models/issue.go
View file

@ -1706,17 +1706,18 @@ func GetRepoIssueStats(repoID, uid int64, filterMode int, isPull bool) (numOpen
func SearchIssueIDsByKeyword(kw string, repoIDs []int64, limit, start int) (int64, []int64, error) { func SearchIssueIDsByKeyword(kw string, repoIDs []int64, limit, start int) (int64, []int64, error) {
var repoCond = builder.In("repo_id", repoIDs) var repoCond = builder.In("repo_id", repoIDs)
var subQuery = builder.Select("id").From("issue").Where(repoCond) var subQuery = builder.Select("id").From("issue").Where(repoCond)
kw = strings.ToUpper(kw)
var cond = builder.And( var cond = builder.And(
repoCond, repoCond,
builder.Or( builder.Or(
builder.Like{"name", kw}, builder.Like{"UPPER(name)", kw},
builder.Like{"content", kw}, builder.Like{"UPPER(content)", kw},
builder.In("id", builder.Select("issue_id"). builder.In("id", builder.Select("issue_id").
From("comment"). From("comment").
Where(builder.And( Where(builder.And(
builder.Eq{"type": CommentTypeComment}, builder.Eq{"type": CommentTypeComment},
builder.In("issue_id", subQuery), builder.In("issue_id", subQuery),
builder.Like{"content", kw}, builder.Like{"UPPER(content)", kw},
)), )),
), ),
), ),