++ here.  I would suggest going just a bit further:

let's fix the other columns in the table while we're there.

  * what is the difference between branchcode and issuingbranch ?
A transaction can only happen at one location, so that should be NOT NULL
and the other should be removed.

issues.return and returndate should be removed.

We've seen a couple of reports of these phantom loans, which would be made
impossible by db constraints.  So far, evidence points to setting the item to LOST
(chargelostitem), but the jury's still out.  regardless of the cause of this particular
bug, I think we should take advantage of the database's integrity features here:
++galen.

Ryan






--
Ryan Higgins

LibLime  *  Open-Source Solutions for Libraries
Featuring KohaZOOM ILS
888-564-2457  x704