[Jifty-commit] r2095 - in jifty/trunk: lib/Jifty/Web

[jifty-commit at lists.jifty.org]

Author: jesse
Date: Mon Oct 30 23:49:11 2006
New Revision: 2095

Modified:
   jifty/trunk/   (props changed)
   jifty/trunk/lib/Jifty/Web/Session.pm

Log:
 r29627 at pinglin:  jesse | 2006-10-30 23:42:19 -0500
 * minor refactoring to enable non-cookie based session sorage


Modified: jifty/trunk/lib/Jifty/Web/Session.pm
==============================================================================
--- jifty/trunk/lib/Jifty/Web/Session.pm	(original)
+++ jifty/trunk/lib/Jifty/Web/Session.pm	Mon Oct 30 23:49:11 2006
@@ -54,12 +54,7 @@
     my $self       = shift;
     my $session_id = shift;
 
-    unless ($session_id) {
-        my %cookies    = CGI::Cookie->fetch();
-        my $cookie_name = $self->cookie_name;
-        $session_id
-            = $cookies{$cookie_name} ? $cookies{$cookie_name}->value() : undef;
-    }
+   $session_id ||= $self->_get_session_id_from_client();
 
     my $session = Jifty::Model::Session->new;
     $session->load_by_cols(
@@ -73,6 +68,14 @@
     $self->{cache} = undef;
 }
 
+sub _get_session_id_from_client {
+        my $self = shift;
+        my %cookies    = CGI::Cookie->fetch();
+        my $cookie_name = $self->cookie_name;
+        my $session_id
+            = $cookies{$cookie_name} ? $cookies{$cookie_name}->value() : undef;
+}
+
 =head2 unload
 
 Flush the session, and leaves the session object blank.