<div dir="ltr">I've just pushed bug 13967 [1]. It introduces a new package (Koha::Config::SysPrefs) that heavily relies on DBIx::Class to deal with the required CRUD for system preferences.<div><br></div><div>It should be no surprise that the first widespread use of DBIx on Koha's codebase raises problems. Widespread means every call to anything is using it :-D.</div><div><br></div><div>The point: it noticeably broke lots of our t/db_dependent/ tests. A first guess would point to the package itself. But such doesn't seem to be the case. It is highlighting an underlaying problem in how we deal with the DB: the errors are related to nested transactions taking place, so I would point to our (abuse?) of $dbh->{AutoCommit} = 0 on the tests.</div><div><br></div><div>Actually, if you pick any db_dependent test that is currently failing on master and set the AutoCommit = 1 you will notice the tests work as expected (with the side effect of altering the DB of course).</div><div><br></div><div>So, this is a call for all of you, whenever you find some time, to try to figure what's going on and possible solutions to this. I ain't no DBIx expert and will be doing my best but really need your help.</div><div><br></div><div>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.</div><div><br></div><div>[1] <a href="http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13967">http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13967</a><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div>Tomás Cohen Arazi</div><div>Prosecretaría de Informática</div><div>Universidad Nacional de Córdoba</div><div>✆ +54 351 5353750 ext 13168</div><div>GPG: B76C 6E7C 2D80 551A C765  E225 0A27 2EA1 B2F3 C15F</div></div></div>
</div></div>