Bug #88550

EXT:form: It's not possible to set the value for reply-to dynamically

Added by Wolfgang Wagner 8 months ago. Updated about 1 month ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Form Framework
Target version:
-
Start date:
2019-06-13
Due date:
% Done:

100%

TYPO3 Version:
10
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

In the current TYPO3-Master 10.0.0-dev it is not possible to use the value for reply-to from the form input in a form.

Due to the increasingly stringent spam checks by e-mail providers, it should be possible to read the value for reply-to from the form. It should therefore be possible to use the e-mail address that a user enters in the frontend.

At the moment this does not seem to work, when sending the form an error message appears:

Swift_RfcComplianceException
Address in mailbox given [{email-1}] does not comply with RFC 2822, 3.6.2.

Attached you find a screenshot of my form configuration.

2019-06-13_08-53-32.jpg View (76.1 KB) Wolfgang Wagner, 2019-06-13 08:58

Associated revisions

Revision 020f22dc (diff)
Added by Christian Eßl 3 months ago

[BUGFIX] Substitute form references for array keys as well in ext:form

In ext:form, string placeholders, like "{text-1}", can be used inside
the finisher to place form inputs, made by the user, inside fields like
the email subject, reploy to email and name, etc. These string
substitutions however were not done for array keys (only array values).

Resolves: #88550
Releases: master
Change-Id: I727850f3e15479215b56be4cd136d93644b0d435
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/61890
Tested-by: TYPO3com <>
Tested-by: Susanne Moog <>
Tested-by: Daniel Goerz <>
Reviewed-by: Susanne Moog <>
Reviewed-by: Daniel Goerz <>

History

#1 Updated by Susanne Moog 7 months ago

  • Status changed from New to Accepted

Research:

\TYPO3\CMS\Form\Domain\Finishers\AbstractFinisher::parseOption uses $this->substituteRuntimeReferences($optionValue, $formRuntime); -> that only substitutes array values, not keys. The email addresses are an associative array where both keys and values might use placeholders.

#2 Updated by Gerrit Code Review 4 months ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61890

#3 Updated by Christian Eßl 3 months ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#4 Updated by Benni Mack about 1 month ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF