8000 Merge branch '4.4' · symfony/symfony@e281087 · GitHub
[go: up one dir, main page]
Skip to content

Commit e281087

Browse files
nicolas-grekasnicolas-grekas
committed
Merge branch '4.4'
* 4.4: [Validator] relax low dep [Validator] fix conflict rule [Serializer] Fix DataUriNormalizer deprecation (MIME type guesser is optional) [DependencyInjection] fix the ValidateEnvPlaceHolderPassTest that was using a deprecated path for TreeBuilder avoid service id conflicts with Swiftmailer [Form] fix usage of legacy TranslatorInterface [Serializer] Fix DataUriNormalizer docblock & composer suggest section [Validator] v4 conflicts with translation v5
2 parents 17edc6a + c5a2927 commit e281087

File tree

5 files changed

+20
-12
lines changed

5 files changed

+20
-12
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1767,7 +1767,7 @@ private function registerMailerConfiguration(array $config, ContainerBuilder $co
17671767
}
17681768

17691769
$loader->load('mailer.xml');
1770-
$container->getDefinition('mailer.transport')->setArgument(0, $config['dsn']);
1770+
$container->getDefinition('mailer.default_transport')->setArgument(0, $config['dsn']);
17711771
}
17721772

17731773
/**

‎src/Symfony/Bundle/FrameworkBundle/Resources/config/mailer.xml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,23 +6,23 @@
66

77
<services>
88
<service id="mailer.mailer" class="Symfony\Component\Mailer\Mailer">
9-
<argument type="service" id="mailer.transport" />
9+
<argument type="service" id="mailer.default_transport" />
1010
<argument type="service" id="messenger.default_bus" on-invalid="ignore" />
1111
</service>
1212
<service id="mailer" alias="mailer.mailer" />
1313
<service id="Symfony\Component\Mailer\MailerInterface" alias="mailer.mailer" />
1414

15-
<service id="mailer.transport" class="Symfony\Component\Mailer\Transport\TransportInterface">
15+
<service id="mailer.default_transport" class="Symfony\Component\Mailer\Transport\TransportInterface">
1616
<factory class="Symfony\Component\Mailer\Transport" method="fromDsn" />
1717
<argument /> <!-- env(MAILER_DSN) -->
1818
<argument type="service" id="event_dispatcher" />
1919
<argument type="service" id="http_client" on-invalid="ignore" />
2020
<argument type="service" id="logger" on-invalid="ignore" />
2121
</service>
22-
<service id="Symfony\Component\Mailer\Transport\TransportInterface" alias="mailer.transport" />
22+
<service id="Symfony\Component\Mailer\Transport\TransportInterface" alias="mailer.default_transport" />
2323

2424
<service id="mailer.messenger.message_handler" class="Symfony\Component\Mailer\Messenger\MessageHandler">
25-
<argument type="service" id="mailer.transport" />
25+
<argument type="service" id="mailer.default_transport" />
2626
<tag name="messenger.message_handler" />
2727
</service>
2828
</services>

‎src/Symfony/Component/Form/Extension/Core/Type/FileType.php

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,14 @@ class FileType extends AbstractType
3535

3636
private $translator;
3737

38-
public function __construct(TranslatorInterface $translator = null)
38+
/**
39+
* @param TranslatorInterface|null $translator
40+
*/
41+
public function __construct($translator = null)
3942
{
43+
if (null !== $translator && !$translator instanceof LegacyTranslatorInterface && !$translator instanceof TranslatorInterface) {
44+
throw new \TypeError(sprintf('Argument 1 passed to %s() must be an instance of %s, %s given.', __METHOD__, TranslatorInterface::class, \is_object($translator) ? \get_class($translator) : \gettype($translator)));
45+
}
4046
$this->translator = $translator;
4147
}
4248

‎src/Symfony/Component/Serializer/Normalizer/DataUriNormalizer.php

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -34,12 +34,12 @@ class DataUriNormalizer implements NormalizerInterface, DenormalizerInterface, C
3434
];
3535

3636
/**
37-
* @var MimeTypeGuesserInterface
37+
* @var MimeTypeGuesserInterface|null
3838
*/
3939
private $mimeTypeGuesser;
4040

4141
/**
42-
* @param MimeTypeGuesserInterface
42+
* @param MimeTypeGuesserInterface|null $mimeTypeGuesser
4343
*/
4444
public function __construct($mimeTypeGuesser = null)
4545
{
@@ -48,8 +48,8 @@ public function __construct($mimeTypeGuesser = null)
4848
} elseif (null === $mimeTypeGuesser) {
4949
if (class_exists(MimeTypes::class)) {
5050
$mimeTypeGuesser = MimeTypes::getDefault();
51-
} else {
52-
@trigger_error(sprintf('Passing null to "%s()" without symfony/mime installed is deprecated since Symfony 4.3, install symfony/mime.', __METHOD__), E_USER_DEPRECATED);
51+
} elseif (class_exists(MimeTypeGuesser::class)) {
52+
@trigger_error(sprintf('Passing null to "%s()" to use a default MIME type guesser without Symfony Mime installed is deprecated since Symfony 4.3. Try running "composer require symfony/mime".', __METHOD__), E_USER_DEPRECATED);
5353
$mimeTypeGuesser = MimeTypeGuesser::getInstance();
5454
}
5555
} elseif (!$mimeTypeGuesser instanceof MimeTypes) {
@@ -156,7 +156,9 @@ private function getMimeType(\SplFileInfo $object)
156156

157157
if ($this->mimeTypeGuesser instanceof DeprecatedMimeTypeGuesserInterface && $mimeType = $this->mimeTypeGuesser->guess($object->getPathname())) {
158158
return $mimeType;
159-
} elseif ($this->mimeTypeGuesser && $mimeType = $this->mimeTypeGuesser->guessMimeType($object->getPathname())) {
159+
}
160+
161+
if ($this->mimeTypeGuesser && $mimeType = $this->mimeTypeGuesser->guessMimeType($object->getPathname())) {
160162
return $mimeType;
161163
}
162164

‎src/Symfony/Component/Serializer/composer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@
4545
"symfony/yaml": "For using the default YAML mapping loader.",
4646
"symfony/config": "For using the XML mapping loader.",
4747
"symfony/property-access": "For using the ObjectNormalizer.",
48-
"symfony/http-foundation": "To use the DataUriNormalizer.",
48+
"symfony/http-foundation": "For using a MIME type guesser within the DataUriNormalizer.",
4949
"doctrine/annotations": "For using the annotation mapping. You will also need doctrine/cache.",
5050
"doctrine/cache": "For using the default cached annotation reader and metadata cache."
5151
},

0 commit comments

Comments
 (0)
0