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

(-)a/Koha/Exceptions/Hold.pm (+4 lines)
Lines 31-36 use Exception::Class ( Link Here
31
    'Koha::Exceptions::Hold::InvalidPickupLocation' => {
31
    'Koha::Exceptions::Hold::InvalidPickupLocation' => {
32
        isa         => 'Koha::Exceptions::Hold',
32
        isa         => 'Koha::Exceptions::Hold',
33
        description => 'The supplied pickup location is not valid'
33
        description => 'The supplied pickup location is not valid'
34
    },
35
    'Koha::Exceptions::Hold::MissingPickupLocation' => {
36
        isa         => 'Koha::Exceptions::Hold',
37
        description => 'You must supply a pickup location when placing a hold'
34
    }
38
    }
35
);
39
);
36
40
(-)a/Koha/Hold.pm (-1 / +2 lines)
Lines 713-718 expirationdate for holds. Link Here
713
sub store {
713
sub store {
714
    my ($self) = @_;
714
    my ($self) = @_;
715
715
716
    Koha::Exceptions::Hold::MissingPickupLocation->throw() unless $self->branchcode;
717
716
    if ( !$self->in_storage ) {
718
    if ( !$self->in_storage ) {
717
        if ( ! $self->expirationdate && $self->patron_expiration_date ) {
719
        if ( ! $self->expirationdate && $self->patron_expiration_date ) {
718
            $self->expirationdate($self->patron_expiration_date);
720
            $self->expirationdate($self->patron_expiration_date);
719
- 

Return to bug 31086