Merge pull request #310 from ifad/bugfix/309-method-visibility

Fix patched association methods visibility

lib/chrono_model/patches/association.rb

@@ -11,10 +11,6 @@ module Patches
     # on the join model's (:through association) one.
     #
     module Association
-      def skip_statement_cache?(*)
-        super || _chrono_target?
-      end
-
       # If the association class or the through association are ChronoModels,
       # then fetches the records from a virtual table using a subquery scope
       # to a specific timestamp.
@@ -36,6 +32,10 @@ def scope
 
       private
 
+      def skip_statement_cache?(*)
+        super || _chrono_target?
+      end
+
       def _chrono_record?
         owner.class.include?(ChronoModel::Patches::AsOfTimeHolder) && owner.as_of_time.present?
       end

lib/chrono_model/patches/preloader.rb

@@ -33,6 +33,8 @@ def chronomodel_scope(given_preload_scope)
       end
 
       module Association
+        private
+
         # Builds the preloader scope taking into account a potential
         # +as_of_time+ passed down the call chain starting at the
         # end user invocation.
@@ -49,6 +51,8 @@ def build_scope
       end
 
       module ThroughAssociation
+        private
+
         # Builds the preloader scope taking into account a potential
         # +as_of_time+ passed down the call chain starting at the
         # end user invocation.

lib/chrono_model/patches/relation.rb

@@ -37,6 +37,20 @@ def merge(*)
         super.as_of_time!(@_as_of_time)
       end
 
+      def find_nth(*)
+        return super unless try(:history?)
+
+        with_hid_pkey { super }
+      end
+
+      def last(*)
+        return super unless try(:history?)
+
+        with_hid_pkey { super }
+      end
+
+      private
+
       def build_arel(*)
         return super unless @_as_of_time
 
@@ -70,20 +84,6 @@ def chrono_join_history(join)
         )
       end
 
-      def find_nth(*)
-        return super unless try(:history?)
-
-        with_hid_pkey { super }
-      end
-
-      def last(*)
-        return super unless try(:history?)
-
-        with_hid_pkey { super }
-      end
-
-      private
-
       def ordered_relation
         return super unless try(:history?)