8000 Memory spooling is not working · Issue #43 · symfony/swiftmailer-bundle · GitHub
[go: up one dir, main page]
Skip to content
8000
This repository was archived by the owner on Feb 6, 2022. It is now read-only.

Memory spooling is not working #43

Closed
smatyas opened this issue Jul 25, 2013 · 2 comments
Closed

Memory spooling is not working #43

smatyas opened this issue Jul 25, 2013 · 2 comments

Comments

@smatyas
Copy link
Contributor
smatyas commented Jul 25, 2013

As of the 'mailers' feature came in, the memory spooling has stopped working.
I found out that the 'mailer' service became just an alias, and that's what is causing the problem.
The 'initialized' method of the Symfony\Component\DependencyInjection\Container does not handle aliases (as it is in the ContainerBuilder), but therefore the EmailSenderListener never sends emails.

I don't really know which is better: if it the Container should handle aliases or the EmailSenderListener should check services in a different way. Anyway, I've made a pull request here which resolves the service checking.
@smatyas smatyas mentioned this issue Jul 25, 2013
Closed
@smatyas
Copy link
Contributor Author
smatyas commented Jul 25, 2013

Well, I'm using Symfony 2.2, where not the Container handles aliases. (I see that in 2.3 it does.)

@smatyas
Copy link
Contributor Author
smatyas commented Jul 26, 2013

So I think #34 made a BC break.

fabpot added a commit that referenced this issue Jul 26, 2013
@fabpot
merged branch smatyas/emailsenderlistener-fix (PR #44)
6b52e41 
This PR was squashed before being merged into the master branch (closes #44).

Discussion
----------

fix checking of mailer services in EmailSenderListener

The 'initialized' method of the Symfony\Component\DependencyInjection\Container
554F
 does not handle aliases (as it is in the ContainerBuilder), but therefore the EmailSenderListener never sends emails.

This fix is related to issue #43.

Commits
-------

33a7015 fix checking of mailer services in EmailSenderListener
@fabpot fabpot closed this as completed Jul 26, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@fabpot @smatyas
0