activeaudit 0.1.4 → 0.1.5

data/README.md

@@ -4,6 +4,7 @@ Audit support for Active Record
 
 [![Build Status](https://travis-ci.org/maintux/activeaudit.png)](https://travis-ci.org/maintux/activeaudit)
 [![Code Climate](https://codeclimate.com/github/maintux/activeaudit.png)](https://codeclimate.com/github/maintux/activeaudit)
+[![Dependency Status](https://gemnasium.com/maintux/activeaudit.png)](https://gemnasium.com/maintux/activeaudit)
 
 ## Install
 

data/lib/active_audit/logger.rb

@@ -14,6 +14,7 @@ module ActiveAudit
       before_destroy :log_activity_on_destroy
 
       attr_accessible :audit_user_id
+      attr_accessible :audit_extras
       attr_accessor :audit_user_id
       attr_accessor :audit_extras
 
@@ -25,24 +26,24 @@ module ActiveAudit
         self.loggable_events.merge! event
       end
 
+      def log_event(activity)
+        ActiveAudit::Audit.create obj_id: id, obj_type: self.class.to_s, user_id: audit_user_id, activity: activity, extras: audit_extras
+      end
+      private :log_event
+
       def log_activity_on_create
         self.class.loggable_events.each do |k,v|
           next unless k.eql?(:create)
           if v.eql?(true)
-            ActiveAudit::Audit.create obj_id: id, obj_type: self.class.to_s, user_id: audit_user_id, activity: k.to_s, extras: audit_extras
+            log_event k.to_s
           elsif v.is_a?(Array)
-            log_event = false
             v.each do |field|
-              if field.is_a?(Hash)
-                key, value = field.first
-                log_event = true and break if self.send(key.to_s).eql?(value)
-              elsif field.is_a?(Symbol)
-                log_event and break if self.send(field.to_s)
+              if field.is_a?(Hash) and self.send(field.keys.first).eql?(field.values.first)
+                log_event(k.to_s) and break
+              elsif field.is_a?(Symbol) and self.send(field.to_s)
+                log_event(k.to_s) and break
               end
             end
-            if log_event
-              ActiveAudit::Audit.create obj_id: id, obj_type: self.class.to_s, user_id: audit_user_id, activity: k.to_s, extras: audit_extras
-            end
           end
         end
       end
@@ -51,20 +52,13 @@ module ActiveAudit
         self.class.loggable_events.each do |k,v|
           next if [:create,:destroy].include?(k)
           if k.eql?(:update) and v.eql?(true)
-            ActiveAudit::Audit.create obj_id: id, obj_type: self.class.to_s, user_id: audit_user_id, activity: k.to_s, extras: audit_extras
-          else
-            if v.is_a?(Array)
-              log_event = false
-              v.each do |field|
-                if field.is_a?(Hash)
-                  key, value = field.first
-                  log_event = true and break if self.send("#{key}_changed?") and self.send(key.to_s).eql?(value)
-                elsif field.is_a?(Symbol)
-                  log_event = true and break if self.send("#{field}_changed?")
-                end
-              end
-              if log_event
-                ActiveAudit::Audit.create obj_id: id, obj_type: self.class.to_s, user_id: audit_user_id, activity: k.to_s, extras: audit_extras
+            log_event k.to_s
+          elsif v.is_a?(Array)
+            v.each do |field|
+              if field.is_a?(Hash) and self.send("#{field.keys.first}_changed?") and self.send(field.keys.first).eql?(field.values.first)
+                log_event(k.to_s) and break
+              elsif field.is_a?(Symbol) and self.send("#{field}_changed?")
+                log_event(k.to_s) and break
               end
             end
           end
@@ -74,7 +68,7 @@ module ActiveAudit
       def log_activity_on_destroy
         self.class.loggable_events.each do |k,v|
           next unless k.eql?(:destroy)
-          ActiveAudit::Audit.create obj_id: id, obj_type: self.class.to_s, user_id: audit_user_id, activity: k.to_s, extras: audit_extras
+          log_event k.to_s
         end
       end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: activeaudit
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.1.5
   prerelease: 
 platform: ruby
 authors:
@@ -18,7 +18,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '3.12'
+        version: 4.0.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -26,7 +26,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '3.12'
+        version: 4.0.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -158,7 +158,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -2222965222075044048
+      hash: 1684660491457008087
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: