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

(-)a/C4/Letters.pm (-1 / +1 lines)
Lines 1054-1060 sub SendQueuedMessages { Link Here
1054
                    _set_message_status( { message_id => $message->{'message_id'}, status => 'failed' } );
1054
                    _set_message_status( { message_id => $message->{'message_id'}, status => 'failed' } );
1055
                    next MESSAGE;
1055
                    next MESSAGE;
1056
                }
1056
                }
1057
                $message->{to_address} = $message->{to_address} ? $message->{to_address} : $member->{'smsalertnumber'};
1057
                $message->{to_address} ||= $member->{'smsalertnumber'};
1058
                unless ( $message->{to_address} && $member->{'smsalertnumber'} ) {
1058
                unless ( $message->{to_address} && $member->{'smsalertnumber'} ) {
1059
                    _set_message_status( { message_id => $message->{'message_id'}, status => 'failed' } );
1059
                    _set_message_status( { message_id => $message->{'message_id'}, status => 'failed' } );
1060
                    warn sprintf( "No smsalertnumber found for patron %s!", $message->{'borrowernumber'} ) if $params->{'verbose'} or $debug;
1060
                    warn sprintf( "No smsalertnumber found for patron %s!", $message->{'borrowernumber'} ) if $params->{'verbose'} or $debug;
(-)a/t/db_dependent/Reserves.t (-3 / +18 lines)
Lines 733-746 subtest '_koha_notify_reserve() tests' => sub { Link Here
733
    };
733
    };
734
734
735
    my $mp = Test::MockModule->new( 'C4::Members::Messaging' );
735
    my $mp = Test::MockModule->new( 'C4::Members::Messaging' );
736
736
    $mp->mock("GetMessagingPreferences",$wants_hold_and_email);
737
    $mp->mock("GetMessagingPreferences",$wants_hold_and_email);
737
738
738
    my $sms_hold_notice = $builder->build_object({
739
    $dbh->do('DELETE FROM letter');
739
            class => 'Koha::Notice::Templates',
740
741
    my $email_hold_notice = $builder->build({
742
            source => 'Letter',
743
            value => {
744
                message_transport_type => 'email',
745
                branchcode => '',
746
                code => 'HOLD',
747
                module => 'reserves',
748
                lang => 'default',
749
            }
750
        });
751
752
    my $sms_hold_notice = $builder->build({
753
            source => 'Letter',
740
            value => {
754
            value => {
741
                message_transport_type => 'sms',
755
                message_transport_type => 'sms',
742
                branchcode => '',
756
                branchcode => '',
743
                code => 'HOLD',
757
                code => 'HOLD',
758
                module => 'reserves',
759
                lang=>'default',
744
            }
760
            }
745
        });
761
        });
746
762
747
- 

Return to bug 18478