state-fu 0.13.3 → 0.13.4

data/lib/persistence/active_record.rb

@@ -12,6 +12,19 @@ module StateFu
                 
         # it's usually a good idea to do this:
         # validates_presence_of _field_name
+                
+        klass.class_eval do
+          # this is a hack to ensure that when you use the same field for the database
+          # column and the machine name, you don't end up with an unserializable 
+          # StateFu::Binding in record#changes() ...
+          def attribute_change(column)
+            change = super
+            if self.class.respond_to?(:machines) && self.class.machines.keys.include?(column.to_sym) 
+              change[1] = read_attribute(column)
+            end
+            change
+          end                
+        end
       end
 
       private

data/spec/integration/active_record_persistence_spec.rb

@@ -26,6 +26,26 @@ describe "an ActiveRecord model with StateFu included:" do
     # end class ExampleRecord
   end
 
+  describe 'a machine with the same name as the database field' do
+    before do
+      make_pristine_class( 'IdentityCrisis', ActiveRecord::Base )
+      IdentityCrisis.class_eval do
+        set_table_name 'example_records'
+        validates_presence_of :name
+        machine(:state_fu_field, :field_name => :state_fu_field) do
+          connect :a, :b, :c
+        end
+      end      
+    end
+    
+    it 'should not include a StateFu::Binding in record#changes' do
+      record = IdentityCrisis.new :name => 'coexisting field and machine name'
+      record.save!
+      record.state_fu_field.next!      
+      record.changes['state_fu_field'].last.class.should == String
+    end
+  end
+
   it "should be a subclass of ActiveRecord::Base" do
     ExampleRecord.superclass.should == ActiveRecord::Base
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: state-fu
 version: !ruby/object:Gem::Version 
-  version: 0.13.3
+  version: 0.13.4
 platform: ruby
 authors: 
 - David Lee