Skip to content

Commit 7ba7a64

Browse files
xabbuhxabbuh
committed
Merge branch '7.0' into 7.1
* 7.0: fix merge [VarDumper] Test intl formatter broken since dumper does not replace the nnbsp character by standard space [WebProfilerBundle] Fix intercept external redirects [Webhook] Added missing XML attribute in config XSD [String] Skip a test when an issue is detected in PCRE2 [ExpressionLanguage] Fix null coalescing propagation [Mailer] Stop using the (local) AWS shared configuration in the PHPUnit tests. detect colors on not windows fix xterm detection refactor: hyper check Missing translations for Slovak (sk) symfony#51954 Remove redundant PHPdoc line properly handle SYMFONY_DOTENV_VARS being the empty string Avoid incompatibility with symfony/console 7 bug symfony#45057 [Messenger] Avoid reconnecting active Redis connections. [HttpKernel] Catch `TypeError` if the wrong type is used in `BackedEnumValueResolver` [Serializer] fix regression where nullable int cannot be serialized do not overwrite an application's default serialization context
2 parents dd3ef59 + 7d89a65 commit 7ba7a64

File tree

24 files changed

+245
-46
lines changed

24 files changed

+245
-46
lines changed

‎src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php‎

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1906,21 +1906,23 @@ private function registerSerializerConfiguration(array $config, ContainerBuilder
19061906
$container->getDefinition('serializer.name_converter.metadata_aware')->setArgument(1, new Reference($config['name_converter']));
19071907
}
19081908

1909+
$defaultContext = $config['default_context'] ?? [];
1910+
1911+
if ($defaultContext) {
1912+
$container->setParameter('serializer.default_context', $defaultContext);
1913+
}
1914+
19091915
if (isset($config['circular_reference_handler']) && $config['circular_reference_handler']) {
19101916
$arguments = $container->getDefinition('serializer.normalizer.object')->getArguments();
1911-
$context = ($arguments[6] ?? []) + ['circular_reference_handler' => new Reference($config['circular_reference_handler'])];
1917+
$context = ($arguments[6] ?? $defaultContext) + ['circular_reference_handler' => new Reference($config['circular_reference_handler'])];
19121918
$container->getDefinition('serializer.normalizer.object')->setArgument(5, null);
19131919
$container->getDefinition('serializer.normalizer.object')->setArgument(6, $context);
19141920
}
19151921

19161922
if ($config['max_depth_handler'] ?? false) {
1917-
$defaultContext = $container->getDefinition('serializer.normalizer.object')->getArgument(6);
1918-
$defaultContext += ['max_depth_handler' => new Reference($config['max_depth_handler'])];
1919-
$container->getDefinition('serializer.normalizer.object')->replaceArgument(6, $defaultContext);
1920-
}
1921-
1922-
if (isset($config['default_context']) && $config['default_context']) {
1923-
$container->setParameter('serializer.default_context', $config['default_context']);
1923+
$arguments = $container->getDefinition('serializer.normalizer.object')->getArguments();
1924+
$context = ($arguments[6] ?? $defaultContext) + ['max_depth_handler' => new Reference($config['max_depth_handler'])];
1925+
$container->getDefinition('serializer.normalizer.object')->setArgument(6, $context);
19241926
}
19251927
}
19261928

‎src/Symfony/Bundle/FrameworkBundle/Resources/config/schema/symfony-1.0.xsd‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -953,6 +953,7 @@
953953
<xsd:element name="service" type="xsd:string" minOccurs="0" maxOccurs="1" />
954954
<xsd:element name="secret" type="xsd:string" minOccurs="0" maxOccurs="1" />
955955
</xsd:sequence>
956+
<xsd:attribute name="type" type="xsd:string" use="required" />
956957
</xsd:complexType>
957958

958959
<xsd:complexType name="remote-event">

‎src/Symfony/Bundle/FrameworkBundle/Tests/Fixtures/Serializer/CircularReferenceHandler.php‎

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
<?php
2+
3+
/*
4+
* This file is part of the Symfony package.
5+
*
6+
* (c) Fabien Potencier <[email protected]>
7+
*
8+
* For the full copyright and license information, please view the LICENSE
9+
* file that was distributed with this source code.
10+
*/
11+
12+
namespace Symfony\Bundle\FrameworkBundle\Tests\Fixtures\Serializer;
13+
14+
class CircularReferenceHandler
15+
{
16+
public function __invoke()
17+
{
18+
}
19+
}

‎src/Symfony/Bundle/FrameworkBundle/Tests/Fixtures/Serializer/MaxDepthHandler.php‎

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
<?php
2+
3+
/*
4+
* This file is part of the Symfony package.
5+
*
6+
* (c) Fabien Potencier <[email protected]>
7+
*
8+
* For the full copyright and license information, please view the LICENSE
9+
* file that was distributed with this source code.
10+
*/
11+
12+
namespace Symfony\Bundle\FrameworkBundle\Tests\Fixtures\Serializer;
13+
14+
class MaxDepthHandler
15+
{
16+
public function __invoke()
17+
{
18+
}
19+
}

‎src/Symfony/Bundle/FrameworkBundle/Tests/Functional/app/Serializer/config.yml‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@ framework:
66
translator: true
77
serializer:
88
enabled: true
9+
circular_reference_handler: Symfony\Bundle\FrameworkBundle\Tests\Fixtures\Serializer\CircularReferenceHandler
10+
max_depth_handler: Symfony\Bundle\FrameworkBundle\Tests\Fixtures\Serializer\MaxDepthHandler
911
default_context:
1012
enable_max_depth: true
1113
fake_context_option: foo
@@ -63,3 +65,7 @@ services:
6365
serializer.encoder.csv.alias:
6466
alias: serializer.encoder.csv
6567
public: true
68+
69+
Symfony\Bundle\FrameworkBundle\Tests\Fixtures\Serializer\CircularReferenceHandler: ~
70+
71+
Symfony\Bundle\FrameworkBundle\Tests\Fixtures\Serializer\MaxDepthHandler: ~

‎src/Symfony/Bundle/WebProfilerBundle/Resources/views/Profiler/toolbar_redirect.html.twig‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,10 +39,10 @@
3939
<div class="block-exception">
4040
<h1>Redirection Intercepted</h1>
4141

42-
{% set absolute_url = host in location ? location : host ~ location %}
42+
{% set absolute_url = absolute_url(location) %}
4343
<p>This request redirects to <strong>{{ absolute_url }}</strong></p>
4444

45-
<p><a class="btn" href="{{ location }}">Follow redirect</a></p>
45+
<p><a class="btn" href="{{ absolute_url }}">Follow redirect</a></p>
4646

4747
<p class="sf-redirection-help">
4848
The redirect was intercepted by the Symfony Web Debug toolbar to help debugging.

‎src/Symfony/Component/Console/Output/StreamOutput.php‎

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -94,18 +94,17 @@ protected function hasColorSupport(): bool
9494
return false;
9595
}
9696

97-
if ('Hyper' === getenv('TERM_PROGRAM')) {
97+
if (\DIRECTORY_SEPARATOR === '\\'
98+
&& \function_exists('sapi_windows_vt100_support')
99+
&& @sapi_windows_vt100_support($this->stream)
100+
) {
98101
return true;
99102
}
100103

101-
if (\DIRECTORY_SEPARATOR === '\\') {
102-
return (\function_exists('sapi_windows_vt100_support')
103-
&& @sapi_windows_vt100_support($this->stream))
104-
|| false !== getenv('ANSICON')
105-
|| 'ON' === getenv('ConEmuANSI')
106-
|| 'xterm' === getenv('TERM');
107-
}
108-
109-
return stream_isatty($this->stream);
104+
return 'Hyper' === getenv('TERM_PROGRAM')
105+
|| false !== getenv('ANSICON')
106+
|| 'ON' === getenv('ConEmuANSI')
107+
|| str_starts_with((string) getenv('TERM'), 'xterm')
108+
|| stream_isatty($this->stream);
110109
}
111110
}

‎src/Symfony/Component/DependencyInjection/EnvVarProcessorInterface.php‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ interface EnvVarProcessorInterface
2323
/**
2424
* Returns the value of the given variable as managed by the current instance.
2525
*
26-
* @param string $prefix The namespace of the variable
2726
* @param string $prefix The namespace of the variable; when the empty string is passed, null values should be kept as is
2827
* @param string $name The name of the variable within the namespace
2928
* @param \Closure(string): mixed $getEnv A closure that allows fetching more env vars

‎src/Symfony/Component/Dotenv/Command/DebugCommand.php‎

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,13 @@ private function getVariables(array $envFiles, ?string $nameFilter): array
141141

142142
private function getAvailableVars(): array
143143
{
144-
$vars = explode(',', $_SERVER['SYMFONY_DOTENV_VARS'] ?? '');
144+
$dotenvVars = $_SERVER['SYMFONY_DOTENV_VARS'] ?? '';
145+
146+
if ('' === $dotenvVars) {
147+
return [];
148+
}
149+
150+
$vars = explode(',', $dotenvVars);
145151
sort($vars);
146152

147153
return $vars;

‎src/Symfony/Component/Dotenv/Tests/Command/DebugCommandTest.php‎

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,8 @@ class DebugCommandTest extends TestCase
2727
*/
2828
public function testErrorOnUninitializedDotenv()
2929
{
30+
unset($_SERVER['SYMFONY_DOTENV_VARS']);
31+
3032
$command = new DebugCommand('dev', __DIR__.'/Fixtures/Scenario1');
3133
$command->setHelperSet(new HelperSet([new FormatterHelper()]));
3234
$tester = new CommandTester($command);
@@ -36,6 +38,30 @@ public function testErrorOnUninitializedDotenv()
3638
$this->assertStringContainsString('[ERROR] Dotenv component is not initialized', $output);
3739
}
3840

41+
/**
42+
* @runInSeparateProcess
43+
*/
44+
public function testEmptyDotEnvVarsList()
45+
{
46+
$_SERVER['SYMFONY_DOTENV_VARS'] = '';
47+
48+
$command = new DebugCommand('dev', __DIR__.'/Fixtures/Scenario1');
49+
$command->setHelperSet(new HelperSet([new FormatterHelper()]));
50+
$tester = new CommandTester($command);
51+
$tester->execute([]);
52+
$expectedFormat = <<<'OUTPUT'
53+
%a
54+
---------- ------- ------------ ------%S
55+
Variable Value .env.local .env%S
56+
---------- ------- ------------ ------%S
57+
58+
// Note that values might be different between web and CLI.%S
59+
%a
60+
OUTPUT;
61+
62+
$this->assertStringMatchesFormat($expectedFormat, $tester->getDisplay());
63+
}
64+
3965
public function testScenario1InDevEnv()
4066
{
4167
$output = $this->executeCommand(__DIR__.'/Fixtures/Scenario1', 'dev');

0 commit comments

Comments
 (0)