Bugzilla – Attachment 58677 Details for
Bug 17866
Change sender for claim and order notices
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 17866: Change sender for serial claim notifications
Bug-17866-Change-sender-for-serial-claim-notificat.patch (text/plain), 1.65 KB, created by
Marcel de Rooy
on 2017-01-09 14:16:04 UTC
(
hide
)
Description:
Bug 17866: Change sender for serial claim notifications
Filename:
MIME Type:
Creator:
Marcel de Rooy
Created:
2017-01-09 14:16:04 UTC
Size:
1.65 KB
patch
obsolete
>From 75cec9e9399f80716387a209dedd40e18355daf8 Mon Sep 17 00:00:00 2001 >From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> >Date: Mon, 9 Jan 2017 15:10:56 +0100 >Subject: [PATCH] Bug 17866: Change sender for serial claim notifications >Content-Type: text/plain; charset=utf-8 > >Currently, the From field uses the emailaddress of the staff client member >from the user context. This is exceptional in Koha. And might very well, as >in our case, be the cause of fraud detection warnings. >We should branch email address or fallback to KohaAdminEmailAddress, as we >do (almost) everywhere else. > >Test plan: >Go to subscription detail in Koha. >Go to Claims. >Select a missing issue and click Send notification. >Verify the sender address in the generated notification. (Make sure that >you receive this mail.) >--- > C4/Letters.pm | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > >diff --git a/C4/Letters.pm b/C4/Letters.pm >index 3ff39b1..3f69477 100644 >--- a/C4/Letters.pm >+++ b/C4/Letters.pm >@@ -572,10 +572,11 @@ sub SendAlerts { > $letter->{content} =~ s/<order>(.*?)<\/order>/$1/gxms; > > # ... then send mail >+ my $branchdetails = GetBranchDetail($userenv->{'branch'}); > my %mail = ( > To => join( ',', @email), > Cc => join( ',', @cc), >- From => $userenv->{emailaddress}, >+ From => $branchdetails->{'branchemail'} || C4::Context->preference('KohaAdminEmailAddress'), > Subject => Encode::encode( "UTF-8", "" . $letter->{title} ), > Message => $letter->{'is_html'} > ? _wrap_html( Encode::encode( "UTF-8", $letter->{'content'} ), >-- >2.1.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 17866
:
58676
|
58677
|
58721
|
59043
|
61473
|
61517
|
61583
|
61584
|
61585
|
61774
|
61776