local-it/mitgliederverwaltung
2
3
Fork 0
Code Issues 56 Pull requests 6 Projects 1 Releases Packages 1 Activity Actions

refactor: remove redundant ilike filter in build_substring_filter

Browse source 
contains(city, ^query) already produces ILIKE '%query%'
ilike(city, ^pattern) with pattern="%query%" is identical
This commit is contained in:
Moritz 2025-12-11 13:35:52 +01:00
parent 265e976d94
commit add855c8cb
Signed by: moritz
GPG key ID: 1020A035E5DD0824
1 changed files with 2 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

5
lib/membership/member.ex
View file

@ -515,14 +515,13 @@ defmodule Mv.Membership.Member do
# Builds substring search filter for structured fields # Builds substring search filter for structured fields
# Note: contains/2 uses ILIKE '%value%' which is not index-optimized # Note: contains/2 uses ILIKE '%value%' which is not index-optimized
# Performance: Good for small datasets, may be slow on large tables # Performance: Good for small datasets, may be slow on large tables
defp build_substring_filter(query, pattern) do defp build_substring_filter(query, _pattern) do
expr( expr(
contains(postal_code, ^query) or contains(postal_code, ^query) or
contains(house_number, ^query) or contains(house_number, ^query) or
contains(phone_number, ^query) or contains(phone_number, ^query) or
contains(email, ^query) or contains(email, ^query) or
contains(city, ^query) or contains(city, ^query)
ilike(city, ^pattern)
) )
end end