From add855c8cb61354c8b98cafff70f0846a3d9e3b2 Mon Sep 17 00:00:00 2001 From: Moritz Date: Thu, 11 Dec 2025 13:35:52 +0100 Subject: [PATCH] refactor: remove redundant ilike filter in build_substring_filter contains(city, ^query) already produces ILIKE '%query%' ilike(city, ^pattern) with pattern="%query%" is identical --- lib/membership/member.ex | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/lib/membership/member.ex b/lib/membership/member.ex index 4c1a156..a558cf0 100644 --- a/lib/membership/member.ex +++ b/lib/membership/member.ex @@ -515,14 +515,13 @@ defmodule Mv.Membership.Member do # Builds substring search filter for structured fields # Note: contains/2 uses ILIKE '%value%' which is not index-optimized # 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( contains(postal_code, ^query) or contains(house_number, ^query) or contains(phone_number, ^query) or contains(email, ^query) or - contains(city, ^query) or - ilike(city, ^pattern) + contains(city, ^query) ) end