Skip to content

Commit a9cbf8b

Browse files
alamiraultalamirault
committed
Standardize the name of the container configurator variable
1 parent a5b8e76 commit a9cbf8b

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

42 files changed

+212
-212
lines changed

‎bundles/best_practices.rst

+2-2
Original file line numberDiff line numberDiff line change
@@ -442,8 +442,8 @@ The end user can provide values in any configuration file:
442442
// config/services.php
443443
namespace Symfony\Component\DependencyInjection\Loader\Configurator;
444444
445-
return static function (ContainerConfigurator $container) {
446-
$container->parameters()
445+
return static function (ContainerConfigurator $containerConfigurator) {
446+
$containerConfigurator->parameters()
447447
->set('acme_blog.author.email', '[email protected]')
448448
;
449449
};

‎bundles/prepend_extension.rst

+3-3
Original file line numberDiff line numberDiff line change
@@ -145,13 +145,13 @@ registered and the ``entity_manager_name`` setting for ``acme_hello`` is set to
145145
// config/packages/acme_something.php
146146
namespace Symfony\Component\DependencyInjection\Loader\Configurator;
147147
148-
return static function (ContainerConfigurator $container) {
149-
$container->extension('acme_something', [
148+
return static function (ContainerConfigurator $containerConfigurator) {
149+
$containerConfigurator->extension('acme_something', [
150150
// ...
151151
'use_acme_goodbye' => false,
152152
'entity_manager_name' => 'non_default',
153153
]);
154-
$container->extension('acme_other', [
154+
$containerConfigurator->extension('acme_other', [
155155
// ...
156156
'use_acme_goodbye' => false,
157157
]);

‎cache.rst

+2-2
Original file line numberDiff line numberDiff line change
@@ -387,8 +387,8 @@ with either :class:`Symfony\\Contracts\\Cache\\CacheInterface` or
387387
// config/services.php
388388
namespace Symfony\Component\DependencyInjection\Loader\Configurator;
389389
390-
return function(ContainerConfigurator $configurator) {
391-
$configurator->services()
390+
return function(ContainerConfigurator $containerConfigurator) {
391+
$containerConfigurator->services()
392392
// ...
393393
394394
->set('app.cache.adapter.redis')

‎components/dependency_injection.rst

+3-3
Original file line numberDiff line numberDiff line change
@@ -291,13 +291,13 @@ config files:
291291
292292
namespace Symfony\Component\DependencyInjection\Loader\Configurator;
293293
294-
return static function (ContainerConfigurator $container) {
295-
$container->parameters()
294+
return static function (ContainerConfigurator $containerConfigurator) {
295+
$containerConfigurator->parameters()
296296
// ...
297297
->set('mailer.transport', 'sendmail')
298298
;
299299
300-
$services = $container->services();
300+
$services = $containerConfigurator->services();
301301
$services->set('mailer', 'Mailer')
302302
->args(['%mailer.transport%'])
303303
;

‎components/dependency_injection/_imports-parameters-note.rst.inc

+2-2
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,6 @@
3131
// config/services.php
3232
namespace Symfony\Component\DependencyInjection\Loader\Configurator;
3333

34-
return static function (ContainerConfigurator $container) {
35-
$container->import('%kernel.project_dir%/somefile.yaml');
34+
return static function (ContainerConfigurator $containerConfigurator) {
35+
$containerConfigurator->import('%kernel.project_dir%/somefile.yaml');
3636
};

‎components/http_foundation/session_configuration.rst

+2-2
Original file line numberDiff line numberDiff line change
@@ -200,8 +200,8 @@ configuration:
200200
// config/packages/framework.php
201201
namespace Symfony\Component\DependencyInjection\Loader\Configurator;
202202
203-
return static function (ContainerConfigurator $container) {
204-
$container->extension('framework', [
203+
return static function (ContainerConfigurator $containerConfigurator) {
204+
$containerConfigurator->extension('framework', [
205205
'session' => [
206206
'gc_probability' => null,
207207
],

‎components/serializer.rst

+2-2
Original file line numberDiff line numberDiff line change
@@ -998,8 +998,8 @@ faster alternative to the
998998
999999
use Symfony\Component\Serializer\Normalizer\GetSetMethodNormalizer;
10001000
1001-
return static function (ContainerConfigurator $container) {
1002-
$container->services()
1001+
return static function (ContainerConfigurator $containerConfigurator) {
1002+
$containerConfigurator->services()
10031003
// ...
10041004
->set('get_set_method_normalizer', GetSetMethodNormalizer::class)
10051005
->tag('serializer.normalizer')

‎components/uid.rst

+1-1
Original file line numberDiff line numberDiff line change
@@ -452,7 +452,7 @@ configuration in your application before using these commands:
452452
use Symfony\Component\Uid\Command\InspectUlidCommand;
453453
use Symfony\Component\Uid\Command\InspectUuidCommand;
454454
455-
return static function (ContainerConfigurator $configurator): void {
455+
return static function (ContainerConfigurator $containerConfigurator): void {
456456
// ...
457457
458458
$services

‎components/var_dumper.rst

+2-2
Original file line numberDiff line numberDiff line change
@@ -147,8 +147,8 @@ the :ref:`dump_destination option <configuration-debug-dump_destination>` of the
147147
// config/packages/debug.php
148148
namespace Symfony\Component\DependencyInjection\Loader\Configurator;
149149
150-
return static function (ContainerConfigurator $container) {
151-
$container->extension('debug', [
150+
return static function (ContainerConfigurator $containerConfigurator) {
151+
$containerConfigurator->extension('debug', [
152152
'dump_destination' => 'tcp://%env(VAR_DUMPER_SERVER)%',
153153
]);
154154
};

‎configuration.rst

+26-26
Original file line numberDiff line numberDiff line change
@@ -78,18 +78,18 @@ shown in these three formats.
7878
{
7979
// ...
8080

81-
private function configureContainer(ContainerConfigurator $container): void
81+
private function configureContainer(ContainerConfigurator $containerConfigurator): void
8282
{
8383
$configDir = $this->getConfigDir();
8484

85-
$container->import($configDir.'/{packages}/*.{yaml,php}');
86-
$container->import($configDir.'/{packages}/'.$this->environment.'/*.{yaml,php}');
85+
$containerConfigurator->import($configDir.'/{packages}/*.{yaml,php}');
86+
$containerConfigurator->import($configDir.'/{packages}/'.$this->environment.'/*.{yaml,php}');
8787

8888
if (is_file($configDir.'/services.yaml')) {
89-
$container->import($configDir.'/services.yaml');
90-
$container->import($configDir.'/{services}_'.$this->environment.'.yaml');
89+
$containerConfigurator->import($configDir.'/services.yaml');
90+
$containerConfigurator->import($configDir.'/{services}_'.$this->environment.'.yaml');
9191
} else {
92-
$container->import($configDir.'/{services}.php');
92+
$containerConfigurator->import($configDir.'/{services}.php');
9393
}
9494
}
9595
}
@@ -163,17 +163,17 @@ configuration files, even if they use a different format:
163163
// config/services.php
164164
namespace Symfony\Component\DependencyInjection\Loader\Configurator;
165165
166-
return static function (ContainerConfigurator $container) {
167-
$container->import('legacy_config.php');
166+
return static function (ContainerConfigurator $containerConfigurator) {
167+
$containerConfigurator->import('legacy_config.php');
168168
169169
// glob expressions are also supported to load multiple files
170-
$container->import('/etc/myapp/*.yaml');
170+
$containerConfigurator->import('/etc/myapp/*.yaml');
171171
172172
// the third optional argument of import() is 'ignore_errors'
173173
// 'ignore_errors' set to 'not_found' silently discards errors if the loaded file doesn't exist
174-
$container->import('my_config_file.yaml', null, 'not_found');
174+
$containerConfigurator->import('my_config_file.yaml', null, 'not_found');
175175
// 'ignore_errors' set to true silently discards all errors (including invalid code and not found)
176-
$container->import('my_config_file.yaml', null, true);
176+
$containerConfigurator->import('my_config_file.yaml', null, true);
177177
};
178178
179179
// ...
@@ -262,8 +262,8 @@ reusable configuration value. By convention, parameters are defined under the
262262
263263
use App\Entity\BlogPost;
264264
265-
return static function (ContainerConfigurator $container) {
266-
$container->parameters()
265+
return static function (ContainerConfigurator $containerConfigurator) {
266+
$containerConfigurator->parameters()
267267
// the parameter name is an arbitrary string (the 'app.' prefix is recommended
268268
// to better differentiate your parameters from Symfony parameters).
269269
->set('app.admin_email', '[email protected]')
@@ -334,8 +334,8 @@ configuration file using a special syntax: wrap the parameter name in two ``%``
334334
// config/packages/some_package.php
335335
namespace Symfony\Component\DependencyInjection\Loader\Configurator;
336336
337-
return static function (ContainerConfigurator $container) {
338-
$container->extension('some_package', [
337+
return static function (ContainerConfigurator $containerConfigurator) {
338+
$containerConfigurator->extension('some_package', [
339339
// any string surrounded by two % is replaced by that parameter value
340340
'email_address' => '%app.admin_email%',
341341
@@ -371,8 +371,8 @@ configuration file using a special syntax: wrap the parameter name in two ``%``
371371
// config/services.php
372372
namespace Symfony\Component\DependencyInjection\Loader\Configurator;
373373
374-
return static function (ContainerConfigurator $container) {
375-
$container->parameters()
374+
return static function (ContainerConfigurator $containerConfigurator) {
375+
$containerConfigurator->parameters()
376376
->set('url_pattern', 'http://symfony.com/?foo=%%s&amp;bar=%%d');
377377
};
378378
@@ -508,20 +508,20 @@ files directly in the ``config/packages/`` directory.
508508
use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
509509
use Symfony\Config\WebpackEncoreConfig;
510510
511-
return static function (WebpackEncoreConfig $webpackEncore, ContainerConfigurator $container) {
511+
return static function (WebpackEncoreConfig $webpackEncore, ContainerConfigurator $containerConfigurator) {
512512
$webpackEncore
513513
->outputPath('%kernel.project_dir%/public/build')
514514
->strictMode(true)
515515
->cache(false)
516516
;
517517
518518
// cache is enabled only in the "prod" environment
519-
if ('prod' === $container->env()) {
519+
if ('prod' === $containerConfigurator->env()) {
520520
$webpackEncore->cache(true);
521521
}
522522
523523
// disable strict mode only in the "test" environment
524-
if ('test' === $container->env()) {
524+
if ('test' === $containerConfigurator->env()) {
525525
$webpackEncore->strictMode(false);
526526
}
527527
};
@@ -642,8 +642,8 @@ This example shows how you could configure the database connection using an env
642642
// config/packages/doctrine.php
643643
namespace Symfony\Component\DependencyInjection\Loader\Configurator;
644644
645-
return static function (ContainerConfigurator $container) {
646-
$container->extension('doctrine', [
645+
return static function (ContainerConfigurator $containerConfigurator) {
646+
$containerConfigurator->extension('doctrine', [
647647
'dbal' => [
648648
// by convention the env var names are always uppercase
649649
'url' => env('DATABASE_URL')->resolve(),
@@ -991,8 +991,8 @@ doesn't work for parameters:
991991
992992
use App\Service\MessageGenerator;
993993
994-
return static function (ContainerConfigurator $container) {
995-
$container->parameters()
994+
return static function (ContainerConfigurator $containerConfigurator) {
995+
$containerConfigurator->parameters()
996996
->set('app.contents_dir', '...');
997997
998998
$container->services()
@@ -1048,8 +1048,8 @@ whenever a service/controller defines a ``$projectDir`` argument, use this:
10481048
10491049
use App\Controller\LuckyController;
10501050
1051-
return static function (ContainerConfigurator $container) {
1052-
$container->services()
1051+
return static function (ContainerConfigurator $containerConfigurator) {
1052+
$containerConfigurator->services()
10531053
->defaults()
10541054
// pass this value to any $projectDir argument for any service
10551055
// that's created in this file (including controller arguments)

‎configuration/micro_kernel_trait.rst

+7-7
Original file line numberDiff line numberDiff line change
@@ -43,10 +43,10 @@ Next, create an ``index.php`` file that defines the kernel class and runs it::
4343
];
4444
}
4545

46-
protected function configureContainer(ContainerConfigurator $c): void
46+
protected function configureContainer(ContainerConfigurator $containerConfigurator): void
4747
{
4848
// PHP equivalent of config/packages/framework.yaml
49-
$c->extension('framework', [
49+
$containerConfigurator->extension('framework', [
5050
'secret' => 'S0ME_SECRET'
5151
]);
5252
}
@@ -88,7 +88,7 @@ that define your bundles, your services and your routes:
8888
**registerBundles()**
8989
This is the same ``registerBundles()`` that you see in a normal kernel.
9090

91-
**configureContainer(ContainerConfigurator $c)**
91+
**configureContainer(ContainerConfigurator $containerConfigurator)**
9292
This method builds and configures the container. In practice, you will use
9393
``extension()`` to configure different bundles (this is the equivalent
9494
of what you see in a normal ``config/packages/*`` file). You can also register
@@ -191,20 +191,20 @@ hold the kernel. Now it looks like this::
191191
return $bundles;
192192
}
193193

194-
protected function configureContainer(ContainerConfigurator $c): void
194+
protected function configureContainer(ContainerConfigurator $containerConfigurator): void
195195
{
196-
$c->import(__DIR__.'/../config/framework.yaml');
196+
$containerConfigurator->import(__DIR__.'/../config/framework.yaml');
197197

198198
// register all classes in /src/ as service
199-
$c->services()
199+
$containerConfigurator->services()
200200
->load('App\\', __DIR__.'/*')
201201
->autowire()
202202
->autoconfigure()
203203
;
204204

205205
// configure WebProfilerBundle only if the bundle is enabled
206206
if (isset($this->bundles['WebProfilerBundle'])) {
207-
$c->extension('web_profiler', [
207+
$containerConfigurator->extension('web_profiler', [
208208
'toolbar' => true,
209209
'intercept_redirects' => false,
210210
]);

‎configuration/multiple_kernels.rst

+6-6
Original file line numberDiff line numberDiff line change
@@ -106,16 +106,16 @@ files so they don't collide with the files from ``src/Kernel.php``::
106106
return $this->getProjectDir().'/var/log/api';
107107
}
108108

109-
protected function configureContainer(ContainerConfigurator $container): void
109+
protected function configureContainer(ContainerConfigurator $containerConfigurator): void
110110
{
111-
$container->import('../config/api/{packages}/*.yaml');
112-
$container->import('../config/api/{packages}/'.$this->environment.'/*.yaml');
111+
$containerConfigurator->import('../config/api/{packages}/*.yaml');
112+
$containerConfigurator->import('../config/api/{packages}/'.$this->environment.'/*.yaml');
113113

114114
if (is_file(\dirname(__DIR__).'/config/api/services.yaml')) {
115-
$container->import('../config/api/services.yaml');
116-
$container->import('../config/api/{services}_'.$this->environment.'.yaml');
115+
$containerConfigurator->import('../config/api/services.yaml');
116+
$containerConfigurator->import('../config/api/{services}_'.$this->environment.'.yaml');
117117
} else {
118-
$container->import('../config/api/{services}.php');
118+
$containerConfigurator->import('../config/api/{services}.php');
119119
}
120120
}
121121

‎controller/argument_value_resolver.rst

+2-2
Original file line numberDiff line numberDiff line change
@@ -233,8 +233,8 @@ and adding a priority.
233233
234234
use App\ArgumentResolver\UserValueResolver;
235235
236-
return static function (ContainerConfigurator $container) {
237-
$services = $configurator->services();
236+
return static function (ContainerConfigurator $containerConfigurator) {
237+
$services = $containerConfigurator->services();
238238
239239
$services->set(UserValueResolver::class)
240240
->tag('controller.argument_value_resolver', ['priority' => 50])

‎controller/upload_file.rst

+2-2
Original file line numberDiff line numberDiff line change
@@ -321,8 +321,8 @@ Then, define a service for this class:
321321
322322
use App\Service\FileUploader;
323323
324-
return static function (ContainerConfigurator $container) {
325-
$services = $configurator->services();
324+
return static function (ContainerConfigurator $containerConfigurator) {
325+
$services = $containerConfigurator->services();
326326
327327
$services->set(FileUploader::class)
328328
->arg('$targetDirectory', '%brochures_directory%')

‎doctrine/events.rst

+6-6
Original file line numberDiff line numberDiff line change
@@ -227,8 +227,8 @@ with the ``doctrine.event_listener`` tag:
227227
228228
use App\EventListener\SearchIndexer;
229229
230-
return static function (ContainerConfigurator $configurator) {
231-
$services = $configurator->services();
230+
return static function (ContainerConfigurator $containerConfigurator) {
231+
$services = $containerConfigurator->services();
232232
233233
// listeners are applied by default to all Doctrine connections
234234
$services->set(SearchIndexer::class)
@@ -360,8 +360,8 @@ with the ``doctrine.orm.entity_listener`` tag:
360360
use App\Entity\User;
361361
use App\EventListener\UserChangedNotifier;
362362
363-
return static function (ContainerConfigurator $container) {
364-
$services = $configurator->services();
363+
return static function (ContainerConfigurator $containerConfigurator) {
364+
$services = $containerConfigurator->services();
365365
366366
$services->set(UserChangedNotifier::class)
367367
->tag('doctrine.orm.entity_listener', [
@@ -501,8 +501,8 @@ Doctrine connection to use) you must do that in the manual service configuration
501501
502502
use App\EventListener\DatabaseActivitySubscriber;
503503
504-
return static function (ContainerConfigurator $container) {
505-
$services = $configurator->services();
504+
return static function (ContainerConfigurator $containerConfigurator) {
505+
$services = $containerConfigurator->services();
506506
507507
$services->set(DatabaseActivitySubscriber::class)
508508
->tag('doctrine.event_subscriber'[

‎event_dispatcher.rst

+2-2
Original file line numberDiff line numberDiff line change
@@ -104,8 +104,8 @@ using a special "tag":
104104
105105
use App\EventListener\ExceptionListener;
106106
107-
return function(ContainerConfigurator $configurator) {
108-
$services = $configurator->services();
107+
return function(ContainerConfigurator $containerConfigurator) {
108+
$services = $containerConfigurator->services();
109109
110110
$services->set(ExceptionListener::class)
111111
->tag('kernel.event_listener', ['event' => 'kernel.exception'])

‎frontend/custom_version_strategy.rst

+2-2
Original file line numberDiff line numberDiff line change
@@ -144,8 +144,8 @@ After creating the strategy PHP class, register it as a Symfony service.
144144
use App\Asset\VersionStrategy\GulpBusterVersionStrategy;
145145
use Symfony\Component\DependencyInjection\Definition;
146146
147-
return function(ContainerConfigurator $configurator) {
148-
$services = $configurator->services();
147+
return function(ContainerConfigurator $containerConfigurator) {
148+
$services = $containerConfigurator->services();
149149
150150
$services->set(GulpBusterVersionStrategy::class)
151151
->args(

0 commit comments

Comments
 (0)