diff options
author | lain <lain@soykaf.club> | 2019-11-04 16:57:41 +0100 |
---|---|---|
committer | lain <lain@soykaf.club> | 2019-11-04 16:57:41 +0100 |
commit | 4e535209172bb5460353fe011c06d127cfaa5847 (patch) | |
tree | ae44b0cbde3bc5d027c26c31127f69a0e2f2a4cd | |
parent | a304a2a845834b73636706ba82e34463a47d9258 (diff) | |
download | pleroma-4e535209172bb5460353fe011c06d127cfaa5847.tar.gz |
User Search: Remove superfluous setweight and random test.
The test tested for a behavior that isn't actually enforced anymore.
-rw-r--r-- | lib/pleroma/user/search.ex | 12 | ||||
-rw-r--r-- | test/user_search_test.exs | 7 |
2 files changed, 1 insertions, 18 deletions
diff --git a/lib/pleroma/user/search.ex b/lib/pleroma/user/search.ex index bab8d92e2..09664db76 100644 --- a/lib/pleroma/user/search.ex +++ b/lib/pleroma/user/search.ex @@ -54,15 +54,7 @@ defmodule Pleroma.User.Search do |> maybe_restrict_local(for_user) end - @nickname_regex ~r/^[a-zA-Z0-9.!#$%&'*+\/=?^_`{|}~\-@]+$/ defp fts_search(query, query_string) do - {nickname_weight, name_weight} = - if String.match?(query_string, @nickname_regex) do - {"A", "B"} - else - {"B", "A"} - end - query_string = to_tsquery(query_string) from( @@ -70,12 +62,10 @@ defmodule Pleroma.User.Search do where: fragment( """ - (setweight(to_tsvector('simple', ?), ?) || setweight(to_tsvector('simple', ?), ?)) @@ to_tsquery('simple', ?) + (to_tsvector('simple', ?) || to_tsvector('simple', ?)) @@ to_tsquery('simple', ?) """, u.name, - ^name_weight, u.nickname, - ^nickname_weight, ^query_string ) ) diff --git a/test/user_search_test.exs b/test/user_search_test.exs index 78a02d536..721af1e5b 100644 --- a/test/user_search_test.exs +++ b/test/user_search_test.exs @@ -51,13 +51,6 @@ defmodule Pleroma.UserSearchTest do end) end - test "finds users, preferring nickname matches over name matches" do - u1 = insert(:user, %{name: "lain", nickname: "nick1"}) - u2 = insert(:user, %{nickname: "lain", name: "nick1"}) - - assert [u2.id, u1.id] == Enum.map(User.search("lain"), & &1.id) - end - test "finds users, considering density of matched tokens" do u1 = insert(:user, %{name: "Bar Bar plus Word Word"}) u2 = insert(:user, %{name: "Word Word Bar Bar Bar"}) |