Skip to content
Browse files

Add ActiveModel requirement to application.rb

Currently, ActiveModel is only loaded by ActiveRecord. If you skip ActiveRecord, ActiveModel will not be required (or even autoloaded) and including `ActiveModel::Model` into a plain Ruby class will raise `NameError`.

To reproduce this:

- create a new app with `rails new my_app -O`
- create a Ruby class that includes `ActiveModel::Model` in `app/models`
- load up a Rails console and try to do anything with the class :-)

Since ActionPack relies so heavily on the ActiveModel API, this should probably be considered a dependency of the app. Another possibility would be to make it a dependency of ActionController.
  • Loading branch information...
1 parent d6d6376 commit ab727743bf8a7e27ab1e1cc6097a07e5792aa7da @jgaskins jgaskins committed May 22, 2013
Showing with 1 addition and 0 deletions.
  1. +1 −0 railties/lib/rails/generators/rails/app/templates/config/application.rb
View
1 railties/lib/rails/generators/rails/app/templates/config/application.rb
@@ -4,6 +4,7 @@
require 'rails/all'
<% else -%>
# Pick the frameworks you want:
+require "active_model/railtie"
<%= comment_if :skip_active_record %>require "active_record/railtie"
require "action_controller/railtie"
require "action_mailer/railtie"

0 comments on commit ab72774

Please sign in to comment.
Something went wrong with that request. Please try again.