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

[Mime]TemplatedEmail::$locale is not serialized #52718

Closed
mkrauser opened this issue Nov 24, 2023 · 0 comments · Fixed by #52719
Closed

[Mime]TemplatedEmail::$locale is not serialized #52718

mkrauser opened this issue Nov 24, 2023 · 0 comments · Fixed by #52719

Comments

@mkrauser
Copy link
Contributor

Symfony version(s) affected

6.4

Description

In #51690 the locale-property was added to TemplatedEmail, but this property is not serialized.

If Mails are sent asynchronously via the Messenger-Component, the locale-value is not available, once the TemplatedEmail is rendered.

How to reproduce

Create and sent a TemplatedEmail with a locale and send it via a message queue. The locale-property is not serialized to the message-queue

Possible Solution

No response

Additional Context

No response

@mkrauser mkrauser added the Bug label Nov 24, 2023
fabpot added a commit that referenced this issue Nov 25, 2023
…ps (mkrauser)

This PR was squashed before being merged into the 6.4 branch.

Discussion
----------

[Mime] Add `TemplatedEmail::$locale` to the serialized props

| Q             | A
| ------------- | ---
| Branch?       | 6.4
| Bug fix?      | yes
| New feature?  |no
| Deprecations? | no
| Issues        | Fix #52718
| License       | MIT

To use the locale-property when working with Symfony Messenger asynchronously it has to be serialized.

Commits
-------

cedb7cc [Mime] Add `TemplatedEmail::$locale` to the serialized props
@fabpot fabpot closed this as completed Nov 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants