diff options
author | Amine Hilaly <hilalyamine@gmail.com> | 2019-04-05 16:28:41 +0200 |
---|---|---|
committer | Amine Hilaly <hilalyamine@gmail.com> | 2019-04-05 16:56:08 +0200 |
commit | 1d00ded250102c23c0a106e14d88875d342f7880 (patch) | |
tree | 2d27d6bdbcd74bffeeed48b62e8c046085ecbb94 | |
parent | 98169d5ad193fe776c29cc40dcd7103bb13e9b7d (diff) | |
download | git-bug-1d00ded250102c23c0a106e14d88875d342f7880.tar.gz |
implement identityExcerpt match method and integrate it into filters
Update docs
-rw-r--r-- | cache/filter.go | 15 | ||||
-rw-r--r-- | cache/identity_excerpt.go | 8 | ||||
-rw-r--r-- | doc/queries.md | 1 |
3 files changed, 12 insertions, 12 deletions
diff --git a/cache/filter.go b/cache/filter.go index ca70bb2a..27e92cf3 100644 --- a/cache/filter.go +++ b/cache/filter.go @@ -33,10 +33,7 @@ func AuthorFilter(query string) Filter { panic("missing identity in the cache") } - return query == author.Id || - query == author.HumanId() || - strings.Contains(strings.ToLower(author.Name), query) || - strings.Contains(strings.ToLower(author.Login), query) + return author.Match(query) } // Legacy identity support @@ -68,10 +65,7 @@ func ActorFilter(query string) Filter { panic("missing identity in the cache") } - if query == identityExcerpt.Id || - query == identityExcerpt.HumanId() || - strings.Contains(strings.ToLower(identityExcerpt.Name), query) || - query == strings.ToLower(identityExcerpt.Login) { + if identityExcerpt.Match(query) { return true } } @@ -90,10 +84,7 @@ func ParticipantFilter(query string) Filter { panic("missing identity in the cache") } - if query == identityExcerpt.Id || - query == identityExcerpt.HumanId() || - strings.Contains(strings.ToLower(identityExcerpt.Name), query) || - query == strings.ToLower(identityExcerpt.Login) { + if identityExcerpt.Match(query) { return true } } diff --git a/cache/identity_excerpt.go b/cache/identity_excerpt.go index 2a13bc60..3ac13903 100644 --- a/cache/identity_excerpt.go +++ b/cache/identity_excerpt.go @@ -3,6 +3,7 @@ package cache import ( "encoding/gob" "fmt" + "strings" "github.com/MichaelMure/git-bug/identity" ) @@ -51,6 +52,13 @@ func (i *IdentityExcerpt) DisplayName() string { panic("invalid person data") } +// Match matches a query with the identity name, login and ID prefixes +func (i *IdentityExcerpt) Match(query string) bool { + return strings.HasPrefix(i.Id, query) || + strings.Contains(strings.ToLower(i.Name), query) || + strings.Contains(strings.ToLower(i.Login), query) +} + /* * Sorting */ diff --git a/doc/queries.md b/doc/queries.md index b93941d1..eb510185 100644 --- a/doc/queries.md +++ b/doc/queries.md @@ -11,6 +11,7 @@ A few tips: - queries are case insensitive. - you can combine as many qualifiers as you want. - you can use double quotes for multi-word search terms. For example, `author:"René Descartes"` searches for bugs opened by René Descartes, whereas `author:René Descartes` will throw an error since full-text search is not yet supported. +- instead of a complete ID, you can use any prefix length. For example `participant=9ed1a`. ## Filtering |