Merge pull request #182 from kbrock/v71

Drop rails version 7.0 patch

Author: jrafanie

activerecord-virtual_attributes.gemspec

@@ -28,9 +28,7 @@ Gem::Specification.new do |spec|
 
   spec.require_paths = ["lib"]
 
-  spec.add_dependency "concurrent-ruby", "< 1.3.5" # Temporary pin down as concurrent-ruby 1.3.5 breaks Rails 7.0, and rails-core doesn't
-                                                   # plan to ship a new 7.0 to fix it. See https://github.com/rails/rails/pull/54264
-  spec.add_runtime_dependency "activerecord", "~> 7.0", ">=7.0.8.7"
+  spec.add_runtime_dependency "activerecord", "~> 7.1", ">=7.1.5.1"
 
   spec.add_development_dependency "byebug"
   spec.add_development_dependency "database_cleaner-active_record", "~> 2.1"

lib/active_record/virtual_attributes/virtual_arel.rb

@@ -118,45 +118,3 @@ def define_virtual_arel(name, arel) # :nodoc:
   end
 end
 
-# fixed in https://github.com/rails/rails/pull/45642
-if ActiveRecord.version < Gem::Version.new(7.1)
-module Arel # :nodoc: all
-  # rubocop:disable Naming/MethodName
-  # rubocop:disable Naming/MethodParameterName
-  # rubocop:disable Style/ConditionalAssignment
-  module Visitors
-    # rails 6.1...
-    class ToSql
-      private
-
-      def visit_Arel_Nodes_HomogeneousIn(o, collector)
-        collector.preparable = false
-
-        # change:
-        # See https://github.com/rails/rails/pull/45642
-        visit(o.left, collector)
-        # /change
-
-        if o.type == :in
-          collector << " IN ("
-        else
-          collector << " NOT IN ("
-        end
-
-        values = o.casted_values
-
-        if values.empty?
-          collector << @connection.quote(nil)
-        else
-          collector.add_binds(values, o.proc_for_binds, &bind_block)
-        end
-
-        collector << ")"
-      end
-    end
-  end
-  # rubocop:enable Naming/MethodName
-  # rubocop:enable Naming/MethodParameterName
-  # rubocop:enable Style/ConditionalAssignment
-end
-end