Add :string_eq condition. Fix #291

n-rodriguez · n-rodriguez · commit 2b2485c661ca · 2018-06-05T00:18:41.000+02:00
diff --git a/README.md b/README.md
@@ -205,7 +205,7 @@ end
 
 `cond` can be :
 
-* `:like`, `:start_with`, `:end_with` for string or full text search
+* `:like`, `:start_with`, `:end_with`, `:string_eq` for string or full text search
 * `:eq`, `:not_eq`, `:lt`, `:gt`, `:lteq`, `:gteq`, `:in` for numeric
 * `:date_range` for date range (only for Rails > 4.2.x, see [here](#daterange-search))
 * `:null_value` for nil field
diff --git a/lib/ajax-datatables-rails/datatable/column/search.rb b/lib/ajax-datatables-rails/datatable/column/search.rb
@@ -57,7 +57,7 @@ def non_regex_search
           when Proc
             filter
           when :eq, :not_eq, :lt, :gt, :lteq, :gteq, :in
-            is_searchable_integer? ? numeric_search : empty_search
+            is_searchable_integer? ? raw_search(cond) : empty_search
           when :null_value
             null_value_search
           when :start_with
@@ -66,6 +66,8 @@ def non_regex_search
             casted_column.matches("%#{formated_value}")
           when :like
             casted_column.matches("%#{formated_value}%")
+          when :string_eq
+            raw_search(:eq)
           end
         end
 
@@ -77,7 +79,7 @@ def null_value_search
           end
         end
 
-        def numeric_search
+        def raw_search(cond)
           if custom_field?
             ::Arel::Nodes::SqlLiteral.new(field).eq(formated_value)
           else
diff --git a/spec/ajax-datatables-rails/orm/active_record_filter_records_spec.rb b/spec/ajax-datatables-rails/orm/active_record_filter_records_spec.rb
@@ -360,6 +360,38 @@
       end
     end
 
+    describe 'it can filter records with condition :like' do
+      let(:datatable) { DatatableCondLike.new(view) }
+
+      before(:each) do
+        create(:user, email: 'john@foo.com')
+        create(:user, email: 'mary@bar.com')
+      end
+
+      it 'should filter records matching' do
+        datatable.params[:columns]['1'][:search][:value] = 'foo'
+        expect(datatable.data.size).to eq 1
+        item = datatable.data.first
+        expect(item[:email]).to eq 'john@foo.com'
+      end
+    end
+
+    describe 'it can filter records with condition :string_eq' do
+      let(:datatable) { DatatableCondStringEq.new(view) }
+
+      before(:each) do
+        create(:user, email: 'john@foo.com')
+        create(:user, email: 'mary@bar.com')
+      end
+
+      it 'should filter records matching' do
+        datatable.params[:columns]['1'][:search][:value] = 'john@foo.com'
+        expect(datatable.data.size).to eq 1
+        item = datatable.data.first
+        expect(item[:email]).to eq 'john@foo.com'
+      end
+    end
+
     describe 'it can filter records with condition :null_value' do
       let(:datatable) { DatatableCondNullValue.new(view) }
 
diff --git a/spec/support/datatable_cond_string.rb b/spec/support/datatable_cond_string.rb
@@ -10,6 +10,18 @@ def view_columns
   end
 end
 
+class DatatableCondLike < ComplexDatatable
+  def view_columns
+    super.deep_merge(email: { cond: :like })
+  end
+end
+
+class DatatableCondStringEq < ComplexDatatable
+  def view_columns
+    super.deep_merge(email: { cond: :string_eq })
+  end
+end
+
 class DatatableCondNullValue < ComplexDatatable
   def view_columns
     super.deep_merge(email: { cond: :null_value })
