scoped_search 4.1.9 → 4.1.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9c1453ebb8fe2391f71ea35171f2115ed89bb7920a1a042d86a4393fe218888c
-  data.tar.gz: 61d2d0e948a952f7e955240747dae7e529103c5871ad3160cb2213574f0f155a
+  metadata.gz: f0f6f98982cba23ec69f2b1adaf0c44832abb19adad7a347d4d78c4b0fbb63a8
+  data.tar.gz: 114ea022f87208dc89b92247687d64c2a0146119afa33f2106b69507271ba210
 SHA512:
-  metadata.gz: 04e61c564c6117d60b6f82b970df5c6c7a68362d828d89d1cf5d2bf7f6b7c9ea777cf641dbb2a21be895905305c0c7305105c9840d391aa702cd69eab706fd4d
-  data.tar.gz: 842c31b1714558d8bd4578d47b87bca4fa4298617a16168b66a77e6ff06ee1f43406b30be3557a9c50222ba5effaa9c5eec27832839e75bb11faf085f95bb7bc
+  metadata.gz: 6a87832ba5990fd45b7116d374a9c2505c7f9fcf62b4364859986ae497b2aa6d3a0890b222b1ad6c07ea999d48b25fbc42303a63cdc8ce3f4ea3a23e1f049eeb
+  data.tar.gz: a5cd950d77712004cb649261f08a8e8e04169622b346de7c3b15b422dbae7343bf8211febbdb988cf390a43c5f4794c93e820b9bc440d4cc9130d7f3146464f9

data/.github/workflows/ruby.yml

@@ -0,0 +1,76 @@
+# This workflow uses actions that are not certified by GitHub.
+# They are provided by a third-party and are governed by
+# separate terms of service, privacy policy, and support
+# documentation.
+# This workflow will download a prebuilt Ruby version, install dependencies and run tests with Rake
+# For more information see: https://github.com/marketplace/actions/setup-ruby-jruby-and-truffleruby
+
+name: Ruby
+
+on: [pull_request]
+
+env:
+  TESTOPTS: --verbose
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        ruby_version:
+          - 2.7.1
+          - 3.0.0
+          - ruby-head
+          - jruby-head
+        gemfile:
+          - Gemfile.activerecord52
+          - Gemfile.activerecord52_with_activesupport52
+          - Gemfile.activerecord60
+          - Gemfile.activerecord60_with_activesupport60
+          - Gemfile.activerecord61
+          - Gemfile.activerecord61_with_activesupport61
+
+        exclude:
+          - ruby_version: 2.7.1
+            gemfile: Gemfile.activerecord52
+          - ruby_version: 2.7.1
+            gemfile: Gemfile.activerecord52_with_activesupport52
+          - ruby_version: 3.0.0
+            gemfile: Gemfile.activerecord52
+          - ruby_version: 3.0.0
+            gemfile: Gemfile.activerecord52_with_activesupport52
+          - ruby_version: ruby-head
+            gemfile: Gemfile.activerecord52
+          - ruby_version: ruby-head
+            gemfile: Gemfile.activerecord52_with_activesupport52
+    services:
+      postgres:
+        image: postgres:12.1
+        ports: ['5432:5432']
+        options: --health-cmd pg_isready --health-interval 10s --health-timeout 5s --health-retries 5
+        env:
+          POSTGRES_DB: scoped_search_test
+      mariadb:
+        image: mariadb:10
+        ports: ['3306:3306']
+        env:
+          MYSQL_ALLOW_EMPTY_PASSWORD: 'yes'
+          MYSQL_DATABASE: scoped_search_test
+
+    env:
+      BUNDLE_GEMFILE: ${{ github.workspace }}/${{ matrix.gemfile }}
+
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Ruby
+    # To automatically get bug fixes and new Ruby versions for ruby/setup-ruby,
+    # change this to (see https://github.com/ruby/setup-ruby#versioning):
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby_version }}
+        bundler-cache: true
+    # - name: Install dependencies
+    #   run: bundle install
+    - name: Run tests
+      run: bundle exec rake

data/.travis.yml

@@ -22,6 +22,7 @@ rvm:
 - "2.5.1"
 - "2.6.0"
 - "2.7.1"
+- "3.0.2"
 - ruby-head
 - jruby-19mode
 - jruby-head
@@ -34,6 +35,8 @@ gemfile:
 - Gemfile.activerecord52_with_activesupport52
 - Gemfile.activerecord60
 - Gemfile.activerecord60_with_activesupport60
+- Gemfile.activerecord61
+- Gemfile.activerecord61_with_activesupport61
 
 matrix:
   allow_failures:
@@ -77,6 +80,26 @@ matrix:
       gemfile: Gemfile.activerecord60_with_activesupport60
     - rvm: "2.4.0"
       gemfile: Gemfile.activerecord60_with_activesupport60
+    - rvm: "2.0"
+      gemfile: Gemfile.activerecord61
+    - rvm: "2.1"
+      gemfile: Gemfile.activerecord61
+    - rvm: "2.2.2"
+      gemfile: Gemfile.activerecord61
+    - rvm: "2.3.7"
+      gemfile: Gemfile.activerecord61
+    - rvm: "2.4.0"
+      gemfile: Gemfile.activerecord61
+    - rvm: "2.0"
+      gemfile: Gemfile.activerecord61_with_activesupport61
+    - rvm: "2.1"
+      gemfile: Gemfile.activerecord61_with_activesupport61
+    - rvm: "2.2.2"
+      gemfile: Gemfile.activerecord61_with_activesupport61
+    - rvm: "2.3.7"
+      gemfile: Gemfile.activerecord61_with_activesupport61
+    - rvm: "2.4.0"
+      gemfile: Gemfile.activerecord61_with_activesupport61
     - rvm: "2.7.1"
       gemfile: Gemfile.activerecord42
     - rvm: "2.7.1"
@@ -87,3 +110,13 @@ matrix:
       gemfile: Gemfile.activerecord52
     - rvm: "2.7.1"
       gemfile: Gemfile.activerecord52_with_activesupport52
+    - rvm: "3.0.2"
+      gemfile: Gemfile.activerecord42
+    - rvm: "3.0.2"
+      gemfile: Gemfile.activerecord50
+    - rvm: "3.0.2"
+      gemfile: Gemfile.activerecord51
+    - rvm: "3.0.2"
+      gemfile: Gemfile.activerecord52
+    - rvm: "3.0.2"
+      gemfile: Gemfile.activerecord52_with_activesupport52

data/CHANGELOG.rdoc

@@ -6,7 +6,16 @@ Please add an entry to the "Unreleased changes" section in your pull requests.
 
 === Unreleased changes
 
-- Nothing yet
+=== Version 4.1.10
+
+- Fix querying through associations in Rails 6.1 (undefined method join_keys) (#201)
+- Fix query generation when going through STI associations (#211)
+- Properly support Ruby 3
+- Add support for postgres citext type
+
+=== Version 4.1.9
+- Fix querying on nested has-many associations (#196)
+- Abide "has_many ..., primary_key" overrides in relation searches
 
 === Version 4.1.8
 

data/Gemfile.activerecord61

@@ -0,0 +1,17 @@
+source 'https://rubygems.org'
+gemspec
+
+gem 'actionview', '~> 6.1.0'
+gem 'activerecord', '~> 6.1.0'
+
+platforms :jruby do
+  gem 'activerecord-jdbcsqlite3-adapter'
+  gem 'activerecord-jdbcmysql-adapter'
+  gem 'activerecord-jdbcpostgresql-adapter'
+end
+
+platforms :ruby do
+  gem 'sqlite3', '~> 1.4'
+  gem 'mysql2', '> 0.5'
+  gem 'pg', '>= 0.18', '< 2.0'
+end

data/Gemfile.activerecord61_with_activesupport61

@@ -0,0 +1,18 @@
+source 'https://rubygems.org'
+gemspec
+
+gem 'actionview', '~> 6.1.0'
+gem 'activerecord', '~> 6.1.0'
+gem 'activesupport', '~> 6.1.0'
+
+platforms :jruby do
+  gem 'activerecord-jdbcsqlite3-adapter'
+  gem 'activerecord-jdbcmysql-adapter'
+  gem 'activerecord-jdbcpostgresql-adapter'
+end
+
+platforms :ruby do
+  gem 'sqlite3', '~> 1.4'
+  gem 'mysql2', '> 0.5'
+  gem 'pg', '>= 0.18', '< 2.0'
+end

data/lib/scoped_search/definition.rb

@@ -157,7 +157,7 @@ module ScopedSearch
 
       # Returns true if this is a textual column.
       def textual?
-        [:string, :text].include?(type)
+        [:string, :text, :citext].include?(type)
       end
 
       def uuid?
@@ -172,7 +172,7 @@ module ScopedSearch
       # Returns the default search operator for this field.
       def default_operator
         @default_operator ||= case type
-          when :string, :text then :like
+          when :string, :text, :citext then :like
           else :eq
         end
       end
@@ -266,7 +266,7 @@ module ScopedSearch
     def default_fields_for(value, operator = nil)
 
       column_types  = [:virtual]
-      column_types += [:string, :text]                if [nil, :like, :unlike, :ne, :eq].include?(operator)
+      column_types += [:string, :text, :citext]       if [nil, :like, :unlike, :ne, :eq].include?(operator)
       column_types += [:double, :float, :decimal]     if value =~ NUMERICAL_REGXP
       column_types += [:integer]                      if value =~ INTEGER_REGXP
       column_types += [:uuid]                         if value =~ UUID_REGXP
@@ -305,8 +305,8 @@ module ScopedSearch
     end
 
     # Defines a new search field for this search definition.
-    def define(*args)
-      Field.new(self, *args)
+    def define(*args, **kwargs)
+      Field.new(self, *args, **kwargs)
     end
 
     # Returns a reflection for a given klass and name

data/lib/scoped_search/query_builder.rb

@@ -38,7 +38,9 @@ module ScopedSearch
 
     # Initializes the instance by setting the relevant parameters
     def initialize(definition, ast, profile)
-      @definition, @ast, @definition.profile = definition, ast, profile
+      @definition = definition
+      @ast = ast
+      @definition.profile = profile
     end
 
     # Actually builds the find parameters hash that should be used in the search_for
@@ -287,15 +289,20 @@ module ScopedSearch
       connection = many_class.connection
       sql = connection.quote_table_name(many_class.table_name)
       join_reflections = nested_has_many(many_class, field.relation)
-      table_names = [many_class.table_name] + join_reflections.map(&:table_name)
+      table_names = [[many_class.table_name, many_class.sti_name.tableize]] + join_reflections.map(&:table_name)
 
       join_reflections.zip(table_names.zip(join_reflections.drop(1))).reduce(sql) do |acc, (reflection, (previous_table, next_reflection))|
-        klass = reflection.method(:join_keys).arity == 1 ? [reflection.klass] : [] # ActiveRecord <5.2 workaround
-        fk1, pk1 = reflection.join_keys(*klass).values # We are joining the tables "in reverse", so the PK and FK are swapped
-
+        fk1, pk1 = if reflection.respond_to?(:join_keys)
+                     klass = reflection.method(:join_keys).arity == 1 ? [reflection.klass] : [] # ActiveRecord <5.2 workaround
+                     reflection.join_keys(*klass).values # We are joining the tables "in reverse", so the PK and FK are swapped
+                   else
+                     [reflection.join_primary_key, reflection.join_foreign_key] #ActiveRecord 6.1
+                   end
+
+        previous_table, sti_name = previous_table
         # primary and foreign keys + optional conditions for the joins
         join_condition = if with_polymorphism?(reflection)
-                           field.reflection_conditions(definition.reflection_by_name(next_reflection.klass, previous_table))
+                           field.reflection_conditions(definition.reflection_by_name(next_reflection.klass, sti_name || previous_table))
                          else
                            ''
                          end

data/lib/scoped_search/version.rb

@@ -1,3 +1,3 @@
 module ScopedSearch
-  VERSION = "4.1.9"
+  VERSION = "4.1.10"
 end

data/lib/scoped_search.rb

@@ -34,9 +34,9 @@ module ScopedSearch
 
       definitions.each do |definition|
         if definition[:on].kind_of?(Array)
-          definition[:on].each { |field| self.scoped_search_definition.define(definition.merge(:on => field)) }
+          definition[:on].each { |field| self.scoped_search_definition.define(**definition.merge(:on => field)) }
         else
-          self.scoped_search_definition.define(definition)
+          self.scoped_search_definition.define(**definition)
         end
       end
       return self.scoped_search_definition

data/spec/database.jruby.yml

@@ -4,7 +4,7 @@ sqlite:
 
 mysql:
   adapter: jdbcmysql
-  host: localhost
+  host: 127.0.0.1
   username: root
   database: scoped_search_test
 

data/spec/database.ruby.yml

@@ -4,7 +4,7 @@ sqlite:
 
 mysql:
   adapter: mysql2
-  host: localhost
+  host: 127.0.0.1
   port: 3306
   username: root
   database: scoped_search_test

metadata

@@ -1,16 +1,16 @@
 --- !ruby/object:Gem::Specification
 name: scoped_search
 version: !ruby/object:Gem::Version
-  version: 4.1.9
+  version: 4.1.10
 platform: ruby
 authors:
 - Amos Benari
 - Willem van Bergen
 - Wes Hays
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-08-24 00:00:00.000000000 Z
+date: 2021-11-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -77,6 +77,7 @@ extra_rdoc_files:
 - CONTRIBUTING.rdoc
 - LICENSE
 files:
+- ".github/workflows/ruby.yml"
 - ".gitignore"
 - ".travis.yml"
 - CHANGELOG.rdoc
@@ -89,6 +90,8 @@ files:
 - Gemfile.activerecord52_with_activesupport52
 - Gemfile.activerecord60
 - Gemfile.activerecord60_with_activesupport60
+- Gemfile.activerecord61
+- Gemfile.activerecord61_with_activesupport61
 - LICENSE
 - README.rdoc
 - Rakefile
@@ -141,7 +144,7 @@ homepage: https://github.com/wvanbergen/scoped_search/wiki
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options:
 - "--title"
 - scoped_search
@@ -162,8 +165,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.1.2
+signing_key:
 specification_version: 4
 summary: Easily search you ActiveRecord models with a simple query language using
   a named scope