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

(-)a/C4/Reserves.pm (-7 / +4 lines)
Lines 1119-1134 sub ModReserve { Link Here
1119
        $hold->cancel({ cancellation_reason => $cancellation_reason });
1119
        $hold->cancel({ cancellation_reason => $cancellation_reason });
1120
    }
1120
    }
1121
    elsif ($hold->found && $hold->priority eq '0' && $date) {
1121
    elsif ($hold->found && $hold->priority eq '0' && $date) {
1122
        logaction( 'HOLDS', 'MODIFY', $hold->reserve_id, $hold )
1123
            if C4::Context->preference('HoldsLog');
1124
1125
        # The only column that can be updated for a found hold is the expiration date
1122
        # The only column that can be updated for a found hold is the expiration date
1126
        $hold->expirationdate($date)->store();
1123
        $hold->expirationdate($date)->store();
1127
    }
1128
    elsif ($rank =~ /^\d+/ and $rank > 0) {
1129
        logaction( 'HOLDS', 'MODIFY', $hold->reserve_id, $hold )
1124
        logaction( 'HOLDS', 'MODIFY', $hold->reserve_id, $hold )
1130
            if C4::Context->preference('HoldsLog');
1125
            if C4::Context->preference('HoldsLog');
1131
1126
    }
1127
    elsif ($rank =~ /^\d+/ and $rank > 0) {
1132
        my $properties = {
1128
        my $properties = {
1133
            priority    => $rank,
1129
            priority    => $rank,
1134
            branchcode  => $branchcode,
1130
            branchcode  => $branchcode,
Lines 1156-1161 sub ModReserve { Link Here
1156
        }
1152
        }
1157
1153
1158
        _FixPriority({ reserve_id => $reserve_id, rank =>$rank });
1154
        _FixPriority({ reserve_id => $reserve_id, rank =>$rank });
1155
        logaction( 'HOLDS', 'MODIFY', $hold->reserve_id, $hold )
1156
            if C4::Context->preference('HoldsLog');
1159
    }
1157
    }
1160
}
1158
}
1161
1159
1162
- 

Return to bug 32306