Veikko,

I am getting confused on your CreoleSessionStorage::initialize method because you have:

       if ($this->getParameter('auto_start', true)) {
           // start our session
           session_start();
       }

at the bottom.  Wouldn't your autoStart method handle this?

-Wes

Veikko Mäkinen wrote:
Veikko Mäkinen wrote:
Hey folks,

SessionStorage::initialize's auto_start implementation causes a small problem when using an inheriting class (like PostgreSQLSessionStorage or my CreoleSessionStorage [http://veikkomakinen.com/agavi/source.php?file=SessionStorage.class.php&hidelineno=true]).

This would require small changes in Context.class.php.

I fixed the auto_start problem by adding autoStart method to SessionStorage and calling it from Context::initialize. Links to source and diffs against trunk rev 262 (newest for now):

http://veikkomakinen.com/agavi/source.php?file=Context.class.php&hidelineno=true http://veikkomakinen.com/agavi/source.php?file=SessionStorage.class.php&hidelineno=true

http://veikkomakinen.com/agavi/Context.diff.txt
http://veikkomakinen.com/agavi/SessionStorage.diff.txt

My SessionStorage also introduces cookie parameter handling and little more documentation.

What do you think?


-veikko

_______________________________________________
agavi-dev mailing list
[email protected]
http://labworkz.com/cgi-bin/mailman/listinfo/agavi-dev

--
James "Wes" Hays
Great Basin Development
P.O.Box 3503, Reno, Nevada 89505
Cell: (775) 745-3013
[EMAIL PROTECTED]
http://www.gbdev.com


_______________________________________________
agavi-dev mailing list
[email protected]
http://labworkz.com/cgi-bin/mailman/listinfo/agavi-dev

Reply via email to