Bug #17061

Mailform with localization ignores the email-address of the translated record

Added by Marcel Alburg about 6 years ago. Updated over 1 year ago.

Status:Resolved Start date:2007-03-01
Priority:Should have Due date:
Assignee:Michael Stucki % Done:

100%

Category:Communication
Target version:-
TYPO3 Version:4.0 Complexity:
PHP Version:5.2
Votes: 1 (View)

Description

If you've a FORM with different languages and different recipients (every language a other recipient) the FORM send's the mail always to the e-mail address which was entered in default language.

(issue imported from #M5104)

mailForm_localized.patch (1.1 kB) Administrator Admin, 2007-03-01 17:09

bug_5104_stucki.diff (1.2 kB) Administrator Admin, 2009-04-29 12:11

Associated revisions

Revision f00eb53a
Added by Michael Stucki over 1 year ago

[BUGFIX] Translated mailform should use localized email

In case mailforms are translated the email adress from the
translated record is ignored.

Change-Id: I184e7b96fe5a0405e3bed6546d4461cd44282ed6
Fixes: #17061
Releases: 4.7, 4.6, 4.5
Reviewed-on: http://review.typo3.org/7817
Reviewed-by: Stefan Rotsch
Tested-by: Stefan Rotsch
Reviewed-by: Thomas Layh
Tested-by: Thomas Layh
Reviewed-by: Wouter Wolters
Reviewed-by: Tolleiv Nietsch
Tested-by: Tolleiv Nietsch

Revision d2eb5843
Added by Michael Stucki about 1 year ago

[BUGFIX] Translated mailform should use localized email

In case mailforms are translated the email adress from the
translated record is ignored.

Change-Id: I55913d42815f7da53a05b3bfd635b589ecc3f1ae
Fixes: #17061
Releases: 4.7, 4.6, 4.5
Reviewed-on: http://review.typo3.org/8957
Reviewed-by: Xavier Perseguers
Tested-by: Xavier Perseguers

Revision eb2dd3db
Added by Michael Stucki about 1 year ago

[BUGFIX] Translated mailform should use localized email

In case mailforms are translated the email adress from the
translated record is ignored.

Change-Id: I4cf05157cff40e68ebe35c1c242af711dc0a3497
Fixes: #17061
Releases: 4.7, 4.6, 4.5
Reviewed-on: http://review.typo3.org/8958
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog

History

Updated by Marcel Alburg about 6 years ago

i'm sorry.

But this problem is for Typo3 4.0.x.

Updated by Ferdinand Kuhl about 4 years ago

This is true for the current 4.2.6 release!
The added Patch solves the problem at its root. The form prints out the wrong ID in its locationData field. Why has this patch not been integrated yet?

Updated by Ferdinand Kuhl about 4 years ago

(As the patch is quite old, I did not apply the patch, but used the informations in it to edit the tslib_content myself)

Updated by Michael Stucki about 4 years ago

@Ferdinand: I acknowledge your solution which seems pretty nice and also works well. However, it is difficult to change the currentRecord since it is not defined whether it should contain the original or the translated record uid. It is likely that your fix would cause breakage at some place.

Instead I have uploaded another patch which fixes the issue without touching $this->currentRecord.

Updated by Ferdinand Kuhl almost 4 years ago

Thanks a lot. Works perfectly at my setup.

Updated by Freddy Tripold about 2 years ago

This problem still exits with version 4.5.2, so why is this this bugfix not in the core?

Updated by Chris topher about 2 years ago

I guess that issue got forgotten somehow. :-(

Can you please push it to Gerrit, so that it can be reviewed?
For more information see here: http://wiki.typo3.org/Git

Updated by Gerrit Code Review over 1 year ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/7817

Updated by Gerrit Code Review over 1 year ago

Patch set 1 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/8957

Updated by Gerrit Code Review over 1 year ago

Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at http://review.typo3.org/8958

Updated by Michael Stucki over 1 year ago

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

Also available in: Atom PDF