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

Martin Renvoize <[email protected]> changed:

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

--- Comment #49 from Martin Renvoize <[email protected]> ---
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.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to