Login | Register
My pages Projects Community openCollabNet

propel
Reply to message

* = Required fields
* Subject
* Body
Attachments
Send reply to
Topic
Author (directly in email)
Please type the letters in the image above.

Original message

Author Cameron Brunner <cameron.brunner@gmail.com>
Full name Cameron Brunner <cameron.brunner@gmail.com>
Date 2006-09-22 08:33:32 PDT
Message I was trying to avoid altering the default pdo functionality, i just
wanted to add what we needed for propel so when users used the
connection directly it was 'as normal'...


Cameron

On 9/23/06, Hans Lellelid <hans at velum dot net> wrote:
> I spend some more time looking at this & at PDO. I'm going to commit a
> slightly modified version of this to 1.3 branch (and later, to trunk) --
> so that API matches default PDO as possible.
>
> Thanks!
>
> Hans
>
> Hans Lellelid wrote:
> > Hi Cameron,
> >
> > I just had an opportunity to look at this patch. I hadn't thought about
> > simply extending the PDO class. Were the begin()/rollback()/commit()
> > methods final (i.e. is that why you created txnBegin(), etc.)?
> >
> > I'd rather use an API that will allow us to revert to using the default
> > PDO driver, once its transaction support is improved.
> >
> > Hans
> >
> > Cameron Brunner wrote:
> >
> >> This needs a beating before it gets commited however i extended PDO
> >> and setup a transaction wrapper in it and altered the generators to
> >> use it. Anyone care to try it out and let me know how it goes for
> >> them? Also does anyone have any objections to a patch along these
> >> lines? I have it active on my sites and it seems fine...
> >>
> >>
> >> Cameron