From a712acc42543b575a623e1a8880c0e9dd5505cfa Mon Sep 17 00:00:00 2001
From: Santosh Wadghule <santosh.wadghule@gmail.com>
Date: Sat, 16 Apr 2016 16:16:17 +0530
Subject: Fix forced cache miss for fetch.

- Raised an argument error if no block is passed to #fetch with
'force: true' option is set.
- Added tests for the same.
---
 activesupport/lib/active_support/cache.rb | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

(limited to 'activesupport/lib/active_support/cache.rb')

diff --git a/activesupport/lib/active_support/cache.rb b/activesupport/lib/active_support/cache.rb
index 85f462a1b1..f27bac3dca 100644
--- a/activesupport/lib/active_support/cache.rb
+++ b/activesupport/lib/active_support/cache.rb
@@ -198,10 +198,16 @@ module ActiveSupport
       #   cache.fetch('city')   # => "Duckburgh"
       #
       # You may also specify additional options via the +options+ argument.
-      # Setting <tt>force: true</tt> will force a cache miss:
+      # Setting <tt>force: true</tt> will force a cache miss and return value of
+      # the block will be written to the cache under the given cache key.
       #
       #   cache.write('today', 'Monday')
-      #   cache.fetch('today', force: true)  # => nil
+      #   cache.fetch('today', force: true) { 'Tuesday' } # => 'Tuesday'
+      #
+      # It will raise an argument error if no block is passed to #fetch with
+      # option <tt>force: true</tt> is set.
+      #
+      #   cache.fetch('today', force: true) # => ArgumentError: Missing block
       #
       # Setting <tt>:compress</tt> will store a large cache entry set by the call
       # in a compressed format.
@@ -292,6 +298,8 @@ module ActiveSupport
           else
             save_block_result_to_cache(name, options) { |_name| yield _name }
           end
+        elsif options && options[:force]
+          raise ArgumentError, 'Missing block'
         else
           read(name, options)
         end
-- 
cgit v1.2.3