From c4bc360e4edcdf8b05e334582d4ab5bc41ed2abc Mon Sep 17 00:00:00 2001 From: Michiel Sikkes Date: Mon, 28 Jan 2013 21:17:48 +0100 Subject: [PATCH] Remove caching_allowed? from ActionController::Caching Where is this used? No other code references to this method and it isn't being tested anywhere. No tests fail when commented out. --- actionpack/lib/action_controller/caching.rb | 4 ---- 1 file changed, 4 deletions(-) diff --git a/actionpack/lib/action_controller/caching.rb b/actionpack/lib/action_controller/caching.rb index cf2cda039d..ea33d975ef 100644 --- a/actionpack/lib/action_controller/caching.rb +++ b/actionpack/lib/action_controller/caching.rb @@ -82,10 +82,6 @@ def view_cache_dependency(&dependency) end end - def caching_allowed? - request.get? && response.status == 200 - end - def view_cache_dependencies self.class._view_cache_dependencies.map { |dep| instance_exec(&dep) }.compact end