[cgiapp] Re: CGI::Application::Plugin::Authorization - perldoc mistake (v. 0.03)

["Cees Hek" <suppressed>]

A new version of the Authorization plugin is on it's way to CPAN now. 
Or you can fetch it from here:

http://cees.crtconsulting.ca/perl/modules/CGI-Application-Plugin-Authorization-0.04.tar.gz

Changes in 0.04

    - New SimpleGroup driver provided by Mark Stosberg (See docs for usage)
    - Implemented the __USERNAME__ and __GROUP__ constraint variables
      in the DBI driver which were already documented in the examples
      given in the Authorization module docs
    - the DBI driver was assuming that a USERNAME option was being passed
      in, which could result in an invalid SQL statement (Hugh Esco)


Cheers,

Cees

On 3/29/06, Cees Hek <suppressed> wrote:
> On 3/26/06, Giannis Economou <suppressed> wrote:
> > Hi...
> >
> > Just want to mention a mistake found on the documentation of
> > CGI::Application::Plugin::Authorization current version.
> > In the docs found here:
> > http://search.cpan.org/~ceeshek/CGI-Application-Plugin-Authorization-0.03/lib/CGI/Application/Plugin/Authorization.pm
> > there is this example:
> >
> > # Configure Authorization (manages runmode authorization)
> >   MyCGIApp->authz->config(
> >       DRIVER => [ 'DBI',
> >           DBH         => $self->dbh,
> >           TABLES      => ['user', 'usergroup', 'group'],
> >           JOIN_ON     => 'user.id = usergroup.user_id AND usergroup.group_id = group.id',
> >           CONSTRAINTS => {
> >              'user.name'  => '__USERNAME__',
> >              'group.name' => '__GROUP__',
> >           }
> >       ],
> >   );
> >
> > which in not working right.
>
> This is my fault.  I wrote the documentation first so that I had some
> examples to work with, and it would be easier to see a working API.
> Then I didn't finish implementing all the features before releasing
> the first versions.
>
> I have actually added the code for __USERNAME__ and __GROUP__ support
> already, but I haven't released a new version yet.
>
> So thanks for the catch and for reminding me of this, but the in the
> new version, that config will actually work.  Sorry for the confusion.
>
> Cheers,
>
> Cees
>

---------------------------------------------------------------------
Web Archive:  http://www.mail-archive.com/suppressed/
              http://marc.theaimsgroup.com/?l=cgiapp&r=1&w=2
To unsubscribe, e-mail: suppressed
For additional commands, e-mail: suppressed