From 7d9fe04b1d87b9f346649c9385a3c95121132653 Mon Sep 17 00:00:00 2001 From: David Heinemeier Hansson Date: Sun, 23 Sep 2007 21:56:52 +0000 Subject: Fixed cache_page to use the request url instead of the routing options when picking a save path (closes #8614) [josh] git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@7598 5ecf4fe2-1ee6-0310-87b1-e25e094e27de --- actionpack/lib/action_controller/caching.rb | 19 +++++++++++-------- 1 file changed, 11 insertions(+), 8 deletions(-) (limited to 'actionpack/lib') diff --git a/actionpack/lib/action_controller/caching.rb b/actionpack/lib/action_controller/caching.rb index 3e01ee12b2..e90e27c388 100644 --- a/actionpack/lib/action_controller/caching.rb +++ b/actionpack/lib/action_controller/caching.rb @@ -136,17 +136,20 @@ module ActionController #:nodoc: end # Manually cache the +content+ in the key determined by +options+. If no content is provided, the contents of response.body is used - # If no options are provided, the current +options+ for this action is used. Example: + # If no options are provided, the requested url is used. Example: # cache_page "I'm the cached content", :controller => "lists", :action => "show" - def cache_page(content = nil, options = {}) + def cache_page(content = nil, options = nil) return unless perform_caching && caching_allowed - - if options.is_a?(Hash) - path = url_for(options.merge(:only_path => true, :skip_relative_url_root => true, :format => params[:format])) - else - path = options + + path = case options + when Hash + url_for(options.merge(:only_path => true, :skip_relative_url_root => true, :format => params[:format])) + when String + options + else + request.path end - + self.class.cache_page(content || response.body, path) end -- cgit v1.2.3