Skip to content

Commit a7c2998

Browse files
authored
Merge pull request #51 from patterninc/search_filter-placement-fix
fixed placement issue for search fields
2 parents 360f1f1 + f521b8e commit a7c2998

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

lib/query_helper.rb

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -280,8 +280,12 @@ def search_filter(column_maps)
280280
raise ArgumentError.new("search_fields not defined") unless @search_fields.length > 0
281281
placement = :where
282282
maps = column_maps.select do |cm|
283-
placement = :having if cm.aggregate
284-
@search_fields.include? cm.alias_name
283+
if @search_fields.include? cm.alias_name
284+
placement = :having if cm.aggregate
285+
true
286+
else
287+
false
288+
end
285289
end
286290
bind_variable = ('a'..'z').to_a.shuffle[0,20].join.to_sym
287291
@bind_variables[bind_variable] = "%#{@search_string}%"

0 commit comments

Comments
 (0)