[Koha-bugs] [Bug 29325] commit_file.pl error 'Already in a transaction'

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29325

Martin Renvoize <martin.renvoize at ptfs-europe.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |Signed Off

--- Comment #49 from Martin Renvoize <martin.renvoize at ptfs-europe.com> ---
The code reads a fair bit better now, though there still some peculiarities..
it would be really nice to clean this up whilst moving it to the more modern
background task queue implementation.

In my testing this appears to all work.. but I'd love to see some more tests to
make sure I'm not imagining positives.. 

We need to confirm the command line version works, the staff client without
pushing to background and the staff client with pushing to background all work
still.

Whilst reading the code, it also felt like we could perhaps drop the following
line.. but it's 'interesting'...

$dbh->{InactiveDestroy}  = 1;

-- 
You are receiving this mail because:
You are watching all bug changes.