Permalink
Browse files

Add missing require for ActionView::AssetPaths

  • Loading branch information...
1 parent d74e1b1 commit 987eb7d4236513978e3cfcd5c3c159d95d2a84c8 @pixeltrix pixeltrix committed Jul 1, 2011
Showing with 1 addition and 0 deletions.
  1. +1 −0 actionpack/lib/sprockets/helpers/rails_helper.rb
@@ -1,3 +1,4 @@
+require "action_view/asset_paths"
require "action_view/helpers"
module Sprockets

5 comments on commit 987eb7d

Contributor

josevalim replied Jul 1, 2011

AV::AssetPaths is autoloaded. requiring a file that may have been autoloaded somewhere else can lead to double requires due to a ruby bug. :(

Owner

pixeltrix replied Jul 1, 2011

I was getting missing constant errors trying to run the sass-rails tests - I'll investigate and look at why they're not autoloading

Contributor

josevalim replied Jul 1, 2011

Maybe sass-rails is not doing: require "action_view"?

Owner

pixeltrix replied Jul 1, 2011

Should it? I don't think it so since Sass doesn't need it.

Contributor

josevalim replied Jul 1, 2011

Sorry. I meant to say sprockets-rails should require action view and not asset paths explicitly.

Please sign in to comment.