On Thu, Feb 03, 2005 at 08:40:48AM -0800, David Wheeler wrote:
> On Feb 3, 2005, at 7:45 AM, Tim Bunce wrote:
>
> >>+ my $no_reset = delete $attr->{NoReset};
> >>
> >>+ %$attr = () if $no_reset;
> >>
> >>If this looks good to you, I can add a test for it and update the docs
> >>and send that patch.
>
> Any thoughts on this, then?
Same as before - not very keen.
> >In http://search.cpan.org/src/TIMB/DBI/ToDo it says (slightly edited):
> >
> >The key point is that attributes passed to $drh->connect and
> >$dbh->prepare are applied to the new 'virgin' handle before the
> >real $dbh->connect or $sth->prepare is called. (In case that's not
> >clear, think about the $drh, $dbh, and $sth variables on those calls.)
>
> I see. So if connect_cached() returns an existing handle, the
> attributes would not be applied to it, yes?
Yes. The current attribute setting post-connect would be done
pre-connect so if the cached handle is okay it's returned unaltered.
> >You'd be most welcome to draw up an informal spec for some callbacks
> >(such as OnConnect and OnConnectCached) for discussion and
> >implementation.
>
> Should be pretty straight-forward. Are there other events for which
> you'd like to add callbacks? OnPrepare, perhaps? OnExecute? OnFetch?
Yes, but let's just start with the connect ones for now.
Tim.
