Handle leading/trailing spaces in search input (#248)

Co-authored-by: Jonatan Kłosko <[email protected]>
thewca · Dec 26, 2024 · 6b7cd5f · 6b7cd5f
1 parent 5f546f5
commit 6b7cd5f
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 0 deletions.
diff --git a/client/src/components/CompetitionSearch/CompetitionSearch.jsx b/client/src/components/CompetitionSearch/CompetitionSearch.jsx
@@ -50,6 +50,7 @@ function CompetitionSearch({ value = null, onChange, TextFieldProps = {} }) {
       forcePopupIcon={false}
       disableClearable={true}
       renderInput={(params) => <TextField {...params} {...TextFieldProps} />}
+      filterOptions={(options) => options}
     />
   );
 }

diff --git a/lib/wca_live/competitions.ex b/lib/wca_live/competitions.ex
@@ -37,6 +37,7 @@ defmodule WcaLive.Competitions do
   defp filter_by_text(query, nil), do: query
 
   defp filter_by_text(query, filter) do
+    filter = filter |> String.trim() |> String.replace(~r/\s+/, " ")
     from c in query, where: ilike(c.name, ^"%#{filter}%")
   end
 

diff --git a/test/wca_live/competitions_test.exs b/test/wca_live/competitions_test.exs
@@ -28,6 +28,12 @@ defmodule WcaLive.CompetitionsTest do
     assert worlds.id in ids(list)
     assert open.id not in ids(list)
     assert euro.id in ids(list)
+
+    list = Competitions.list_competitions(%{filter: " championship  2020  "})
+
+    assert worlds.id in ids(list)
+    assert open.id not in ids(list)
+    assert euro.id in ids(list)
   end
 
   test "list_competitions/1 given :from returns competitions from that date inclusive" do