Skip to content
This repository has been archived by the owner on Feb 6, 2022. It is now read-only.

Commit

Permalink
bug #295 Remove not Symfony 5 compliant service argument %kernel.root…
Browse files Browse the repository at this point in the history
…_dir% (l-vo)

This PR was merged into the 3.3-dev branch.

Discussion
----------

Remove not Symfony 5 compliant service argument %kernel.root_dir%

The service swiftmailer.spool.file.abstract uses %kernel.root_dir% in a path as its first argument. I think this argument is useless since it's replaced in `SwiftMailerExtension`. https://github.com/symfony/swiftmailer-bundle/blob/6b895bc0a5e815d1bf2d444869415a7c752516aa/DependencyInjection/SwiftmailerExtension.php#L262

Can anyone confirm it's not a BC break since a CompilerPass can't be registered and pass (and use this useless value) before `SwiftMailerExtension` code execution ?

Commits
-------

3653248 Remove useless swiftmailer.spool.file.abstract argument value
  • Loading branch information
fabpot committed Nov 7, 2019
2 parents 6b895bc + 3653248 commit defa9bd
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Resources/config/swiftmailer.xml
Expand Up @@ -48,7 +48,7 @@
<service id="swiftmailer.transport.spool.abstract" class="Swift_Transport_SpoolTransport" public="false" abstract="true" />

<service id="swiftmailer.spool.file.abstract" class="Swift_FileSpool" public="false" abstract="true">
<argument>%kernel.root_dir%/../data/swiftmailer/spool</argument>
<argument />
</service>

<service id="swiftmailer.spool.memory.abstract" class="Swift_MemorySpool" public="false" abstract="true" />
Expand Down

0 comments on commit defa9bd

Please sign in to comment.