baby_squeel 1.4.4 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 54dbc98299ab8f9909d8c312e9663cc615543ea6421b81f2aab6efdd2bd9fe59
-  data.tar.gz: 97cc39dced57e0bd6bf16c854859f47902a2d6550e71c4120643fb01d0de9529
+  metadata.gz: e317da76921bc04f342b3cd1f219fa6585f642c4b96e936a18a442a1eb51c18b
+  data.tar.gz: 6a25d3d64d518f5b43124b4dd9083636f10ac0fad1e0ba4e9634f0dded17cb23
 SHA512:
-  metadata.gz: 8f5e68cf4ac0e85d2dc8fce1741f2d64c2bd5b5ecc08c335d71ad3fae335fb3b580f5c3aa2c5f98053ec445d9a7fbc1d3ddec6c7218c99bcb9869a6767fa68c8
-  data.tar.gz: ca66ea54876dfb670177ccea3be1808c90b4d1affea3b7bc1cb5c515e6e8208a8b55fb9fcd06202c6411ea102a358ceb0aa7d0468f82ee7c857045fbc0c5e946
+  metadata.gz: f1bcaadbb4fc5894a5cd1e4c797ac3d6cfbbaee0cad994caf360fc2ac347f9a30019c297bcd254cae7642574b56bd22d927ea492bad76be02905379847af5330
+  data.tar.gz: 012e06a95f564e7502a4aa73a5629969f82e76a4abf475fa7364f25891f3345710334848313d6c3fc4c3842da29c5fc3220d95d58a1829258445886be8ac143e

data/CHANGELOG.md

@@ -1,5 +1,8 @@
 ## [Unreleased]
 
+- AR 6.1: fix - FrozenError: can't modify frozen object: []
+- Drop support for ActiveRecord older than 6.0.
+
 ## [1.4.4] - 2022-02-07
 
 ### Fixed

data/ISSUE_TEMPLATE.md

@@ -11,7 +11,7 @@ require 'minitest/autorun'
 
 gemfile true do
   source 'https://rubygems.org'
-  gem 'activerecord', '~> 5.2.0' # which Active Record version?
+  gem 'activerecord', '~> 6.0.0' # which Active Record version?
   gem 'sqlite3'
   gem 'baby_squeel', github: 'rzane/baby_squeel'
 end

data/baby_squeel.gemspec

@@ -19,7 +19,7 @@ Gem::Specification.new do |spec|
 
   spec.files = Dir.glob('{lib/**/*,*.{md,txt,gemspec}}')
 
-  spec.add_dependency 'activerecord', '>= 5.2', '< 7.1'
+  spec.add_dependency 'activerecord', '>= 6.0', '< 7.1'
   spec.add_dependency 'ransack', '~> 2.3'
 
   spec.add_development_dependency 'bundler', '~> 2'

data/lib/baby_squeel/active_record/query_methods.rb

@@ -64,10 +64,12 @@ module BabySqueel
 
         # https://github.com/rails/rails/commit/c0c53ee9d28134757cf1418521cb97c4a135f140
         def select_association_list(*args)
-          args[0].extend(BabySqueel::ActiveRecord::QueryMethods::Injector6_1)
+          if args[0].any? { |join| join.is_a?(BabySqueel::Join) }
+            args[0].extend(BabySqueel::ActiveRecord::QueryMethods::Injector6_1)
+          end
           super *args
         end
-      elsif BabySqueel::ActiveRecord::VersionHelper.at_least_6_0?
+      else
         private
 
         # Active Record will call `each` on the `joins`. The
@@ -77,16 +79,6 @@ module BabySqueel
           args[1] = BabySqueel::JoinDependency::Injector6_0.new(args.second)
           super(*args)
         end
-      else
-        private
-
-        # Active Record will call `group_by` on the `joins`. The
-        # Injector has a custom `group_by` method that handles
-        # BabySqueel::Join nodes.
-        def build_joins(*args)
-          args[1] = BabySqueel::JoinDependency::Injector5_2.new(args.second)
-          super(*args)
-        end
       end
     end
   end

data/lib/baby_squeel/active_record/version_helper.rb

@@ -7,15 +7,6 @@ module BabySqueel
         ::ActiveRecord::VERSION::MAJOR > 6 ||
           ::ActiveRecord::VERSION::MAJOR == 6 && ::ActiveRecord::VERSION::MINOR >= 1
       end
-
-      def self.at_least_6_0?
-        ::ActiveRecord::VERSION::MAJOR >= 6
-      end
-
-      def self.at_least_5_2_3?
-        at_least_6_0? ||
-          ::ActiveRecord::VERSION::MAJOR >= 5 && ::ActiveRecord::VERSION::MINOR >= 2 && ::ActiveRecord::VERSION::TINY >= 3
-      end
     end
   end
 end

data/lib/baby_squeel/join_dependency.rb

@@ -2,26 +2,6 @@ require 'baby_squeel/active_record/version_helper'
 
 module BabySqueel
   module JoinDependency
-    # This class allows BabySqueel to slip custom
-    # joins_values into Active Record's JoinDependency
-    class Injector5_2 < Array # :nodoc:
-      # Active Record will call group_by on this object
-      # in ActiveRecord::QueryMethods#build_joins. This
-      # allows BabySqueel::Joins to be treated
-      # like typical join hashes until Polyamorous can
-      # deal with them.
-      def group_by
-        super do |join|
-          case join
-          when BabySqueel::Join
-            :association_join
-          else
-            yield join
-          end
-        end
-      end
-    end
-
     # This class allows BabySqueel to slip custom
     # joins_values into Active Record's JoinDependency
     class Injector6_0 < Array # :nodoc:
@@ -75,7 +55,7 @@ module BabySqueel
           join_root.each_children do |parent, child|
             join_dependency.construct_tables_for_association!(parent, child)
           end
-        elsif BabySqueel::ActiveRecord::VersionHelper.at_least_5_2_3?
+        else
           # If we tell join_dependency to construct its tables, Active Record
           # handles building the correct aliases and attaching them to its
           # JoinDepenencies.
@@ -147,16 +127,9 @@ module BabySqueel
         join_list = join_nodes + joins
 
         alias_tracker = Associations::AliasTracker.create(relation.klass.connection, relation.table.name, join_list)
-        if BabySqueel::ActiveRecord::VersionHelper.at_least_6_0?
-          join_dependency = Associations::JoinDependency.new(relation.klass, relation.table, association_joins, Arel::Nodes::InnerJoin)
-          join_dependency.instance_variable_set(:@alias_tracker, alias_tracker)
-        elsif BabySqueel::ActiveRecord::VersionHelper.at_least_5_2_3?
-          join_dependency = Associations::JoinDependency.new(relation.klass, relation.table, association_joins)
-          join_dependency.instance_variable_set(:@alias_tracker, alias_tracker)
-        else
-          # Rails 5.2.0 - 5.2.2
-          join_dependency = Associations::JoinDependency.new(relation.klass, relation.table, association_joins, alias_tracker)
-        end
+        join_dependency = Associations::JoinDependency.new(relation.klass, relation.table, association_joins, Arel::Nodes::InnerJoin)
+        join_dependency.instance_variable_set(:@alias_tracker, alias_tracker)
+
         join_nodes.each do |join|
           join_dependency.send(:alias_tracker).aliases[join.left.name.downcase] = 1
         end

data/lib/baby_squeel/version.rb

@@ -1,3 +1,3 @@
 module BabySqueel
-  VERSION = '1.4.4'.freeze
+  VERSION = '2.0.0'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: baby_squeel
 version: !ruby/object:Gem::Version
-  version: 1.4.4
+  version: 2.0.0
 platform: ruby
 authors:
 - Ray Zane
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-02-08 00:00:00.000000000 Z
+date: 2022-08-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -16,7 +16,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.2'
+        version: '6.0'
     - - "<"
       - !ruby/object:Gem::Version
         version: '7.1'
@@ -26,7 +26,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.2'
+        version: '6.0'
     - - "<"
       - !ruby/object:Gem::Version
         version: '7.1'
@@ -156,7 +156,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.3
+rubygems_version: 3.3.7
 signing_key:
 specification_version: 4
 summary: An expressive query DSL for Active Record 4 and 5.