https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19530
Marcel de Rooy <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] Status|Signed Off |Failed QA --- Comment #5 from Marcel de Rooy <[email protected]> --- $dbh->do("UPDATE branchtransfers SET datearrived = NOW(), comments = ? WHERE itemnumber = ?", undef, "Canceled, new transfer from $frombranch to $tobranch created", $itemnumber); We should at least limit this update by adding datearrived IS NULL. No need to update here all arrived items. In this case you might even wonder if you should not DELETE those records instead? I am wondering if transferbook (the sub calling ModItemTransfer) should check if there is no open transfer in the table. It is possible to add multiple transfers by selecting different destinations. What do you think? -- 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/
