From: Jay Kuri <jayk@cpan.org>
Date: Sun, 22 Jul 2007 03:22:06 +0000 (+0000)
Subject: $user->store() should NOT be set by C::P::Auth - if it's needed - it
X-Git-Tag: v0.10009_01~55
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=daed2d141fbe18a85c4c9d040cfa9bb03138988e;hp=57751a69aa035a1c3937b54229822aa7c61a70b0;p=catagits%2FCatalyst-Plugin-Authentication.git

$user->store() should NOT be set by C::P::Auth - if it's needed - it
should be set by whatever module creates the user.  We use realm for
saving into the session.  that's the important bit
---

diff --git a/lib/Catalyst/Plugin/Authentication.pm b/lib/Catalyst/Plugin/Authentication.pm
index 6c69a90..2b81662 100644
--- a/lib/Catalyst/Plugin/Authentication.pm
+++ b/lib/Catalyst/Plugin/Authentication.pm
@@ -40,7 +40,6 @@ sub set_authenticated {
         $c->save_user_in_session($user, $realmname);
     }
     $user->auth_realm($realmname);
-    $user->store(ref($c->auth_realms->{$realmname}{'store'}));
     
     $c->NEXT::set_authenticated($user, $realmname);
 }
@@ -150,10 +149,7 @@ sub auth_restore_user {
     
     # this sets the realm the user originated in.
     $user->auth_realm($realmname);
-    ## compatibility - some pre 0.10 store / credentials may need the store name,
-    ## this is not used by the current api in any form.
-    $user->store(ref($c->auth_realms->{$realmname}{'store'}));
-    
+        
     return $user;
 
 }