Skip to content
Browse files

Deprecate ActionView::Base.cache_template_extensions

This commit deprecates unused ActionView::Base.cache_template_extensions option
and adds a deprecation warning for those having it in env specific config files.

Thanks to José Valim for pointing this out.
  • Loading branch information...
1 parent 2183c22 commit 2463e38efd3cbcc10e7b0a93ad9c2d2224340668 @lifo lifo committed May 17, 2008
Showing with 5 additions and 5 deletions.
  1. +5 −5 actionpack/lib/action_view/base.rb
10 actionpack/lib/action_view/base.rb
@@ -168,12 +168,12 @@ class Base
# Specify whether file modification times should be checked to see if a template needs recompilation
@@cache_template_loading = false
cattr_accessor :cache_template_loading
- # Specify whether file extension lookup should be cached, and whether template base path lookup should be cached.
- # Should be +false+ for development environments. Defaults to +true+.
- @@cache_template_extensions = true
- cattr_accessor :cache_template_extensions
+ def self.cache_template_extensions=(*args)
+ ActiveSupport::Deprecation.warn("config.action_view.cache_template_extensions option has been deprecated and has no affect. " <<
+ "Please remove it from your config files.", caller)
+ end
# Specify whether RJS responses should be wrapped in a try/catch block
# that alert()s the caught exception (and then re-raises it).
@@debug_rjs = false

0 comments on commit 2463e38

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