Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correction of keyword #1248

Merged
merged 1 commit into from
Jan 11, 2021
Merged

Correction of keyword #1248

merged 1 commit into from
Jan 11, 2021

Conversation

Footrotflat
Copy link
Contributor

In the section "Send email with FluidEmail"

This keyword is wrong: use Symfony\Component\Mailer\Mailer;
This keyword is correct : use TYPO3\CMS\Core\Mail\Mailer;

In the section "Send email with FluidEmail"

This keyword is wrong: use Symfony\Component\Mailer\Mailer;
This keyword is correct : use TYPO3\CMS\Core\Mail\Mailer;
@mbrodala mbrodala merged commit af8127c into TYPO3-Documentation:10.4 Jan 11, 2021
@mbrodala mbrodala added the backport-required needs cherry-pick into other branches label Jan 11, 2021
@linawolf
Copy link
Member

@mbrodala which versions do we need to backport this to?

@mbrodala
Copy link
Member

@linawolf I didn't notice that this did target 10.4 but I can see that master is missing the import completely:

use Symfony\Component\Mime\Address;
use TYPO3\CMS\Core\Mail\FluidEmail;

$email = GeneralUtility::makeInstance(FluidEmail::class);
$email
    ->to('contact@acme.com')
    ->from(new Address('jeremy@acme.com', 'Jeremy'))
    ->subject('TYPO3 loves you - here is why')
    ->format('html') // only HTML mail
    ->setTemplate('TipsAndTricks')
    ->assign('mySecretIngredient', 'Tomato and TypoScript');
GeneralUtility::makeInstance(Mailer::class)->send($email);

So we basically need a "forward-port". ;-)

linawolf pushed a commit that referenced this pull request Jan 19, 2021
In the section "Send email with FluidEmail"

This keyword is wrong: use Symfony\Component\Mailer\Mailer;
This keyword is correct : use TYPO3\CMS\Core\Mail\Mailer;
@linawolf linawolf added backport-done and removed backport-required needs cherry-pick into other branches labels Jan 19, 2021
@linawolf
Copy link
Member

forwardported to master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants