t/db_dependent/Letters.t

   1 #!/usr/bin/perl
   2
   3 # This file is part of Koha.
   4 #
   5 # Copyright (C) 2013 Equinox Software, Inc.
   6 #
   7 # Koha is free software; you can redistribute it and/or modify it
   8 # under the terms of the GNU General Public License as published by
   9 # the Free Software Foundation; either version 3 of the License, or
  10 # (at your option) any later version.
  11 #
  12 # Koha is distributed in the hope that it will be useful, but
  13 # WITHOUT ANY WARRANTY; without even the implied warranty of
  14 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
  15 # GNU General Public License for more details.
  16 #
  17 # You should have received a copy of the GNU General Public License
  18 # along with Koha; if not, see <http://www.gnu.org/licenses>.
  19
  20 use Modern::Perl;
  21 use Test::More tests => 65;
  22 use Test::MockModule;
  23 use Test::Warn;
  24
  25 use MARC::Record;
  26
  27 my %mail;
  28 my $module = new Test::MockModule('Mail::Sendmail');
  29 $module->mock(
  30     'sendmail',
  31     sub {
  32         warn "Fake sendmail";
  33         %mail = @_;
  34     }
  35 );
  36
  37 use_ok('C4::Context');
  38 use_ok('C4::Members');
  39 use_ok('C4::Acquisition');
  40 use_ok('C4::Biblio');
  41 use_ok('C4::Letters');
  42 use t::lib::Mocks;
  43 use t::lib::TestBuilder;
  44 use Koha::Database;
  45 use Koha::DateUtils qw( dt_from_string output_pref );
  46 use Koha::Acquisition::Booksellers;
  47 use Koha::Acquisition::Bookseller::Contacts;
  48 use Koha::Acquisition::Orders;
  49 use Koha::Libraries;
  50 use Koha::Notice::Templates;
  51 use Koha::Patrons;
  52 use Koha::Subscriptions;
  53 my $schema = Koha::Database->schema;
  54 $schema->storage->txn_begin();
  55
  56 my $builder = t::lib::TestBuilder->new;
  57 my $dbh = C4::Context->dbh;
  58 $dbh->{RaiseError} = 1;
  59
  60 $dbh->do(q|DELETE FROM letter|);
  61 $dbh->do(q|DELETE FROM message_queue|);
  62 $dbh->do(q|DELETE FROM message_transport_types|);
  63
  64 my $library = $builder->build({
  65     source => 'Branch',
  66 });
  67 my $patron_category = $builder->build({ source => 'Category' })->{categorycode};
  68 my $date = dt_from_string;
  69 my $borrowernumber = AddMember(
  70     firstname    => 'Jane',
  71     surname      => 'Smith',
  72     categorycode => $patron_category,
  73     branchcode   => $library->{branchcode},
  74     dateofbirth  => $date,
  75     smsalertnumber => undef,
  76 );
  77
  78 my $marc_record = MARC::Record->new;
  79 my( $biblionumber, $biblioitemnumber ) = AddBiblio( $marc_record, '' );
  80
  81
  82
  83 # GetMessageTransportTypes
  84 my $mtts = C4::Letters::GetMessageTransportTypes();
  85 is( @$mtts, 0, 'GetMessageTransportTypes returns the correct number of message types' );
  86
  87 $dbh->do(q|
  88     INSERT INTO message_transport_types( message_transport_type ) VALUES ('email'), ('phone'), ('print'), ('sms')
  89 |);
  90 $mtts = C4::Letters::GetMessageTransportTypes();
  91 is_deeply( $mtts, ['email', 'phone', 'print', 'sms'], 'GetMessageTransportTypes returns all values' );
  92
  93
  94 # EnqueueLetter
  95 is( C4::Letters::EnqueueLetter(), undef, 'EnqueueLetter without argument returns undef' );
  96
  97 my $my_message = {
  98     borrowernumber         => $borrowernumber,
  99     message_transport_type => 'sms',
 100     to_address             => undef,
 101     from_address           => 'from@example.com',
 102 };
 103 my $message_id = C4::Letters::EnqueueLetter($my_message);
 104 is( $message_id, undef, 'EnqueueLetter without the letter argument returns undef' );
 105
 106 delete $my_message->{message_transport_type};
 107 $my_message->{letter} = {
 108     content      => 'a message',
 109     title        => 'message title',
 110     metadata     => 'metadata',
 111     code         => 'TEST_MESSAGE',
 112     content_type => 'text/plain',
 113 };
 114
 115 $message_id = C4::Letters::EnqueueLetter($my_message);
 116 is( $message_id, undef, 'EnqueueLetter without the message type argument argument returns undef' );
 117
 118 $my_message->{message_transport_type} = 'sms';
 119 $message_id = C4::Letters::EnqueueLetter($my_message);
 120 ok(defined $message_id && $message_id > 0, 'new message successfully queued');
 121
 122
 123 # GetQueuedMessages
 124 my $messages = C4::Letters::GetQueuedMessages();
 125 is( @$messages, 1, 'GetQueuedMessages without argument returns all the entries' );
 126
 127 $messages = C4::Letters::GetQueuedMessages({ borrowernumber => $borrowernumber });
 128 is( @$messages, 1, 'one message stored for the borrower' );
 129 is( $messages->[0]->{message_id}, $message_id, 'EnqueueLetter returns the message id correctly' );
 130 is( $messages->[0]->{borrowernumber}, $borrowernumber, 'EnqueueLetter stores the borrower number correctly' );
 131 is( $messages->[0]->{subject}, $my_message->{letter}->{title}, 'EnqueueLetter stores the subject correctly' );
 132 is( $messages->[0]->{content}, $my_message->{letter}->{content}, 'EnqueueLetter stores the content correctly' );
 133 is( $messages->[0]->{message_transport_type}, $my_message->{message_transport_type}, 'EnqueueLetter stores the message type correctly' );
 134 is( $messages->[0]->{status}, 'pending', 'EnqueueLetter stores the status pending correctly' );
 135
 136
 137 # SendQueuedMessages
 138 my $messages_processed = C4::Letters::SendQueuedMessages( { type => 'email' });
 139 is($messages_processed, 0, 'No queued messages processed if type limit passed with unused type');
 140 $messages_processed = C4::Letters::SendQueuedMessages( { type => 'sms' });
 141 is($messages_processed, 1, 'All queued messages processed, found correct number of messages with type limit');
 142 $messages = C4::Letters::GetQueuedMessages({ borrowernumber => $borrowernumber });
 143 is(
 144     $messages->[0]->{status},
 145     'failed',
 146     'message marked failed if tried to send SMS message for borrower with no smsalertnumber set (bug 11208)'
 147 );
 148
 149 # ResendMessage
 150 my $resent = C4::Letters::ResendMessage($messages->[0]->{message_id});
 151 my $message = C4::Letters::GetMessage( $messages->[0]->{message_id});
 152 is( $resent, 1, 'The message should have been resent' );
 153 is($message->{status},'pending', 'ResendMessage sets status to pending correctly (bug 12426)');
 154 $resent = C4::Letters::ResendMessage($messages->[0]->{message_id});
 155 is( $resent, 0, 'The message should not have been resent again' );
 156 $resent = C4::Letters::ResendMessage();
 157 is( $resent, undef, 'ResendMessage should return undef if not message_id given' );
 158
 159 # GetLetters
 160 my $letters = C4::Letters::GetLetters();
 161 is( @$letters, 0, 'GetLetters returns the correct number of letters' );
 162
 163 my $title = q|<<branches.branchname>> - <<status>>|;
 164 my $content = q{Dear <<borrowers.firstname>> <<borrowers.surname>>,
 165 According to our current records, you have items that are overdue.Your library does not charge late fines, but please return or renew them at the branch below as soon as possible.
 166
 167 <<branches.branchname>>
 168 <<branches.branchaddress1>>
 169 URL: <<OPACBaseURL>>
 170
 171 The following item(s) is/are currently <<status>>:
 172
 173 <item> <<count>>. <<items.itemcallnumber>>, Barcode: <<items.barcode>> </item>
 174
 175 Thank-you for your prompt attention to this matter.
 176 Don't forget your date of birth: <<borrowers.dateofbirth>>.
 177 Look at this wonderful biblio timestamp: <<biblio.timestamp>>.
 178 };
 179
 180 $dbh->do( q|INSERT INTO letter(branchcode,module,code,name,is_html,title,content,message_transport_type) VALUES (?,'my module','my code','my name',1,?,?,'email')|, undef, $library->{branchcode}, $title, $content );
 181 $letters = C4::Letters::GetLetters();
 182 is( @$letters, 1, 'GetLetters returns the correct number of letters' );
 183 is( $letters->[0]->{module}, 'my module', 'GetLetters gets the module correctly' );
 184 is( $letters->[0]->{code}, 'my code', 'GetLetters gets the code correctly' );
 185 is( $letters->[0]->{name}, 'my name', 'GetLetters gets the name correctly' );
 186
 187
 188 # getletter
 189 subtest 'getletter' => sub {
 190     plan tests => 16;
 191     t::lib::Mocks::mock_preference('IndependentBranches', 0);
 192     my $letter = C4::Letters::getletter('my module', 'my code', $library->{branchcode}, 'email');
 193     is( $letter->{branchcode}, $library->{branchcode}, 'GetLetters gets the branch code correctly' );
 194     is( $letter->{module}, 'my module', 'GetLetters gets the module correctly' );
 195     is( $letter->{code}, 'my code', 'GetLetters gets the code correctly' );
 196     is( $letter->{name}, 'my name', 'GetLetters gets the name correctly' );
 197     is( $letter->{is_html}, 1, 'GetLetters gets the boolean is_html correctly' );
 198     is( $letter->{title}, $title, 'GetLetters gets the title correctly' );
 199     is( $letter->{content}, $content, 'GetLetters gets the content correctly' );
 200     is( $letter->{message_transport_type}, 'email', 'GetLetters gets the message type correctly' );
 201
 202     my $context = Test::MockModule->new('C4::Context');
 203     $context->mock( 'userenv', sub {
 204         return {
 205             flags  => 1,
 206             branch => "anotherlib" }
 207     });
 208
 209     t::lib::Mocks::mock_preference('IndependentBranches', 1);
 210     $letter = C4::Letters::getletter('my module', 'my code', $library->{branchcode}, 'email');
 211     is( $letter->{branchcode}, $library->{branchcode}, 'GetLetters gets the branch code correctly' );
 212     is( $letter->{module}, 'my module', 'GetLetters gets the module correctly' );
 213     is( $letter->{code}, 'my code', 'GetLetters gets the code correctly' );
 214     is( $letter->{name}, 'my name', 'GetLetters gets the name correctly' );
 215     is( $letter->{is_html}, 1, 'GetLetters gets the boolean is_html correctly' );
 216     is( $letter->{title}, $title, 'GetLetters gets the title correctly' );
 217     is( $letter->{content}, $content, 'GetLetters gets the content correctly' );
 218     is( $letter->{message_transport_type}, 'email', 'GetLetters gets the message type correctly' );
 219
 220     $context->unmock('userenv');
 221 };
 222
 223
 224
 225 # Regression test for Bug 14206
 226 $dbh->do( q|INSERT INTO letter(branchcode,module,code,name,is_html,title,content,message_transport_type) VALUES ('FFL','my module','my code','my name',1,?,?,'print')|, undef, $title, $content );
 227 my $letter14206_a = C4::Letters::getletter('my module', 'my code', 'FFL' );
 228 is( $letter14206_a->{message_transport_type}, 'print', 'Bug 14206 - message_transport_type not passed, correct mtt detected' );
 229 my $letter14206_b = C4::Letters::getletter('my module', 'my code', 'FFL', 'print');
 230 is( $letter14206_b->{message_transport_type}, 'print', 'Bug 14206 - message_transport_type passed, correct mtt detected'  );
 231
 232 # test for overdue_notices.pl
 233 my $overdue_rules = {
 234     letter1         => 'my code',
 235 };
 236 my $i = 1;
 237 my $branchcode = 'FFL';
 238 my $letter14206_c = C4::Letters::getletter('my module', $overdue_rules->{"letter$i"}, $branchcode);
 239 is( $letter14206_c->{message_transport_type}, 'print', 'Bug 14206 - correct mtt detected for call from overdue_notices.pl' );
 240
 241 # GetPreparedLetter
 242 t::lib::Mocks::mock_preference('OPACBaseURL', 'http://thisisatest.com');
 243
 244 my $sms_content = 'This is a SMS for an <<status>>';
 245 $dbh->do( q|INSERT INTO letter(branchcode,module,code,name,is_html,title,content,message_transport_type) VALUES (?,'my module','my code','my name',1,'my title',?,'sms')|, undef, $library->{branchcode}, $sms_content );
 246
 247 my $tables = {
 248     borrowers => $borrowernumber,
 249     branches => $library->{branchcode},
 250     biblio => $biblionumber,
 251 };
 252 my $substitute = {
 253     status => 'overdue',
 254 };
 255 my $repeat = [
 256     {
 257         itemcallnumber => 'my callnumber1',
 258         barcode        => '1234',
 259     },
 260     {
 261         itemcallnumber => 'my callnumber2',
 262         barcode        => '5678',
 263     },
 264 ];
 265 my $prepared_letter = GetPreparedLetter((
 266     module      => 'my module',
 267     branchcode  => $library->{branchcode},
 268     letter_code => 'my code',
 269     tables      => $tables,
 270     substitute  => $substitute,
 271     repeat      => $repeat,
 272 ));
 273 my $retrieved_library = Koha::Libraries->find($library->{branchcode});
 274 my $my_title_letter = $retrieved_library->branchname . qq| - $substitute->{status}|;
 275 my $biblio_timestamp = dt_from_string( GetBiblioData($biblionumber)->{timestamp} );
 276 my $my_content_letter = qq|Dear Jane Smith,
 277 According to our current records, you have items that are overdue.Your library does not charge late fines, but please return or renew them at the branch below as soon as possible.
 278
 279 |.$retrieved_library->branchname.qq|
 280 |.$retrieved_library->branchaddress1.qq|
 281 URL: http://thisisatest.com
 282
 283 The following item(s) is/are currently $substitute->{status}:
 284
 285 <item> 1. $repeat->[0]->{itemcallnumber}, Barcode: $repeat->[0]->{barcode} </item>
 286 <item> 2. $repeat->[1]->{itemcallnumber}, Barcode: $repeat->[1]->{barcode} </item>
 287
 288 Thank-you for your prompt attention to this matter.
 289 Don't forget your date of birth: | . output_pref({ dt => $date, dateonly => 1 }) . q|.
 290 Look at this wonderful biblio timestamp: | . output_pref({ dt => $biblio_timestamp })  . ".\n";
 291
 292 is( $prepared_letter->{title}, $my_title_letter, 'GetPreparedLetter returns the title correctly' );
 293 is( $prepared_letter->{content}, $my_content_letter, 'GetPreparedLetter returns the content correctly' );
 294
 295 $prepared_letter = GetPreparedLetter((
 296     module                 => 'my module',
 297     branchcode             => $library->{branchcode},
 298     letter_code            => 'my code',
 299     tables                 => $tables,
 300     substitute             => $substitute,
 301     repeat                 => $repeat,
 302     message_transport_type => 'sms',
 303 ));
 304 $my_content_letter = qq|This is a SMS for an $substitute->{status}|;
 305 is( $prepared_letter->{content}, $my_content_letter, 'GetPreparedLetter returns the content correctly' );
 306
 307 $dbh->do(q{INSERT INTO letter (module, code, name, title, content) VALUES ('test_date','TEST_DATE','Test dates','A title with a timestamp: <<biblio.timestamp>>','This one only contains the date: <<biblio.timestamp | dateonly>>.');});
 308 $prepared_letter = GetPreparedLetter((
 309     module                 => 'test_date',
 310     branchcode             => '',
 311     letter_code            => 'test_date',
 312     tables                 => $tables,
 313     substitute             => $substitute,
 314     repeat                 => $repeat,
 315 ));
 316 is( $prepared_letter->{content}, q|This one only contains the date: | . output_pref({ dt => $date, dateonly => 1 }) . q|.|, 'dateonly test 1' );
 317
 318 $dbh->do(q{UPDATE letter SET content = 'And also this one:<<timestamp | dateonly>>.' WHERE code = 'test_date';});
 319 $prepared_letter = GetPreparedLetter((
 320     module                 => 'test_date',
 321     branchcode             => '',
 322     letter_code            => 'test_date',
 323     tables                 => $tables,
 324     substitute             => $substitute,
 325     repeat                 => $repeat,
 326 ));
 327 is( $prepared_letter->{content}, q|And also this one:| . output_pref({ dt => $date, dateonly => 1 }) . q|.|, 'dateonly test 2' );
 328
 329 $dbh->do(q{UPDATE letter SET content = 'And also this one:<<timestamp|dateonly >>.' WHERE code = 'test_date';});
 330 $prepared_letter = GetPreparedLetter((
 331     module                 => 'test_date',
 332     branchcode             => '',
 333     letter_code            => 'test_date',
 334     tables                 => $tables,
 335     substitute             => $substitute,
 336     repeat                 => $repeat,
 337 ));
 338 is( $prepared_letter->{content}, q|And also this one:| . output_pref({ dt => $date, dateonly => 1 }) . q|.|, 'dateonly test 3' );
 339
 340 t::lib::Mocks::mock_preference( 'TimeFormat', '12hr' );
 341 my $yesterday_night = $date->clone->add( days => -1 )->set_hour(22);
 342 $dbh->do(q|UPDATE biblio SET timestamp = ? WHERE biblionumber = ?|, undef, $yesterday_night, $biblionumber );
 343 $dbh->do(q{UPDATE letter SET content = 'And also this one:<<timestamp>>.' WHERE code = 'test_date';});
 344 $prepared_letter = GetPreparedLetter((
 345     module                 => 'test_date',
 346     branchcode             => '',
 347     letter_code            => 'test_date',
 348     tables                 => $tables,
 349     substitute             => $substitute,
 350     repeat                 => $repeat,
 351 ));
 352 is( $prepared_letter->{content}, q|And also this one:| . output_pref({ dt => $yesterday_night }) . q|.|, 'dateonly test 3' );
 353
 354 $dbh->do(q{INSERT INTO letter (module, code, name, title, content) VALUES ('claimacquisition','TESTACQCLAIM','Acquisition Claim','Item Not Received','<<aqbooksellers.name>>|<<aqcontacts.name>>|<order>Ordernumber <<aqorders.ordernumber>> (<<biblio.title>>) (<<aqorders.quantity>> ordered)</order>');});
 355 $dbh->do(q{INSERT INTO letter (module, code, name, title, content) VALUES ('orderacquisition','TESTACQORDER','Acquisition Order','Order','<<aqbooksellers.name>>|<<aqcontacts.name>>|<order>Ordernumber <<aqorders.ordernumber>> (<<biblio.title>>) (<<aqorders.quantity>> ordered)</order>');});
 356
 357 # Test that _parseletter doesn't modify its parameters bug 15429
 358 {
 359     my $values = { dateexpiry => '2015-12-13', };
 360     C4::Letters::_parseletter($prepared_letter, 'borrowers', $values);
 361     is( $values->{dateexpiry}, '2015-12-13', "_parseletter doesn't modify its parameters" );
 362 }
 363
 364 my $bookseller = Koha::Acquisition::Bookseller->new(
 365     {
 366         name => "my vendor",
 367         address1 => "bookseller's address",
 368         phone => "0123456",
 369         active => 1,
 370         deliverytime => 5,
 371     }
 372 )->store;
 373 my $booksellerid = $bookseller->id;
 374
 375 Koha::Acquisition::Bookseller::Contact->new( { name => 'John Smith',  phone => '0123456x1', claimacquisition => 1, orderacquisition => 1, booksellerid => $booksellerid } )->store;
 376 Koha::Acquisition::Bookseller::Contact->new( { name => 'Leo Tolstoy', phone => '0123456x2', claimissues      => 1, booksellerid => $booksellerid } )->store;
 377 my $basketno = NewBasket($booksellerid, 1);
 378
 379 my $budgetid = C4::Budgets::AddBudget({
 380     budget_code => "budget_code_test_letters",
 381     budget_name => "budget_name_test_letters",
 382 });
 383
 384 my $bib = MARC::Record->new();
 385 if (C4::Context->preference('marcflavour') eq 'UNIMARC') {
 386     $bib->append_fields(
 387         MARC::Field->new('200', ' ', ' ', a => 'Silence in the library'),
 388     );
 389 } else {
 390     $bib->append_fields(
 391         MARC::Field->new('245', ' ', ' ', a => 'Silence in the library'),
 392     );
 393 }
 394
 395 ($biblionumber, $biblioitemnumber) = AddBiblio($bib, '');
 396 my $order = Koha::Acquisition::Order->new(
 397     {
 398         basketno => $basketno,
 399         quantity => 1,
 400         biblionumber => $biblionumber,
 401         budget_id => $budgetid,
 402     }
 403 )->store;
 404 my $ordernumber = $order->ordernumber;
 405
 406 C4::Acquisition::CloseBasket( $basketno );
 407 my $err;
 408 warning_like {
 409     $err = SendAlerts( 'claimacquisition', [ $ordernumber ], 'TESTACQCLAIM' ) }
 410     qr/^Bookseller .* without emails at/,
 411     "SendAlerts prints a warning";
 412 is($err->{'error'}, 'no_email', "Trying to send an alert when there's no e-mail results in an error");
 413
 414 $bookseller = Koha::Acquisition::Booksellers->find( $booksellerid );
 415 $bookseller->contacts->next->email('testemail@mydomain.com')->store;
 416
 417 # Ensure that the preference 'LetterLog' is set to logging
 418 t::lib::Mocks::mock_preference( 'LetterLog', 'on' );
 419
 420 # SendAlerts needs branchemail or KohaAdminEmailAddress as sender
 421 C4::Context->_new_userenv('DUMMY');
 422 C4::Context->set_userenv( 0, 0, 0, 'firstname', 'surname', $library->{branchcode}, 'My Library', 0, '', '');
 423 t::lib::Mocks::mock_preference( 'KohaAdminEmailAddress', 'library@domain.com' );
 424
 425 {
 426 warning_is {
 427     $err = SendAlerts( 'orderacquisition', $basketno , 'TESTACQORDER' ) }
 428     "Fake sendmail",
 429     "SendAlerts is using the mocked sendmail routine (orderacquisition)";
 430 is($err, 1, "Successfully sent order.");
 431 is($mail{'To'}, 'testemail@mydomain.com', "mailto correct in sent order");
 432 is($mail{'Message'}, 'my vendor|John Smith|Ordernumber ' . $ordernumber . ' (Silence in the library) (1 ordered)', 'Order notice text constructed successfully');
 433
 434 $dbh->do(q{DELETE FROM letter WHERE code = 'TESTACQORDER';});
 435 warning_like {
 436     $err = SendAlerts( 'orderacquisition', $basketno , 'TESTACQORDER' ) }
 437     qr/No orderacquisition TESTACQORDER letter transported by email/,
 438     "GetPreparedLetter warns about missing notice template";
 439 is($err->{'error'}, 'no_letter', "No TESTACQORDER letter was defined.");
 440 }
 441
 442 {
 443 warning_is {
 444     $err = SendAlerts( 'claimacquisition', [ $ordernumber ], 'TESTACQCLAIM' ) }
 445     "Fake sendmail",
 446     "SendAlerts is using the mocked sendmail routine";
 447
 448 is($err, 1, "Successfully sent claim");
 449 is($mail{'To'}, 'testemail@mydomain.com', "mailto correct in sent claim");
 450 is($mail{'Message'}, 'my vendor|John Smith|Ordernumber ' . $ordernumber . ' (Silence in the library) (1 ordered)', 'Claim notice text constructed successfully');
 451 }
 452
 453 {
 454 use C4::Serials;
 455
 456 my $notes = 'notes';
 457 my $internalnotes = 'intnotes';
 458 $dbh->do(q|UPDATE subscription_numberpatterns SET numberingmethod='No. {X}' WHERE id=1|);
 459 my $subscriptionid = NewSubscription(
 460      undef,      "",     undef, undef, undef, $biblionumber,
 461     '2013-01-01', 1, undef, undef,  undef,
 462     undef,      undef,  undef, undef, undef, undef,
 463     1,          $notes,undef, '2013-01-01', undef, 1,
 464     undef,       undef,  0,    $internalnotes,  0,
 465     undef, undef, 0,          undef,         '2013-12-31', 0
 466 );
 467 $dbh->do(q{INSERT INTO letter (module, code, name, title, content) VALUES ('serial','RLIST','Serial issue notification','Serial issue notification','<<biblio.title>>,<<subscription.subscriptionid>>,<<serial.serialseq>>');});
 468 my ($serials_count, @serials) = GetSerials($subscriptionid);
 469 my $serial = $serials[0];
 470
 471 my $borrowernumber = AddMember(
 472     firstname    => 'John',
 473     surname      => 'Smith',
 474     categorycode => $patron_category,
 475     branchcode   => $library->{branchcode},
 476     dateofbirth  => $date,
 477     email        => 'john.smith@test.de',
 478 );
 479 my $subscription = Koha::Subscriptions->find( $subscriptionid );
 480 $subscription->add_subscriber( scalar Koha::Patrons->find( $borrowernumber ) );
 481
 482 my $err2;
 483 warning_is {
 484 $err2 = SendAlerts( 'issue', $serial->{serialid}, 'RLIST' ) }
 485     "Fake sendmail",
 486     "SendAlerts is using the mocked sendmail routine";
 487 is($err2, 1, "Successfully sent serial notification");
 488 is($mail{'To'}, 'john.smith@test.de', "mailto correct in sent serial notification");
 489 is($mail{'Message'}, 'Silence in the library,'.$subscriptionid.',No. 0', 'Serial notification text constructed successfully');
 490 }
 491
 492 subtest 'SendAlerts - claimissue' => sub {
 493     plan tests => 8;
 494
 495     use C4::Serials;
 496
 497     $dbh->do(q{INSERT INTO letter (module, code, name, title, content) VALUES ('claimissues','TESTSERIALCLAIM','Serial claim test','Serial claim test','<<serial.serialid>>|<<subscription.startdate>>|<<biblio.title>>|<<biblioitems.issn>>');});
 498
 499     my $bookseller = Koha::Acquisition::Bookseller->new(
 500         {
 501             name => "my vendor",
 502             address1 => "bookseller's address",
 503             phone => "0123456",
 504             active => 1,
 505             deliverytime => 5,
 506         }
 507     )->store;
 508     my $booksellerid = $bookseller->id;
 509
 510     Koha::Acquisition::Bookseller::Contact->new( { name => 'Leo Tolstoy', phone => '0123456x2', claimissues => 1, booksellerid => $booksellerid } )->store;
 511
 512     my $bib = MARC::Record->new();
 513     if (C4::Context->preference('marcflavour') eq 'UNIMARC') {
 514         $bib->append_fields(
 515             MARC::Field->new('011', ' ', ' ', a => 'xxxx-yyyy'),
 516             MARC::Field->new('200', ' ', ' ', a => 'Silence in the library'),
 517         );
 518     } else {
 519         $bib->append_fields(
 520             MARC::Field->new('022', ' ', ' ', a => 'xxxx-yyyy'),
 521             MARC::Field->new('245', ' ', ' ', a => 'Silence in the library'),
 522         );
 523     }
 524     my ($biblionumber) = AddBiblio($bib, '');
 525
 526     $dbh->do(q|UPDATE subscription_numberpatterns SET numberingmethod='No. {X}' WHERE id=1|);
 527     my $subscriptionid = NewSubscription(
 528          undef, "", $booksellerid, undef, undef, $biblionumber,
 529         '2013-01-01', 1, undef, undef,  undef,
 530         undef,  undef,  undef, undef, undef, undef,
 531         1, 'public',undef, '2013-01-01', undef, 1,
 532         undef, undef,  0, 'internal',  0,
 533         undef, undef, 0,  undef, '2013-12-31', 0
 534     );
 535
 536     my ($serials_count, @serials) = GetSerials($subscriptionid);
 537     my  @serialids = ($serials[0]->{serialid});
 538
 539     my $err;
 540     warning_like {
 541         $err = SendAlerts( 'claimissues', \@serialids, 'TESTSERIALCLAIM' ) }
 542         qr/^Bookseller .* without emails at/,
 543         "Warn on vendor without email address";
 544
 545     $bookseller = Koha::Acquisition::Booksellers->find( $booksellerid );
 546     $bookseller->contacts->next->email('testemail@mydomain.com')->store;
 547
 548     # Ensure that the preference 'LetterLog' is set to logging
 549     t::lib::Mocks::mock_preference( 'LetterLog', 'on' );
 550
 551     # SendAlerts needs branchemail or KohaAdminEmailAddress as sender
 552     C4::Context->_new_userenv('DUMMY');
 553     C4::Context->set_userenv( 0, 0, 0, 'firstname', 'surname', $library->{branchcode}, 'My Library', 0, '', '');
 554     t::lib::Mocks::mock_preference( 'KohaAdminEmailAddress', 'library@domain.com' );
 555
 556     {
 557     warning_is {
 558         $err = SendAlerts( 'claimissues', \@serialids , 'TESTSERIALCLAIM' ) }
 559         "Fake sendmail",
 560         "SendAlerts is using the mocked sendmail routine (claimissues)";
 561     is($err, 1, "Successfully sent claim");
 562     is($mail{'To'}, 'testemail@mydomain.com', "mailto correct in sent claim");
 563     is($mail{'Message'}, "$serialids[0]|2013-01-01|Silence in the library|xxxx-yyyy", 'Serial claim letter for 1 issue constructed successfully');
 564     }
 565
 566     {
 567     my $publisheddate = output_pref({ dt => dt_from_string, dateformat => 'iso', dateonly => 1 });
 568     my $serialexpected = ( C4::Serials::findSerialsByStatus( 1, $subscriptionid ) )[0];
 569     ModSerialStatus( $serials[0]->{serialid}, "No. 1", $publisheddate, $publisheddate, $publisheddate, '3', 'a note' );
 570     ($serials_count, @serials) = GetSerials($subscriptionid);
 571     push @serialids, ($serials[1]->{serialid});
 572
 573     $err = SendAlerts( 'claimissues', \@serialids , 'TESTSERIALCLAIM' );
 574     is($mail{'Message'}, "$serialids[0]|2013-01-01|Silence in the library|xxxx-yyyy\n$serialids[1]|2013-01-01|Silence in the library|xxxx-yyyy", "Serial claim letter for 2 issues constructed successfully");
 575
 576     $dbh->do(q{DELETE FROM letter WHERE code = 'TESTSERIALCLAIM';});
 577     warning_like {
 578         $err = SendAlerts( 'orderacquisition', $basketno , 'TESTSERIALCLAIM' ) }
 579         qr/No orderacquisition TESTSERIALCLAIM letter transported by email/,
 580         "GetPreparedLetter warns about missing notice template";
 581     is($err->{'error'}, 'no_letter', "No TESTSERIALCLAIM letter was defined");
 582     }
 583
 584 };
 585
 586 subtest 'GetPreparedLetter' => sub {
 587     plan tests => 4;
 588
 589     Koha::Notice::Template->new(
 590         {
 591             module                 => 'test',
 592             code                   => 'test',
 593             branchcode             => '',
 594             message_transport_type => 'email'
 595         }
 596     )->store;
 597     my $letter;
 598     warning_like {
 599         $letter = C4::Letters::GetPreparedLetter(
 600             module      => 'test',
 601             letter_code => 'test',
 602         );
 603     }
 604     qr{^ERROR: nothing to substitute},
 605 'GetPreparedLetter should warn if tables, substiture and repeat are not set';
 606     is( $letter, undef,
 607 'No letter should be returned by GetPreparedLetter if something went wrong'
 608     );
 609
 610     warning_like {
 611         $letter = C4::Letters::GetPreparedLetter(
 612             module      => 'test',
 613             letter_code => 'test',
 614             substitute  => {}
 615         );
 616     }
 617     qr{^ERROR: nothing to substitute},
 618 'GetPreparedLetter should warn if tables, substiture and repeat are not set, even if the key is passed';
 619     is( $letter, undef,
 620 'No letter should be returned by GetPreparedLetter if something went wrong'
 621     );
 622
 623 };
 624
 625
 626
 627 subtest 'TranslateNotices' => sub {
 628     plan tests => 4;
 629
 630     t::lib::Mocks::mock_preference( 'TranslateNotices', '1' );
 631
 632     $dbh->do(
 633         q|
 634         INSERT INTO letter (module, code, branchcode, name, title, content, message_transport_type, lang) VALUES
 635         ('test', 'code', '', 'test', 'a test', 'just a test', 'email', 'default'),
 636         ('test', 'code', '', 'test', 'una prueba', 'solo una prueba', 'email', 'es-ES');
 637     | );
 638     my $substitute = {};
 639     my $letter = C4::Letters::GetPreparedLetter(
 640             module                 => 'test',
 641             tables                 => $tables,
 642             letter_code            => 'code',
 643             message_transport_type => 'email',
 644             substitute             => $substitute,
 645     );
 646     is(
 647         $letter->{title},
 648         'a test',
 649         'GetPreparedLetter should return the default one if the lang parameter is not provided'
 650     );
 651
 652     $letter = C4::Letters::GetPreparedLetter(
 653             module                 => 'test',
 654             tables                 => $tables,
 655             letter_code            => 'code',
 656             message_transport_type => 'email',
 657             substitute             => $substitute,
 658             lang                   => 'es-ES',
 659     );
 660     is( $letter->{title}, 'una prueba',
 661         'GetPreparedLetter should return the required notice if it exists' );
 662
 663     $letter = C4::Letters::GetPreparedLetter(
 664             module                 => 'test',
 665             tables                 => $tables,
 666             letter_code            => 'code',
 667             message_transport_type => 'email',
 668             substitute             => $substitute,
 669             lang                   => 'fr-FR',
 670     );
 671     is(
 672         $letter->{title},
 673         'a test',
 674         'GetPreparedLetter should return the default notice if the one required does not exist'
 675     );
 676
 677     t::lib::Mocks::mock_preference( 'TranslateNotices', '' );
 678
 679     $letter = C4::Letters::GetPreparedLetter(
 680             module                 => 'test',
 681             tables                 => $tables,
 682             letter_code            => 'code',
 683             message_transport_type => 'email',
 684             substitute             => $substitute,
 685             lang                   => 'es-ES',
 686     );
 687     is( $letter->{title}, 'a test',
 688         'GetPreparedLetter should return the default notice if pref disabled but additional language exists' );
 689
 690 };
 691
 692 subtest 'SendQueuedMessages' => sub {
 693
 694     plan tests => 4;
 695     t::lib::Mocks::mock_preference( 'SMSSendDriver', 'Email' );
 696     my $patron = Koha::Patrons->find($borrowernumber);
 697     $dbh->do(q|
 698         INSERT INTO message_queue(borrowernumber, subject, content, message_transport_type, status, letter_code)
 699         VALUES (?, 'subject', 'content', 'sms', 'pending', 'just_a_code')
 700         |, undef, $borrowernumber
 701     );
 702     eval { C4::Letters::SendQueuedMessages(); };
 703     is( $@, '', 'SendQueuedMessages should not explode if the patron does not have a sms provider set' );
 704
 705     my $sms_pro = $builder->build_object({ class => 'Koha::SMS::Providers', value => { domain => 'kidclamp.rocks' } });
 706     ModMember( borrowernumber => $borrowernumber, smsalertnumber => '5555555555', sms_provider_id => $sms_pro->id() );
 707     $message_id = C4::Letters::EnqueueLetter($my_message); #using datas set around line 95 and forward
 708     C4::Letters::SendQueuedMessages();
 709     my $sms_message_address = $schema->resultset('MessageQueue')->search({
 710         borrowernumber => $borrowernumber,
 711         status => 'sent'
 712     })->next()->to_address();
 713     is( $sms_message_address, '5555555555@kidclamp.rocks', 'SendQueuedMessages populates the to address correctly for SMS by email when to_address not set' );
 714     $schema->resultset('MessageQueue')->search({borrowernumber => $borrowernumber,status => 'sent'})->delete(); #clear borrower queue
 715     $my_message->{to_address} = 'fixme@kidclamp.iswrong';
 716     $message_id = C4::Letters::EnqueueLetter($my_message);
 717
 718     my $number_attempted = C4::Letters::SendQueuedMessages({
 719         borrowernumber => -1, # -1 still triggers the borrowernumber condition
 720         letter_code    => 'PASSWORD_RESET',
 721     });
 722     is ( $number_attempted, 0, 'There were no password reset messages for SendQueuedMessages to attempt.' );
 723
 724     C4::Letters::SendQueuedMessages();
 725     $sms_message_address = $schema->resultset('MessageQueue')->search({
 726         borrowernumber => $borrowernumber,
 727         status => 'sent'
 728     })->next()->to_address();
 729     is( $sms_message_address, '5555555555@kidclamp.rocks', 'SendQueuedMessages populates the to address correctly for SMS by email when to_address is set incorrectly' );
 730
 731 };
 732
 733 subtest 'get_item_content' => sub {
 734     plan tests => 2;
 735
 736     t::lib::Mocks::mock_preference('dateformat', 'metric');
 737     t::lib::Mocks::mock_preference('timeformat', '24hr');
 738     my @items = (
 739         {date_due => '2041-01-01 12:34', title => 'a first title', barcode => 'a_first_barcode', author => 'a_first_author', itemnumber => 1 },
 740         {date_due => '2042-01-02 23:45', title => 'a second title', barcode => 'a_second_barcode', author => 'a_second_author', itemnumber => 2 },
 741     );
 742     my @item_content_fields = qw( date_due title barcode author itemnumber );
 743
 744     my $items_content;
 745     for my $item ( @items ) {
 746         $items_content .= C4::Letters::get_item_content( { item => $item, item_content_fields => \@item_content_fields } );
 747     }
 748
 749     my $expected_items_content = <<EOF;
 750 01/01/2041 12:34\ta first title\ta_first_barcode\ta_first_author\t1
 751 02/01/2042 23:45\ta second title\ta_second_barcode\ta_second_author\t2
 752 EOF
 753     is( $items_content, $expected_items_content, 'get_item_content should return correct items info with time (default)' );
 754
 755
 756     $items_content = q||;
 757     for my $item ( @items ) {
 758         $items_content .= C4::Letters::get_item_content( { item => $item, item_content_fields => \@item_content_fields, dateonly => 1, } );
 759     }
 760
 761     $expected_items_content = <<EOF;
 762 01/01/2041\ta first title\ta_first_barcode\ta_first_author\t1
 763 02/01/2042\ta second title\ta_second_barcode\ta_second_author\t2
 764 EOF
 765     is( $items_content, $expected_items_content, 'get_item_content should return correct items info without time (if dateonly => 1)' );
 766 };
 767
 768 subtest 'Test limit parameter for SendQueuedMessages' => sub {
 769     plan tests => 3;
 770
 771     my $dbh = C4::Context->dbh;
 772
 773     my $borrowernumber = AddMember(
 774         firstname    => 'Jane',
 775         surname      => 'Smith',
 776         categorycode => $patron_category,
 777         branchcode   => $library->{branchcode},
 778         dateofbirth  => $date,
 779         smsalertnumber => undef,
 780     );
 781
 782     $dbh->do(q|DELETE FROM message_queue|);
 783     $my_message = {
 784         'letter' => {
 785             'content'      => 'a message',
 786             'metadata'     => 'metadata',
 787             'code'         => 'TEST_MESSAGE',
 788             'content_type' => 'text/plain',
 789             'title'        => 'message title'
 790         },
 791         'borrowernumber'         => $borrowernumber,
 792         'to_address'             => undef,
 793         'message_transport_type' => 'sms',
 794         'from_address'           => 'from@example.com'
 795     };
 796     C4::Letters::EnqueueLetter($my_message);
 797     C4::Letters::EnqueueLetter($my_message);
 798     C4::Letters::EnqueueLetter($my_message);
 799     C4::Letters::EnqueueLetter($my_message);
 800     C4::Letters::EnqueueLetter($my_message);
 801     my $messages_processed = C4::Letters::SendQueuedMessages( { limit => 1 } );
 802     is( $messages_processed, 1,
 803         'Processed 1 message with limit of 1 and 5 unprocessed messages' );
 804     $messages_processed = C4::Letters::SendQueuedMessages( { limit => 2 } );
 805     is( $messages_processed, 2,
 806         'Processed 2 message with limit of 2 and 4 unprocessed messages' );
 807     $messages_processed = C4::Letters::SendQueuedMessages( { limit => 3 } );
 808     is( $messages_processed, 2,
 809         'Processed 2 message with limit of 3 and 2 unprocessed messages' );
 810 };