Merge pull request #1126 from itsalongstory/fix_1115

Strip Leading & Trailing Whitespace Before Searching
activerecord-hackery · Nov 27, 2020 · dc3eb66 · dc3eb66
2 parents 216d909 + 05750ce
commit dc3eb66
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/lib/ransack/search.rb b/lib/ransack/search.rb
@@ -18,6 +18,7 @@ def initialize(object, params = {}, options = {})
       params = params.to_unsafe_h if params.respond_to?(:to_unsafe_h)
       if params.is_a? Hash
         params = params.dup
+        params = params.transform_values { |v| v.is_a?(String) ? v.strip : v }
         params.delete_if { |k, v| [*v].all?{ |i| i.blank? && i != false } }
       else
         params = {}

diff --git a/spec/ransack/search_spec.rb b/spec/ransack/search_spec.rb
@@ -20,6 +20,12 @@ module Ransack
         Search.new(Person, name_eq: 'foobar')
       end
 
+      it 'strip leading & trailing whitespace before building' do
+        expect_any_instance_of(Search).to receive(:build)
+        .with({ 'name_eq' => 'foobar' })
+        Search.new(Person, name_eq: '   foobar     ')
+      end
+
       it 'removes empty suffixed conditions before building' do
         expect_any_instance_of(Search).to receive(:build).with({})
         Search.new(Person, name_eq_any: [''])