View | Details | Raw Unified | Return to bug 9842
Collapse All | Expand All

(-)a/C4/SIP/ILS.pm (-3 / +2 lines)
Lines 302-315 sub cancel_hold { Link Here
302
    $trans->patron($patron);
302
    $trans->patron($patron);
303
    $trans->item($item);
303
    $trans->item($item);
304
	$trans->drop_hold;
304
	$trans->drop_hold;
305
	unless ($trans->ok) {
305
	unless ($trans->ok(1)) {
306
		$trans->screen_msg("Error with transaction drop_hold: " . $trans->screen_msg);
306
		$trans->screen_msg("Error with transaction drop_hold: " . $trans->screen_msg);
307
		return $trans;
307
		return $trans;
308
	}
308
	}
309
    # Remove the hold from the patron's record first
309
    # Remove the hold from the patron's record first
310
    $trans->ok($patron->drop_hold($item_id));	# different than the transaction drop!
310
    $trans->ok($patron->drop_hold($item_id));	# different than the transaction drop!
311
311
312
    unless ($trans->ok) {
312
    unless ($trans->ok(1)) {
313
		# We didn't find it on the patron record
313
		# We didn't find it on the patron record
314
		$trans->screen_msg("No such hold on patron record.");
314
		$trans->screen_msg("No such hold on patron record.");
315
		return $trans;
315
		return $trans;
316
- 

Return to bug 9842