activerecord-cached_at 2.0.13 → 5.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c744adeef1d92b25ca053e9553e23d107317b0db
-  data.tar.gz: 07d579d29d299964580a6025509c756d01fddb3f
+  metadata.gz: c9738b748326936b6c9ca40a335f3ff903ddbdb2
+  data.tar.gz: d2a4acd817f4f27deaa58984ab285946c711dbd8
 SHA512:
-  metadata.gz: 8b32156ffebc82c9fc2a75e1244edfafe76b51bee868eb8126de844220e4afe75c11415cd8d169b28c6138fb2b6aef4f87ee9d05e25a8f94936464756f3511cc
-  data.tar.gz: '0706495432358f4a0b816e93b0d9fd1b5bf284acd36cdd4fe71f9c9969fbec238a1ae69e92a07ae3db3c975ba53e9b6c3fc0596f9a0c6628d3d67e5495893d62'
+  metadata.gz: 0b16e4afd909f8091721348e23d54f2d71427bcb550016d77677574bb19a525c2f12cb8fbccb25b1de39d82604f954be5674ec7e99611bfb0e776c21398bdcb0
+  data.tar.gz: 49bc4f9883cd48034e0dcad03b8525e750dd59244e143946d3c33b4f88a54d19705183b69492dba045dee6dc9e4729ca9b144817603ce718f734a11b3b671f0e

data/lib/cached_at/associations/belongs_to_association.rb

@@ -13,8 +13,8 @@ module CachedAt
 
       cache_column = "#{options[:inverse_of]}_cached_at"
       if options[:polymorphic]
-        oldtype = owner.send("#{reflection.foreign_type}_was")
-        oldid = owner.send("#{reflection.foreign_key}_was")
+        oldtype = owner.send("#{reflection.foreign_type}_before_last_save")
+        oldid = owner.send("#{reflection.foreign_key}_before_last_save")
         newtype = owner.send(reflection.foreign_type)
         newid = owner.send(reflection.foreign_key)
         if !oldtype.nil? && oldtype == newtype
@@ -38,7 +38,7 @@ module CachedAt
           end
         end
       else
-        ids = [owner.send(reflection.foreign_key), owner.send("#{reflection.foreign_key}_was")].compact.uniq
+        ids = [owner.send(reflection.foreign_key), owner.send("#{reflection.foreign_key}_before_last_save")].compact.uniq
         query = klass.where({ reflection.association_primary_key => ids })
         query.update_all({ cache_column => timestamp })
         traverse_relationships(klass, options[:cached_at], query, cache_column, timestamp)

data/lib/cached_at/associations/collection_association.rb

@@ -10,7 +10,7 @@ module CachedAt
       end
 
       cache_column = "#{reflection.inverse_of.name}_cached_at"
-      ids = [owner.send(reflection.association_primary_key), owner.send("#{reflection.association_primary_key}_was")].compact.uniq
+      ids = [owner.send(reflection.association_primary_key), owner.send("#{reflection.association_primary_key}_before_last_save")].compact.uniq
       query = klass.where({ reflection.foreign_key => ids })
     
       if loaded?

data/lib/cached_at/associations/collection_proxy.rb

@@ -0,0 +1,10 @@
+module CachedAt
+  module CollectionProxy
+    def delete(*records)
+      @association.touch_records_cached_at(records, Time.now) unless @association.owner.new_record?
+      super
+    end
+  end
+end
+
+ActiveRecord::Associations::CollectionProxy.prepend(CachedAt::CollectionProxy)

data/lib/cached_at/associations/has_many_through_association.rb

@@ -18,7 +18,7 @@ module CachedAt
           target.each { |r| r.raw_write_attribute(cache_column, timestamp) }
           query = klass.where(klass.primary_key => target.map(&:id))
         else
-          ids = [owner.send(using_reflection.association_primary_key), owner.send("#{using_reflection.association_primary_key}_was")].compact.uniq
+          ids = [owner.send(using_reflection.association_primary_key), owner.send("#{using_reflection.association_primary_key}_before_last_save")].compact.uniq
           arel_table = klass._reflections[using_reflection.inverse_of.options[:through].to_s].klass.arel_table
           query = klass.joins(using_reflection.inverse_of.options[:through])
           query = if using_reflection.is_a?(ActiveRecord::Reflection::HasAndBelongsToManyReflection)

data/lib/cached_at/associations/has_one_association.rb

@@ -10,7 +10,7 @@ module CachedAt
       end
   
       cache_column = "#{reflection.inverse_of.name}_cached_at"
-      ids = [owner.send(reflection.association_primary_key), owner.send("#{reflection.association_primary_key}_was")].compact.uniq
+      ids = [owner.send(reflection.association_primary_key), owner.send("#{reflection.association_primary_key}_before_last_save")].compact.uniq
       query = klass.where({ reflection.foreign_key => ids })
   
       case options[:dependent]

data/lib/cached_at/base.rb

@@ -3,6 +3,7 @@ require File.expand_path(File.join(__FILE__, '../associations/association'))
 require File.expand_path(File.join(__FILE__, '../associations/has_one_association'))
 require File.expand_path(File.join(__FILE__, '../associations/belongs_to_association'))
 require File.expand_path(File.join(__FILE__, '../associations/collection_association'))
+require File.expand_path(File.join(__FILE__, '../associations/collection_proxy'))
 require File.expand_path(File.join(__FILE__, '../associations/has_many_through_association'))
 
 require File.expand_path(File.join(__FILE__, '../reflections/abstract_reflection'))
@@ -31,8 +32,9 @@ module CachedAt
     def update_relations_cached_at(timestamp: nil, method: nil)
       method = instance_variable_defined?(:@new_record_before_save) && @new_record_before_save ? :create : :update if method.nil?
       
-      return if method == :create && changes.empty?
-      return if method == :update && changes.empty?
+      diff = saved_changes.transform_values(&:first)
+      return if method == :create && diff.empty?
+      return if method == :update && diff.empty?
       
       timestamp ||= current_time_from_proper_timezone
 

data/lib/cached_at/timestamp.rb

@@ -1,15 +1,22 @@
 module ActiveRecord
-  module Timestamp
-
-    private
+  module CachedAt
+    module Timestamp
+      extend ActiveSupport::Concern
+      
+      class_methods do
+        private
     
-    def timestamp_attributes_for_update
-      [:updated_at, :cached_at]
-    end
+        def timestamp_attributes_for_update
+          ['updated_at', 'cached_at']
+        end
+
+        def timestamp_attributes_for_create
+          ['created_at', 'udpated_at', 'cached_at'] + column_names.select{|c| c.end_with?('_cached_at') }
+        end
 
-    def timestamp_attributes_for_create
-      [:created_at, :udpated_at, :cached_at] + self.class.column_names.select{|c| c.end_with?('_cached_at') }.map(&:to_sym)
+      end
     end
-    
   end
-end
+end
+
+ActiveRecord::Base.include(ActiveRecord::CachedAt::Timestamp)

data/lib/cached_at/version.rb

@@ -1,3 +1,3 @@
 module CachedAt
-  VERSION = '2.0.13'
+  VERSION = '5.1.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: activerecord-cached_at
 version: !ruby/object:Gem::Version
-  version: 2.0.13
+  version: 5.1.0
 platform: ruby
 authors:
 - Jon Bracy
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-01-11 00:00:00.000000000 Z
+date: 2017-08-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 5.0.0
+        version: 5.1.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 5.0.0
+        version: 5.1.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -139,6 +139,7 @@ files:
 - lib/cached_at/associations/association.rb
 - lib/cached_at/associations/belongs_to_association.rb
 - lib/cached_at/associations/collection_association.rb
+- lib/cached_at/associations/collection_proxy.rb
 - lib/cached_at/associations/has_many_through_association.rb
 - lib/cached_at/associations/has_one_association.rb
 - lib/cached_at/base.rb
@@ -170,7 +171,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.2
+rubygems_version: 2.6.11
 signing_key: 
 specification_version: 4
 summary: Allows ActiveRecord and Rails to use a `cached_at` column for the `cache_key`