diff --git a/lib/acts_as_versioned.rb b/lib/acts_as_versioned.rb index 4be13aa..baf7d9e 100644 --- a/lib/acts_as_versioned.rb +++ b/lib/acts_as_versioned.rb @@ -355,7 +355,7 @@ def versioned_attributes end def altered? - self.track_altered_attributes ? (self.version_if_changed.map(&:to_s) - changed).length > 0 : changed? + self.track_altered_attributes ? (self.version_if_changed.map(&:to_s) - self.changed).length == 0 : changed? end # Clones a model. Used when saving a new version or reverting a model's version. diff --git a/test/versioned_test.rb b/test/versioned_test.rb index a7bc208..554e06a 100644 --- a/test/versioned_test.rb +++ b/test/versioned_test.rb @@ -184,10 +184,11 @@ def test_version_if_condition_with_block def test_version_no_limit p = Page.create! :title => "title", :body => 'first body' - p.save - p.save 5.times do |i| - assert_page_title p, i + p.title = "title#{i}" + p.save + assert_equal "title#{i}", p.title + assert_equal i+2, p.version end end @@ -196,7 +197,10 @@ def test_version_max_limit p.update_attributes(:title => "title1") p.update_attributes(:title => "title2") 5.times do |i| - assert_page_title p, i, :lock_version + p.title = "title#{i}" + p.save + assert_equal "title#{i}", p.title + assert_equal (i+4), p.send(:lock_version) assert p.versions(true).size <= 2, "locked version can only store 2 versions" end end @@ -236,13 +240,6 @@ def test_track_altered_attributes assert_equal 2, p.versions(true).size # version 1 deleted end - def assert_page_title(p, i, version_field = :version) - p.title = "title#{i}" - p.save - assert_equal "title#{i}", p.title - assert_equal (i+4), p.send(version_field) - end - def test_find_versions assert_equal 2, locked_pages(:welcome).versions.size assert_equal 1, locked_pages(:welcome).versions.find(:all, :conditions => ['title LIKE ?', '%weblog%']).length