From 0efeeba7802b3e5137cd62cbb6fcf303a8a238a2 Mon Sep 17 00:00:00 2001
From: Jamis Buck <jamis@37signals.com>
Date: Tue, 11 Oct 2005 01:53:31 +0000
Subject: Speed improvement for session_options. #2287. [skaes@web.de]

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@2527 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
---
 .../lib/action_controller/session_management.rb    | 47 +++++++++++++---------
 1 file changed, 29 insertions(+), 18 deletions(-)

(limited to 'actionpack/lib/action_controller/session_management.rb')

diff --git a/actionpack/lib/action_controller/session_management.rb b/actionpack/lib/action_controller/session_management.rb
index b0a457932d..6752d66e85 100644
--- a/actionpack/lib/action_controller/session_management.rb
+++ b/actionpack/lib/action_controller/session_management.rb
@@ -77,26 +77,36 @@ module ActionController #:nodoc:
         write_inheritable_array("session_options", [options])
       end
 
+      def cached_session_options
+        @session_options ||= read_inheritable_attribute("session_options") || []
+      end
+
       def session_options_for(request, action) #:nodoc:
-        options = {}
+        if (session_options = cached_session_options).empty?
+          {}
+        else
+          options = {}
 
-        action = action.to_s
-        (read_inheritable_attribute("session_options") || []).each do |opts|
-          next if opts[:if] && !opts[:if].call(request)
-          if opts[:only] && opts[:only].include?(action)
-            options.merge!(opts)
-          elsif opts[:except] && !opts[:except].include?(action)
-            options.merge!(opts)
-          elsif !opts[:only] && !opts[:except]
-            options.merge!(opts)
+          action = action.to_s
+          session_options.each do |opts|
+            next if opts[:if] && !opts[:if].call(request)
+            if opts[:only] && opts[:only].include?(action)
+              options.merge!(opts)
+            elsif opts[:except] && !opts[:except].include?(action)
+              options.merge!(opts)
+            elsif !opts[:only] && !opts[:except]
+              options.merge!(opts)
+            end
+          end
+          
+          if options.empty? then options
+          else
+            options.delete :only
+            options.delete :except
+            options.delete :if
+            options[:disabled] ? false : options
           end
         end
-
-        options.delete :only
-        options.delete :except
-        options.delete :if
-
-        options[:disabled] ? false : options
       end
     end
 
@@ -108,8 +118,9 @@ module ActionController #:nodoc:
   
     private
       def clear_persistant_model_associations #:doc:
-        session = @session.instance_variable_get("@data")
-        session.each { |key, obj| obj.clear_association_cache if obj.respond_to?(:clear_association_cache) } if session
+        if session = @session.instance_variable_get("@data")
+          session.each { |key, obj| obj.clear_association_cache if obj.respond_to?(:clear_association_cache) }
+        end
       end
   end
 end
-- 
cgit v1.2.3