Bug 5929

Summary: Use branch admin email for advance_notice.pl emails
Product: Koha Reporter: Shelley <shelley>
Component: System AdministrationAssignee: Srdjan Jankovic <srdjan>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: enhancement    
Priority: PATCH-Sent (DO NOT USE) CC: chris, f.demians, gmcharlt, koha.sekjal
Version: 3.6   
Hardware: All   
OS: All   
Change sponsored?: Sponsored Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on:    
Bug Blocks: 5544    
Attachments: patch
patch
Squashed, Signed off commit for fix

Description Shelley 2011-03-22 01:15:46 UTC
Instead of using the KohaAdminEmailAddress system preference, the PREDUE and DUE notices should send 'from' the email address configured in the branch management area, just like Overdues. KohaAdminEmailAddress should be a fallback if no address is configured for the branch. 

The branch selected should be the same as is selected for overdues.


Work to be done: 
  -  Find the patch that added the functionality to overdues 
  -  Duplicate the functionality for DUE and PREDUE templates
Comment 1 Srdjan Jankovic 2011-03-22 06:07:58 UTC Comment hidden (obsolete)
Comment 2 Srdjan Jankovic 2011-03-28 23:44:56 UTC Comment hidden (obsolete)
Comment 3 Ian Walls 2011-06-03 20:32:55 UTC
Created attachment 4355 [details] [review]
Squashed, Signed off commit for fix

Takes the previous two attachments, squashes, and adds a minor fix to template toolkit compatibility.
Comment 4 Chris Cormack 2011-06-04 07:41:28 UTC
Pushed to master please test
Comment 5 Ian Walls 2011-08-02 20:23:14 UTC
Tested on production system, works.  Marking as resolved.