http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6835
--- Comment #30 from M. de Rooy <[email protected]> --- Still two questions looking at the code again: 1) $email = C4::Members::GetFirstValidEmailAddress($borrowernumber); push @emails_to_use, $email; Shouldn't you test what you get back from this routine? Could be nothing. Could you use a temp variable instead of $email? (see point 2) 2) $email = 1 if (@emails_to_use); Does this work later on? I would rather test later on @emails_to_use and not change a variable that is a record value used in the while loop (l.488). See also line 535. And the template vars on 591/620. Thanks. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
