[Koha-bugs] [Bug 12802] Sent notices using several email addresses

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12802

Jonathan Druart <jonathan.druart at biblibre.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Failed QA

--- Comment #42 from Jonathan Druart <jonathan.druart at 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.