logidze 0.5.3 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a553a435446577ff945a979411dcd5d36c6185e7
-  data.tar.gz: 4c06374961f3073969d5f7d16184bd7e7e8d5dd9
+  metadata.gz: eac2c78af58349b132099e71a9a3f6b780a432c5
+  data.tar.gz: cab59c998349bb26b5affe84574bfc5509243542
 SHA512:
-  metadata.gz: 9bbdf6fcec9cb2e4a5686f7f80972aa43889bda5e74e7711d4e80364d580279e9f947859aa76342b3018dcf5fcc471ab6b998c5a54e9b836654dab1e94e9bdeb
-  data.tar.gz: 991804f34a2f90601794418789fde42aff8d7031a8edd6b115fb79c4e0fd59e3b14277b8931a509e19c01200d4cd830776484e6d4622059c4bca3b321d2ecfc7
+  metadata.gz: acd1debc4bc436ae1e1142dd2d760c7a614f9ea6be8e46f751a2497ebbdc2d8054243f3a05d0e00aa0b07a243762ece3834a78f77b744fc9c96ee62da53fac87
+  data.tar.gz: 756bebbaac83f9bb0d205b7208c43424cdbdda32e054e1a978839dff72441ce9083e88f870919bf475eac1cb2b70727bc0f40b93dc4f954bdedb1fddc5b41fbe

data/.rubocop.yml

@@ -9,11 +9,14 @@ AllCops:
     - 'spec/dummy/**/*'
     - 'tmp/**/*'
     - 'bench/**/*'
+    - 'Rakefile'
+    - 'Gemfile'
+    - '*.gemspec'
   DisplayCopNames: true
   StyleGuideCopsOnly: false
   TargetRubyVersion: 2.3
 
-Style/AccessorMethodName:
+Naming/AccessorMethodName:
   Enabled: false
 
 Style/TrivialAccessors:
@@ -26,20 +29,30 @@ Style/Documentation:
 Style/StringLiterals:
   Enabled: false
 
-Style/SpaceInsideStringInterpolation:
+Layout/SpaceInsideStringInterpolation:
   EnforcedStyle: no_space
 
 Style/BlockDelimiters:
   Exclude:
     - 'spec/**/*.rb'
 
+Style/PercentLiteralDelimiters:
+  Enabled: false
+
 Lint/AmbiguousRegexpLiteral:
   Enabled: false
 
+Lint/MissingCopEnableDirective:
+  Enabled: false
+
 Metrics/MethodLength:
   Exclude:
     - 'spec/**/*.rb'
 
+Metrics/BlockLength:
+  Exclude:
+    - 'spec/**/*.rb'
+
 Metrics/LineLength:
   Max: 100
   Exclude:
@@ -59,5 +72,11 @@ Lint/HandleExceptions:
   Exclude:
     - 'spec/**/*.rb'
 
-Style/DotPosition:
+Layout/DotPosition:
   EnforcedStyle: leading
+
+Layout/IndentHeredoc:
+  Enabled: false
+
+Layout/EmptyLineAfterMagicComment:
+  Enabled: false

data/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 ## master
 
+## 0.6.0 (2017-12-27)
+
+- [Fixes [#33](https://github.com/palkan/logidze/issues/33)] Support attributes types. ([@palkan][])
+
+  Added deserialization of complex types (such as `jsonb`, arrays, whatever).
+
+- Use positional arguments in `at`/`diff_from` methods and allow passing version. ([@palkan][])
+
+  Now you can write `post.diff_from(time: ts)`, `post.diff_from(version: x)`, Post.at(time: 1.day.ago)`, etc.
+
+  NOTE: the previous behaviour is still supported (but gonna be deprecated),
+  i.e. you still can use `post.diff_from(ts)` if you don't mind the deprecation warning.
+
 ## 0.5.3 (2017-08-22)
 
 - Add `--update` flag to model migration. ([@palkan][])

data/Gemfile

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in logidze.gemspec

data/README.md

@@ -134,35 +134,35 @@ post.log_version #=> 3
 post.log_size #=> 3
 
 # Get copy of a record at a given time
-old_post = post.at(2.days.ago)
+old_post = post.at(time: 2.days.ago)
 
 # or revert the record itself to the previous state (without committing to DB)
-post.at!('201-04-15 12:00:00')
+post.at!(time: '201-04-15 12:00:00')
 
 # If no version found
-post.at('1945-05-09 09:00:00') #=> nil
+post.at(time: '1945-05-09 09:00:00') #=> nil
 ```
 
 You can also get revision by version number:
 
 ```ruby
-post.at_version(2)
+post.at(version: 2)
 ```
 
 It is also possible to get version for relations:
 
 ```ruby
-Post.where(active: true).at(1.month.ago)
+Post.where(active: true).at(time: 1.month.ago)
 ```
 
 You can also get diff from specified time:
 
 ```ruby
-post.diff_from(1.hour.ago)
+post.diff_from(time: 1.hour.ago)
 #=> { "id" => 27, "changes" => { "title" => { "old" => "Logidze sucks!", "new" => "Logidze rulz!" } } }
 
 # the same for relations
-Post.where(created_at: Time.zone.today.all_day).diff_from(1.hour.ago)
+Post.where(created_at: Time.zone.today.all_day).diff_from(time: 1.hour.ago)
 ```
 
 There are also `#undo!` and `#redo!` options (and more general `#switch_to!`):

data/Rakefile

@@ -1,6 +1,9 @@
+# frozen_string_literal: true
 require "bundler/gem_tasks"
 require "rspec/core/rake_task"
+require "rubocop/rake_task"
 
+RuboCop::RakeTask.new
 RSpec::Core::RakeTask.new(:spec)
 
 namespace :dummy do

data/lib/generators/logidze/model/model_generator.rb

@@ -29,7 +29,7 @@ module Logidze
 
       def generate_migration
         if options[:blacklist] && options[:whitelist]
-          $stderr.puts "Use only one: --whitelist or --blacklist"
+          warn "Use only one: --whitelist or --blacklist"
           exit(1)
         end
         migration_template "migration.rb.erb", "db/migrate/#{migration_file_name}"

data/lib/logidze/has_logidze.rb

@@ -9,7 +9,7 @@ module Logidze
     module ClassMethods # :nodoc:
       # Include methods to work with history.
       #
-      # rubocop:disable Style/PredicateName
+      # rubocop:disable Naming/PredicateName, Style/MixinUsage
       def has_logidze
         include Logidze::Model
       end

data/lib/logidze/history.rb

@@ -124,8 +124,8 @@ module Logidze
     private
 
     def build_changes(a, b)
-      b.each_with_object({}) do |kv, acc|
-        acc[kv.first] = { "old" => a[kv.first], "new" => kv.last } unless kv.last == a[kv.first]
+      b.each_with_object({}) do |(k, v), acc|
+        acc[k] = { "old" => a[k], "new" => v } unless v == a[k]
       end
     end
 

data/lib/logidze/model.rb

@@ -2,7 +2,17 @@
 require 'active_support'
 
 module Logidze
+  module Deprecations # :nodoc:
+    def self.show_ts_deprecation_for(meth)
+      warn(
+        "[Deprecation] Usage of #{meth}(time) will be removed in the future releases, "\
+        "use #{meth}(time: ts) instead"
+      )
+    end
+  end
+
   # Extends model with methods to browse history
+  # rubocop: disable Metrics/ModuleLength
   module Model
     require 'logidze/history/type' if Rails::VERSION::MAJOR >= 5
 
@@ -20,13 +30,17 @@ module Logidze
 
     module ClassMethods # :nodoc:
       # Return records reverted to specified time
-      def at(ts)
-        all.map { |record| record.at(ts) }.compact
+      def at(ts = nil, time: nil, version: nil)
+        Deprecations.show_ts_deprecation_for(".at") if ts
+        time ||= ts
+        all.map { |record| record.at(time: time, version: version) }.compact
       end
 
       # Return changes made to records since specified time
-      def diff_from(ts)
-        all.map { |record| record.diff_from(ts) }
+      def diff_from(ts = nil, time: nil, version: nil)
+        Deprecations.show_ts_deprecation_for(".diff_from") if ts
+        time ||= ts
+        all.map { |record| record.diff_from(time: time, version: version) }
       end
 
       # Alias for Logidze.without_logging
@@ -34,9 +48,11 @@ module Logidze
         Logidze.without_logging(&block)
       end
 
+      # rubocop: disable Naming/PredicateName
       def has_logidze?
         true
       end
+      # rubocop: enable Naming/PredicateName
     end
 
     # Use this to convert Ruby time to milliseconds
@@ -45,35 +61,48 @@ module Logidze
     attr_accessor :logidze_requested_ts
 
     # Return a dirty copy of record at specified time
-    # If time is less then the first version, then return nil.
-    # If time is greater then the last version, then return self.
-    def at(ts)
-      ts = parse_time(ts)
+    # If time/version is less then the first version, then return nil.
+    # If time/version is greater then the last version, then return self.
+    # rubocop: disable Metrics/AbcSize, Metrics/MethodLength
+    def at(ts = nil, time: nil, version: nil)
+      Deprecations.show_ts_deprecation_for("#at") if ts
 
-      return nil unless log_data.exists_ts?(ts)
+      return at_version(version) if version
 
-      if log_data.current_ts?(ts)
-        self.logidze_requested_ts = ts
+      time ||= ts
+      time = parse_time(time)
+
+      return nil unless log_data.exists_ts?(time)
+
+      if log_data.current_ts?(time)
+        self.logidze_requested_ts = time
         return self
       end
 
-      version = log_data.find_by_time(ts).version
+      version = log_data.find_by_time(time).version
 
       object_at = dup
       object_at.apply_diff(version, log_data.changes_to(version: version))
       object_at.id = id
-      object_at.logidze_requested_ts = ts
+      object_at.logidze_requested_ts = time
 
       object_at
     end
+    # rubocop: enable Metrics/AbcSize, Metrics/MethodLength
 
     # Revert record to the version at specified time (without saving to DB)
-    def at!(ts)
-      ts = parse_time(ts)
-      return self if log_data.current_ts?(ts)
-      return false unless log_data.exists_ts?(ts)
+    def at!(ts = nil, time: nil, version: nil)
+      Deprecations.show_ts_deprecation_for("#at!") if ts
+
+      return at_version!(version) if version
 
-      version = log_data.find_by_time(ts).version
+      time ||= ts
+      time = parse_time(time)
+
+      return self if log_data.current_ts?(time)
+      return false unless log_data.exists_ts?(time)
+
+      version = log_data.find_by_time(time).version
 
       apply_diff(version, log_data.changes_to(version: version))
     end
@@ -99,11 +128,17 @@ module Logidze
     #
     # @example
     #
-    #   post.diff_from(2.days.ago)
+    #   post.diff_from(time: 2.days.ago) # or post.diff_from(version: 2)
     #   #=> { "id" => 1, "changes" => { "title" => { "old" => "Hello!", "new" => "World" } } }
-    def diff_from(ts)
-      ts = parse_time(ts)
-      { "id" => id, "changes" => log_data.diff_from(time: ts) }
+    def diff_from(ts = nil, version: nil, time: nil)
+      Deprecations.show_ts_deprecation_for("#diff_from") if ts
+      time ||= ts
+      time = parse_time(time) if time
+      changes = log_data.diff_from(time: time, version: version).tap do |v|
+        deserialize_changes!(v)
+      end
+
+      { "id" => id, "changes" => changes }
     end
 
     # Restore record to the previous version.
@@ -135,6 +170,7 @@ module Logidze
       end
     end
 
+    # rubocop: disable Metrics/MethodLength
     def association(name)
       association = super
 
@@ -157,15 +193,40 @@ module Logidze
 
       association
     end
+    # rubocop: enable Metrics/MethodLength
 
     protected
 
     def apply_diff(version, diff)
-      diff.each { |k, v| send("#{k}=", v) }
+      diff.each do |k, v|
+        apply_column_diff(k, v)
+      end
+
       log_data.version = version
       self
     end
 
+    def apply_column_diff(column, value)
+      write_attribute column, deserialize_value(column, value)
+    end
+
+    if Rails::VERSION::MAJOR < 5
+      def deserialize_value(column, value)
+        @attributes[column].type.type_cast_from_database(value)
+      end
+    else
+      def deserialize_value(column, value)
+        @attributes[column].type.deserialize(value)
+      end
+    end
+
+    def deserialize_changes!(diff)
+      diff.each do |k, v|
+        v["old"] = deserialize_value(k, v["old"])
+        v["new"] = deserialize_value(k, v["new"])
+      end
+    end
+
     def logidze_past?
       return false unless @logidze_requested_ts
 

data/lib/logidze/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module Logidze
-  VERSION = "0.5.3"
+  VERSION = "0.6.0"
 end

data/lib/logidze/versioned_association.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
-module Logidze
+module Logidze # :nodoc: all
   module VersionedAssociation
+    # rubocop: disable Metrics/MethodLength, Metrics/AbcSize
     def load_target
       target = super
 
@@ -10,10 +11,10 @@ module Logidze
 
       if target.is_a? Array
         target.map! do |object|
-          object.at(time)
+          object.at(time: time)
         end.compact!
       else
-        target.at!(time)
+        target.at!(time: time)
       end
 
       target
@@ -26,9 +27,7 @@ module Logidze
     def logidze_stale?
       return false if !loaded? || inversed
 
-      unless target.is_a?(Array)
-        return owner.logidze_requested_ts != target.logidze_requested_ts
-      end
+      return owner.logidze_requested_ts != target.logidze_requested_ts unless target.is_a?(Array)
 
       return false if target.empty?
 

data/logidze.gemspec

@@ -1,4 +1,4 @@
-# coding: utf-8
+# frozen_string_literal: true
 lib = File.expand_path('../lib', __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'logidze/version'
@@ -19,13 +19,14 @@ Gem::Specification.new do |spec|
 
   spec.add_dependency "rails", ">= 4.2"
 
-  spec.add_development_dependency "pg", "~>0.18"
+  spec.add_development_dependency "ammeter", "~> 1.1.3"
   spec.add_development_dependency "bundler", "~> 1"
+  spec.add_development_dependency "database_cleaner", "~> 1.5"
+  spec.add_development_dependency "pg", "~>0.18"
+  spec.add_development_dependency "pry-byebug"
   spec.add_development_dependency "rake", ">= 10.0"
   spec.add_development_dependency "rspec-rails", ">= 3.4"
-  spec.add_development_dependency "database_cleaner", "~> 1.5"
+  spec.add_development_dependency "rubocop", "~> 0.52"
   spec.add_development_dependency "simplecov", ">= 0.3.8"
-  spec.add_development_dependency "ammeter", "~> 1.1.3"
-  spec.add_development_dependency "pry-byebug"
   spec.add_development_dependency "timecop", "~> 0.8"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logidze
 version: !ruby/object:Gem::Version
-  version: 0.5.3
+  version: 0.6.0
 platform: ruby
 authors:
 - palkan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-08-22 00:00:00.000000000 Z
+date: 2017-12-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -25,19 +25,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '4.2'
 - !ruby/object:Gem::Dependency
-  name: pg
+  name: ammeter
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.18'
+        version: 1.1.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.18'
+        version: 1.1.3
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -53,89 +53,103 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: database_cleaner
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '1.5'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '1.5'
 - !ruby/object:Gem::Dependency
-  name: rspec-rails
+  name: pg
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.18'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.18'
+- !ruby/object:Gem::Dependency
+  name: pry-byebug
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.4'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.4'
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: database_cleaner
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: '10.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: '10.0'
 - !ruby/object:Gem::Dependency
-  name: simplecov
+  name: rspec-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.3.8
+        version: '3.4'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.3.8
+        version: '3.4'
 - !ruby/object:Gem::Dependency
-  name: ammeter
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.1.3
+        version: '0.52'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.1.3
+        version: '0.52'
 - !ruby/object:Gem::Dependency
-  name: pry-byebug
+  name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.3.8
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.3.8
 - !ruby/object:Gem::Dependency
   name: timecop
   requirement: !ruby/object:Gem::Requirement
@@ -226,7 +240,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.11
+rubygems_version: 2.6.13
 signing_key: 
 specification_version: 4
 summary: PostgreSQL JSON-based auditing