@@ -, +, @@ --- C4/SIP/ILS/Transaction/Checkin.pm | 18 ++++++++++++------ 1 file changed, 12 insertions(+), 6 deletions(-) --- a/C4/SIP/ILS/Transaction/Checkin.pm +++ a/C4/SIP/ILS/Transaction/Checkin.pm @@ -74,7 +74,13 @@ sub do_checkin { if ( $checked_in_ok ) { delete $messages->{NotIssued}; delete $messages->{LocalUse}; - $return = 1 unless keys %$messages; + if( keys %$messages ) { + $self->alert(!$return); + } else { + $return = 1; + } + } else { + $self->alert(!$return); } # biblionumber, biblioitemnumber, itemnumber @@ -130,11 +136,11 @@ sub do_checkin { } # ignoring messages: NotIssued, WasTransfered - if ($cv_triggers_alert) { - $self->alert( defined $self->alert_type ); # Overwrites existing alert value, should set to 0 if there is no alert type - } - else { - $self->alert( !$return || defined $self->alert_type ); + my $alert = defined $self->alert_type; + if ( $cv_triggers_alert ) { + $self->alert($alert); # Overwrites existing alert value, should set to 0 if there is no alert type + } else { + $self->alert($alert) if $alert; # Doesn't affect alert value unless an alert type is set } $self->ok($return); --