rails_state_machine 2.2.0 → 3.0.0

data/Gemfile.7.0.pg.lock

@@ -1,75 +1,80 @@
 PATH
   remote: .
   specs:
-    rails_state_machine (2.2.0)
+    rails_state_machine (3.0.0)
       activerecord
 
 GEM
   remote: https://rubygems.org/
   specs:
-    actioncable (7.0.0)
-      actionpack (= 7.0.0)
-      activesupport (= 7.0.0)
+    actioncable (7.0.1)
+      actionpack (= 7.0.1)
+      activesupport (= 7.0.1)
       nio4r (~> 2.0)
       websocket-driver (>= 0.6.1)
-    actionmailbox (7.0.0)
-      actionpack (= 7.0.0)
-      activejob (= 7.0.0)
-      activerecord (= 7.0.0)
-      activestorage (= 7.0.0)
-      activesupport (= 7.0.0)
+    actionmailbox (7.0.1)
+      actionpack (= 7.0.1)
+      activejob (= 7.0.1)
+      activerecord (= 7.0.1)
+      activestorage (= 7.0.1)
+      activesupport (= 7.0.1)
       mail (>= 2.7.1)
-    actionmailer (7.0.0)
-      actionpack (= 7.0.0)
-      actionview (= 7.0.0)
-      activejob (= 7.0.0)
-      activesupport (= 7.0.0)
+      net-imap
+      net-pop
+      net-smtp
+    actionmailer (7.0.1)
+      actionpack (= 7.0.1)
+      actionview (= 7.0.1)
+      activejob (= 7.0.1)
+      activesupport (= 7.0.1)
       mail (~> 2.5, >= 2.5.4)
+      net-imap
+      net-pop
+      net-smtp
       rails-dom-testing (~> 2.0)
-    actionpack (7.0.0)
-      actionview (= 7.0.0)
-      activesupport (= 7.0.0)
+    actionpack (7.0.1)
+      actionview (= 7.0.1)
+      activesupport (= 7.0.1)
       rack (~> 2.0, >= 2.2.0)
       rack-test (>= 0.6.3)
       rails-dom-testing (~> 2.0)
       rails-html-sanitizer (~> 1.0, >= 1.2.0)
-    actiontext (7.0.0)
-      actionpack (= 7.0.0)
-      activerecord (= 7.0.0)
-      activestorage (= 7.0.0)
-      activesupport (= 7.0.0)
+    actiontext (7.0.1)
+      actionpack (= 7.0.1)
+      activerecord (= 7.0.1)
+      activestorage (= 7.0.1)
+      activesupport (= 7.0.1)
       globalid (>= 0.6.0)
       nokogiri (>= 1.8.5)
-    actionview (7.0.0)
-      activesupport (= 7.0.0)
+    actionview (7.0.1)
+      activesupport (= 7.0.1)
       builder (~> 3.1)
       erubi (~> 1.4)
       rails-dom-testing (~> 2.0)
       rails-html-sanitizer (~> 1.1, >= 1.2.0)
-    activejob (7.0.0)
-      activesupport (= 7.0.0)
+    activejob (7.0.1)
+      activesupport (= 7.0.1)
       globalid (>= 0.3.6)
-    activemodel (7.0.0)
-      activesupport (= 7.0.0)
-    activerecord (7.0.0)
-      activemodel (= 7.0.0)
-      activesupport (= 7.0.0)
-    activestorage (7.0.0)
-      actionpack (= 7.0.0)
-      activejob (= 7.0.0)
-      activerecord (= 7.0.0)
-      activesupport (= 7.0.0)
+    activemodel (7.0.1)
+      activesupport (= 7.0.1)
+    activerecord (7.0.1)
+      activemodel (= 7.0.1)
+      activesupport (= 7.0.1)
+    activestorage (7.0.1)
+      actionpack (= 7.0.1)
+      activejob (= 7.0.1)
+      activerecord (= 7.0.1)
+      activesupport (= 7.0.1)
       marcel (~> 1.0)
       mini_mime (>= 1.1.0)
-    activesupport (7.0.0)
+    activesupport (7.0.1)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 1.6, < 2)
       minitest (>= 5.1)
       tzinfo (~> 2.0)
     builder (3.2.4)
-    byebug (11.1.3)
     coderay (1.1.3)
-    concurrent-ruby (1.1.9)
+    concurrent-ruby (1.2.2)
     crass (1.0.6)
     database_cleaner (2.0.1)
       database_cleaner-active_record (~> 2.0.0)
@@ -77,62 +82,74 @@ GEM
       activerecord (>= 5.a)
       database_cleaner-core (~> 2.0.0)
     database_cleaner-core (2.0.1)
+    date (3.3.4)
     diff-lcs (1.4.4)
-    erubi (1.10.0)
-    gemika (0.6.1)
-    globalid (1.0.0)
-      activesupport (>= 5.0)
-    i18n (1.8.11)
+    erubi (1.12.0)
+    gemika (0.8.2)
+    globalid (1.2.1)
+      activesupport (>= 6.1)
+    i18n (1.14.1)
       concurrent-ruby (~> 1.0)
-    loofah (2.13.0)
+    loofah (2.22.0)
       crass (~> 1.0.2)
-      nokogiri (>= 1.5.9)
-    mail (2.7.1)
+      nokogiri (>= 1.12.0)
+    mail (2.8.1)
       mini_mime (>= 0.1.1)
+      net-imap
+      net-pop
+      net-smtp
     marcel (1.0.2)
     method_source (1.0.0)
-    mini_mime (1.1.2)
-    mini_portile2 (2.6.1)
-    minitest (5.15.0)
-    nio4r (2.5.8)
-    nokogiri (1.12.5)
-      mini_portile2 (~> 2.6.1)
+    mini_mime (1.1.5)
+    mini_portile2 (2.8.5)
+    minitest (5.20.0)
+    net-imap (0.3.7)
+      date
+      net-protocol
+    net-pop (0.1.2)
+      net-protocol
+    net-protocol (0.2.2)
+      timeout
+    net-smtp (0.4.0)
+      net-protocol
+    nio4r (2.7.0)
+    nokogiri (1.15.5)
+      mini_portile2 (~> 2.8.2)
       racc (~> 1.4)
-    nokogiri (1.12.5-x86_64-linux)
+    nokogiri (1.15.5-x86_64-linux)
       racc (~> 1.4)
-    pg (1.2.3)
-    pry (0.13.1)
+    pg (1.5.4)
+    pry (0.14.2)
       coderay (~> 1.1)
       method_source (~> 1.0)
-    pry-byebug (3.9.0)
-      byebug (~> 11.0)
-      pry (~> 0.13.0)
-    racc (1.6.0)
-    rack (2.2.3)
-    rack-test (1.1.0)
-      rack (>= 1.0, < 3)
-    rails (7.0.0)
-      actioncable (= 7.0.0)
-      actionmailbox (= 7.0.0)
-      actionmailer (= 7.0.0)
-      actionpack (= 7.0.0)
-      actiontext (= 7.0.0)
-      actionview (= 7.0.0)
-      activejob (= 7.0.0)
-      activemodel (= 7.0.0)
-      activerecord (= 7.0.0)
-      activestorage (= 7.0.0)
-      activesupport (= 7.0.0)
+    racc (1.7.3)
+    rack (2.2.8)
+    rack-test (2.1.0)
+      rack (>= 1.3)
+    rails (7.0.1)
+      actioncable (= 7.0.1)
+      actionmailbox (= 7.0.1)
+      actionmailer (= 7.0.1)
+      actionpack (= 7.0.1)
+      actiontext (= 7.0.1)
+      actionview (= 7.0.1)
+      activejob (= 7.0.1)
+      activemodel (= 7.0.1)
+      activerecord (= 7.0.1)
+      activestorage (= 7.0.1)
+      activesupport (= 7.0.1)
       bundler (>= 1.15.0)
-      railties (= 7.0.0)
-    rails-dom-testing (2.0.3)
-      activesupport (>= 4.2.0)
+      railties (= 7.0.1)
+    rails-dom-testing (2.2.0)
+      activesupport (>= 5.0.0)
+      minitest
       nokogiri (>= 1.6)
-    rails-html-sanitizer (1.4.2)
-      loofah (~> 2.3)
-    railties (7.0.0)
-      actionpack (= 7.0.0)
-      activesupport (= 7.0.0)
+    rails-html-sanitizer (1.6.0)
+      loofah (~> 2.21)
+      nokogiri (~> 1.14)
+    railties (7.0.1)
+      actionpack (= 7.0.1)
+      activesupport (= 7.0.1)
       method_source
       rake (>= 12.2)
       thor (~> 1.0)
@@ -151,26 +168,28 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.10.0)
     rspec-support (3.10.3)
-    thor (1.1.0)
-    tzinfo (2.0.4)
+    thor (1.3.0)
+    timeout (0.4.1)
+    tzinfo (2.0.6)
       concurrent-ruby (~> 1.0)
-    websocket-driver (0.7.5)
+    websocket-driver (0.7.6)
       websocket-extensions (>= 0.1.0)
     websocket-extensions (0.1.5)
-    zeitwerk (2.5.1)
+    zeitwerk (2.6.12)
 
 PLATFORMS
   ruby
   x86_64-linux
 
 DEPENDENCIES
+  activerecord (= 7.0.1)
   database_cleaner
-  gemika
-  pg
-  pry-byebug
-  rails (~> 7.0.0)
+  gemika (>= 0.8.1)
+  pg (> 1.2.3)
+  pry
+  rails (= 7.0.1)
   rails_state_machine!
-  rake
+  rake (> 10.0)
   rspec (~> 3.5)
 
 BUNDLED WITH

data/Gemfile.lock

@@ -1 +1 @@
-./Gemfile.5.1.pg.lock
+Gemfile.7.0.pg.lock

data/README.md

@@ -56,8 +56,8 @@ A model instance offers these state machine methods:
 - `<event_name>` call an event and transition into a new state. The record will be `save`d, if valid.
 - `<event_name>!` call an event and transition into a new state. Calls `save!` to save the record.
 - `may_<event_name>?` to find out if an event transition could be taken. Note that this will not validate if the model is valid afterwards.
-- `state_event=` to take a state event, but not save yet. Commonly used for forms where the controller takes a "state_event" param and saves.
-- `state_event` to get the name of the event that will be called
+- `<state_name>_event=` to set the event the record tries to call on save. If the event is invalid, an `:invalid` error on the `<state_name>_event` is set. Commonly used for forms where the controller takes a "<state_name>_event" param and saves.
+- `<state_name>_event` to get the name of the event that will be called
 
 Should you ever need to query the state machine for its states or events, it is accessible via `state_machine` class or instance methods on the model. See [`state_machine.rb`](https://github.com/makandra/rails_state_machine/blob/master/lib/rails_state_machine/state_machine.rb) for a list of available methods. This is mostly helpful in tests.
 
@@ -127,6 +127,11 @@ state_machine :review_state, prefix: 'some_prefix' do
 end
 ```
 
+Note: The `prefix` option is designed to apply only to constants and state methods **defined by the gem**.
+
+**State event names** are manually defined by the developer and thus not altered even if the `prefix` 
+option is used. It's advised to review them regarding potential naming collision and clarity when
+ introducing the `prefix` method.
 
 ## Taking multiple transitions
 

data/bin/console

@@ -2,7 +2,7 @@
 
 require 'bundler/setup'
 require 'rails_state_machine'
-require 'pry-byebug'
+require 'pry'
 
 # You can add fixtures and/or initialization code here to make experimenting
 # with your gem easier. You can also use a different console, if you like.

data/lib/rails_state_machine/callbacks.rb

@@ -10,6 +10,7 @@ module RailsStateMachine
 
       def register_callbacks(model)
         model.class_eval do
+          before_validation :set_state_from_state_event, prepend: true
           before_validation :run_state_events_before_validation
           before_save :register_state_events_for_callbacks
           before_save { flush_state_event_callbacks(:before_save) }
@@ -25,6 +26,16 @@ module RailsStateMachine
       end
     end
 
+    def set_state_from_state_event
+      state_machine_state_managers.each do |state_manager|
+        next if state_manager.state_event.blank?
+
+        unless state_manager.transition_to(state_manager.state_event)
+          errors.add(:"#{state_manager.state_attribute}_event", :invalid)
+        end
+      end
+    end
+
     def run_state_events_before_validation
       # Since validations may be skipped, we will not register validation callbacks in @state_event_callbacks,
       # but call them explicitly when before_validation callbacks are triggered.

data/lib/rails_state_machine/state_machine.rb

@@ -26,7 +26,7 @@ module RailsStateMachine
       register_initial_state
 
       define_event_methods
-      define_model_methods
+      define_attributes
     end
 
     def states
@@ -46,7 +46,7 @@ module RailsStateMachine
     end
 
     def find_event(name)
-      @events_by_name.fetch(name.to_sym)
+      @events_by_name[name.to_sym]
     end
 
     def has_state?(name)
@@ -148,17 +148,11 @@ module RailsStateMachine
       end
     end
 
-    def define_model_methods
+    def define_attributes
       state_attribute = @state_attribute
 
-      model_module_eval do
-        define_method :"#{state_attribute}_event=" do |event_name|
-          state_machine_state_manager(state_attribute).transition_to(event_name)
-        end
-
-        define_method :"#{state_attribute}_event" do
-          state_machine_state_manager(state_attribute).next_event&.name
-        end
+      @model.instance_eval do
+        attribute :"#{state_attribute}_event", :string
       end
     end
   end

data/lib/rails_state_machine/state_manager.rb

@@ -1,6 +1,7 @@
 module RailsStateMachine
   class StateManager
     attr_accessor :next_event, :state_before_state_event
+    attr_reader :state_attribute
 
     def initialize(record, state_machine, state_attribute)
       @record = record
@@ -21,7 +22,10 @@ module RailsStateMachine
     end
 
     def revert
-      self.state = @state_before_state_event if @next_event
+      if @next_event
+        self.state = @state_before_state_event
+        self.state_event = @next_event.name
+      end
     end
 
     def source_state
@@ -32,16 +36,30 @@ module RailsStateMachine
       end
     end
 
-    def transition_to(event_name)
-      @next_event = @state_machine.find_event(event_name)
-      @state_before_state_event = source_state
+    def state_event
+      @record.public_send(:"#{@state_machine.state_attribute}_event")
+    end
 
-      # If the event can not transition from source_state, a TransitionNotFoundError will be raised
-      self.state = @next_event.future_state_name(source_state).to_s
+    def state_event=(value)
+      @record.public_send(:"#{@state_machine.state_attribute}_event=", value)
     end
 
     def transition_allowed_for?(event_name)
-      @state_machine.find_event(event_name).allowed_from?(source_state)
+      !!@state_machine.find_event(event_name)&.allowed_from?(state)
+    end
+
+    def transition_to(event_name)
+      if transition_allowed_for?(event_name)
+        self.state_before_state_event = source_state
+        event = @state_machine.find_event(event_name)
+        self.state = event.future_state_name(state).to_s
+        self.state_event = nil
+        @next_event = event
+
+        true
+      else
+        false
+      end
     end
   end
 end

data/lib/rails_state_machine/version.rb

@@ -1,3 +1,3 @@
 module RailsStateMachine
-  VERSION = '2.2.0'
+  VERSION = '3.0.0'
 end

data/rails_state_machine.gemspec

@@ -26,5 +26,4 @@ Gem::Specification.new do |spec|
 
   spec.add_development_dependency 'bundler', '~> 1.16'
   spec.add_development_dependency 'rake', '~> 10.0'
-  spec.add_development_dependency 'pry-byebug', '~> 3.5'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails_state_machine
 version: !ruby/object:Gem::Version
-  version: 2.2.0
+  version: 3.0.0
 platform: ruby
 authors:
 - Arne Hartherz
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-12-07 00:00:00.000000000 Z
+date: 2024-06-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -53,20 +53,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '10.0'
-- !ruby/object:Gem::Dependency
-  name: pry-byebug
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.5'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.5'
 description: ActiveRecord-bound state machine
 email:
 - arne.hartherz@makandra.de
@@ -80,8 +66,6 @@ files:
 - ".ruby-version"
 - CHANGELOG.md
 - Gemfile
-- Gemfile.5.1.pg
-- Gemfile.5.1.pg.lock
 - Gemfile.5.2.pg
 - Gemfile.5.2.pg.lock
 - Gemfile.6.0.pg
@@ -126,7 +110,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
+rubygems_version: 3.4.14
 signing_key: 
 specification_version: 4
 summary: ActiveRecord-bound state machine

data/Gemfile.5.1.pg

@@ -1,15 +0,0 @@
-source 'https://rubygems.org'
-
-# Runtime dependencies
-gem 'rails', '~>5.1.0'
-gem 'pg'
-
-# Development dependencies
-gem 'rspec', '~>3.5'
-gem 'rake'
-gem 'pry-byebug'
-gem 'gemika'
-gem 'database_cleaner'
-
-# Gem under test
-gem 'rails_state_machine', :path => '.'

data/Gemfile.5.1.pg.lock

@@ -1,149 +0,0 @@
-PATH
-  remote: .
-  specs:
-    rails_state_machine (2.2.0)
-      activerecord
-
-GEM
-  remote: https://rubygems.org/
-  specs:
-    actioncable (5.1.4)
-      actionpack (= 5.1.4)
-      nio4r (~> 2.0)
-      websocket-driver (~> 0.6.1)
-    actionmailer (5.1.4)
-      actionpack (= 5.1.4)
-      actionview (= 5.1.4)
-      activejob (= 5.1.4)
-      mail (~> 2.5, >= 2.5.4)
-      rails-dom-testing (~> 2.0)
-    actionpack (5.1.4)
-      actionview (= 5.1.4)
-      activesupport (= 5.1.4)
-      rack (~> 2.0)
-      rack-test (>= 0.6.3)
-      rails-dom-testing (~> 2.0)
-      rails-html-sanitizer (~> 1.0, >= 1.0.2)
-    actionview (5.1.4)
-      activesupport (= 5.1.4)
-      builder (~> 3.1)
-      erubi (~> 1.4)
-      rails-dom-testing (~> 2.0)
-      rails-html-sanitizer (~> 1.0, >= 1.0.3)
-    activejob (5.1.4)
-      activesupport (= 5.1.4)
-      globalid (>= 0.3.6)
-    activemodel (5.1.4)
-      activesupport (= 5.1.4)
-    activerecord (5.1.4)
-      activemodel (= 5.1.4)
-      activesupport (= 5.1.4)
-      arel (~> 8.0)
-    activesupport (5.1.4)
-      concurrent-ruby (~> 1.0, >= 1.0.2)
-      i18n (~> 0.7)
-      minitest (~> 5.1)
-      tzinfo (~> 1.1)
-    arel (8.0.0)
-    builder (3.2.3)
-    byebug (9.1.0)
-    coderay (1.1.2)
-    concurrent-ruby (1.0.5)
-    crass (1.0.3)
-    database_cleaner (1.6.2)
-    diff-lcs (1.3)
-    erubi (1.7.0)
-    gemika (0.5.0)
-    globalid (0.4.1)
-      activesupport (>= 4.2.0)
-    i18n (0.9.1)
-      concurrent-ruby (~> 1.0)
-    loofah (2.1.1)
-      crass (~> 1.0.2)
-      nokogiri (>= 1.5.9)
-    mail (2.7.0)
-      mini_mime (>= 0.1.1)
-    method_source (0.9.0)
-    mini_mime (1.0.0)
-    mini_portile2 (2.3.0)
-    minitest (5.10.3)
-    nio4r (2.1.0)
-    nokogiri (1.8.1)
-      mini_portile2 (~> 2.3.0)
-    pg (0.21.0)
-    pry (0.11.3)
-      coderay (~> 1.1.0)
-      method_source (~> 0.9.0)
-    pry-byebug (3.5.0)
-      byebug (~> 9.1)
-      pry (~> 0.10)
-    rack (2.0.3)
-    rack-test (0.7.0)
-      rack (>= 1.0, < 3)
-    rails (5.1.4)
-      actioncable (= 5.1.4)
-      actionmailer (= 5.1.4)
-      actionpack (= 5.1.4)
-      actionview (= 5.1.4)
-      activejob (= 5.1.4)
-      activemodel (= 5.1.4)
-      activerecord (= 5.1.4)
-      activesupport (= 5.1.4)
-      bundler (>= 1.3.0)
-      railties (= 5.1.4)
-      sprockets-rails (>= 2.0.0)
-    rails-dom-testing (2.0.3)
-      activesupport (>= 4.2.0)
-      nokogiri (>= 1.6)
-    rails-html-sanitizer (1.0.3)
-      loofah (~> 2.0)
-    railties (5.1.4)
-      actionpack (= 5.1.4)
-      activesupport (= 5.1.4)
-      method_source
-      rake (>= 0.8.7)
-      thor (>= 0.18.1, < 2.0)
-    rake (12.2.1)
-    rspec (3.7.0)
-      rspec-core (~> 3.7.0)
-      rspec-expectations (~> 3.7.0)
-      rspec-mocks (~> 3.7.0)
-    rspec-core (3.7.0)
-      rspec-support (~> 3.7.0)
-    rspec-expectations (3.7.0)
-      diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.7.0)
-    rspec-mocks (3.7.0)
-      diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.7.0)
-    rspec-support (3.7.0)
-    sprockets (3.7.1)
-      concurrent-ruby (~> 1.0)
-      rack (> 1, < 3)
-    sprockets-rails (3.2.1)
-      actionpack (>= 4.0)
-      activesupport (>= 4.0)
-      sprockets (>= 3.0.0)
-    thor (0.20.0)
-    thread_safe (0.3.6)
-    tzinfo (1.2.4)
-      thread_safe (~> 0.1)
-    websocket-driver (0.6.5)
-      websocket-extensions (>= 0.1.0)
-    websocket-extensions (0.1.3)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  database_cleaner
-  gemika
-  pg
-  pry-byebug
-  rails (~> 5.1.0)
-  rails_state_machine!
-  rake
-  rspec (~> 3.5)
-
-BUNDLED WITH
-   2.3.0