activerecord-bitemporal 5.1.0 → 5.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 682d7ade7951759a2237d6395aab4dccc06d5c5ea6d1820b20e1bd1301d2efc7
-  data.tar.gz: a58bb4c8b01a5f7724ad367ab69e45a54f715b96a0d789e7690689798d0f0a88
+  metadata.gz: 690af251479740f03e0dafd504a9b906672e8860c6f8734b37b24f0d2c0c2b5d
+  data.tar.gz: 2b41e3d362d267e4b573911ec75a1b03de9c4259bf8312c6fae9759aae29bf34
 SHA512:
-  metadata.gz: a1761e5e97fc83beac29d7e23edec2ab5bb4d02958b5a7600073b099ca800a6ad3ffa3f8f9bc423aafb412c4c11a2bb1cb03d09d4fd36b6d77e7dd02560a9493
-  data.tar.gz: efc543c3eb6b2ab89d3556eb25f342983d8e5854ebbec77ee6cf853c01dd0ccd64945264c53f30bacb9d370a6083ba65b3791e88b26c0c5c4ecea4481ad66e0c
+  metadata.gz: d86c42c61e77843ef9a76e45868d71722f08e8af28f6c1747f2d4ea55ecd1bf5d5731d3e4b41ad82827c59e0f8e21f3b9ab714a7dc2a442c387c17c97858c8bc
+  data.tar.gz: e53538e79ab1b48563e804080a23f8e8aa6f67f6fa71a54880fa7c93696a9e8778fc5248937ea653cefe72f8cb215e84aec223559f1436506354f65ade735f8d

data/.circleci/config.yml

@@ -33,7 +33,6 @@ workflows:
                 - rails_6.1
                 - rails_7.0
                 - rails_7.1
-                - rails_main
             exclude:
               - ruby-version: '3.2'
                 gemfile: rails_6.1

data/Appraisals

@@ -1,9 +1,5 @@
 # frozen_string_literal: true
 
-appraise "rails-main" do
-  gem "rails", git: 'https://github.com/rails/rails.git', branch: "main"
-end
-
 appraise "rails-6.1" do
   gem "rails", "~> 6.1.0"
 end

data/CHANGELOG.md

@@ -1,5 +1,24 @@
 # Changelog
 
+## 5.2.0
+
+### Added
+
+### Changed
+
+### Deprecated
+
+### Removed
+
+### Fixed
+
+- [Delegate CollectionProxy#bitemporal_value to Relation #168](https://github.com/kufu/activerecord-bitemporal/pull/168)
+- [Fix unintended valid_datetime set when `CollectionProxy#load` #169](https://github.com/kufu/activerecord-bitemporal/pull/169)
+
+### Chores
+
+- [Do not run CI against rails_main #166](https://github.com/kufu/activerecord-bitemporal/pull/166)
+
 ## 5.1.0
 
 ### Added

data/lib/activerecord-bitemporal/scope.rb

@@ -141,6 +141,22 @@ module ActiveRecord::Bitemporal
     end
   end
 
+  module CollectionProxy
+    # Delegate to ActiveRecord::Bitemporal::Relation
+    # @see https://github.com/rails/rails/blob/v7.1.3.4/activerecord/lib/active_record/associations/collection_proxy.rb#L1115-L1124
+    #
+    # In order to update the CollectionProxy state, `load` needs to be excluded.
+    # The reason for using `@association` instead of delegating this method is to preserve state such as `loaded`.
+    # @see https://github.com/rails/rails/blob/v7.1.3.4/activerecord/lib/active_record/associations/collection_proxy.rb#L44
+    #
+    # There is no need to delegate to `scope` as `ActiveRecord::Bitemporal::Relation::Finder`'s methods are delegated
+    # by `ActiveRecord::Delegation`. This is not a problem because `scoping` used in this is delegated to `scope`.
+    # @see https://github.com/rails/rails/blob/v7.1.3.4/activerecord/lib/active_record/relation/delegation.rb#L117
+    delegate :bitemporal_value, :bitemporal_value=, :valid_datetime, :valid_date,
+             :transaction_datetime, :bitemporal_option, :bitemporal_option_merge!,
+             :build_arel, :primary_key, to: :scope
+  end
+
   module Scope
     extend ActiveSupport::Concern
 

data/lib/activerecord-bitemporal/version.rb

@@ -2,6 +2,6 @@
 
 module ActiveRecord
   module Bitemporal
-    VERSION = "5.1.0"
+    VERSION = "5.2.0"
   end
 end

data/lib/activerecord-bitemporal.rb

@@ -29,7 +29,6 @@ module ActiveRecord::Bitemporal::Bitemporalize
         def prepend_relation_delegate_class(mod)
           relation_delegate_class(ActiveRecord::Relation).prepend mod
           relation_delegate_class(ActiveRecord::AssociationRelation).prepend mod
-          relation_delegate_class(ActiveRecord::Associations::CollectionProxy).prepend mod
         end
       end
     end
@@ -58,6 +57,7 @@ module ActiveRecord::Bitemporal::Bitemporalize
     def inherited(klass)
       super
       klass.prepend_relation_delegate_class ActiveRecord::Bitemporal::Relation
+      klass.relation_delegate_class(ActiveRecord::Associations::CollectionProxy).prepend ActiveRecord::Bitemporal::CollectionProxy
       if relation_delegate_class(ActiveRecord::Relation).ancestors.include? ActiveRecord::Bitemporal::Relation::MergeWithExceptBitemporalDefaultScope
         klass.relation_delegate_class(ActiveRecord::Relation).prepend ActiveRecord::Bitemporal::Relation::MergeWithExceptBitemporalDefaultScope
       end
@@ -169,6 +169,7 @@ module ActiveRecord::Bitemporal::Bitemporalize
     validates bitemporal_id_key, uniqueness: true, allow_nil: true, strict: enable_strict_by_validates_bitemporal_id
 
     prepend_relation_delegate_class ActiveRecord::Bitemporal::Relation
+    relation_delegate_class(ActiveRecord::Associations::CollectionProxy).prepend ActiveRecord::Bitemporal::CollectionProxy
   end
 end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: activerecord-bitemporal
 version: !ruby/object:Gem::Version
-  version: 5.1.0
+  version: 5.2.0
 platform: ruby
 authors:
 - SmartHR
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2024-06-05 00:00:00.000000000 Z
+date: 2024-06-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -148,7 +148,6 @@ files:
 - gemfiles/rails_6.1.gemfile
 - gemfiles/rails_7.0.gemfile
 - gemfiles/rails_7.1.gemfile
-- gemfiles/rails_main.gemfile
 - lib/activerecord-bitemporal.rb
 - lib/activerecord-bitemporal/bitemporal.rb
 - lib/activerecord-bitemporal/callbacks.rb

data/gemfiles/rails_main.gemfile

@@ -1,8 +0,0 @@
-# This file was generated by Appraisal
-
-source "https://rubygems.org"
-
-gem "appraisal"
-gem "rails", git: "https://github.com/rails/rails.git", branch: "main"
-
-gemspec path: "../"