Bug 32455: Use from_email_address for 'from' field for hold notices
authorNick Clemens <nick@bywatersolutions.com>
Tue, 13 Dec 2022 12:01:04 +0000 (12:01 +0000)
committerJacob O'Mara <jacob.omara@ptfs-europe.com>
Wed, 8 Feb 2023 16:23:12 +0000 (16:23 +0000)
commiteb32f03103872ede5946f0c60caeb4c3020c2983
treee4cc7b2f96443db5108d8001682ec8851c5fbf32
parentff9b5f1275c50dad5b77b39df15c065a8b58bf8f
Bug 32455: Use from_email_address for 'from' field for hold notices

This patch updates two occurrences where the inbound library email is
used as the from address

To test:
 1 - Set a unique 'Email' and 'Reply to' address for a library
 2 - Find a patron of that library, ensure they have an email
 3 - Ensure their messaging preference for holds is 'email'
 4 - Set system preference ReservesMaxPickupDelay to -1
 5 - Set system preference  ExpireReservesMaxPickUpDelay to Allow
 6 - Set system preference ExpireReservesAutoFill to Do
 7 - Place and fill a hold for that patron at that library
 8 - Check the patron's notification tab, confirm the from address is the 'Reply to'
 9 - Place a hold for another patron on the same item
10 - Run the expired holds cronjob:
     perl misc/cronjobs/holds/cancel_expired_holds.pl --reason=whatever
11 - Check the message_queue - notice the from address is the 'reply to'
12 - Apply patch
13 - Repeat 1-11, confirm the from addresses are correct now

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit ed30887924896b883b3cba9ddc50543af3f2636e)
Signed-off-by: Jacob O'Mara <jacob.omara@ptfs-europe.com>
C4/Reserves.pm