auditor 2.1.0 → 2.1.1

data/lib/auditor/recorder.rb

@@ -24,8 +24,11 @@ module Auditor
       audit = Audit.new
       audit.auditable_id = model.id
       audit.auditable_type = model.class.name
-      audit.audited_changes = prepare_changes(model.changes) if changes_available?(action)
+      audit.audited_changes = prepare_changes(model.changes) if model.changed?
       audit.action = action
+
+      return if noop?(audit)
+
       audit.comment = @blk.call(model, user, action) if @blk
 
       without_auditing do
@@ -44,9 +47,8 @@ module Auditor
       chg.empty? ? nil : chg
     end
 
-    def changes_available?(action)
-      [:create, :update].include?(action)
+    def noop?(audit)
+      audit.action == :update && !audit.audited_changes.present?
     end
-
   end
 end

data/lib/auditor/version.rb

@@ -1,3 +1,3 @@
 module Auditor
-  VERSION = "2.1.0"
+  VERSION = "2.1.1"
 end

data/spec/auditable_spec.rb

@@ -97,6 +97,15 @@ describe Auditor::Auditable do
     m.attributes_at(ts4).should == {'name' => '2', 'id' => m.id}
   end
 
+  it 'should not save an update record that does not have any audited changes' do
+    redefine_model { audit!(:update, :except => :name) }
+
+    lambda {
+      m = Model.create(:name => 'new')
+      m.update_attributes({:name => 'newer'})
+    }.should_not change(Audit, :count)
+  end
+
   def verify_audit(audit, model, changes=nil)
     audit.should_not be_nil
     audit.auditable.should == model unless audit.action == 'destroy'

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: auditor
 version: !ruby/object:Gem::Version 
-  hash: 11
+  hash: 9
   prerelease: 
   segments: 
   - 2
   - 1
-  - 0
-  version: 2.1.0
+  - 1
+  version: 2.1.1
 platform: ruby
 authors: 
 - Jeff Kunkle
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-06-20 00:00:00 -04:00
+date: 2011-07-05 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency