Skip to content

Commit d6392af

Browse files
committed
minor #17199 Fix dependency injection PHP sample code (jdecool)
This PR was merged into the 5.4 branch. Discussion ---------- Fix dependency injection PHP sample code I've just noticed that there is an error in the code sample: ```php namespace Symfony\Component\DependencyInjection\Loader\Configurator; return static function (ContainerConfigurator $container) { $container->parameters() // ... ->set('mailer.transport', 'sendmail') ; $container->services() ->set('mailer', 'Mailer') ->args(['%mailer.transport%']) $services->set('mailer', 'Mailer') // the param() method was introduced in Symfony 5.2. ->args([param('mailer.transport')]) ; $services->set('newsletter_manager', 'NewsletterManager') // In versions earlier to Symfony 5.1 the service() function was called ref() ->call('setMailer', [service('mailer')]) ; }; ``` The `$services` is not defined. Commits ------- 087773a Fix dependency injection PHP sample code
2 parents fc087b8 + 087773a commit d6392af

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

components/dependency_injection.rst

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -297,9 +297,9 @@ config files:
297297
->set('mailer.transport', 'sendmail')
298298
;
299299
300-
$container->services()
301-
->set('mailer', 'Mailer')
302-
->args(['%mailer.transport%'])
300+
$services = $container->services();
301+
$services->set('mailer', 'Mailer')
302+
->args(['%mailer.transport%'])
303303
304304
$services->set('mailer', 'Mailer')
305305
// the param() method was introduced in Symfony 5.2.

0 commit comments

Comments
 (0)