bugfix: initialize the aasm state column after initialization of the …

…ActiveRecord instance only if the attribute has been loaded #193

CHANGELOG.md

@@ -5,6 +5,10 @@
  * `aasm_column` has been removed. Use `aasm.attribute_name` instead
  * `aasm_human_event_name` has been removed. Use `aasm.human_event_name` instead
 
+## 4.0.5
+
+ * bugfix: initialize the aasm state column after initialization of the _ActiveRecord_ instance only if the attribute has been loaded (see [issue #193](https://github.com/aasm/aasm/issues/193) for details)
+
 ## 4.0.4
 
  * corrected callback order in README

aasm.gemspec

@@ -19,7 +19,7 @@ Gem::Specification.new do |s|
 
   # debugging
   # s.add_development_dependency 'debugger'
-  # s.add_development_dependency 'pry'
+  s.add_development_dependency 'pry'
 
   # test coverage
   # s.add_development_dependency 'mime-types', '~> 1.25' # needed by coveralls (>= 2.0 needs Ruby >=1.9.2)

lib/aasm/persistence/active_record_persistence.rb

@@ -164,7 +164,9 @@ def aasm_raw_attribute_value(state)
         #   foo.aasm_state # => nil
         #
         def aasm_ensure_initial_state
-          aasm.enter_initial_state if send(self.class.aasm.attribute_name).blank?
+          if respond_to?(self.class.aasm.attribute_name) && send(self.class.aasm.attribute_name).blank?
+            aasm.enter_initial_state
+          end
         end
 
         def aasm_fire_event(name, options, *args, &block)

spec/unit/persistence/active_record_persistence_spec.rb

@@ -252,9 +252,20 @@
     expect(gate.aasm.current_state).to be_nil
   end
 
-  it "should initialize the aasm state on instantiation" do
-    expect(Gate.new.aasm_state).to eql 'opened'
-    expect(Gate.new.aasm.current_state).to eql :opened
+  context 'on initialization' do
+    it "should initialize the aasm state" do
+      expect(Gate.new.aasm_state).to eql 'opened'
+      expect(Gate.new.aasm.current_state).to eql :opened
+    end
+
+    it "should not initialize the aasm state if it has not been loaded" do
+      # we have to create a gate in the database, for which we only want to
+      # load the id, and not the state
+      gate = Gate.create!
+
+      # then we just load the gate ids
+      Gate.select(:id).where(id: gate.id).first
+    end
   end
 
 end