0
@@ -1,5 +1,9 @@
0
 *Edge*
0
 
0
+* Allow caches_action to accept cache store options. #416. [José Valim]. Example:
0
+  
0
+  caches_action :index, :redirected, :if => Proc.new { |c| !c.request.format.json? }, :expires_in => 1.hour
0
+
0
 * Remove define_javascript_functions, javascript_include_tag and friends are far superior. [Michael Koziarski]
0
 
0
 * Deprecate :use_full_path render option. The supplying the option no longer has an effect [Josh Peek]

0
@@ -27,13 +27,15 @@ module ActionController #:nodoc:
0
     # You can set modify the default action cache path by passing a :cache_path option.  This will be passed directly to ActionCachePath.path_for.  This is handy
0
     # for actions with multiple possible routes that should be cached differently.  If a block is given, it is called with the current controller instance.
0
     #
0
-    # And you can also use :if to pass a Proc that specifies when the action should be cached.
0
+    # And you can also use :if (or :unless) to pass a Proc that specifies when the action should be cached.
0
+    #
0
+    # Finally, if you are using memcached, you can also pass :expires_in.
0
     #
0
     #   class ListsController < ApplicationController
0
     #     before_filter :authenticate, :except => :public
0
     #     caches_page   :public
0
     #     caches_action :index, :if => Proc.new { |c| !c.request.format.json? } # cache if is not a JSON request
0
-    #     caches_action :show, :cache_path => { :project => 1 }
0
+    #     caches_action :show, :cache_path => { :project => 1 }, :expires_in => 1.hour
0
     #     caches_action :feed, :cache_path => Proc.new { |controller|
0
     #       controller.params[:user_id] ?
0
     #         controller.send(:user_list_url, c.params[:user_id], c.params[:id]) :
0
@@ -56,8 +58,10 @@ module ActionController #:nodoc:
0
         def caches_action(*actions)
0
           return unless cache_configured?
0
           options = actions.extract_options!
0
-          cache_filter = ActionCacheFilter.new(:layout => options.delete(:layout), :cache_path => options.delete(:cache_path))
0
-          around_filter(cache_filter, {:only => actions}.merge(options))
0
+          filter_options = { :only => actions, :if => options.delete(:if), :unless => options.delete(:unless) }
0
+
0
+          cache_filter = ActionCacheFilter.new(:layout => options.delete(:layout), :cache_path => options.delete(:cache_path), :store_options => options)
0
+          around_filter(cache_filter, filter_options)
0
         end
0
       end
0
 
0
@@ -80,8 +84,8 @@ module ActionController #:nodoc:
0
         end
0
 
0
         def before(controller)
0
-          cache_path = ActionCachePath.new(controller, path_options_for(controller, @options))
0
-          if cache = controller.read_fragment(cache_path.path)
0
+          cache_path = ActionCachePath.new(controller, path_options_for(controller, @options.slice(:cache_path)))
0
+          if cache = controller.read_fragment(cache_path.path, @options[:store_options])
0
             controller.rendered_action_cache = true
0
             set_content_type!(controller, cache_path.extension)
0
             options = { :text => cache }
0
@@ -96,7 +100,7 @@ module ActionController #:nodoc:
0
         def after(controller)
0
           return if controller.rendered_action_cache || !caching_allowed(controller)
0
           action_content = cache_layout? ? content_for_layout(controller) : controller.response.body
0
-          controller.write_fragment(controller.action_cache_path.path, action_content)
0
+          controller.write_fragment(controller.action_cache_path.path, action_content, @options[:store_options])
0
         end
0
 
0
         private

0
@@ -152,7 +152,7 @@ end
0
 
0
 
0
 class ActionCachingTestController < ActionController::Base
0
-  caches_action :index, :redirected, :forbidden, :if => Proc.new { |c| !c.request.format.json? }
0
+  caches_action :index, :redirected, :forbidden, :if => Proc.new { |c| !c.request.format.json? }, :expires_in => 1.hour
0
   caches_action :show, :cache_path => 'http://test.host/custom/show'
0
   caches_action :edit, :cache_path => Proc.new { |c| c.params[:id] ? "http://test.host/#{c.params[:id]};edit" : "http://test.host/edit" }
0
   caches_action :with_layout
0
@@ -188,6 +188,7 @@ class ActionCachingTestController < ActionController::Base
0
     expire_action :controller => 'action_caching_test', :action => 'index'
0
     render :nothing => true
0
   end
0
+
0
   def expire_xml
0
     expire_action :controller => 'action_caching_test', :action => 'index', :format => 'xml'
0
     render :nothing => true
0
@@ -289,6 +290,13 @@ class ActionCacheTest < Test::Unit::TestCase
0
     assert !fragment_exist?('hostname.com/action_caching_test')
0
   end
0
 
0
+  def test_action_cache_with_store_options
0
+    MockTime.expects(:now).returns(12345).once
0
+    @controller.expects(:read_fragment).with('hostname.com/action_caching_test', :expires_in => 1.hour).once
0
+    @controller.expects(:write_fragment).with('hostname.com/action_caching_test', '12345.0', :expires_in => 1.hour).once
0
+    get :index
0
+  end
0
+
0
   def test_action_cache_with_custom_cache_path
0
     get :show
0
     cached_time = content_to_cache