On Sat, Jun 19, 2010 at 8:50 AM, Jens Rehsack <rehs...@googlemail.com> wrote: > Hi Todd, hi Lincoln, > > at first I want to express my acknowledgment and thank for your great > work at Tie::DBI. > I'm unsure who is the primary maintainer now (does Todd just apply > some work, but main development is still at Lincoln or has Todd took > over the maintainership), so I address both. > > While I take a short time out from updating DBD::AnyData to the > current DBD::File requirements, I start some work on Template::DBI > which supports Tie::DBI, too (just for the history of my suggestions). > I detected some errors in the Tie::DBI tests - and so I investigated some > time. > > At first I've seen, your tests didn't realize that DBI ships a DBD > which is for most tests fully usable: DBD::DBM. > So I updated t/DBI.t (patch-aa) to allow DBI_DSN (DBI_USER, DBI_PASS) > environment variables can be used, too (and remove a "Useless > localization of scalar assignment at t/DBI.t line 130." warning). > > Then I updated lib/Tie/DBI.pm (patch-ab) to support DBD::DBM (and all > other DBD::File based DBD's a bit better).: > bind and bindselect is fully supported by DBD::File (and derived > drivers), so I added all I know. > LISTFIELDS is not an ANSI SQL command (neither my PostgreSQL reference > book nor my SQL Bible name that command), - I added DBD::File derived > drivers to the list of DBD's which do not support them. > I suggest to convert this list into a CAN_LISTFIELDS (is expected to > be shorted and will simplify logical expressions evaluating it). > > Tie::DBI::STORE has a minor logical issue: > eval { local $dbh->{PrintError} = 0; ... }; will return the result of > $dbh->STORE( "PrintError", $origValue ); at the end of the eval block. > Fixed that. > > _run_query need to $dbh->quote() when not $self->{CanBindSelect} > (Tie::DBI::_quote seems not to work - I didn't investigate why). > > That's all and it runs fine now \o/ > > I would be glad if you join us at dbi-dev@perl.org to keep up-to-date > on DBI and DBD development. > Further you're welcome to join daily debates at irc://irc.perl.org/dbi > Thanks for the info. I haven't looked closeley at your email but will later.
Repos are managed at github and RT takes tickets. both can be gotten to from the link below. http://search.cpan.org/~toddr/Tie-DBI-1.05/ Todd