search_cop 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bf37a33f6c9998a09a09817fe5f14024947a1f83ed63682c066dcf35c385ac57
-  data.tar.gz: 36df6f6110f2126240702636d93c0ed54322794ff5f558fa6dcb27484c3b2b28
+  metadata.gz: 8cb7e91b305057b2ea3df0c96643f8cd2cc726583f2362db9d576ef128826a75
+  data.tar.gz: 419ee38ec698795d478f7749486e63b533c9cc497057aa820b5627b343cbdeb4
 SHA512:
-  metadata.gz: 4c5233ee11b096ec18e1a53f1e76a6dbb0880a99de9ac41ec836993f317612cf728d81a1332c3f7bfeb42766cb7e28e388061d85cabec8355e7c32e969468761
-  data.tar.gz: f766262804751519e3566ea80af0358dba148540883d74b5947804e70876ae90555dcb539032f5e59ff1d64fb03b6b10e293e487842b47eeea31246c14a69024
+  metadata.gz: 19b765a13b9a21a5b798b60618253729b4a8c1279babe47644d5c030b93b09c87706138ab11cd5bfd9daee757d8d04bff95fc1bc3ce50c5f6f7399aaaabc09d1
+  data.tar.gz: a65e72e3d6e5500b986bb312c45dbbd00432419b344765c71387432dd127efd5eb463cd42f71c3e405913a66b8d45201b9d909b9e66da17bf28f11ea3b7aa5c8

data/.github/workflows/test.yml

@@ -0,0 +1,42 @@
+name: test
+on: [push, pull_request]
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        ruby: ["2.5", "2.6", "2.7"]
+        rails: ["rails5", "rails6"]
+        database: ["sqlite", "postgres", "mysql"]
+    services:
+      postgres:
+        image: postgres
+        env:
+          POSTGRES_USER: search_cop
+          POSTGRES_PASSWORD: secret
+          POSTGRES_DB: search_cop
+        ports:
+          - 5432:5432
+      mysql:
+        image: mysql
+        env:
+          MYSQL_ALLOW_EMPTY_PASSWORD: "yes"
+          MYSQL_ROOT_PASSWORD: ""
+          MYSQL_DATABASE: search_cop
+        ports:
+          - 3306:3306
+    steps:
+      - uses: actions/checkout@v1
+      - uses: actions/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+      - name: test
+        env:
+          DATABASE: ${{ matrix.database }}
+        run: |
+          gem install bundler
+          bundle config set --local gemfile "gemfiles/${{ matrix.rails }}.gemfile"
+          bundle config set --local path "../vendor/bundle"
+          bundle install
+          bundle exec rake test
+          bundle exec rubocop

data/.rubocop.yml

@@ -0,0 +1,128 @@
+AllCops:
+  NewCops: enable
+
+Gemspec/RequiredRubyVersion:
+  Enabled: false
+
+Style/CaseLikeIf:
+  Enabled: false
+
+Style/RedundantArgument:
+  Enabled: false
+
+Lint/EmptyBlock:
+  Enabled: false
+
+Layout/EmptyLineBetweenDefs:
+  Enabled: false
+
+Style/FrozenStringLiteralComment:
+  Enabled: false
+
+Lint/RedundantRequireStatement:
+  Enabled: false
+
+Layout/ArgumentAlignment:
+  EnforcedStyle: with_fixed_indentation
+
+Layout/FirstArrayElementIndentation:
+  EnforcedStyle: consistent
+
+Style/PercentLiteralDelimiters:
+  Enabled: false
+
+Style/SpecialGlobalVars:
+  EnforcedStyle: use_english_names
+
+Security/Eval:
+  Enabled: false
+
+Style/WordArray:
+  EnforcedStyle: brackets
+
+Style/ClassAndModuleChildren:
+  Enabled: false
+
+Style/TrivialAccessors:
+  Enabled: false
+
+Style/Alias:
+  Enabled: false
+
+Style/StringLiteralsInInterpolation:
+  EnforcedStyle: double_quotes
+
+Metrics/ClassLength:
+  Enabled: false
+
+Naming/MethodParameterName:
+  Enabled: false
+
+Style/SymbolArray:
+  EnforcedStyle: brackets
+
+Layout/RescueEnsureAlignment:
+  Enabled: false
+
+Layout/LineLength:
+  Enabled: false
+
+Metrics/MethodLength:
+  Enabled: false
+
+Metrics/ModuleLength:
+  Enabled: false
+
+Style/ZeroLengthPredicate:
+  Enabled: false
+
+Metrics/PerceivedComplexity:
+  Enabled: false
+
+Metrics/AbcSize:
+  Enabled: false
+
+Metrics/CyclomaticComplexity:
+  Enabled: false
+
+Metrics/BlockLength:
+  Enabled: false
+
+Metrics/BlockNesting:
+  Enabled: false
+
+Style/NumericPredicate:
+  Enabled: false
+
+Naming/AccessorMethodName:
+  Enabled: false
+
+Naming/MemoizedInstanceVariableName:
+  Enabled: false
+
+Style/StringLiterals:
+  EnforcedStyle: double_quotes
+
+Style/Documentation:
+  Enabled: false
+
+Naming/ConstantName:
+  Enabled: false
+
+Style/MutableConstant:
+  Enabled: false
+
+Layout/MultilineMethodCallIndentation:
+  EnforcedStyle: indented
+
+Layout/ParameterAlignment:
+  EnforcedStyle: with_fixed_indentation
+
+Lint/UnusedMethodArgument:
+  Enabled: false
+
+Style/IfUnlessModifier:
+  Enabled: false
+
+Style/RedundantBegin:
+  Enabled: false

data/CHANGELOG.md

@@ -3,13 +3,18 @@
 
 Version 1.2.0:
 
+* Added support for disabling the right wildcard
+* Fixed escaping of wildcard chars (`_`, `%`)
+
+Version 1.1.0:
+
 * Adds customizable default operator to concatenate conditions (#49)
 * Make the postgis adapter use the postgres extensions
 
 Version 1.0.9:
 
-* Use [:blank:] instead of \s for space (#46)
-* Updated SearchCop::Visitors::Visitor to check the connection's adapter_name when extending. (#47)
+* Use `[:blank:]` instead of `\s` for space (#46)
+* Updated `SearchCop::Visitors::Visitor` to check the connection's `adapter_name` when extending. (#47)
 * Fix for negative numeric values
 * allow searching for relative dates, like hours, days, weeks, months or years ago
 
@@ -19,8 +24,8 @@ Version 1.0.8:
 
 Version 1.0.7:
 
-* Bugfix regarding NOT queries in fulltext mode #32
-* Safely handle NULL values for match queries
+* Bugfix regarding `NOT` queries in fulltext mode #32
+* Safely handle `NULL` values for match queries
 * Added coalesce option
 
 Version 1.0.6:
@@ -58,7 +63,7 @@ Version 1.0.0:
 
 Version 0.0.5:
 
-* Supporting :default => false
+* Supporting `:default => false`
 * Datetime/Date greater operator fix
 * Use reflection to find associated models
 * Providing reflection
@@ -67,16 +72,16 @@ Version 0.0.4:
 
 * Fixed date attributes
 * Fail softly for mixed datatype attributes
-* Support custom table, class and alias names via attr_searchable_alias
+* Support custom table, class and alias names via `attr_searchable_alias`
 
 Version 0.0.3:
 
-* belongs_to association fixes
+* `belongs_to` association fixes
 
 Version 0.0.2:
 
 * Arel abstraction layer added
-* count() queries resulting in "Cannot visit AttrSearchableGrammar::Nodes..." fixed
+* `count()` queries resulting in "Cannot visit AttrSearchableGrammar::Nodes..." fixed
 * Better error messages
-* Model#unsafe_search added
+* `Model#unsafe_search` added
 

data/Gemfile

@@ -1,11 +1,10 @@
-source 'https://rubygems.org'
+source "https://rubygems.org"
 
 # Specify your gem's dependencies in search_cop.gemspec
 gemspec
 
 platforms :ruby do
-  gem 'sqlite3'
-  gem 'mysql2'
-  gem 'pg'
+  gem "mysql2"
+  gem "pg"
+  gem "sqlite3"
 end
-

data/README.md

@@ -1,6 +1,6 @@
 # SearchCop
 
-[![Build Status](https://secure.travis-ci.org/mrkamel/search_cop.png?branch=master)](http://travis-ci.org/mrkamel/search_cop)
+[![Build Status](https://github.com/mrkamel/search_cop/workflows/test/badge.svg?branch=master)](https://github.com/mrkamel/search_cop/actions?query=workflow%3Atest)
 [![Code Climate](https://codeclimate.com/github/mrkamel/search_cop.png)](https://codeclimate.com/github/mrkamel/search_cop)
 [![Gem Version](https://badge.fury.io/rb/search_cop.svg)](http://badge.fury.io/rb/search_cop)
 
@@ -329,6 +329,16 @@ User.search("admin")
 # ... WHERE users.username LIKE 'admin%'
 ```
 
+Similarly, you can disable the right wildcard as well:
+
+```ruby
+search_scope :search do
+  attributes :username
+
+  options :username, right_wildcard: false
+end
+```
+
 ## Default operator
 
 When you define multiple fields on a search scope, SearcCop will use

data/Rakefile

@@ -6,4 +6,3 @@ Rake::TestTask.new(:test) do |t|
   t.pattern = "test/**/*_test.rb"
   t.verbose = true
 end
-

data/docker-compose.yml

@@ -8,4 +8,11 @@ services:
       - MYSQL_ROOT_PASSWORD=
     ports:
       - 3306:3306
-
+  postgres:
+    image: postgres:9.6.6
+    environment:
+      POSTGRES_DB: search_cop
+      POSTGRES_USER: search_cop
+      POSTGRES_PASSWORD: secret
+    ports:
+      - 5432:5432

data/gemfiles/{5.1.gemfile → rails5.gemfile}

@@ -5,9 +5,9 @@ source "https://rubygems.org"
 gem "activerecord", "~> 5.1"
 
 platforms :ruby do
-  gem "sqlite3"
   gem "mysql2"
   gem "pg"
+  gem "sqlite3"
 end
 
-gemspec :path => "../"
+gemspec path: "../"

data/gemfiles/rails6.gemfile

@@ -0,0 +1,13 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "activerecord", "~> 6.0"
+
+platforms :ruby do
+  gem "mysql2"
+  gem "pg"
+  gem "sqlite3"
+end
+
+gemspec path: "../"

data/lib/search_cop.rb

@@ -1,5 +1,5 @@
-
 require "search_cop/version"
+require "search_cop/helpers"
 require "search_cop/search_scope"
 require "search_cop/query_info"
 require "search_cop/query_builder"
@@ -23,7 +23,7 @@ module SearchCop
   module Parser
     def self.parse(query, query_info, query_options = {})
       if query.is_a?(Hash)
-        SearchCop::HashParser.new(query_info).parse(query)
+        SearchCop::HashParser.new(query_info).parse(query, query_options)
       else
         SearchCop::GrammarParser.new(query_info).parse(query, query_options)
       end
@@ -44,8 +44,8 @@ module SearchCop
       search_scopes[name] = SearchScope.new(name, self)
       search_scopes[name].instance_exec(&block)
 
-      self.send(:define_singleton_method, name) { |query, query_options={}| search_cop query, name, query_options }
-      self.send(:define_singleton_method, "unsafe_#{name}") { |query, query_options={}| unsafe_search_cop query, name, query_options }
+      send(:define_singleton_method, name) { |query, query_options = {}| search_cop(query, name, query_options) }
+      send(:define_singleton_method, "unsafe_#{name}") { |query, query_options = {}| unsafe_search_cop(query, name, query_options) }
     end
 
     def search_reflection(scope_name)
@@ -53,7 +53,7 @@ module SearchCop
     end
 
     def search_cop(query, scope_name, query_options)
-      unsafe_search_cop query, scope_name, query_options
+      unsafe_search_cop(query, scope_name, query_options)
     rescue SearchCop::RuntimeError
       respond_to?(:none) ? none : where("1 = 0")
     end
@@ -69,22 +69,4 @@ module SearchCop
       (scope || self).where(query_builder.sql)
     end
   end
-
-  module Helpers
-    def self.sanitize_default_operator(hash, delete_hash_option=false)
-      default_operator = :and
-      if hash.member?(:default_operator)
-        unless [:and, :or].include?(hash[:default_operator])
-          raise(SearchCop::UnknownDefaultOperator, "Unknown default operator value #{hash[:default_operator]}")
-        end
-
-        default_operator = hash[:default_operator]
-      end
-
-      hash.delete(:default_operator) if delete_hash_option
-
-      default_operator
-    end
-  end
 end
-

data/lib/search_cop/grammar_parser.rb

@@ -1,8 +1,7 @@
-
 require "search_cop_grammar"
 require "treetop"
 
-Treetop.load File.expand_path("../../search_cop_grammar.treetop", __FILE__)
+Treetop.load File.expand_path("../search_cop_grammar.treetop", __dir__)
 
 module SearchCop
   class GrammarParser
@@ -20,4 +19,3 @@ module SearchCop
     end
   end
 end
-

data/lib/search_cop/hash_parser.rb

@@ -1,4 +1,3 @@
-
 class SearchCop::HashParser
   attr_reader :query_info
 
@@ -6,25 +5,25 @@ class SearchCop::HashParser
     @query_info = query_info
   end
 
-  def parse(hash)
-    default_operator = SearchCop::Helpers.sanitize_default_operator(hash, true)
+  def parse(hash, query_options = {})
+    default_operator = SearchCop::Helpers.sanitize_default_operator(query_options)
 
     res = hash.collect do |key, value|
       case key
-        when :and
-          value.collect { |val| parse val }.inject(:and)
-        when :or
-          value.collect { |val| parse val }.inject(:or)
-        when :not
-          parse(value).not
-        when :query
-          SearchCop::Parser.parse value, query_info
-        else
-          parse_attribute key, value
+      when :and
+        value.collect { |val| parse val }.inject(:and)
+      when :or
+        value.collect { |val| parse val }.inject(:or)
+      when :not
+        parse(value).not
+      when :query
+        SearchCop::Parser.parse(value, query_info)
+      else
+        parse_attribute(key, value)
       end
     end
 
-    res.inject default_operator
+    res.inject(default_operator)
   end
 
   private
@@ -35,14 +34,15 @@ class SearchCop::HashParser
     if value.is_a?(Hash)
       raise(SearchCop::ParseError, "Unknown operator #{value.keys.first}") unless collection.valid_operator?(value.keys.first)
 
-      if generator = collection.generator_for(value.keys.first)
-        collection.generator generator, value.values.first
+      generator = collection.generator_for(value.keys.first)
+
+      if generator
+        collection.generator(generator, value.values.first)
       else
-        collection.send value.keys.first, value.values.first.to_s
+        collection.send(value.keys.first, value.values.first.to_s)
       end
     else
-      collection.send :matches, value.to_s
+      collection.send(:matches, value.to_s)
     end
   end
 end
-