write_state_without_persistence confusion about transitions

[venetanji]

Totally agree! I'm using mongoid. Consider something like:

  before_save :expire_free_trial

  def expire_free_trial
    begin
      trial_expired if Time.now  - created_at > FREE_TRIAL_PERIOD
      self.state = self.current_state
    rescue Transitions::InvalidTransition
    end
  end

It took me quite a while to figure out that line that sets the state attribute. I'd expect that to be set when the non-bang event fires. Perhaps that works only for active record?
Anyways I'm ok with it, but it should be more clear. Specially if you want to call a transition in a before_save...

[troessner]

@robolson, @venetanji could you submit a pull request which updates our documentation?
If not, I'll update when I find the time.

[zmillman]

As someone who stumbled on the project and tried to use it with ActiveRecord, this issue caused me a lot of confusion.

I expected the behavior of calling an event like article.publish to update the article.state attribute and was surprised when only current_state changed. Perhaps it would be best to define write_state_without_persistence as @robolson wrote it in the ActiveRecord::Transitions plugin?

[troessner]

Closed thanks to @zmillman