From 7db29f900400e9f35142ede94148d3a71e651adb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Caina=CC=83=20Costa?= <cainan.costa@gmail.com>
Date: Mon, 30 Jul 2012 02:40:13 -0300
Subject: [PATCH] Refactor
 ActionDispatch::Http::Cache::Response#cache_control_headers.

---
 actionpack/lib/action_dispatch/http/cache.rb | 30 +++++++++++++-------
 1 file changed, 19 insertions(+), 11 deletions(-)

diff --git a/actionpack/lib/action_dispatch/http/cache.rb b/actionpack/lib/action_dispatch/http/cache.rb
index 0b895e7860..647a9e3c19 100644
--- a/actionpack/lib/action_dispatch/http/cache.rb
+++ b/actionpack/lib/action_dispatch/http/cache.rb
@@ -86,21 +86,29 @@ def etag=(etag)
         CACHE_CONTROL = "Cache-Control".freeze
         SPESHUL_KEYS  = %w[extras no-cache max-age public must-revalidate]
 
+        def cache_control_segments
+          if cache_control = self[CACHE_CONTROL]
+            cache_control.delete(' ').split(',')
+          else
+            []
+          end
+        end
+
         def cache_control_headers
           cache_control = {}
-          if cc = self[CACHE_CONTROL]
-            cc.delete(' ').split(',').each do |segment|
-              directive, argument = segment.split('=', 2)
-              case directive
-              when *SPESHUL_KEYS
-                key = directive.tr('-', '_')
-                cache_control[key.to_sym] = argument || true
-              else
-                cache_control[:extras] ||= []
-                cache_control[:extras] << segment
-              end
+
+          cache_control_segments.each do |segment|
+            directive, argument = segment.split('=', 2)
+
+            if SPESHUL_KEYS.include? directive
+              key = directive.tr('-', '_')
+              cache_control[key.to_sym] = argument || true
+            else
+              cache_control[:extras] ||= []
+              cache_control[:extras] << segment
             end
           end
+
           cache_control
         end
 
-- 
GitLab