[Catalyst-commits] r6590 - trunk/Catalyst-Plugin-Authentication/lib/Catalyst/Plugin

jayk at dev.catalyst.perl.org jayk at dev.catalyst.perl.org
Sun Jul 22 04:22:07 GMT 2007


Author: jayk
Date: 2007-07-22 04:22:06 +0100 (Sun, 22 Jul 2007)
New Revision: 6590

Modified:
   trunk/Catalyst-Plugin-Authentication/lib/Catalyst/Plugin/Authentication.pm
Log:
$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


Modified: trunk/Catalyst-Plugin-Authentication/lib/Catalyst/Plugin/Authentication.pm
===================================================================
--- trunk/Catalyst-Plugin-Authentication/lib/Catalyst/Plugin/Authentication.pm	2007-07-21 02:59:04 UTC (rev 6589)
+++ trunk/Catalyst-Plugin-Authentication/lib/Catalyst/Plugin/Authentication.pm	2007-07-22 03:22:06 UTC (rev 6590)
@@ -40,7 +40,6 @@
         $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 @@
     
     # 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;
 
 }




More information about the Catalyst-commits mailing list