Core

Core Advanced Frontend Editing DBAL Extbase MVC Framework Fluid Indexed Search Linkvalidator Scheduler t3editor Team Resources Workspaces & Versioning Community Extensions Incubator Distributions TYPO3 4.5 Projects TYPO3 4.7 Projects TYPO3 6.0 Projects TYPO3 6.1 Projects TYPO3 6.2 Projects (+)

Bug #25301

Setting "Reply-To" fails in class.t3lib_formmail.php

Added by Jan Radecker over 2 years ago. Updated about 2 years ago.

Status:

Resolved

Start date:

2011-03-10

Priority:

Should have

Due date:

Assignee:

Jigal van Hemert

% Done:

100%

Category:

Target version:

TYPO3 Version:

4.5

Complexity:

PHP Version:

5.2

Votes:

Description

t3lib_formmail tries to set a "Reply-To:" Header but fails doing so because of a wrong call of method "addReplyTo()" from swiftmailer.

The method "addReplyTo()" of class "t3lib_mail_Message" (which extends Swift_Message) expects to parameters of "string" type.
But t3lib_formmail sets an array as argument to "addReplyTo()".

Original Code (line 232...):
$replyTo = $this->replyToName ? array($this->replyToAddress => $this->replyToName) : array($this->replyToAddress);
$this->mailMessage->addReplyTo($replyTo);

This can be fixed by using this line instead:
$this->mailMessage->addReplyTo($this->replyToAddress, $this->replyToName);

(issue imported from #M17916)

Related issues

duplicated by Core - Bug #25415: t3lib_formmail: addReplyTo method of Swift Mailer SimpleM...

Resolved

2011-03-29

Associated revisions

Revision 515d8b31
Added by Jigal van Hemert about 2 years ago

[BUGFIX] Use setReplyTo and setCc instead of add*

The methods addReplyTo and addCc accept strings as parameters, but we
need the setReplyTo and setCc which accept arrays. This will make the
reply-to and recipient-copy options work in formMail.

Change-Id: Ia3826d51f53ffe497f4c3b25957f0332fb89e81e
Resolves: #25301
Releases: 4.6
Reviewed-on: http://review.typo3.org/1442
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn