Fix reproducible "no transaction to rollback" when loading bookbag. user/dyrcona/lp1060953
authorJason Stephenson <jason@sigio.com>
Sat, 6 Oct 2012 19:33:16 +0000 (15:33 -0400)
committerJason Stephenson <jason@sigio.com>
Sat, 6 Oct 2012 19:48:39 +0000 (15:48 -0400)
commit4cbc94386a2bb62cc3b8c4edb43bc29876909d82
treed3c434676005f2670bf6d6a0ffb9a7ee3ba4fd67
parentee8d5613cd3bc469678deece795fb43adb0e8f91
Fix reproducible "no transaction to rollback" when loading bookbag.

Remove most transaction code from load_myopac_bookbags in OpenILS::WWW::
EGCatLoader::Account, only doing xact_begin when an update is called for.

It looks like the original transaction may have been created to prevent
the bookbags from changing while being loaded.  The comment on the line
where the xact_begin call originally was simply said 'replication...'
I don't know what problem, if any, that transaction was created to resolve,
but dropping most of the transaction code from that method does resolve a
reproducible bug.

(I also don't know exactly what in the patron's bookbags caused the no
transaction to rollback error, but it always appears with the same bookbag
with regularity.)

Signed-off-by: Jason Stephenson <jason@sigio.com>
Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Account.pm