filterable-by 0.5.0 → 0.5.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1369751aa86d400aa27ab43ad4cdbbcc92ff06a2ca19162582f8c95914882814
-  data.tar.gz: ca8d93901d61b20377196045430f22e63bf622e05abdab58ec4f85a578526458
+  metadata.gz: 7df6ffda90f61fe76374ea6f338c2857a262f7a824620719fb36df05ba22fa0e
+  data.tar.gz: 31cec22f5a8c62759ca8b5155ea9060ff84a85f5dc702baf9237bc3aa3219baa
 SHA512:
-  metadata.gz: 5e487f8f1e5337faaaf0a25c009ded133b78f21a89c07b49b4d81ead360b6cbfb1a21acb4787ff96b2e5d35931ed126f42c1d1acdfa6b23b8d8ad190da260a41
-  data.tar.gz: e68b17a74b428364b6bb46bf92da38f18b0fda824eef32823af567b336f172ef011c2dd15a904e41accad29ea0e0c103f078e8949a30dedd08c7a6b96407152b
+  metadata.gz: c3e36d0fb257cb048470825bc2669b050759fec897588a0bdad51e2f5b9a7f4702d68abf32fad5412f718ef233d9a6ca5a5c4a8c64c2a6768a1140a09cb57a88
+  data.tar.gz: 6a488f6969ab64b3dad1091966a5da7aa072da42b973f714fdb85934a4fbfe555a9543d2d25bfad2a035b5d542033fb1411dbdb324faa921a70e41520653e3c3

data/.github/workflows/test.yml

@@ -0,0 +1,21 @@
+name: Test
+
+on:
+  push:
+    branches: [main]
+  pull_request:
+    branches: [main]
+
+jobs:
+  ruby:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        ruby-version: ["2.7", "3.0", "3.1"]
+    steps:
+      - uses: actions/checkout@v2
+      - uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby-version }}
+          bundler-cache: true
+      - run: bundle exec rake

data/.rubocop.yml

@@ -1,6 +1,11 @@
-inherit_from:
-  - https://gitlab.com/bsm/misc/raw/master/rubocop/default.yml
+inherit_gem:
+  rubocop-bsm:
+    - default.yml
+inherit_mode:
+  merge:
+    - Exclude
+
 AllCops:
-  TargetRubyVersion: "2.5"
+  TargetRubyVersion: "2.7"
 Naming/FileName:
   Exclude: [lib/filterable-by.rb]

data/Gemfile.lock

@@ -1,63 +1,77 @@
 PATH
   remote: .
   specs:
-    filterable-by (0.5.0)
+    filterable-by (0.5.3)
       activerecord
       activesupport
 
 GEM
   remote: http://rubygems.org/
   specs:
-    activemodel (6.0.0)
-      activesupport (= 6.0.0)
-    activerecord (6.0.0)
-      activemodel (= 6.0.0)
-      activesupport (= 6.0.0)
-    activesupport (6.0.0)
+    activemodel (7.0.2.2)
+      activesupport (= 7.0.2.2)
+    activerecord (7.0.2.2)
+      activemodel (= 7.0.2.2)
+      activesupport (= 7.0.2.2)
+    activesupport (7.0.2.2)
       concurrent-ruby (~> 1.0, >= 1.0.2)
-      i18n (>= 0.7, < 2)
-      minitest (~> 5.1)
-      tzinfo (~> 1.1)
-      zeitwerk (~> 2.1, >= 2.1.8)
-    ast (2.4.0)
-    concurrent-ruby (1.1.5)
-    diff-lcs (1.3)
-    i18n (1.6.0)
+      i18n (>= 1.6, < 2)
+      minitest (>= 5.1)
+      tzinfo (~> 2.0)
+    ast (2.4.2)
+    concurrent-ruby (1.1.9)
+    diff-lcs (1.5.0)
+    i18n (1.10.0)
       concurrent-ruby (~> 1.0)
-    jaro_winkler (1.5.3)
-    minitest (5.11.3)
-    parallel (1.17.0)
-    parser (2.6.3.0)
-      ast (~> 2.4.0)
-    rainbow (3.0.0)
-    rake (12.3.3)
-    rspec (3.8.0)
-      rspec-core (~> 3.8.0)
-      rspec-expectations (~> 3.8.0)
-      rspec-mocks (~> 3.8.0)
-    rspec-core (3.8.2)
-      rspec-support (~> 3.8.0)
-    rspec-expectations (3.8.4)
+    minitest (5.15.0)
+    parallel (1.21.0)
+    parser (3.1.1.0)
+      ast (~> 2.4.1)
+    rainbow (3.1.1)
+    rake (13.0.6)
+    regexp_parser (2.2.1)
+    rexml (3.2.5)
+    rspec (3.11.0)
+      rspec-core (~> 3.11.0)
+      rspec-expectations (~> 3.11.0)
+      rspec-mocks (~> 3.11.0)
+    rspec-core (3.11.0)
+      rspec-support (~> 3.11.0)
+    rspec-expectations (3.11.0)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.8.0)
-    rspec-mocks (3.8.1)
+      rspec-support (~> 3.11.0)
+    rspec-mocks (3.11.0)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.8.0)
-    rspec-support (3.8.2)
-    rubocop (0.74.0)
-      jaro_winkler (~> 1.5.1)
+      rspec-support (~> 3.11.0)
+    rspec-support (3.11.0)
+    rubocop (1.25.1)
       parallel (~> 1.10)
-      parser (>= 2.6)
+      parser (>= 3.1.0.0)
       rainbow (>= 2.2.2, < 4.0)
+      regexp_parser (>= 1.8, < 3.0)
+      rexml
+      rubocop-ast (>= 1.15.1, < 2.0)
       ruby-progressbar (~> 1.7)
-      unicode-display_width (>= 1.4.0, < 1.7)
-    ruby-progressbar (1.10.1)
-    sqlite3 (1.4.1)
-    thread_safe (0.3.6)
-    tzinfo (1.2.5)
-      thread_safe (~> 0.1)
-    unicode-display_width (1.6.0)
-    zeitwerk (2.1.9)
+      unicode-display_width (>= 1.4.0, < 3.0)
+    rubocop-ast (1.16.0)
+      parser (>= 3.1.1.0)
+    rubocop-bsm (0.6.0)
+      rubocop (~> 1.0)
+      rubocop-performance
+      rubocop-rake
+      rubocop-rspec
+    rubocop-performance (1.13.3)
+      rubocop (>= 1.7.0, < 2.0)
+      rubocop-ast (>= 0.4.0)
+    rubocop-rake (0.6.0)
+      rubocop (~> 1.0)
+    rubocop-rspec (2.9.0)
+      rubocop (~> 1.19)
+    ruby-progressbar (1.11.0)
+    sqlite3 (1.4.2)
+    tzinfo (2.0.4)
+      concurrent-ruby (~> 1.0)
+    unicode-display_width (2.1.0)
 
 PLATFORMS
   ruby
@@ -67,8 +81,8 @@ DEPENDENCIES
   filterable-by!
   rake
   rspec
-  rubocop
+  rubocop-bsm
   sqlite3
 
 BUNDLED WITH
-   2.0.2
+   2.3.6

data/README.md

@@ -1,5 +1,7 @@
 # Filterable By
 
+[![Test](https://github.com/bsm/filterable-by/actions/workflows/test.yml/badge.svg)](https://github.com/bsm/filterable-by/actions/workflows/test.yml)
+
 ActiveRecord plugin to parse e.g. a `filter` query parameter apply scopes. Useful for [JSON-API][jsonapi] compatibility.
 
 [jsonapi]: http://jsonapi.org/format/#fetching-filtering
@@ -16,22 +18,33 @@ class Comment < ActiveRecord::Base
 
   filterable_by :post_id, :user_id
   filterable_by :post_author_id do |scope, value|
-    scope.joins(:posts).where(:"posts.author_id" => value)
+    scope.joins(:posts).where(:'posts.author_id' => value)
+  end
+  filterable_by :only do |scope, value, **opts|
+    case value
+    when 'mine'
+      scope.where(user_id: opts[:user_id]) if opts[:user_id]
+    else
+      scope
+    end
   end
 end
 
-Comment.filter_by(params[:filter])  # => ActiveRecord::Relation
+Comment.filter_by(params[:filter], user_id: current_user.id)  # => ActiveRecord::Relation
 ```
 
 Simple use cases:
 
 ```ruby
-Comment.filter_by({ "post_id" => "1" })
+Comment.filter_by({ 'post_id' => '1' })
 # => WHERE post_id = 1
 
-Comment.filter_by({ "user_id" => "2", "ignored" => "3" })
+Comment.filter_by({ 'user_id' => '2', 'ignored' => '3' })
 # => WHERE user_id = 2
 
-Comment.filter_by({ "post_author_id" => "5" })
+Comment.filter_by({ 'only' => 'mine' }, user_id: 4)
+# => WHERE user_id = 4
+
+Comment.filter_by({ 'post_author_id' => '5' })
 # => JOINS posts ON posts.id = comments.post_id WHERE posts.author_id = 5
 ```

data/filterable-by.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name        = 'filterable-by'
-  s.version     = '0.5.0'
+  s.version     = '0.5.3'
   s.authors     = ['Dimitrij Denissenko']
   s.email       = ['dimitrij@blacksquaremedia.com']
   s.summary     = 'Generate white-listed filter scopes from URL parameter values'
@@ -11,7 +11,7 @@ Gem::Specification.new do |s|
   s.files         = `git ls-files -z`.split("\x0").reject {|f| f.match(%r{^spec/}) }
   s.test_files    = `git ls-files -z -- spec/*`.split("\x0")
   s.require_paths = ['lib']
-  s.required_ruby_version = '>= 2.5'
+  s.required_ruby_version = '>= 2.7'
 
   s.add_dependency 'activerecord'
   s.add_dependency 'activesupport'
@@ -19,6 +19,7 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'bundler'
   s.add_development_dependency 'rake'
   s.add_development_dependency 'rspec'
-  s.add_development_dependency 'rubocop'
+  s.add_development_dependency 'rubocop-bsm'
   s.add_development_dependency 'sqlite3'
+  s.metadata['rubygems_mfa_required'] = 'true'
 end

data/lib/filterable_by.rb

@@ -27,15 +27,19 @@ module ActiveRecord
 
       def filterable_by(*names, &block)
         names.each do |name|
-          _filterable_by_config[name.to_s] = block || ->(scope, v) { scope.where(name.to_sym => v) }
+          _filterable_by_config[name.to_s] = block || ->(scope, value, **) { scope.where(name.to_sym => value) }
         end
       end
 
-      # @param [Hash] hash the filter params
+      # @param [Hash|ActionController::Parameters] the filter params
       # @return [ActiveRecord::Relation] the scoped relation
-      def filter_by(hash)
+      def filter_by(hash = nil, **opts)
+        if hash.nil?
+          hash = opts
+          opts = {}
+        end
         scope = all
-        return scope unless hash.is_a?(Hash)
+        return scope unless hash.respond_to?(:key?) && hash.respond_to?(:[])
 
         _filterable_by_config.each do |name, block|
           next unless hash.key?(name)
@@ -43,9 +47,10 @@ module ActiveRecord
           value = FilterableBy.normalize(hash[name])
           next if value.blank?
 
-          scope = block.call(scope, value)
+          scope = block.call(scope, value, **opts)
         end
-        scope
+
+        scope || none
       end
     end
   end

data/spec/{filterable_by_spec.rb → active_record/filterable_by_spec.rb}

@@ -1,4 +1,4 @@
-require File.dirname(__FILE__) + '/spec_helper'
+require 'spec_helper'
 
 describe ActiveRecord::FilterableBy do
   let(:alice) { AUTHORS[:alice] }
@@ -7,21 +7,23 @@ describe ActiveRecord::FilterableBy do
   let(:apost) { POSTS[:alices] }
   let(:bpost) { POSTS[:bobs] }
 
-  it 'should have config' do
+  it 'has config' do
     expect(Comment.send(:_filterable_by_config).count).to eq(3)
     expect(Rating.send(:_filterable_by_config).count).to eq(2)
-    expect(Post.send(:_filterable_by_config).count).to eq(1)
+    expect(Post.send(:_filterable_by_config).count).to eq(2)
   end
 
-  it 'should ignore bad inputs' do
+  it 'ignores bad inputs' do
+    expect(Comment.filter_by.count).to eq(4)
     expect(Comment.filter_by(nil).count).to eq(4)
-    expect(Comment.filter_by({}).count).to eq(4)
+    expect(Comment.filter_by(nil, extra: true).count).to eq(4)
+    expect(Comment.filter_by('bad').count).to eq(4)
 
     expect(Comment.filter_by('author_id' => '').count).to eq(4)
     expect(Comment.filter_by('author_id' => []).count).to eq(4)
   end
 
-  it 'should generate simple scopes' do
+  it 'generates simple scopes' do
     expect(Comment.filter_by('author_id' => alice.id).pluck(:title)).to match_array(%w[AA AB])
     expect(Comment.filter_by('author_id' => bob.id).pluck(:title)).to match_array(%w[BA BB])
     expect(Comment.filter_by('author_id' => [alice.id, '']).pluck(:title)).to match_array(%w[AA AB])
@@ -38,7 +40,7 @@ describe ActiveRecord::FilterableBy do
     expect(Post.filter_by('author_id' => bob.id).count).to eq(1)
   end
 
-  it 'should generate combined scopes' do
+  it 'generates combined scopes' do
     expect(Comment.filter_by('author_id' => alice.id, 'post_id' => apost.id).pluck(:title)).to match_array(['AA'])
     expect(Comment.filter_by('author_id' => alice.id, 'post_id' => bpost.id).pluck(:title)).to match_array(['AB'])
     expect(Comment.filter_by('author_id' => bob.id, 'post_id' => apost.id).pluck(:title)).to match_array(['BA'])
@@ -48,12 +50,29 @@ describe ActiveRecord::FilterableBy do
     expect(scope.pluck(:title)).to match_array(%w[AB BB])
   end
 
-  it 'should combine with other scopes' do
+  it 'combines with other scopes' do
     scope = Comment.where(author_id: alice.id).filter_by('post_id' => apost.id)
     expect(scope.pluck(:title)).to match_array(['AA'])
   end
 
-  it 'should ignore invalid scopes' do
+  it 'allows custom options' do
+    scope = Post.filter_by({ 'only' => 'me' }, user_id: alice.id)
+    expect(scope).to match_array([apost])
+
+    scope = Post.filter_by({ 'only' => '??' }, user_id: alice.id)
+    expect(scope.count).to eq(2)
+
+    scope = Post.filter_by({ 'only' => 'me' })
+    expect(scope.count).to eq(0)
+  end
+
+  it 'allows custom options from params' do
+    filter = { 'only' => 'me' }
+    expect(Post.filter_by(filter, user_id: alice.id)).to match_array([apost])
+    expect(Post.filter_by(filter).count).to eq(0)
+  end
+
+  it 'ignores invalid scopes' do
     expect(Comment.filter_by('invalid' => 1).count).to eq(4)
     expect(Post.filter_by('post_id' => bpost.id).count).to eq(2)
     expect(Rating.filter_by('post_author_id' => bob.id).count).to eq(1)

data/spec/spec_helper.rb

@@ -2,11 +2,12 @@ ENV['RACK_ENV'] ||= 'test'
 require 'filterable-by'
 require 'rspec'
 
-ActiveRecord::Base.configurations['test'] = { 'adapter' => 'sqlite3', 'database' => ':memory:' }
+ActiveRecord::Base.configurations = { 'test' => { 'adapter' => 'sqlite3', 'database' => ':memory:' } }
 ActiveRecord::Base.establish_connection :test
 
 ActiveRecord::Base.connection.instance_eval do
   create_table :authors do |_|
+    # no columns
   end
   create_table :posts do |t|
     t.integer :author_id, null: false
@@ -27,6 +28,14 @@ class Post < ActiveRecord::Base
   belongs_to :author
 
   filterable_by :author_id
+  filterable_by :only do |scope, value, **opts|
+    case value
+    when 'me'
+      scope.where(author_id: opts[:user_id]) if opts[:user_id]
+    else
+      scope
+    end
+  end
 end
 
 class Feedback < ActiveRecord::Base

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: filterable-by
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.5.3
 platform: ruby
 authors:
 - Dimitrij Denissenko
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-08-27 00:00:00.000000000 Z
+date: 2022-03-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -81,7 +81,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: rubocop-bsm
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -115,9 +115,9 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/test.yml"
 - ".gitignore"
 - ".rubocop.yml"
-- ".travis.yml"
 - Gemfile
 - Gemfile.lock
 - LICENSE
@@ -126,12 +126,13 @@ files:
 - filterable-by.gemspec
 - lib/filterable-by.rb
 - lib/filterable_by.rb
-- spec/filterable_by_spec.rb
+- spec/active_record/filterable_by_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/bsm/filterable-by
 licenses:
 - MIT
-metadata: {}
+metadata:
+  rubygems_mfa_required: 'true'
 post_install_message: 
 rdoc_options: []
 require_paths:
@@ -140,17 +141,17 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.5'
+      version: '2.7'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.3.3
 signing_key: 
 specification_version: 4
 summary: Generate white-listed filter scopes from URL parameter values
 test_files:
-- spec/filterable_by_spec.rb
+- spec/active_record/filterable_by_spec.rb
 - spec/spec_helper.rb

data/.travis.yml

@@ -1,5 +0,0 @@
-rvm:
-  - 2.5
-  - 2.6
-gemfile:
-  - Gemfile