Skip to content

Commit 8558b9e

Browse files
Merge branch '6.0' into 6.1
* 6.0: [HttpClient] fix resetting DNS/etc when calling CurlHttpClient::reset() Fix invalid guess with enumType [HttpClient] Remove deprecated usage of GuzzleHttp\Promise\promise_for [HttpClient] Remove deprecated usage of `GuzzleHttp\Promise\queue` [PropertyAccess] Fix handling of uninitialized property of anonymous class [FrameworkBundle] Allow default cache pools to be overwritten by user [DependencyInjection] fix test ResolveBindingsPass remove loading of class iterable [Serializer] Fix AbstractObjectNormalizer TypeError on denormalization [FrameworkBundle] Avoid calling rtrim(null, '/') in AssetsInstallCommand Optimization of resolveEnvPlaceholders Fix incorrect format [DependencyInjection] Fix nested env var with resolve processor Allow OutputFormatter::escape() to be used for escaping URLs used in <href> allow a zero time-limit Use correct tag for ExpoTransportFactory service [DependencyInjection] Ignore argument type check in CheckTypeDeclarationsPass if it's a Definition with a factory [Validators] Add translations for Slovak #43735
2 parents a8de615 + f8edbc9 commit 8558b9e

File tree

3 files changed

+27
-27
lines changed

3 files changed

+27
-27
lines changed

Command/AssetsInstallCommand.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -94,8 +94,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
9494
{
9595
/** @var KernelInterface $kernel */
9696
$kernel = $this->getApplication()->getKernel();
97-
$targetArg = rtrim($input->getArgument('target'), '/');
98-
97+
$targetArg = rtrim($input->getArgument('target') ?? '', '/');
9998
if (!$targetArg) {
10099
$targetArg = $this->getPublicDirectory($kernel->getContainer());
101100
}

DependencyInjection/FrameworkExtension.php

Lines changed: 25 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -336,34 +336,10 @@ public function load(array $configs, ContainerBuilder $container)
336336
}
337337
}
338338

339-
// register cache before session so both can share the connection services
340-
$this->registerCacheConfiguration($config['cache'], $container);
341-
342-
if ($this->isConfigEnabled($container, $config['session'])) {
343-
if (!\extension_loaded('session')) {
344-
throw new LogicException('Session support cannot be enabled as the session extension is not installed. See https://php.net/session.installation for instructions.');
345-
}
346-
347-
$this->sessionConfigEnabled = true;
348-
$this->registerSessionConfiguration($config['session'], $container, $loader);
349-
if (!empty($config['test'])) {
350-
// test listener will replace the existing session listener
351-
// as we are aliasing to avoid duplicated registered events
352-
$container->setAlias('session_listener', 'test.session.listener');
353-
}
354-
} elseif (!empty($config['test'])) {
355-
$container->removeDefinition('test.session.listener');
356-
}
357-
358339
if ($this->isConfigEnabled($container, $config['request'])) {
359340
$this->registerRequestConfiguration($config['request'], $container, $loader);
360341
}
361342

362-
if (null === $config['csrf_protection']['enabled']) {
363-
$config['csrf_protection']['enabled'] = $this->sessionConfigEnabled && !class_exists(FullStack::class) && ContainerBuilder::willBeAvailable('symfony/security-csrf', CsrfTokenManagerInterface::class, ['symfony/framework-bundle']);
364-
}
365-
$this->registerSecurityCsrfConfiguration($config['csrf_protection'], $container, $loader);
366-
367343
if ($this->isConfigEnabled($container, $config['form'])) {
368344
if (!class_exists(Form::class)) {
369345
throw new LogicException('Form support cannot be enabled as the Form component is not installed. Try running "composer require symfony/form".');
@@ -494,6 +470,31 @@ public function load(array $configs, ContainerBuilder $container)
494470
$this->registerUidConfiguration($config['uid'], $container, $loader);
495471
}
496472

473+
// register cache before session so both can share the connection services
474+
$this->registerCacheConfiguration($config['cache'], $container);
475+
476+
if ($this->isConfigEnabled($container, $config['session'])) {
477+
if (!\extension_loaded('session')) {
478+
throw new LogicException('Session support cannot be enabled as the session extension is not installed. See https://php.net/session.installation for instructions.');
479+
}
480+
481+
$this->sessionConfigEnabled = true;
482+
$this->registerSessionConfiguration($config['session'], $container, $loader);
483+
if (!empty($config['test'])) {
484+
// test listener will replace the existing session listener
485+
// as we are aliasing to avoid duplicated registered events
486+
$container->setAlias('session_listener', 'test.session.listener');
487+
}
488+
} elseif (!empty($config['test'])) {
489+
$container->removeDefinition('test.session.listener');
490+
}
491+
492+
// csrf depends on session being registered
493+
if (null === $config['csrf_protection']['enabled']) {
494+
$config['csrf_protection']['enabled'] = $this->sessionConfigEnabled && !class_exists(FullStack::class) && ContainerBuilder::willBeAvailable('symfony/security-csrf', CsrfTokenManagerInterface::class, ['symfony/framework-bundle']);
495+
}
496+
$this->registerSecurityCsrfConfiguration($config['csrf_protection'], $container, $loader);
497+
497498
$this->addAnnotatedClassesToCompile([
498499
'**\\Controller\\',
499500
'**\\Entity\\',

Resources/config/notifier_transports.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -240,6 +240,6 @@
240240

241241
->set('notifier.transport_factory.expo', ExpoTransportFactory::class)
242242
->parent('notifier.transport_factory.abstract')
243-
->tag('chatter.transport_factory')
243+
->tag('texter.transport_factory')
244244
;
245245
};

0 commit comments

Comments
 (0)