Revert #9, go back to generating sequential ids on save (fixes #10)

derrickreimer · Oct 24, 2014 · c12726f · c12726f
1 parent 192a02d
commit c12726f
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 7 deletions.
diff --git a/lib/sequenced/acts_as_sequenced.rb b/lib/sequenced/acts_as_sequenced.rb
@@ -34,7 +34,7 @@ def acts_as_sequenced(options = {})
         cattr_accessor :sequenced_options
         self.sequenced_options = options
 
-        before_validation :set_sequential_id
+        before_save :set_sequential_id
         include Sequenced::ActsAsSequenced::InstanceMethods
       end
     end

diff --git a/test/acts_as_sequenced_test.rb b/test/acts_as_sequenced_test.rb
@@ -129,10 +129,4 @@ class ActsAsSequencedTest < ActiveSupport::TestCase
     assert_equal 1, zombie.sequential_id
     assert_equal 2, werewolf.sequential_id
   end
-
-  test "validation passing" do
-    obj = Policeman.create
-    assert_empty obj.errors[:sequential_id]
-    assert obj.valid?
-  end
 end