Bug 9815 - <<branches.branchemail>> can't contain "The Library" <library@example.com>
Summary: <<branches.branchemail>> can't contain "The Library" <library@example.com>
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: Notices (show other bugs)
Version: master
Hardware: All All
: P5 - low trivial (vote)
Assignee: Bugs List
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-03-14 04:06 UTC by wajasu
Modified: 2021-08-23 14:06 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Text to go in the release notes:
Version(s) released in:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description wajasu 2013-03-14 04:06:06 UTC
I had a situation where I used syspref
KohaAdminEmailAddress:     "The Library" <thelibrary@example.com>
and my overdue notices went out fine.

Then at a later point the branch details email field had
thelibrary@example.com
and was put in the overdue notice  as   <<branches.branchemail>>
and the notice generated fine.
BUT
the overdue notice From: now became  thelibrary@example.com
which wasn't as pretty.
THEN
I put   "The Library" <thelibrary@example.com>
in the branch details email field
BUT the notice showed:   "The Library"
as <thelibrary@example.com> got gobbled up by the Letters template code.
It thought it was a stragling  <tag> that had no substitution.

Workaround:
a) blank out  branch details email field, so KohaAdminEmailAddress is used.
b) replace  <<branches.branchemail>>   witht he literal branch email address.

Just providing a workaround.
Comment 1 Katrin Fischer 2021-07-13 14:19:35 UTC
*** Bug 28706 has been marked as a duplicate of this bug. ***
Comment 2 Katrin Fischer 2021-07-13 14:19:43 UTC
*** Bug 24233 has been marked as a duplicate of this bug. ***
Comment 3 Katrin Fischer 2021-07-13 14:22:21 UTC
Hm, realizing this bug is about the notices, but still interesting in context of bugs 24233 and 28706 and I just KNOW there was another bug where I discussed this change with Owen on :(
Comment 4 Katrin Fischer 2021-07-13 14:26:47 UTC
Found it - it was 15349. Redirecting the duplicates.