benzelano / test_lh_import

1 stars 0 forks source link

require qualified name for state_event=? #121

Closed benzelano closed 13 years ago

benzelano commented 13 years ago

Aaron Gibralter opened this issue

So it seems like the following code: @@@ state_machine :state_news, :initial => :unsubscribed, :namespace => :news do event :toggle do transition :unsubscribed => :subscribed transition :subscribed => :unsubscribed end end

state_machine :state_standing, :initial => :neutral, :namespace => :standing do event :toggle do transition :neutral => :suspended transition :suspended => :neutral end end @@@ requires the following: @@@ @user.state_news_event = ’toggle_news’ @user.state_standing_event = ’toggle_standing’ @@@ Should the following be allowed since the first way seems a bit redundant? @@@ @user.state_news_event = ’toggle’ @user.state_standing_event = ’toggle’ @@@

original LH ticket

This ticket has 0 attachment(s).

benzelano commented 13 years ago

(from [9207069dfcc8c990d53888c69fd269133c48be01]) Don’t use qualified name for event attributes [#21 state:resolved] http://github.com/pluginaweek/state_machine/commit/9207069dfcc8c990d53888c69fd269133c48be01

benzelano commented 13 years ago

Nice catch :) Thanks!