mincer 0.2.3 → 0.2.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 977d1742d488e2ec263b062b0ebd6ff27335a377
-  data.tar.gz: 436e3e27dc8664fd945af1ba045b58bfff108f57
+  metadata.gz: c74759caf1e6c25531831b8826956f646100c748
+  data.tar.gz: 9aeaeef5b0179d8a5ac4abf8eaedd685babab45b
 SHA512:
-  metadata.gz: f984595c4b855119c56707fff4e997f3d1790d4a4d46f4f53b8480d4b4527e69c17ba0d30aef198d79d623155fd9769dc7c47553149bbf57e2a33c183a4139b0
-  data.tar.gz: e3bc8da6242291d4786574a4b5654c86148b96e3012f6583d7ff256b11751d2db54364af4193edf9203843a15c1a3cba2cb61431d6f74f54b5e4ab06b7c862d6
+  metadata.gz: e5a0292afa6da4666085e69d7300aa8c1767a7a622d8152c52586384529dc5d14390008b967bd2f39774636b60e0deb6156f3a50e63db6f77341d916f561f165
+  data.tar.gz: f9a42a835a506a4d36894f4ac8d42c232fd31672e09479080b1f2f0aa2316fd6a899448d80b813b0e3336c7038384809893e66fa0882cd8a06ab73fbb22fb099

data/lib/mincer/base.rb

@@ -27,10 +27,10 @@ module Mincer
     # Allows enumerable methods to be called directly on object
     def each(&block)
       @collection ||= if @relation.is_a?(ActiveRecord::Relation)
-                        @relation.to_a
-                      else
-                        @relation.all
-                      end
+        @relation.to_a
+      else
+        @relation.all
+      end
       @collection.each(&block)
     end
 

data/lib/mincer/processors/helpers.rb

@@ -3,6 +3,7 @@ module Mincer
     module Helpers
 
       def join_expressions(expressions, join_with)
+        return expressions.first if expressions.size < 2
         case join_with
         when :and then Arel::Nodes::And.new(expressions)
         when :or then expressions.inject { |accumulator, expression| Arel::Nodes::Or.new(accumulator, expression) }

data/lib/mincer/processors/pg_search/sanitizer.rb

@@ -15,7 +15,11 @@ module Mincer
 
         def sanitize_string(options = {})
           if sanitizers.empty?
-            return options[:quote] ? Mincer.connection.quote(@term) : @term
+            if defined?(Arel::Nodes::Quoted)
+              return self.class.quote(@term)
+            elsif options[:quote]
+              return Mincer.connection.quote(@term)
+            end
           end
           @sanitized_string ||= sanitize(@term)
         end
@@ -40,21 +44,29 @@ module Mincer
         end
 
         def self.ignore_case(term)
-          Arel::Nodes::NamedFunction.new('lower', [term])
+          Arel::Nodes::NamedFunction.new('lower', [quote(term)])
         end
 
         def self.ignore_accent(term)
-          Arel::Nodes::NamedFunction.new('unaccent', [term])
+          Arel::Nodes::NamedFunction.new('unaccent', [quote(term)])
         end
 
         def self.coalesce(term, val = '')
           if Mincer.pg_extension_installed?(:unaccent)
-            Arel::Nodes::NamedFunction.new('coalesce', [term, val])
+            Arel::Nodes::NamedFunction.new('coalesce', [quote(term), quote(val)])
           else
             term
           end
         end
 
+        def self.quote(string)
+          if defined?(Arel::Nodes::Quoted) && !string.is_a?(Arel::Nodes::Quoted) && !string.is_a?(Arel::Nodes::NamedFunction)
+            Arel::Nodes::Quoted.new(string)
+          else
+            string
+          end
+        end
+
       end
     end
   end

data/lib/mincer/processors/pg_search/search_engines/base.rb

@@ -12,6 +12,7 @@ module Mincer
         def arel_group(sql_string = nil)
           sql_string = yield if block_given?
           arel_query = sql_string.is_a?(String) ? Arel.sql(sql_string) : sql_string
+          return arel_query if arel_query.is_a?(Arel::Nodes::Grouping)
           Arel::Nodes::Grouping.new(arel_query)
         end
 
@@ -50,6 +51,10 @@ module Mincer
           nil
         end
 
+        def quote(string)
+          Mincer::Processors::PgSearch::Sanitizer.quote(string)
+        end
+
       end
     end
   end

data/lib/mincer/processors/pg_search/search_engines/fulltext.rb

@@ -47,14 +47,14 @@ module Mincer
           # sanitizers += [:coalesce] if (search_statement.columns.size > 1)
           documents =  search_statement.columns.map do |search_column|
             sanitized_term = sanitize_column(search_column, sanitizers)
-            ts_vector = Arel::Nodes::NamedFunction.new('to_tsvector', [search_statement.dictionary, sanitized_term])
+            ts_vector = Arel::Nodes::NamedFunction.new('to_tsvector', [quote(search_statement.dictionary), sanitized_term])
           end
         end
 
         def ts_query_for(search_statement)
           terms_delimiter = search_statement.options[:any_word] ? '|' : '&'
           tsquery_sql = Arel.sql(search_statement.terms.map { |term| sanitize_string_quoted(term, search_statement.sanitizers(:query)).to_sql }.join(" || ' #{terms_delimiter} ' || "))
-          Arel::Nodes::NamedFunction.new('to_tsquery', [search_statement.dictionary, tsquery_sql])
+          Arel::Nodes::NamedFunction.new('to_tsquery', [quote(search_statement.dictionary), tsquery_sql])
         end
       end
 

data/lib/mincer/version.rb

@@ -1,7 +1,7 @@
 module Mincer
 
   def self.version
-    Gem::Version.new '0.2.3'
+    Gem::Version.new '0.2.4'
   end
 
   module VERSION #:nodoc:

data/mincer.gemspec

@@ -18,7 +18,7 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ['lib']
 
-  spec.add_dependency 'activerecord', '>= 4.0'
+  spec.add_dependency 'activerecord', '= 4.2'
 
   spec.add_development_dependency 'bundler', '~> 1.3'
   spec.add_development_dependency 'rake'

data/spec/lib/mincer/processors/pg_search/search_engines/array_spec.rb

@@ -31,38 +31,38 @@ describe ::Mincer::PgSearch::SearchEngines::Array do
     it 'generates search condition with one column, one term and no options' do
       search_statement1 = search_statement_class.new(['"records"."text"'], engines: [:array])
       search_engine = search_engine_class.new({ pattern: 'search' }, [search_statement1])
-      search_engine.conditions.to_sql.should == %{((("records"."text"::text[]) @> ARRAY['search']))}
+      search_engine.conditions.to_sql.should == %{(("records"."text"::text[]) @> ARRAY['search'])}
     end
 
     it 'generates search condition with two columns, one term and no options' do
       search_statement1 = search_statement_class.new(['"records"."text"', '"records"."text2"'], engines: [:array])
       search_engine = search_engine_class.new({ pattern: 'search' }, [search_statement1])
-      search_engine.conditions.to_sql.should == %{((("records"."text"::text[] || "records"."text2"::text[]) @> ARRAY['search']))}
+      search_engine.conditions.to_sql.should == %{(("records"."text"::text[] || "records"."text2"::text[]) @> ARRAY['search'])}
     end
 
     it 'generates search condition with two columns, two terms and no options' do
       search_statement1 = search_statement_class.new(['"records"."text"', '"records"."text2"'], engines: [:array])
       search_engine = search_engine_class.new({ pattern: 'search word' }, [search_statement1])
-      search_engine.conditions.to_sql.should == %{((("records"."text"::text[] || "records"."text2"::text[]) @> ARRAY['search','word']))}
+      search_engine.conditions.to_sql.should == %{(("records"."text"::text[] || "records"."text2"::text[]) @> ARRAY['search','word'])}
     end
 
     it 'generates search condition with two columns, two terms and option "ignore_accent" set to true ' do
       search_statement1 = search_statement_class.new(['"records"."text"', '"records"."text2"'], engines: [:array], ignore_accent: true)
       search_engine = search_engine_class.new({ pattern: 'search word' }, [search_statement1])
-      search_engine.conditions.to_sql.should == %{((("records"."text"::text[] || "records"."text2"::text[]) @> ARRAY[unaccent('search'),unaccent('word')]))}
+      search_engine.conditions.to_sql.should == %{(("records"."text"::text[] || "records"."text2"::text[]) @> ARRAY[unaccent('search'),unaccent('word')])}
     end
 
     #TODO: sanitizer can not be set on array columns since we ned to unpack an reconstruct those arrays. Find a solution
     it 'generates search condition with two columns, two terms and option "any_word" set to true ' do
       search_statement1 = search_statement_class.new(['"records"."text"', '"records"."text2"'], engines: [:array], any_word: true)
       search_engine = search_engine_class.new({ pattern: 'search word' }, [search_statement1])
-      search_engine.conditions.to_sql.should == %{((("records"."text"::text[] || "records"."text2"::text[]) && ARRAY['search','word']))}
+      search_engine.conditions.to_sql.should == %{(("records"."text"::text[] || "records"."text2"::text[]) && ARRAY['search','word'])}
     end
 
     it 'generates search condition with two columns, two terms and option "ignore_accent" and "ignore_case" set to true ' do
       search_statement1 = search_statement_class.new(['"records"."text"', '"records"."text2"'], engines: [:array], ignore_accent: true, ignore_case: true)
       search_engine = search_engine_class.new({ pattern: 'search word' }, [search_statement1])
-      search_engine.conditions.to_sql.should == %{((("records"."text"::text[] || "records"."text2"::text[]) @> ARRAY[unaccent(lower('search')),unaccent(lower('word'))]))}
+      search_engine.conditions.to_sql.should == %{(("records"."text"::text[] || "records"."text2"::text[]) @> ARRAY[unaccent(lower('search')),unaccent(lower('word'))])}
     end
 
     it 'generates search condition with one column, one term, two statements and no options' do

data/spec/lib/mincer/processors/pg_search/search_engines/fulltext_spec.rb

@@ -33,7 +33,7 @@ describe ::Mincer::PgSearch::SearchEngines::Fulltext do
       it 'generates search condition with one column, one term and no options without columns wrapped with coalesce' do
         search_statement1 = search_statement_class.new(['"records"."text"'], engines: [:fulltext])
         search_engine = search_engine_class.new({ pattern: 'search' }, [search_statement1])
-        search_engine.conditions.to_sql.should == %{((to_tsvector('simple', "records"."text") @@ to_tsquery('simple', 'search')))}
+        search_engine.conditions.to_sql.should == %{(to_tsvector('simple', "records"."text") @@ to_tsquery('simple', 'search'))}
       end
     end
 
@@ -86,7 +86,7 @@ describe ::Mincer::PgSearch::SearchEngines::Fulltext do
     it 'generates search condition with one column, one term and option "dictionary" set to :english' do
       search_statement1 = search_statement_class.new(['"records"."text"'], engines: [:fulltext], dictionary: :english)
       search_engine = search_engine_class.new({ pattern: 'search' }, [search_statement1])
-      search_engine.conditions.to_sql.should == %{((to_tsvector('english', "records"."text") @@ to_tsquery('english', 'search')))}
+      search_engine.conditions.to_sql.should == %{(to_tsvector('english', "records"."text") @@ to_tsquery('english', 'search'))}
     end
 
     it 'generates search condition with two search statements one column, one term and no options' do

data/spec/lib/mincer/processors/pg_search/search_engines/trigram_spec.rb

@@ -31,13 +31,13 @@ describe ::Mincer::PgSearch::SearchEngines::Trigram do
     it 'generates search condition with one column, one term and no options' do
       search_statement1 = search_statement_class.new(['"records"."text"'], engines: [:trigram])
       search_engine = search_engine_class.new({ pattern: 'search' }, [search_statement1])
-      search_engine.conditions.to_sql.should == %{(("records"."text" % 'search'))}
+      search_engine.conditions.to_sql.should == %{("records"."text" % 'search')}
     end
 
     it 'generates search condition with one column, one term and "threshold" option set to 0.5' do
       search_statement1 = search_statement_class.new(['"records"."text"'], engines: [:trigram], threshold: 0.5)
       search_engine = search_engine_class.new({ pattern: 'search' }, [search_statement1])
-      search_engine.conditions.to_sql.should == %{((similarity("records"."text", 'search') >= 0.5))}
+      search_engine.conditions.to_sql.should == %{(similarity("records"."text", 'search') >= 0.5)}
     end
 
     it 'generates search condition with two columns, one term and no options' do

metadata

@@ -1,29 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: mincer
 version: !ruby/object:Gem::Version
-  version: 0.2.3
+  version: 0.2.4
 platform: ruby
 authors:
 - Alex Krasinsky
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-11 00:00:00.000000000 Z
+date: 2015-01-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '4.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '4.2'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement