http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12802 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #42 from Jonathan Druart <jonathan.druart@biblibre.com> --- (In reply to Matthias Meusburger from comment #41)
Letters.t tests are ok without the patch, but Reserves.t are not. (cf http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12802#c36 ).
This is because when sending a print message, if the email address is not gathered, this loop in C4::Letters::EnqueueLetter is never entered:
for my $to_address ( @to_addresses ) { $rs->create({ ... }
The to_addresses should only be populated for email. Otherwise if the patron does not have an email defined, the print/sms/phone/whatever letters won't be sent. Actually there is something wrong, but it should be fixed cleanly. -- You are receiving this mail because: You are watching all bug changes.