---- Reported by patrick.langer@easycheck.org 2009-10-12 10:19:48 ---- Checking in a not checked out item returns an error (which is correct) without giving a reason for the failure as screen_msg (aka AF field in SIP2) is empty. Checkin in koha SIP is generally quite sparse when it comes to error handling and reasons for the failure of the checkin. Most times no screen_msg is filled which makes it hard for the self service terminal to display an understandable reason to the patron for the failure. In this case I suggest to insert in SIP::ILS Checkin around row 183 right after $circ->ok($item && $item->{patron}); if (!defined($item->{patron})) { $circ->screen_msg("Item not checked out"); } else { [rest of the transaction part] } --- Bug imported by chris@bigballofwax.co.nz 2010-05-21 01:13 UTC --- This bug was previously known as _bug_ 3704 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=3704 Unknown operating system Windows 2000. Setting to default OS "All". Actual time not defined. Setting to 0.0 The original reporter of this bug does not have an account here. Reassigning to the person who moved it here: chris@bigballofwax.co.nz. Previous reporter was patrick.langer@easycheck.org.
Created attachment 5414 [details] [review] bug 3704 patch
Adds additional if/then conditional check around whether a patron is defined, and returns "Item not checked out" as a screen_msg if not. Otherwise, proceed as normal. Easy to read, marking as Passed QA
Pushed, please test
There have been no further reports of problems so I am marking this bug resolved.