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

(-)a/C4/SMS.pm (+4 lines)
Lines 89-94 sub send_sms { Link Here
89
    my $driver = exists $params->{'driver'} ? $params->{'driver'} : $self->driver();
89
    my $driver = exists $params->{'driver'} ? $params->{'driver'} : $self->driver();
90
    return unless $driver;
90
    return unless $driver;
91
91
92
    if ( ! defined C4::Context->config('sms_send_config') ) {
93
        # Should I warn in some way?
94
        return;
95
    }
92
96
93
    my ($sent, $sender);
97
    my ($sent, $sender);
94
98
(-)a/t/SMS.t (-2 / +10 lines)
Lines 19-25 use Modern::Perl; Link Here
19
19
20
use t::lib::Mocks;
20
use t::lib::Mocks;
21
21
22
use Test::More tests => 7;
22
use Test::More tests => 8;
23
23
24
BEGIN {
24
BEGIN {
25
    use_ok('C4::SMS');
25
    use_ok('C4::SMS');
Lines 53-58 $send_sms = C4::SMS->send_sms({ Link Here
53
});
53
});
54
is( $send_sms, undef, 'send_sms with an undef driver returns undef' );
54
is( $send_sms, undef, 'send_sms with an undef driver returns undef' );
55
55
56
t::lib::Mocks::mock_config('sms_send_config',undef);
57
$send_sms = C4::SMS->send_sms({
58
    destination => '+33123456789',
59
    message => 'my message',
60
    driver => 'Test',
61
});
62
is( $send_sms, undef, 'send_sms without sms_send_config defined returns undef' );
63
64
t::lib::Mocks::mock_config('sms_send_config','\tmp');
56
$send_sms = C4::SMS->send_sms({
65
$send_sms = C4::SMS->send_sms({
57
    destination => '+33123456789',
66
    destination => '+33123456789',
58
    message => 'my message',
67
    message => 'my message',
59
- 

Return to bug 18732