diff options
author | Aaron Patterson <aaron.patterson@gmail.com> | 2012-05-02 15:33:20 -0700 |
---|---|---|
committer | Aaron Patterson <aaron.patterson@gmail.com> | 2012-05-02 15:33:20 -0700 |
commit | dc2352c58f0f05d9df171538483f31d42e9f0668 (patch) | |
tree | 54866bf9aad86dc882cb83b2d37a1f23491dca30 /actionpack | |
parent | 1ef527f9f96dd6cb9d6f0a93886a28838885513e (diff) | |
download | rails-dc2352c58f0f05d9df171538483f31d42e9f0668.tar.gz rails-dc2352c58f0f05d9df171538483f31d42e9f0668.tar.bz2 rails-dc2352c58f0f05d9df171538483f31d42e9f0668.zip |
initialize instance variables
Diffstat (limited to 'actionpack')
-rw-r--r-- | actionpack/lib/action_dispatch/middleware/session/abstract_store.rb | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/actionpack/lib/action_dispatch/middleware/session/abstract_store.rb b/actionpack/lib/action_dispatch/middleware/session/abstract_store.rb index 48f1006e5e..3710667238 100644 --- a/actionpack/lib/action_dispatch/middleware/session/abstract_store.rb +++ b/actionpack/lib/action_dispatch/middleware/session/abstract_store.rb @@ -124,6 +124,7 @@ module ActionDispatch @env = env @delegate = {} @loaded = false + @exists = nil # we haven't checked yet end def destroy @@ -181,7 +182,7 @@ module ActionDispatch end def exists? - return @exists if instance_variable_defined?(:@exists) + return @exists unless @exists.nil? @exists = @by.send(:session_exists?, @env) end @@ -205,18 +206,16 @@ module ActionDispatch end def load! - id, session = @by.send(:load_session, @env) + id, session = @by.load_session @env @env[ENV_SESSION_OPTIONS_KEY][:id] = id @delegate.replace(stringify_keys(session)) @loaded = true end def stringify_keys(other) - hash = {} - other.each do |key, value| + other.each_with_object({}) { |(key, value), hash| hash[key.to_s] = value - end - hash + } end end end |