Re: [RDBO] Catalyst::Plugin::Session::Store::DBI

2007-03-15 Thread John Siracusa
On 3/15/07 9:14 AM, Peter Karman wrote:
 The latest (0.09) CPAN version of this Catalyst plugin now contains support
 for RDBO. You can specify a RDBO class name as the 'dbi_dbh' config param, and
 it will return the dbh like:
 
   My::RDBO::Session-new-dbh

Shouldn't that be this instead?

My::RDBO::Session-new-db-retain_dbh;

Otherwise, the $dbh may be disconnected as soon as it's returned.

-John



-
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT  business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.phpp=sourceforgeCID=DEVDEV
___
Rose-db-object mailing list
Rose-db-object@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/rose-db-object


Re: [RDBO] Catalyst::Plugin::Session::Store::DBI

2007-03-15 Thread Peter Karman


John Siracusa scribbled on 3/15/07 8:48 AM:
 On 3/15/07 9:14 AM, Peter Karman wrote:
 The latest (0.09) CPAN version of this Catalyst plugin now contains support
 for RDBO. You can specify a RDBO class name as the 'dbi_dbh' config param, 
 and
 it will return the dbh like:

   My::RDBO::Session-new-dbh
 
 Shouldn't that be this instead?
 
 My::RDBO::Session-new-db-retain_dbh;
 
 Otherwise, the $dbh may be disconnected as soon as it's returned.
 

ah. of course you're right (as the author usually is... :)).

I wrote the plugin patch against my own RDBO subclass, which overrides 
Rose::DB::release_dbh() in order to support my DBI handle caching system. I 
should have tested it against a vanilla RDBO. My bad.

I'll submit a patch to Andy and see if he can get a 0.10 out as fast as he did 
the 0.09. :)

thanks, John.

pek

-- 
Peter Karman  .  http://peknet.com/  .  [EMAIL PROTECTED]

-
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT  business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.phpp=sourceforgeCID=DEVDEV
___
Rose-db-object mailing list
Rose-db-object@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/rose-db-object