filterameter 0.5.0 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f44f6d30fd92e15bc7652df41d75e2d620740fb19f146cbb253274460c3686f2
-  data.tar.gz: 84109580ba0079787e3bcc0e20054a71c2eb2ae7efc512d511124172f19c69fc
+  metadata.gz: 8b2ac8a6c211408669122ab5bf2a131ade58736fa145508305f5ebe3dc9c0bdb
+  data.tar.gz: 9216ddecfedab830f9a3a82cf9c1936f040e64664b23b2d16a4ccd71f1f4923e
 SHA512:
-  metadata.gz: 4b8606c20e87f96b25876da9b670f555b1c95b40bc5fa13368174d1e4264ea1a535a93ea714111171710de12e301876aa59081f327a9cbd8e4f5a6735f2c0406
-  data.tar.gz: 1f848be8b42e68845cebaabadd04b0e9019dab54d9cfdfa2628048215cb6ed5c5e37360fa99a473208cf4539a5963a065f0790a75ecc5d795e45037ae35f7347
+  metadata.gz: ad640c098660b98a33f54378f16dced7d3c2dd278c6d6b6db120d2026cb7f03bda5f0d5033653e697af153eff4bdb3c9c84e39d263a18eedba5075098b7354ef
+  data.tar.gz: 3e775bbe1d6ed769bdfb1a83ac3f4a74c69128517a5bf6ea3fc140e50a8b025214d87d544c9263ad4bf0495f34d477f7a2128836f81e7b6a82d4ec037b9df938

data/README.md

@@ -7,7 +7,7 @@
 Declarative filter parameters provide clean and clear filters for Rails controllers.
 
 ## Usage
-Declare filters in controllers to increase readability and reduce boilerplate code. Filters can be declared for attributes, scopes, or attributes from singular associations (`belongs_to` or `has_one`). Validations can also be assigned.
+Declare filters in controllers to increase readability and reduce boilerplate code. Filters can be declared for attributes or scopes, either directly on the model or on an associated model. Validations can also be assigned.
 
 ```ruby
   filter :color
@@ -28,13 +28,25 @@ If the name of the parameter is different than the name of the attribute or scop
 filter :status, name: :current_status
 ```
 
+This option can also be helpful with nested filters so that the query parameter can be prefixed with the model name. See the `association` option for an example.
+
 #### association
-If the attribute or scope is nested, it can be referenced by naming the association. Only singular associations are valid. For example, if the manager_id attribute lives on an employee's department record, use the following:
+If the attribute or scope is nested, it can be referenced by naming the association. For example, if the manager_id attribute lives on an employee's department record, use the following:
 
 ```ruby
 filter :manager_id, association: :department
 ```
 
+The attribute or scope can be nested more than one level. Declare the filter with an array specifying the associations in order. For example, if an employee belongs to a department and a department belongs to a business unit, use the following to query on the business unit name:
+
+```ruby
+filter :business_unit_name, name: :name, association: [:department, :business_unit]
+```
+
+If an association is a `has_many` [the distinct method](https://api.rubyonrails.org/classes/ActiveRecord/QueryMethods.html#method-i-distinct) is called on the query.
+
+_Limitation:_ If there is more than one association to the same table _and_ both associations can be part of the query, then you cannot use a nested filter directly. Instead, build a scope that disambiguates the associations then build a filter against that scope.
+
 #### validates
 If the filter value should be validated, use the `validates` option along with [ActiveModel validations](https://api.rubyonrails.org/classes/ActiveModel/Validations/ClassMethods.html#method-i-validates). Here's an example of the inclusion validator being used to restrict sizes:
 

data/lib/filterameter/filter_declaration.rb

@@ -16,7 +16,7 @@ module Filterameter
 
       validate_options(options)
       @name = options.fetch(:name, parameter_name).to_s
-      @association = options[:association]
+      @association = Array.wrap(options[:association]).presence
       @filter_on_empty = options.fetch(:filter_on_empty, false)
       @validations = Array.wrap(options[:validates])
       @raw_partial_options = options.fetch(:partial, false)
@@ -24,7 +24,7 @@ module Filterameter
     end
 
     def nested?
-      @association.present?
+      !@association.nil?
     end
 
     def validations?

data/lib/filterameter/filter_factory.rb

@@ -10,14 +10,23 @@ module Filterameter
     end
 
     def build(declaration)
-      model = declaration.nested? ? model_from_association(declaration.association) : @model_class
-      filter = build_filter(model, declaration)
-
-      declaration.nested? ? Filterameter::Filters::NestedFilter.new(declaration.association, model, filter) : filter
+      if declaration.nested?
+        build_nested_filter(declaration)
+      else
+        build_filter(@model_class, declaration)
+      end
     end
 
     private
 
+    def build_nested_filter(declaration)
+      model = model_from_association(declaration.association)
+      filter = build_filter(model, declaration)
+      clazz = filter_class(declaration.association)
+
+      clazz.new(declaration.association, model, filter)
+    end
+
     def build_filter(model, declaration) # rubocop:disable Metrics/MethodLength, Metrics/AbcSize
       # checking dangerous_class_method? excludes any names that cannot be scope names, such as "name"
       if model.respond_to?(declaration.name) && !model.dangerous_class_method?(declaration.name)
@@ -33,9 +42,28 @@ module Filterameter
       end
     end
 
+    def filter_class(association_names)
+      if any_collections?(association_names)
+        Filters::NestedCollectionFilter
+      else
+        Filters::NestedFilter
+      end
+    end
+
+    def any_collections?(association_names)
+      association_names.reduce(@model_class) do |model, name|
+        association = model.reflect_on_association(name)
+        return true if association.collection?
+
+        association.klass
+      end
+
+      false
+    end
+
     # TODO: rescue then raise custom error with cause
     def model_from_association(association)
-      [association].flatten.reduce(@model_class) { |memo, name| memo.reflect_on_association(name).klass }
+      association.flatten.reduce(@model_class) { |memo, name| memo.reflect_on_association(name).klass }
       # rescue StandardError => e
     end
   end

data/lib/filterameter/filter_registry.rb

@@ -51,8 +51,8 @@ module Filterameter
     end
 
     def add_range_maximum(parameter_name, options)
-      parameter_name_min = "#{parameter_name}_max"
-      @declarations[parameter_name_min] = Filterameter::FilterDeclaration.new(parameter_name_min,
+      parameter_name_max = "#{parameter_name}_max"
+      @declarations[parameter_name_max] = Filterameter::FilterDeclaration.new(parameter_name_max,
                                                                               options.merge(range: :max_only))
     end
 

data/lib/filterameter/filters/nested_collection_filter.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+module Filterameter
+  module Filters
+    # = Nested Collection Filter
+    #
+    # Class NestedCollectionFilter joins the nested table(s), merges the filter to the association's model, then makes
+    # the results distinct.
+    class NestedCollectionFilter < NestedFilter
+      def apply(*)
+        super.distinct
+      end
+    end
+  end
+end

data/lib/filterameter/filters/nested_filter.rb

@@ -6,8 +6,8 @@ module Filterameter
     #
     # Class NestedFilter joins the nested table(s) then merges the filter to the association's model.
     class NestedFilter
-      def initialize(joins_values, association_model, attribute_filter)
-        @joins_values = joins_values
+      def initialize(association_names, association_model, attribute_filter)
+        @joins_values = build_joins_values_argument(association_names)
         @association_model = association_model
         @attribute_filter = attribute_filter
       end
@@ -16,6 +16,20 @@ module Filterameter
         query.joins(@joins_values)
              .merge(@attribute_filter.apply(@association_model, value))
       end
+
+      private
+
+      def build_joins_values_argument(association_names)
+        return association_names.first if association_names.size == 1
+
+        convert_to_nested_hash(association_names)
+      end
+
+      def convert_to_nested_hash(association_names)
+        {}.tap do |nested_hash|
+          association_names.reduce(nested_hash) { |memo, name| memo.store(name, {}) }
+        end
+      end
     end
   end
 end

data/lib/filterameter/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Filterameter
-  VERSION = '0.5.0'
+  VERSION = '0.6.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: filterameter
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.6.0
 platform: ruby
 authors:
 - Todd Kummer
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-05-14 00:00:00.000000000 Z
+date: 2024-05-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -114,14 +114,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.60.2
+        version: '1.64'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.60.2
+        version: '1.64'
 - !ruby/object:Gem::Dependency
   name: rubocop-packaging
   requirement: !ruby/object:Gem::Requirement
@@ -142,14 +142,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.23.1
+        version: '2.25'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.23.1
+        version: '2.25'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -164,7 +164,8 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.18'
-description: Enable filter parameters to be declared in query classes or controllers.
+description: Declare filters in Rails controllers to increase readability and reduce
+  boilerplate code.
 email:
 - todd@rockridgesolutions.com
 executables: []
@@ -191,6 +192,7 @@ files:
 - lib/filterameter/filters/matches_filter.rb
 - lib/filterameter/filters/maximum_filter.rb
 - lib/filterameter/filters/minimum_filter.rb
+- lib/filterameter/filters/nested_collection_filter.rb
 - lib/filterameter/filters/nested_filter.rb
 - lib/filterameter/filters/scope_filter.rb
 - lib/filterameter/log_subscriber.rb