Created attachment 3304 [details] [review] [PATCH] Improved the email checking when sending "Hold filled" notices. The subroutine for sending HOLD and HOLD_PRINT notices only checked the "email" field of the borrowers table and didn't check the value of the AutoEmailPrimaryAddress preference. With this patch, "Hold filled" notices can now be sent using the other email addresses of the member, depending on the value of the AutoEmailPrimaryAddress preference. Also, if AutoEmailPrimaryAddress is set to "first valid", the other email fields of the user are checked before falling back to a print notice. Patch attached.
Created attachment 4302 [details] [review] Bug 5867 : Improved the email checking when sending "Hold filled" notices. The subroutine for sending HOLD and HOLD_PRINT notices only checked the "email" field of the borrowers table and didn't check the value of the AutoEmailPrimaryAddress preference. With this patch, "Hold filled" notices can now be sent using the other email addresses of the member. http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5867 Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed off patch attached
Easy patch to read, uses existing subroutines correctly. Marking as Passed QA
Pushed, please test