[Pagekit-devel] Re: no_sessions?

[Russell D. W. <rw...@in...>]

Hi Boris,

Somehow I didn't see the patch on the last e-mail you sent.  Thanks!  This
patch "almost" works -- it didn't work, but I realized it's because of order
of operations / combining of conditions...

The line:
if ($model->pkit_get_config_attr( 'PAGE', $pk->{page_id}, 'use_sessions' )
|| 'yes' eq 'yes' ) {

should be changed to... (the 'yes' was OR'd in the wrong order)... perl was
saying "Of course 'yes' equals 'yes'!

if ( ($model->pkit_get_config_attr( 'PAGE', $pk->{page_id}, 'use_sessions' )
|| 'yes') eq 'yes' ) {

Also, I notice it's only a PAGE attribute.  Can this be extended to SECTION
and possibly even GLOBAL.  I can see the potential usefulness of having it
be a SECTION attribute, and believe it or not, some people might put up
sites that don't need use sessions at all.

Thanks for the great work and the quick response!

Russell Weiss
Founder & Technical Manager
InfoRelay Online Systems, Inc.
http://www.inforelay.com/



----- Original Message -----
From: "Boris Zentner" <bo...@id...>
To: "Russell D. Weiss" <rw...@in...>
Sent: Wednesday, October 02, 2002 9:18 AM
Subject: Re: no_sessions?


>
> Hi Russell,
>
> Am Mittwoch, 2. Oktober 2002 14:55 hast Du geschrieben:
> > Hey Boris, any word on my no_sessions question / idea?  I'm assuming the
> > feature isn't there? I'm hoping you can at least see the usefulness of
it?
> > Since you definitely seem to be the resident expert on PageKit, I'm
> > wondering if you can suggest the best way to hack the session
authorization
> > methods to achieve this sort of functionality.
>
> Im not a native english speaker as you might know already. So maybe i
missed
> what you asked for. But on my answer to your first request for a
no_session
> attribute, i attached a patch that do what you want for the PAGE and
SESSION
> config options. also i have not tested the patch. If it works the way you
> need/like it i put it in the next version of pageKit.
>
> to be sure you get the patch now, i attch it again. And i renamed he
> no_session attribute to use_sessions it defaults to 'yes'
>
> please try it and report your results.
>
> >
> > Unless there are plans to add this sort of thing in as a PageKit feature
> > :-).
> >
> > Thanks,
> > Russell
>
> --
> cu boris
>
> Kommt ein Mann in ein Herrenbekleidungsgeschäft:
> "Ich hätte gern einen neuen schwarzen Blazer!""
> Der Verkäufer mit leicht näselnder Stimme:
> "Oh ja, ich auch..."
>