[Catalyst-commits] r9778 - in Catalyst-Authentication-Store-LDAP/trunk: . lib/Catalyst/Authentication/Store/LDAP

[t0m at dev.catalyst.perl.org]

Author: t0m
Date: 2009-04-21 16:23:42 +0100 (Tue, 21 Apr 2009)
New Revision: 9778

Removed:
   Catalyst-Authentication-Store-LDAP/trunk/META.yml
Modified:
   Catalyst-Authentication-Store-LDAP/trunk/
   Catalyst-Authentication-Store-LDAP/trunk/Changes
   Catalyst-Authentication-Store-LDAP/trunk/lib/Catalyst/Authentication/Store/LDAP/Backend.pm
   Catalyst-Authentication-Store-LDAP/trunk/lib/Catalyst/Authentication/Store/LDAP/User.pm
Log:
Remove META.yml from svn, it's a generated file. Update svn ignore props.
Patch docs to not mention the deprecated authentication methods from the old
authentication code.



Property changes on: Catalyst-Authentication-Store-LDAP/trunk
___________________________________________________________________
Name: svn:ignore
   + META.yml
pm_to_blib
blib
inc
Makefile



Modified: Catalyst-Authentication-Store-LDAP/trunk/Changes
===================================================================
--- Catalyst-Authentication-Store-LDAP/trunk/Changes	2009-04-21 13:46:45 UTC (rev 9777)
+++ Catalyst-Authentication-Store-LDAP/trunk/Changes	2009-04-21 15:23:42 UTC (rev 9778)
@@ -1,3 +1,7 @@
+   - Change documentation which still refers to the old ::Plugin:: style
+     auth system to use ->authenticate instead of ->login, and not say that
+     you need to do things manually to have multiple stores. (t0m)
+
 0.1004  21 Oct 2008
    - Add the ability to have the user inflated into a custom
      user class with the user_class option (t0m)

Deleted: Catalyst-Authentication-Store-LDAP/trunk/META.yml
===================================================================
--- Catalyst-Authentication-Store-LDAP/trunk/META.yml	2009-04-21 13:46:45 UTC (rev 9777)
+++ Catalyst-Authentication-Store-LDAP/trunk/META.yml	2009-04-21 15:23:42 UTC (rev 9778)
@@ -1,22 +0,0 @@
---- 
-abstract: Authenticate Users against LDAP Directories
-author: 
-  - Adam Jacob <holoway at cpan.org>
-build_requires: 
-  Net::LDAP::Server::Test: 0.07
-  Test::More: 0
-distribution_type: module
-generated_by: Module::Install version 0.68
-license: perl
-meta-spec: 
-  url: http://module-build.sourceforge.net/META-spec-v1.3.html
-  version: 1.3
-name: Catalyst-Authentication-Store-LDAP
-no_index: 
-  directory: 
-    - inc
-    - t
-requires: 
-  Catalyst::Plugin::Authentication: 0.10003
-  Net::LDAP: 0
-version: 0.1003

Modified: Catalyst-Authentication-Store-LDAP/trunk/lib/Catalyst/Authentication/Store/LDAP/Backend.pm
===================================================================
--- Catalyst-Authentication-Store-LDAP/trunk/lib/Catalyst/Authentication/Store/LDAP/Backend.pm	2009-04-21 13:46:45 UTC (rev 9777)
+++ Catalyst-Authentication-Store-LDAP/trunk/lib/Catalyst/Authentication/Store/LDAP/Backend.pm	2009-04-21 15:23:42 UTC (rev 9778)
@@ -53,19 +53,11 @@
     
     our $users = Catalyst::Authentication::Store::LDAP::Backend->new(\%config);
 
-    sub action : Local {
-        my ( $self, $c ) = @_;
-
-        $c->login( $users->get_user( $c->req->param("login") ),
-            $c->req->param("password") );
-    }
-
 =head1 DESCRIPTION
 
-You probably want L<Catalyst::Authentication::Store::LDAP>, unless
-you are mixing several stores in a single app and one of them is LDAP.
+You probably want L<Catalyst::Authentication::Store::LDAP>.
 
-Otherwise, this lets you create a store manually. 
+Otherwise, this lets you create a store manually.
 
 See the L<Catalyst::Authentication::Store::LDAP> documentation for
 an explanation of the configuration options.

Modified: Catalyst-Authentication-Store-LDAP/trunk/lib/Catalyst/Authentication/Store/LDAP/User.pm
===================================================================
--- Catalyst-Authentication-Store-LDAP/trunk/lib/Catalyst/Authentication/Store/LDAP/User.pm	2009-04-21 13:46:45 UTC (rev 9777)
+++ Catalyst-Authentication-Store-LDAP/trunk/lib/Catalyst/Authentication/Store/LDAP/User.pm	2009-04-21 15:23:42 UTC (rev 9778)
@@ -8,12 +8,15 @@
 
 =head1 SYNOPSIS
 
-You should be creating these objects through L<Catalyst::Authentication::Store::LDAP::Backend>'s "get_user" method, or just letting $c->login do
+You should be creating these objects through L<Catalyst::Authentication::Store::LDAP::Backend>'s "get_user" method, or just letting $c->authenticate do
 it for you.
 
     sub action : Local {
         my ( $self, $c ) = @_;
-        $c->login($c->req->param(username), $c->req->param(password));
+        $c->authenticate({
+            id => $c->req->param(username),
+            password => $c->req->param(password)
+        );
         $c->log->debug($c->user->username . "is really neat!");
     }