To recreate: 1. Set a overdue trigger for print notice 2. Set up an overdue to be captured by overdue_notices.pl 3. Make sure the patron has an email address 4. Run overdue_notices.pl 5. Run gather_print_notices.pl 6. Now go to the patron's notice page (members/notices.pl?borrowernumber=x) 7. Find the print row for the overdue just generated. 8. See that under the 'Delivery note' it says "To: test@test.com". This is confusing to staff as it is a print notice and wasn't delivered to an email address.
Created attachment 177718 [details] [review] Bug 39089: Do not show an email address when the tranport type is print To test: 1. Set a overdue trigger for print notice 2. Set up an overdue to be captured by overdue_notices.pl 3. Make sure the patron has an email address 4. Run overdue_notices.pl 5. Run gather_print_notices.pl 6. Now go to the patron's notice page (members/notices.pl?borrowernumber=x) 7. Find the print row for the overdue just generated. 8. See that under the 'Delivery note' it says "To: test@test.com". 9. APPLY PATCH 10. Try again, you should now see no "To:" field when the transport type is print 11. Make sure that when the transport type is anything but print, you do see it.
Created attachment 177915 [details] [review] Bug 39089: Do not show an email address when the tranport type is print To test: 1. Set a overdue trigger for print notice 2. Set up an overdue to be captured by overdue_notices.pl 3. Make sure the patron has an email address 4. Run overdue_notices.pl 5. Run gather_print_notices.pl 6. Now go to the patron's notice page (members/notices.pl?borrowernumber=x) 7. Find the print row for the overdue just generated. 8. See that under the 'Delivery note' it says "To: test@test.com". 9. APPLY PATCH 10. Try again, you should now see no "To:" field when the transport type is print 11. Make sure that when the transport type is anything but print, you do see it.
Rebased using the auto-rebase script.