diff options
author | unarist <m.unarist@gmail.com> | 2017-06-06 23:07:06 +0900 |
---|---|---|
committer | Eugen Rochko <eugen@zeonfederated.com> | 2017-06-06 16:07:06 +0200 |
commit | 004672aa6c482b212e36b9e794d107be456a11da (patch) | |
tree | 7bde1904fe421f63adb8210179ec8eb458fe16a3 /spec/models | |
parent | ad4a28f4f689cc9a37a5c2d5dd8c012e964903b7 (diff) |
Fix tag search order and not to use tsvector (#3611)
* Sort results by the name * Switch search method to simple `LIKE` matching instead of tsvector/tsquery Previously we used scores from ts_rank_cd() to sort results, but it didn't work because the function returns same score for all results. It's not for calculate similarity of single words. Sometimes this bug even push out exact matching tag from results. Additionally, PostgreSQL supports prefix searching with standard btree index. Using it offers simpler code, but also less index size and some speed.
Diffstat (limited to 'spec/models')
-rw-r--r-- | spec/models/tag_spec.rb | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/spec/models/tag_spec.rb b/spec/models/tag_spec.rb index 7a5b8ec89..2496946cb 100644 --- a/spec/models/tag_spec.rb +++ b/spec/models/tag_spec.rb @@ -22,5 +22,14 @@ RSpec.describe Tag, type: :model do expect(results).to eq [tag] end + + it 'finds the exact matching tag as the first item' do + similar_tag = Fabricate(:tag, name: "matchlater") + tag = Fabricate(:tag, name: "match") + + results = Tag.search_for("match") + + expect(results).to eq [tag, similar_tag] + end end end |