Skip to content

Update service_container.rst #9337

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

Closed
wants to merge 2 commits into from
Closed

Conversation

ilmoralito
Copy link
Contributor

Since 6 use $message = (new \Swift_Message('message')) instead of $message = \Swift_Message::newInstance()

Please check https://github.com/swiftmailer/swiftmailer/blob/master/CHANGES#L24 and https://swiftmailer.symfony.com/docs/introduction.html#basic-usage

Since 6 use `$message = (new \Swift_Message('message'))` instead of `$message = \Swift_Message::newInstance()`

Please check https://github.com/swiftmailer/swiftmailer/blob/master/CHANGES#L24 and https://swiftmailer.symfony.com/docs/introduction.html#basic-usage
@javiereguiluz javiereguiluz added this to the 3.4 milestone Feb 26, 2018
@javiereguiluz
Copy link
Member

@ilmoralito thanks for helping us improve Symfony Docs and congrts on your first contribution! I've tweaked your code a bit because the argument of \Swift_Message is the subject of the email, not its body. Also, we've merged this on 3.4, the oldest maintained branch that contains the error. That's why GitHub shows it as closed instead of merged, but it's merged.

javiereguiluz added a commit that referenced this pull request Feb 26, 2018
This PR was submitted for the 4.0 branch but it was merged into the 3.4 branch instead (closes #9337).

Discussion
----------

Update service_container.rst

Since 6 use `$message = (new \Swift_Message('message'))` instead of `$message = \Swift_Message::newInstance()`

Please check https://github.com/swiftmailer/swiftmailer/blob/master/CHANGES#L24 and https://swiftmailer.symfony.com/docs/introduction.html#basic-usage

Commits
-------

e4ba061 Fixed the code
17da22a Update service_container.rst
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.

3 participants