Swiftmailer spool collector #84
Conversation
|
A great thing would be to have a DataCollector showing the email sent as it would be easier to debug the mailing. Your implementation is not good as it does not collect the data about the current request. The spooled messages can concern another request. |
|
Mmm ok I see, so you are saying that showing spooled messages is not required but you prefer to have the webprofiler showing the current email sent ? |
|
Yes. The profiler is about collecting data on the current request: http://docs.symfony-reloaded.org/master/guides/internals/profiler.html so a Swiftmailer DataCollector should respect this design. But showing the email sent can be more difficult. I don't know if Swiftmailer provide an extension point to log the email sent (which would allow to collect them by logging in an array just like Doctrine and the Logger do for the profiler). |
|
I'm closing this PR as the current implementation cannot be merged as is. |
…ndre) This PR was merged into the master branch. Discussion ---------- Added Brazilian Portuguese translations Hi! Here is a complete Brazilian Portuguese translation Commits ------- 233ade0 Added Brazilian Portuguese translations
…sories#12-yaml-syntax-fixes Correcting YAML syntax for branch names
…ersion (iluuu1994) This PR was merged into the 2.0-dev branch. Discussion ---------- Fix BinaryFileResponse with range to psr response conversion Closes symfony#84 As requested by [Fabien](symfony#38280 (comment)). I think using the slightly less optimal version of checking for a `Content-Range` header is better than relying on reflection. In theory, this could be slightly sub optimal when streaming whole files and setting the `Content-Range` manually but I'm assuming that's very rare in practice. Commits ------- 5d5932d Fix BinaryFileResponse with range to psr response conversion
Increase minimum version minor version of PHPUnit version 5
This commit add a DataCollector for Swiftmailer spool with the number of messages in the spool.
I choose an icon in iconfinder.com, if you have a better one :)