Bug 18775

Summary: The "Password Reset" notice should use the patron's homebranch's email as "from" address
Product: Koha Reporter: David Kuhn <techservspec>
Component: OPACAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: delcastillovictorandres, fridolin.somers, katrin.fischer, paul.poulain, techservspec
Version: MainKeywords: Academy
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18570
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18988
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 18775 - The "Password Reset" notice should use the patron's homebranch's email as "from" address
Bug 18775 - The "Password Reset" notice should use the patron's homebranch's email as "from" address
Bug 18775 - The "Password Reset" notice should use the patron's homebranch's email as "from" address
Bug 18775: Use Koha::Patron->library instead of Koha::Libraries->find

Description David Kuhn 2017-06-10 18:37:30 UTC
The password reset notice currently uses the email address stored in the KohaAdminEmailAddress system preference as the "from" address. This can be confusing to patrons (or even make them suspicious) since it may not be the same address (or even the same domain) as the rest of their email notices.

This enhancement would use the "from" address set for the patron's home branch and only use the Koha Admin Email Address as a fallback if there is not an address set for the home branch.
Comment 1 David Kuhn 2017-06-10 20:12:14 UTC
Created attachment 64173 [details] [review]
Bug 18775 - The "Password Reset" notice should use the patron's homebranch's email as "from" address

To test:
1. Enable the password recovery feature.
2. In the OPAC, click on "Forgot you password?" link and enter requested information.
3. The email with the password reset link has the Koha Admin Email Address listed as the sender.
4. Apply the patch.
5. Repeat step 2. The email with the password reset link now has the the "from" address set for the patron's home branch listed as the sender.
Comment 2 Jessica 2017-07-06 23:17:37 UTC
Created attachment 64875 [details] [review]
Bug 18775 - The "Password Reset" notice should use the patron's homebranch's email as "from" address

To test:
1. Enable the password recovery feature.
2. In the OPAC, click on "Forgot you password?" link and enter requested information.
3. The email with the password reset link has the Koha Admin Email Address listed as the sender.
4. Apply the patch.
5. Repeat step 2. The email with the password reset link now has the the "from" address set for the patron's home branch listed as the sender.

Signed-off-by: Jessica Freeman <jessicafreeman@catalyst.net.nz>
Comment 3 Jonathan Druart 2017-07-13 19:41:03 UTC
Created attachment 65024 [details] [review]
Bug 18775 - The "Password Reset" notice should use the patron's homebranch's email as "from" address

To test:
1. Enable the password recovery feature.
2. In the OPAC, click on "Forgot you password?" link and enter requested information.
3. The email with the password reset link has the Koha Admin Email Address listed as the sender.
4. Apply the patch.
5. Repeat step 2. The email with the password reset link now has the the "from" address set for the patron's home branch listed as the sender.

Signed-off-by: Jessica Freeman <jessicafreeman@catalyst.net.nz>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 4 Jonathan Druart 2017-07-13 19:41:07 UTC
Created attachment 65025 [details] [review]
Bug 18775: Use Koha::Patron->library instead of Koha::Libraries->find

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 5 Jonathan Druart 2017-07-13 21:42:39 UTC
Pushed to master for 17.11, thanks to everybody involved!
Comment 6 Fridolin Somers 2017-07-18 12:37:09 UTC
Enhancement not pushed to 17.05.x
Comment 7 Katrin Fischer 2017-08-15 15:56:05 UTC
Hm, I think we should really use Koha::Email here to have the same logic as other emails. There is another related bug 18988, also commenting there.
Comment 8 Katrin Fischer 2017-08-15 16:07:19 UTC
(In reply to Katrin Fischer from comment #7)
> Hm, I think we should really use Koha::Email here to have the same logic as
> other emails. There is another related bug 18988, also commenting there.

Sigh - after looking at the code more closely, that is nonsense. The password reset emails are using the message_queue which makes them work just like they should.
Comment 9 VikDC 2018-01-22 21:58:28 UTC
hi, i've made all set that you guys mentioned there but i can't make it work, if i use this command: "echo "TEST MAIL" | mail -s "Email test subject" myemail@domain.com"  I got email in my account gmail, but when i try to use "forgotten password" koha (opac) says that i'll recieve an email, but i got nothing, i hope you can help me, i have more than a week trying to make it work, thanks
Comment 10 Owen Leonard 2018-02-12 16:34:04 UTC
VikDC: 

This sounds like a support request rather than a bug.

Questions about Koha should be asked on the Koha mailing list (http://koha-community.org/support/koha-mailing-lists/) or in the #koha IRC channel (http://koha-community.org/get-involved/irc/).

Hopefully you've already seen the Koha documentation: http://koha-community.org/documentation/

You can also find options for paid support here: https://koha-community.org/support/