Skip to content

Commit

Permalink
Revert "Merge pull request #2647 from dmathieu/no_rescue"
Browse files Browse the repository at this point in the history
This reverts commit 125b1b0.
  • Loading branch information
spastorino committed Aug 30, 2011
1 parent 60d8e4a commit e5f2b0d
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
8 changes: 6 additions & 2 deletions actionpack/lib/sprockets/helpers/rails_helper.rb
Expand Up @@ -58,8 +58,12 @@ def asset_path(source, default_ext = nil, body = false, protocol = nil)

private
def debug_assets?
config = Rails.application.config.assets
config.allow_debugging && (config.debug || params[:debug_assets])
begin
config = Rails.application.config.assets
config.allow_debugging && (config.debug || params[:debug_assets])
rescue NoMethodError
false
end
end

# Override to specify an alternative prefix for asset path generation.
Expand Down
6 changes: 1 addition & 5 deletions actionpack/test/abstract_unit.rb
Expand Up @@ -142,11 +142,7 @@ def call(env)
end

class BasicController
attr_accessor :request, :params

def initialize
@params = {}
end
attr_accessor :request

def config
@config ||= ActiveSupport::InheritableOptions.new(ActionController::Base.config).tap do |config|
Expand Down

2 comments on commit e5f2b0d

@dmathieu
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm sorry I didn't follow this. Why the revert ?

@spastorino
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dmathieu because of #2177 dbca49b

Please sign in to comment.