[Koha-devel] Issues with DBIx and nested transactions

Mark Tompsett mtompset at hotmail.com
Thu Jun 4 20:41:18 CEST 2015


Greetings,

Has someone started a bugzilla report for this? If so, great. Have fun. If not, given that Galen’s solution is relatively easy, just tedious, to implement across the board, I’ll work on it.

GPML,
Mark Tompsett

From: Tomas Cohen Arazi 
Sent: Thursday, June 04, 2015 2:20 PM
To: koha-devel 
Subject: Re: [Koha-devel] Issues with DBIx and nested transactions



2015-06-04 14:48 GMT-03:00 Galen Charlton <gmc at esilibrary.com>:
>
> Hi,
>
> On Thu, Jun 4, 2015 at 1:13 PM, Tomas Cohen Arazi <tomascohen at gmail.com> wrote:
> > to our (abuse?) of $dbh->{AutoCommit} = 0 on the tests.
>
> Not abuse - one way or another, the DB-dependent tests really ought
> not to be make permanent changes to the database.


Agreed.

> > Before someone says it: reverting the commits will put us back to the
> > previous state, of course. But it will just hide the problem for a while.
>
> Time to revert regardless. :)


Unless we do what you propose :)

> However, here's a diff that shows one way to get DBIx::Class and DBI
> to stop fighting over who gets to control the transaction:
> http://paste.lisp.org/display/149194

Thanks for the solution.

--
Tomás Cohen Arazi
Prosecretaría de Informática
Universidad Nacional de Córdoba
✆ +54 351 5353750 ext 13168
GPG: B76C 6E7C 2D80 551A C765  E225 0A27 2EA1 B2F3 C15F



--------------------------------------------------------------------------------
_______________________________________________
Koha-devel mailing list
Koha-devel at lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.koha-community.org/pipermail/koha-devel/attachments/20150604/b7f7fbc4/attachment.html>


More information about the Koha-devel mailing list