#20 new
Gaius Centus Novus

State Transitions Return true but do Nothing

Reported by Gaius Centus Novus | June 22nd, 2009 @ 10:40 AM

I have a class:

class Item < ActiveRecord::Base
  include AASM
  aasm_initial_state :available
  aasm_state :available
  aasm_state :in_use
  aasm_event(:check_out) do
    transitions :from => :available, :to => :in_use
  end
  aasm_event(:check_in) do
    transitions :from => :in_use, :to => :available
  end
end

This works:

i = Item.create!
i.write_attribute(:state, 'in_use')
# => "in_use"
i.save!
# => true
i.state
# => "in_use"

But this doesn't do anything:

i = Item.create!
i.check_out!
# => true
i.state
# => "available"

I'm on AASM 2.0.5 and Rails 2.3.2. I tried applying the patch for a (seemingly) related bug at http://www.pastie.org/444480 but that did nothing.

Comments and changes to this ticket

  • Gaius Centus Novus

    Gaius Centus Novus June 22nd, 2009 @ 11:54 AM

    An oddity: after a state transition, read_attribute returns the correct value even though state doesn't:

    i = Item.create!
i.check_out!
i.state
# => 'available'

i.read_attribute(i.class.aasm_column)
# => 'in_use'
  • Gaius Centus Novus

    Gaius Centus Novus June 22nd, 2009 @ 11:59 AM

    The problem was that I was using alias_attribute instead of aasm_column:

    # wrong
class Item
  include AASM
  alias_attribute :aasm_state, :state
end

# right
class Item
  include AASM
  self.aasm_column :state
end

    AASM wrote to ":aasm_state," which I assumed would write through, but didn't.

    This ticket can be closed.

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

Library for adding state machines to Ruby classes. Includes persistence layers for things like ActiveRecord. Formerly known as acts_as_state_machine.

People watching this ticket

Pages