https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28514
--- Comment #8 from Marcel de Rooy <[email protected]> --- Just seeing a mail problem here (not from this patch): sub inbound_email_address { my ($self) = @_; return $self->branchreplyto || $self->branchemail || C4::Context->preference('ReplytoDefault') || C4::Context->preference('KohaAdminEmailAddress') || undef; ReplyToDefault should imo not be used as the from address. Note that the preferences advertize: KohaAdminEmailAddress Email address for the administrator of Koha: (This is the default From: address for emails unless there is one for the particular library, and is referred to when an internal error occurs.) ReplytoDefault Email address to be set as the replyto in emails: Same for branchreplyto. Note that my server may send mails as library.rijksmuseum.nl and wants replies to rijksmuseum.nl but is not allowed to send as rijksmuseum.nl. So this code is bad and triggers spam lists etc. So actually we should do: return $self->branchemail || C4::Context->preference('KohaAdminEmailAddress'); An || undef makes not much sense? Will open a new report too. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
