https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18651
Marcel de Rooy <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #29 from Marcel de Rooy <[email protected]> --- Generally looks good. But still some things: Patch 4 seems to be just for debugging purposes (Data::Printer). The search is saved in account_lines but it is not used any more? Patch 6 adds locking. You lock the message queue. But you should lock old issues, I guess. Furthermore, how long do you really need the lock? Your lock now extends to the very end of MarkIssueReturned, but isn't the lock actually needed only when calculating max+1 and storing it in old issues ? -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] http://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/
