logidze 1.2.3 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ac315542fe9ec1866a4395ed04f1e431d65fe6c9dfc72874a423224c0b52b3cb
-  data.tar.gz: deb4efcba8179273185738fbe395458f6bd4165bf8ed1263b0348bf9bc4b45ab
+  metadata.gz: c4ab83a61c2685b93d8fe58199eac1fb889c4459453db9315bd6a9880abb2bc9
+  data.tar.gz: f856bd257d81c6ad1fc3cbe6328bbd1adbf77670335a07a0f5234f572f8a7145
 SHA512:
-  metadata.gz: bffd04a3005c169a56e3280eaceb7c22623b7d7cfdf82719d65f16804fd86adee32f9afdccd6e767923dd5f6b0b63c0354455ad5daba7a608a420190cae4c2fd
-  data.tar.gz: c7296b56fed241dc7f55a16a5307b596f1ec016f334cb743c47272295db3ee4d1236bbda0f97c78a9b4a8e50309e07757322611c41bb5386bdbee3500fa57a1b
+  metadata.gz: d462f59ba20d402c5c75bb15ea40b709f278088dcd11be7e769640640450bd35a558794884fca89ccdaa403e09fb0d05c40d43bed8dd86698a41ecb870272a4b
+  data.tar.gz: f5bf80de51ffaff0f611e84bbf99f4291ebb82655ff23ce555d042c3ae29f3d79c172e1b7de7c1f19083892a847473ce6252364973e31eecb41d25fe8e1de65b

data/CHANGELOG.md

@@ -2,6 +2,21 @@
 
 ## master (unreleased)
 
+## 1.3.0 (2024-01-09)
+
+- Add retrieving list of versions support. ([@tagirahmad][])
+
+```ruby
+post.versions # => Enumerator
+post.versions.find do
+  _1.title == "old title"
+end
+```
+
+- Add `--after-trigger` option to generate _after_ triggers for partitioned tables in older PostgreSQL versions. ([@SparLaimor][], [@prog-supdex][], [@palkan][])
+
+- **Breaking**. Ruby 2.7, Rails 6.0, PostgreSQL 10.0+ are required.
+
 ## 1.2.3 (2023-01-03)
 
 - [Fixes [#217](https://github.com/palkan/logidze/issues/217)] Fix switch_to with `append: true` when there are changes on JSONB columns. ([@miharekar][])
@@ -376,3 +391,7 @@ This is a quick fix for a more general problem (see [#59](https://github.com/pal
 [@cavi21]: https://github.com/cavi21
 [@danielmklein]: https://github.com/danielmklein
 [@baygeldin]: https://github.com/baygeldin
+[@miharekar]: https://github.com/miharekar
+[@prog-supdex]: https://github.com/prog-supdex
+[@SparLaimor]: https://github.com/SparLaimor
+[@tagirahmad]: https://github.com/tagirahmad

data/README.md

@@ -5,7 +5,7 @@
 
 # Logidze
 
-Logidze provides tools for logging DB records changes when using PostgreSQL (>=9.6). Just like [audited](https://github.com/collectiveidea/audited) and [paper_trail](https://github.com/airblade/paper_trail) do (but [faster](bench/performance)).
+Logidze provides tools for logging DB records changes when using PostgreSQL. Just like [audited](https://github.com/collectiveidea/audited) and [paper_trail](https://github.com/airblade/paper_trail) do (but [faster](bench/performance)).
 
 Logidze allows you to create a DB-level log (using triggers) and gives you an API to browse this log.
 The log is stored with the record itself in JSONB column. No additional tables required.
@@ -14,8 +14,9 @@ The log is stored with the record itself in JSONB column. No additional tables r
 
 Other requirements:
 
-- Ruby ~> 2.5
-- Rails >= 5.0 (for Rails 4.2 use version <=0.12.0)
+- Ruby ~> 2.7
+- Rails >= 6.0 (for Rails 4.2 use version <=0.12.0, for Rails 5.x use version <= 1.2.3)
+- PostgreSQL >= 10.0
 
 <a href="https://evilmartians.com/">
 <img src="https://evilmartians.com/badges/sponsored-by-evil-martians.svg" alt="Sponsored by Evil Martians" width="236" height="54"></a>
@@ -35,6 +36,7 @@ Other requirements:
   - [Tracking only selected columns](#tracking-only-selected-columns)
   - [Logs timestamps](#logs-timestamps)
   - [Undoing a Generated Invocation](#undoing-a-generated-invocation)
+  - [Using with partitioned tables](#using-with-partitioned-tables)
 - [Usage](#usage)
   - [Basic API](#basic-api)
   - [Track meta information](#track-meta-information)
@@ -189,6 +191,18 @@ bundle exec rails destroy logidze:model Post
 
 **IMPORTANT**: If you use non-UTC time zone for Active Record (`config.active_record.default_timezone`), you MUST always infer log timestamps from a timestamp column (e.g., when back-filling data); otherwise, you may end up with inconsistent logs ([#199](https://github.com/palkan/logidze/issues/199)). In general, we recommend using UTC as the database time unless there is a very strong reason not to.
 
+### Using with partitioned tables
+
+Logidze supports partitioned tables for PostgreSQL 13+ without any additional configuration. For PostgreSQL 11/12, you should use _after_ triggers. To do that, provide the `--after-trigger` option to the migration:
+
+```sh
+bundle exec rails generate logidze:model Post --after-trigger
+```
+
+**NOTE:** Record changes are written as a full snapshot if the partition has changed during the update.
+
+**IMPORTANT:** Using Logidze for partitioned tables in PostgreSQL 10 is not supported.
+
 ## Usage
 
 ### Basic API
@@ -243,6 +257,25 @@ Post.where(created_at: Time.zone.today.all_day).diff_from(time: 1.hour.ago)
 
 **NOTE:** If `log_data` is nil, `#diff_from` returns an empty Hash as `"changes"`.
 
+Also, it is possible to retrieve list of model's `versions`:
+
+```ruby
+post.versions # => Enumerator
+
+# you can use Enumerator's #take to return all
+post.versions.take
+
+# or you take a few or call any Enumerable method
+post.versions.take(2)
+post.versions.find do
+  _1.title == "old title"
+end
+
+# we can also add options
+post.versions(reverse: true) # from older to newer
+post.versions(include_self: true) # returns self as the first one (default) or the last one record (if reverse: true)
+```
+
 There are also `#undo!` and `#redo!` options (and more general `#switch_to!`):
 
 ```ruby
@@ -302,6 +335,10 @@ Logidze.with_meta({ip: request.ip}, transactional: false) do
 end
 ```
 
+**Important:** If you use connection pooling (e.g., PgBouncer), using `.with_meta` without a transaction may lead to unexpected results (since meta is set for a connection). Without a transaction, we cannot guarantee that the same connection will be used for queries (including metadata cleanup).
+
+**Important**: In Rails, `after_commit` callbacks are executed after transaction is committed, and, thus, after `with_meta` block is executed—the meta wouldn't be added to changes captured in the `after_commit` phase. One particular scenario is having associations with `touch: true` (_touch_ updates are executed after commit).
+
 ### Track responsibility
 
 A special application of meta information is storing the author of the change, which is called _Responsible ID_. There is more likely that you would like to store the `current_user.id` that way.

data/lib/generators/logidze/fx_helper.rb

@@ -6,7 +6,7 @@ module Logidze
     module FxHelper
       def self.included(base)
         base.class_option :fx, type: :boolean, optional: true,
-                               desc: "Define whether to use fx gem functionality"
+          desc: "Define whether to use fx gem functionality"
       end
 
       def fx?

data/lib/generators/logidze/install/functions/logidze_logger.sql

@@ -1,9 +1,10 @@
 CREATE OR REPLACE FUNCTION logidze_logger() RETURNS TRIGGER AS $body$
-  -- version: 3
+  -- version: 4
   DECLARE
     changes jsonb;
     version jsonb;
-    snapshot jsonb;
+    full_snapshot boolean;
+    log_data jsonb;
     new_v integer;
     size integer;
     history_limit integer;
@@ -30,59 +31,56 @@ CREATE OR REPLACE FUNCTION logidze_logger() RETURNS TRIGGER AS $body$
     columns := NULLIF(TG_ARGV[2], 'null');
     include_columns := NULLIF(TG_ARGV[3], 'null');
 
-    IF TG_OP = 'INSERT' THEN
+    IF NEW.log_data is NULL OR NEW.log_data = '{}'::jsonb
+    THEN
       IF columns IS NOT NULL THEN
-        snapshot = logidze_snapshot(to_jsonb(NEW.*), ts_column, columns, include_columns);
+        log_data = logidze_snapshot(to_jsonb(NEW.*), ts_column, columns, include_columns);
       ELSE
-        snapshot = logidze_snapshot(to_jsonb(NEW.*), ts_column);
+        log_data = logidze_snapshot(to_jsonb(NEW.*), ts_column);
       END IF;
 
-      IF snapshot#>>'{h, -1, c}' != '{}' THEN
-        NEW.log_data := snapshot;
+      IF log_data#>>'{h, -1, c}' != '{}' THEN
+        NEW.log_data := log_data;
       END IF;
 
-    ELSIF TG_OP = 'UPDATE' THEN
+    ELSE
 
-      IF OLD.log_data is NULL OR OLD.log_data = '{}'::jsonb THEN
-        IF columns IS NOT NULL THEN
-          snapshot = logidze_snapshot(to_jsonb(NEW.*), ts_column, columns, include_columns);
-        ELSE
-          snapshot = logidze_snapshot(to_jsonb(NEW.*), ts_column);
-        END IF;
-
-        IF snapshot#>>'{h, -1, c}' != '{}' THEN
-          NEW.log_data := snapshot;
-        END IF;
-        RETURN NEW;
+      IF TG_OP = 'UPDATE' AND (to_jsonb(NEW.*) = to_jsonb(OLD.*)) THEN
+        RETURN NEW; -- pass
       END IF;
 
       history_limit := NULLIF(TG_ARGV[0], 'null');
       debounce_time := NULLIF(TG_ARGV[4], 'null');
 
-      current_version := (NEW.log_data->>'v')::int;
+      log_data := NEW.log_data;
+
+      current_version := (log_data->>'v')::int;
 
       IF ts_column IS NULL THEN
         ts := statement_timestamp();
-      ELSE
-        ts := (to_jsonb(NEW.*)->>ts_column)::timestamp with time zone;
-        IF ts IS NULL OR ts = (to_jsonb(OLD.*)->>ts_column)::timestamp with time zone THEN
+      ELSEIF TG_OP = 'UPDATE' THEN
+        ts := (to_jsonb(NEW.*) ->> ts_column)::timestamp with time zone;
+        IF ts IS NULL OR ts = (to_jsonb(OLD.*) ->> ts_column)::timestamp with time zone THEN
+          ts := statement_timestamp();
+        END IF;
+      ELSEIF TG_OP = 'INSERT' THEN
+        ts := (to_jsonb(NEW.*) ->> ts_column)::timestamp with time zone;
+        IF ts IS NULL OR (extract(epoch from ts) * 1000)::bigint = (NEW.log_data #>> '{h,-1,ts}')::bigint THEN
           ts := statement_timestamp();
         END IF;
       END IF;
 
-      IF to_jsonb(NEW.*) = to_jsonb(OLD.*) THEN
-        RETURN NEW;
-      END IF;
+      full_snapshot := (coalesce(current_setting('logidze.full_snapshot', true), '') = 'on') OR (TG_OP = 'INSERT');
 
-      IF current_version < (NEW.log_data#>>'{h,-1,v}')::int THEN
+      IF current_version < (log_data#>>'{h,-1,v}')::int THEN
         iterator := 0;
-        FOR item in SELECT * FROM jsonb_array_elements(NEW.log_data->'h')
+        FOR item in SELECT * FROM jsonb_array_elements(log_data->'h')
         LOOP
           IF (item.value->>'v')::int > current_version THEN
-            NEW.log_data := jsonb_set(
-              NEW.log_data,
+            log_data := jsonb_set(
+              log_data,
               '{h}',
-              (NEW.log_data->'h') - iterator
+              (log_data->'h') - iterator
             );
           END IF;
           iterator := iterator + 1;
@@ -91,7 +89,7 @@ CREATE OR REPLACE FUNCTION logidze_logger() RETURNS TRIGGER AS $body$
 
       changes := '{}';
 
-      IF (coalesce(current_setting('logidze.full_snapshot', true), '') = 'on') THEN
+      IF full_snapshot THEN
         BEGIN
           changes = hstore_to_jsonb_loose(hstore(NEW.*));
         EXCEPTION
@@ -132,48 +130,50 @@ CREATE OR REPLACE FUNCTION logidze_logger() RETURNS TRIGGER AS $body$
       END IF;
 
       IF changes = '{}' THEN
-        RETURN NEW;
+        RETURN NEW; -- pass
       END IF;
 
-      new_v := (NEW.log_data#>>'{h,-1,v}')::int + 1;
+      new_v := (log_data#>>'{h,-1,v}')::int + 1;
 
-      size := jsonb_array_length(NEW.log_data->'h');
+      size := jsonb_array_length(log_data->'h');
       version := logidze_version(new_v, changes, ts);
 
       IF (
         debounce_time IS NOT NULL AND
-        (version->>'ts')::bigint - (NEW.log_data#>'{h,-1,ts}')::text::bigint <= debounce_time
+        (version->>'ts')::bigint - (log_data#>'{h,-1,ts}')::text::bigint <= debounce_time
       ) THEN
         -- merge new version with the previous one
-        new_v := (NEW.log_data#>>'{h,-1,v}')::int;
-        version := logidze_version(new_v, (NEW.log_data#>'{h,-1,c}')::jsonb || changes, ts);
+        new_v := (log_data#>>'{h,-1,v}')::int;
+        version := logidze_version(new_v, (log_data#>'{h,-1,c}')::jsonb || changes, ts);
         -- remove the previous version from log
-        NEW.log_data := jsonb_set(
-          NEW.log_data,
+        log_data := jsonb_set(
+          log_data,
           '{h}',
-          (NEW.log_data->'h') - (size - 1)
+          (log_data->'h') - (size - 1)
         );
       END IF;
 
-      NEW.log_data := jsonb_set(
-        NEW.log_data,
+      log_data := jsonb_set(
+        log_data,
         ARRAY['h', size::text],
         version,
         true
       );
 
-      NEW.log_data := jsonb_set(
-        NEW.log_data,
+      log_data := jsonb_set(
+        log_data,
         '{v}',
         to_jsonb(new_v)
       );
 
       IF history_limit IS NOT NULL AND history_limit <= size THEN
-        NEW.log_data := logidze_compact_history(NEW.log_data, size - history_limit + 1);
+        log_data := logidze_compact_history(log_data, size - history_limit + 1);
       END IF;
+
+      NEW.log_data := log_data;
     END IF;
 
-    return NEW;
+    RETURN NEW; -- result
   EXCEPTION
     WHEN OTHERS THEN
       GET STACKED DIAGNOSTICS err_sqlstate = RETURNED_SQLSTATE,

data/lib/generators/logidze/install/functions/logidze_logger_after.sql

@@ -0,0 +1,3 @@
+CREATE OR REPLACE FUNCTION logidze_logger_after() RETURNS TRIGGER AS $body$
+  -- version: 4
+<%= generate_logidze_logger_after %>

data/lib/generators/logidze/install/install_generator.rb

@@ -6,8 +6,6 @@ require "logidze/utils/function_definitions"
 require_relative "../inject_sql"
 require_relative "../fx_helper"
 
-using RubyNext
-
 module Logidze
   module Generators
     class InstallGenerator < ::Rails::Generators::Base # :nodoc:
@@ -19,7 +17,7 @@ module Logidze
       source_paths << File.expand_path("functions", __dir__)
 
       class_option :update, type: :boolean, optional: true,
-                            desc: "Define whether this is an update migration"
+        desc: "Define whether this is an update migration"
 
       def generate_migration
         migration_template = fx? ? "migration_fx.rb.erb" : "migration.rb.erb"
@@ -81,6 +79,17 @@ module Logidze
         def function_definitions
           @function_definitions ||= Logidze::Utils::FunctionDefinitions.from_fs
         end
+
+        # Generate `logidze_logger_after.sql` from the regular `logidze_logger.sql`
+        # by find-and-replacing a few lines
+        def generate_logidze_logger_after
+          source = File.read(File.join(__dir__, "functions", "logidze_logger.sql"))
+          source.sub!(/^CREATE OR REPLACE FUNCTION logidze_logger.*$/, "")
+          source.sub!(/^  -- version.*$/, "")
+          source.gsub!("RETURN NEW; -- pass", "RETURN NULL;")
+          source.gsub!("RETURN NEW; -- result", "    EXECUTE format('UPDATE %I.%I SET \"log_data\" = $1 WHERE ctid = %L', TG_TABLE_SCHEMA, TG_TABLE_NAME, NEW.CTID) USING NEW.log_data;\n    RETURN NULL;")
+          source
+        end
       end
 
       def self.next_migration_number(dir)

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

@@ -5,8 +5,6 @@ require "rails/generators/active_record/migration/migration_generator"
 require_relative "../inject_sql"
 require_relative "../fx_helper"
 
-using RubyNext
-
 module Logidze
   module Generators
     class ModelGenerator < ::ActiveRecord::Generators::Base # :nodoc:
@@ -19,13 +17,13 @@ module Logidze
       class_option :limit, type: :numeric, optional: true, desc: "Specify history size limit"
 
       class_option :debounce_time, type: :numeric, optional: true,
-                                   desc: "Specify debounce time in millisecond"
+        desc: "Specify debounce time in millisecond"
 
       class_option :backfill, type: :boolean, optional: true,
-                              desc: "Add query to backfill existing records history"
+        desc: "Add query to backfill existing records history"
 
       class_option :only_trigger, type: :boolean, optional: true,
-                                  desc: "Create trigger-only migration"
+        desc: "Create trigger-only migration"
 
       class_option :path, type: :string, optional: true, desc: "Specify path to the model file"
 
@@ -33,12 +31,14 @@ module Logidze
       class_option :only, type: :array, optional: true
 
       class_option :timestamp_column, type: :string, optional: true,
-                                      desc: "Specify timestamp column"
+        desc: "Specify timestamp column"
 
       class_option :name, type: :string, optional: true, desc: "Migration name"
 
       class_option :update, type: :boolean, optional: true,
-                            desc: "Define whether this is an update migration"
+        desc: "Define whether this is an update migration"
+
+      class_option :after_trigger, type: :boolean, optional: true, desc: "Use after trigger"
 
       def generate_migration
         if options[:except] && options[:only]
@@ -51,7 +51,9 @@ module Logidze
       def generate_fx_trigger
         return unless fx?
 
-        template "logidze.sql", "db/triggers/logidze_on_#{table_name}_v#{next_version.to_s.rjust(2, "0")}.sql"
+        template_name = after_trigger? ? "logidze_after.sql" : "logidze.sql"
+
+        template template_name, "db/triggers/logidze_on_#{table_name}_v#{next_version.to_s.rjust(2, "0")}.sql"
       end
 
       def inject_logidze_to_model
@@ -94,6 +96,10 @@ module Logidze
           options[:update]
         end
 
+        def after_trigger?
+          options[:after_trigger]
+        end
+
         def filtered_columns
           format_pgsql_array(options[:only] || options[:except])
         end

data/lib/generators/logidze/model/templates/migration.rb.erb

@@ -37,7 +37,7 @@ class <%= @migration_class_name %> < ActiveRecord::Migration[<%= ActiveRecord::M
 
   <%- end -%>
         execute <<~SQL
-<%= inject_sql("logidze.sql", indent: 10) %>
+<%= inject_sql(after_trigger? ? "logidze_after.sql" : "logidze.sql", indent: 10) %>
         SQL
       end
 

data/lib/generators/logidze/model/triggers/logidze_after.sql

@@ -0,0 +1,6 @@
+CREATE TRIGGER <%= %Q("logidze_on_#{full_table_name}") %>
+AFTER UPDATE OR INSERT ON <%= %Q("#{full_table_name}") %> FOR EACH ROW
+WHEN (coalesce(current_setting('logidze.disabled', true), '') <> 'on' AND pg_trigger_depth() < 1)
+-- Parameters: history_size_limit (integer), timestamp_column (text), filtered_columns (text[]),
+-- include_columns (boolean), debounce_time_ms (integer)
+EXECUTE PROCEDURE logidze_logger_after(<%= logidze_logger_parameters %>);

data/lib/logidze/ignore_log_data.rb

@@ -5,11 +5,6 @@ module Logidze
     extend ActiveSupport::Concern
 
     included do
-      if Rails::VERSION::MAJOR == 5
-        require "logidze/ignore_log_data/cast_attribute_patch"
-        include CastAttributePatch
-      end
-
       scope :with_log_data, lambda {
         if ignored_columns == ["log_data"]
           select(arel_table[Arel.star])

data/lib/logidze/model.rb

@@ -3,8 +3,6 @@
 require "active_support"
 
 module Logidze
-  using RubyNext
-
   # Extends model with methods to browse history
   module Model
     require "logidze/history/type"
@@ -96,6 +94,20 @@ module Logidze
 
       build_dup(log_entry, time)
     end
+
+    def logidze_versions(reverse: false, include_self: false)
+      versions_meta = log_data.versions.dup
+
+      if reverse
+        versions_meta.reverse!
+        versions_meta.shift unless include_self
+      else
+        versions_meta.pop unless include_self
+      end
+
+      Enumerator.new { |yielder| versions_meta.each { yielder << at(version: _1.version) } }
+    end
+
     # rubocop: enable Metrics/MethodLength
 
     # Revert record to the version at specified time (without saving to DB)
@@ -116,6 +128,7 @@ module Logidze
 
     # Return a dirty copy of specified version of record
     def at_version(version)
+      return nil unless log_data
       return self if log_data.version == version
 
       log_entry = log_data.find_by_version(version)
@@ -217,7 +230,7 @@ module Logidze
 
     # Loads log_data field from the database, stores to the attributes hash and returns it
     def reload_log_data
-      self.log_data = self.class.where(self.class.primary_key => id).pluck("#{self.class.table_name}.log_data".to_sym).first
+      self.log_data = self.class.where(self.class.primary_key => id).pluck(:"#{self.class.table_name}.log_data").first
     end
 
     # Nullify log_data column for a single record

data/lib/logidze/utils/check_pending.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require_relative "./function_definitions"
-require_relative "./pending_migration_error"
+require_relative "function_definitions"
+require_relative "pending_migration_error"
 
 module Logidze
   module Utils
@@ -31,10 +31,10 @@ module Logidze
       def notify_or_raise!
         case Logidze.on_pending_upgrade
         when :warn
-          warn "\n**************************************************\n"\
-               "⛔️  WARNING: Logidze needs an upgrade and might not work correctly.\n"\
-               "Please, make sure to run `bundle exec rails generate logidze:install --update` "\
-               "and apply generated migration."\
+          warn "\n**************************************************\n" \
+               "⛔️  WARNING: Logidze needs an upgrade and might not work correctly.\n" \
+               "Please, make sure to run `bundle exec rails generate logidze:install --update` " \
+               "and apply generated migration." \
                "\n**************************************************\n\n"
         when :raise
           raise Logidze::Utils::PendingMigrationError, "Logidze needs upgrade. Run `bundle exec rails generate logidze:install --update` and apply generated migration."

data/lib/logidze/utils/pending_migration_error.rb

@@ -5,19 +5,17 @@ require "rails/generators"
 module Logidze
   module Utils
     class PendingMigrationError < StandardError
-      if Rails::VERSION::MAJOR >= 6
-        require "active_record"
-        require "active_support/actionable_error"
-        include ActiveSupport::ActionableError
+      require "active_record"
+      require "active_support/actionable_error"
+      include ActiveSupport::ActionableError
 
-        action "Upgrade Logidze" do
-          Rails::Generators.invoke("logidze:install", ["--update"])
-          ActiveRecord::Tasks::DatabaseTasks.migrate
-          if ActiveRecord::Base.dump_schema_after_migration
-            ActiveRecord::Tasks::DatabaseTasks.dump_schema(
-              ActiveRecord::Base.connection_db_config
-            )
-          end
+      action "Upgrade Logidze" do
+        Rails::Generators.invoke("logidze:install", ["--update"])
+        ActiveRecord::Tasks::DatabaseTasks.migrate
+        if ActiveRecord::Base.dump_schema_after_migration
+          ActiveRecord::Tasks::DatabaseTasks.dump_schema(
+            ActiveRecord::Base.connection_db_config
+          )
         end
       end
     end

data/lib/logidze/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Logidze
-  VERSION = "1.2.3"
+  VERSION = "1.3.0"
 end

data/lib/logidze.rb

@@ -5,7 +5,6 @@ require "logidze/version"
 # Logidze provides tools for adding in-table JSON-based audit to DB tables
 # and ActiveRecord extensions to work with changes history.
 module Logidze
-  require "ruby-next"
   require "logidze/history"
   require "logidze/model"
   require "logidze/versioned_association"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logidze
 version: !ruby/object:Gem::Version
-  version: 1.2.3
+  version: 1.3.0
 platform: ruby
 authors:
 - palkan
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-01-04 00:00:00.000000000 Z
+date: 2024-01-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: railties
@@ -16,42 +16,28 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '6.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '6.0'
 - !ruby/object:Gem::Dependency
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '6.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.0'
-- !ruby/object:Gem::Dependency
-  name: ruby-next-core
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.9'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '6.0'
 - !ruby/object:Gem::Dependency
   name: ammeter
   requirement: !ruby/object:Gem::Requirement
@@ -100,14 +86,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.18'
+        version: '1.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.18'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -128,14 +114,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.4'
+        version: '4.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.4'
+        version: '4.0'
 - !ruby/object:Gem::Dependency
   name: timecop
   requirement: !ruby/object:Gem::Requirement
@@ -167,6 +153,7 @@ files:
 - lib/generators/logidze/install/functions/logidze_compact_history.sql
 - lib/generators/logidze/install/functions/logidze_filter_keys.sql
 - lib/generators/logidze/install/functions/logidze_logger.sql
+- lib/generators/logidze/install/functions/logidze_logger_after.sql
 - lib/generators/logidze/install/functions/logidze_snapshot.sql
 - lib/generators/logidze/install/functions/logidze_version.sql
 - lib/generators/logidze/install/install_generator.rb
@@ -177,6 +164,7 @@ files:
 - lib/generators/logidze/model/model_generator.rb
 - lib/generators/logidze/model/templates/migration.rb.erb
 - lib/generators/logidze/model/triggers/logidze.sql
+- lib/generators/logidze/model/triggers/logidze_after.sql
 - lib/logidze.rb
 - lib/logidze/engine.rb
 - lib/logidze/has_logidze.rb
@@ -184,7 +172,6 @@ files:
 - lib/logidze/history/type.rb
 - lib/logidze/history/version.rb
 - lib/logidze/ignore_log_data.rb
-- lib/logidze/ignore_log_data/cast_attribute_patch.rb
 - lib/logidze/meta.rb
 - lib/logidze/model.rb
 - lib/logidze/utils/check_pending.rb
@@ -209,14 +196,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.5.0
+      version: 2.7.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.11
+rubygems_version: 3.4.20
 signing_key:
 specification_version: 4
 summary: PostgreSQL JSONB-based model changes tracking

data/lib/logidze/ignore_log_data/cast_attribute_patch.rb

@@ -1,17 +0,0 @@
-# frozen_string_literal: true
-
-module Logidze
-  module IgnoreLogData
-    # Fixes unexpected behavior (see more https://github.com/rails/rails/pull/34528):
-    # instead of using a type passed to `attribute` call, ignored column uses
-    # a type coming from the DB (in this case `.log_data` would return a plain hash
-    # instead of `Logidze::History`)
-    module CastAttributePatch
-      def log_data
-        return attributes["log_data"] if attributes["log_data"].is_a?(Logidze::History)
-
-        self.log_data = Logidze::History::Type.new.cast_value(super)
-      end
-    end
-  end
-end