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

(-)a/C4/SIP/ILS/Transaction/Checkin.pm (-7 / +12 lines)
Lines 74-80 sub do_checkin { Link Here
74
    if ( $checked_in_ok ) {
74
    if ( $checked_in_ok ) {
75
        delete $messages->{NotIssued};
75
        delete $messages->{NotIssued};
76
        delete $messages->{LocalUse};
76
        delete $messages->{LocalUse};
77
        $return = 1 unless keys %$messages;
77
        if( keys %$messages ) {
78
            $self->alert(!$return);
79
        } else {
80
            $return = 1;
81
        }
82
    } else {
83
        $self->alert(!$return);
78
    }
84
    }
79
85
80
    # biblionumber, biblioitemnumber, itemnumber
86
    # biblionumber, biblioitemnumber, itemnumber
Lines 130-140 sub do_checkin { Link Here
130
    }
136
    }
131
    # ignoring messages: NotIssued, WasTransfered
137
    # ignoring messages: NotIssued, WasTransfered
132
138
133
    if ($cv_triggers_alert) {
139
    my $alert = defined $self->alert_type;
134
        $self->alert( defined $self->alert_type ); # Overwrites existing alert value, should set to 0 if there is no alert type
140
    if ( $cv_triggers_alert ) {
135
    }
141
        $self->alert($alert); # Overwrites existing alert value, should set to 0 if there is no alert type
136
    else {
142
    } else {
137
        $self->alert( !$return || defined $self->alert_type );
143
        $self->alert($alert) if $alert; # Doesn't affect alert value unless an alert type is set
138
    }
144
    }
139
145
140
    $self->ok($return);
146
    $self->ok($return);
141
- 

Return to bug 23057