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.php&p=sourceforge&CID=DEVDEV _______________________________________________ Rose-db-object mailing list Rose-db-object@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/rose-db-object