immosquare-active-record-change-tracker 0.1.4 → 0.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 277ddb50fbedfc0569e1f7e4a61d54252df1ef5402c398eee5dfb0bd469d90a9
-  data.tar.gz: 81ba5ed9aae9a90c05a9be743cf1adf6f3cf027ac4859275f6a650678dc8a500
+  metadata.gz: c45fff508dbb3ba604cd8f4edb7ed21011adf6eacf2ce3c31c0037bb7cbaf6a5
+  data.tar.gz: e03e4dbd003973dc0899f93649433f2d8dde8ada78e1b1ffb087e52578642947
 SHA512:
-  metadata.gz: b82d2dcd49840b346f1550e5f80b552c52111fdcf6c08ba8f5234d0814ff66bb3ad1763cd2c9226282b26d34f19a3ff36245e7ce0424cebbc8a0aa342edd9c2a
-  data.tar.gz: af57bca752c1092ae5f346e6f3c6a2189166780e2fa9efaae441d318152e90a57fb38906994b225956050a1863b7afab19686e7e7eee957bb12c72250382eb91
+  metadata.gz: 9be8b85fbba788bc7af1cdb9ddb5f8489b4ca311e5f60a60438a2078c240a84772a08eb99e7190b032e76e9cd9e6c253a6577af5ce3b8be608b5bb476c04b676
+  data.tar.gz: 26cf27788d62ee7451c9986cd1d604dca7963cf926abdf387cf7261883b940822449cf200abf813e84255ebc401156fcbd93686844365b03567cd861b19f1eda

data/lib/immosquare-active-record-change-tracker/models/history_record.rb

@@ -8,8 +8,8 @@ module ImmosquareActiveRecordChangeTracker
     self.table_name = "active_record_change_trackers"
 
     belongs_to :recordable, :polymorphic => true
-    belongs_to :modifier,   :polymorphic => true
-    serialize :data, :coder => JSON
+    belongs_to :modifier,   :polymorphic => true, :optional => true
+    serialize(:data, :coder => JSON)
 
   end
 end

data/lib/immosquare-active-record-change-tracker/version.rb

@@ -1,3 +1,3 @@
 module ImmosquareActiveRecordChangeTracker
-  VERSION = "0.1.4".freeze
+  VERSION = "0.1.5".freeze
 end

data/lib/immosquare-active-record-change-tracker.rb

@@ -10,10 +10,12 @@ module ImmosquareActiveRecordChangeTracker
 
   module ClassMethods
     ##============================================================##
-    ## Can be improved with other gems like paranoia
+    ## True si le modèle hôte utilise paranoia (acts_as_paranoid).
+    ## La gem paranoia est optionnelle : si elle n'est pas chargée,
+    ## paranoid? n'existe pas — d'où le respond_to?.
     ##============================================================##
-    def kept_in_db
-      paranoid?
+    def kept_in_db?
+      respond_to?(:paranoid?) && paranoid?
     end
 
     def track_active_record_changes(options = {}, &modifier_block)
@@ -31,11 +33,11 @@ module ImmosquareActiveRecordChangeTracker
       }
 
       ##============================================================##
-      ## Ajouter :dependent => :destroy si acts_as_paranoid n'est pas utilisé
-      ## on se base sur paranoia_column car acts_as_paranoid répond true
-      ## sur tous les modèles du temps que la gem est incluse
+      ## Ajouter :dependent => :destroy si acts_as_paranoid n'est pas utilisé.
+      ## Avec paranoia on garde l'historique au soft-delete et on le
+      ## nettoie via after_real_destroy.
       ##============================================================##
-      association_options[:dependent] = :destroy if !kept_in_db
+      association_options[:dependent] = :destroy if !kept_in_db?
 
       ##============================================================##
       ## Ajout de l'association has_many :history_records
@@ -58,7 +60,7 @@ module ImmosquareActiveRecordChangeTracker
       ##============================================================##
       after_save(:save_change_history)
       after_destroy(:delete_change_history)
-      after_real_destroy(:delete_all_change_histories) if paranoid?
+      after_real_destroy(:delete_all_change_histories) if kept_in_db?
     end
   end
 
@@ -74,20 +76,31 @@ module ImmosquareActiveRecordChangeTracker
     ## Stocker les changements après un create ou save ou update
     ##============================================================##
     def save_change_history
-      history_options = self.class.history_options
+      options = self.class.history_options
 
       ##============================================================##
       ## Récupérer les champs à observer
       ##============================================================##
       changes_to_save =
-        if history_options[:only].present?
-          previous_changes.slice(*history_options[:only].map(&:to_s))
+        if options[:only].present?
+          previous_changes.slice(*options[:only].map(&:to_s))
         else
-          excluded_fields = history_options[:except] || []
+          excluded_fields = options[:except] || []
           excluded_fields += [:created_at, :updated_at]
           previous_changes.except(*excluded_fields.uniq.map(&:to_s))
         end
 
+      ##============================================================##
+      ## On regarde si jamais ce que l'on essaye de sauvegarder contient
+      ## des valeurs identiques. Doit s'exécuter AVANT le merge Globalize :
+      ## les entrées de traduction sont des hash {locale => diff} qui ne
+      ## ressemblent pas à un [old, new] et casseraient l'indexation.
+      ## ex: quand on met true dans un integer, rails le convertit
+      ## automatiquement en 1 si le champ était déjà en bdd
+      ## -> {"cellar"=>[1, 1]} ou {"cellar"=>[0, 0]}
+      ##============================================================##
+      changes_to_save = changes_to_save.reject {|_k, change_array| change_array[0] == change_array[1] }
+
       ##============================================================##
       ## Gestion de Globalize
       ##============================================================##
@@ -115,42 +128,12 @@ module ImmosquareActiveRecordChangeTracker
         changes_to_save.merge!(globalize_changes)
       end
 
-
-      ##============================================================##
-      ## On regarde si jamais ce que l'on essaye de sauvegarder contient
-      ## des valeurs identiques.
-      ## ex: quand on met true dans un integer, rails le convertit
-      ## automatiquement en 1 si le champ était déjà en bdd
-      ## -> {"cellar"=>[1, 1]} ou {"cellar"=>[0, 0]}
-      ##============================================================##
-      changes_to_save = changes_to_save.reject {|_k, change_array| change_array[0] == change_array[1] }
-
       ##============================================================##
       ## Si aucun changement à sauvegarder, on sort
       ##============================================================##
       return if changes_to_save.none?
 
-
-      ##============================================================##
-      ## Récupération du modificateur en exécutant le bloc s'il est défini
-      ##============================================================##
-      modifier = history_options[:modifier_block]&.call
-
-      ##============================================================##
-      ## Gestion de l'événement (create ou update)
-      ##============================================================##
-      event = previously_new_record? ? "create" : "update"
-
-      ##============================================================##
-      ## On crée un enregistrement dans la table d'historique
-      ##============================================================##
-      ImmosquareActiveRecordChangeTracker::HistoryRecord.create!(
-        :recordable => self,
-        :modifier   => modifier,
-        :data       => changes_to_save,
-        :event      => event,
-        :created_at => DateTime.now
-      )
+      write_history_record(:event => previously_new_record? ? "create" : "update", :data => changes_to_save)
     end
 
     ##============================================================##
@@ -159,22 +142,23 @@ module ImmosquareActiveRecordChangeTracker
     ## de suppression
     ##============================================================##
     def delete_change_history
-      return if !self.class.kept_in_db
+      return if !self.class.kept_in_db?
 
-      ##============================================================##
-      ## Récupéreration du modificateur en exécutant le bloc s'il est défini
-      ##============================================================##
-      modifier = history_options[:modifier_block]&.call
+      write_history_record(:event => "destroy", :data => nil)
+    end
 
-      ##============================================================##
-      ## On crée un enregistrement dans la table d'historique
-      ##============================================================##
+    ##============================================================##
+    ## Écrit une entrée dans la table d'historique. Le modifier est
+    ## résolu à la volée via le bloc passé à track_active_record_changes
+    ## (souvent un Current.user/admin).
+    ##============================================================##
+    def write_history_record(event:, data:)
       ImmosquareActiveRecordChangeTracker::HistoryRecord.create!(
         :recordable => self,
-        :modifier   => modifier,
-        :data       => nil,
-        :event      => "destroy",
-        :created_at => DateTime.now
+        :modifier   => self.class.history_options[:modifier_block]&.call,
+        :data       => data,
+        :event      => event,
+        :created_at => Time.current
       )
     end
   end

metadata

@@ -1,15 +1,42 @@
 --- !ruby/object:Gem::Specification
 name: immosquare-active-record-change-tracker
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.1.5
 platform: ruby
 authors:
 - immosquare
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-12-05 00:00:00.000000000 Z
-dependencies: []
+date: 1980-01-02 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: activerecord
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '6.1'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '6.1'
+- !ruby/object:Gem::Dependency
+  name: railties
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '6.1'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '6.1'
 description: A gem to track changes on ActiveRecord models
 email:
 - jules@immosquare.com
@@ -27,7 +54,6 @@ homepage: https://github.com/immosquare/immosquare-active-record-change-tracker
 licenses:
 - MIT
 metadata: {}
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -35,15 +61,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.7.2
+      version: 3.2.6
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.22
-signing_key:
+rubygems_version: 4.0.10
 specification_version: 4
 summary: ActiveRecord Change Tracker
 test_files: []