[DependencyInjection][Routing][Serializer][Validator] Deprecate XML configuration format #60568
Conversation
2169ea4 to
c5c98cb
Compare
c5c98cb to
e7849d9
Compare
f21defc to
d725030
Compare
|
Let's wait on the discussion on the issue to know what should actually be deprecated. This PR deprecates a lot more than what was mentioned in the issue it fixes. |
| public function testInlineServicesAreNotCandidates() | ||
| { | ||
| $this->expectUserDeprecationMessage('Since symfony/dependency-injection 7.4: XML configuration format is deprecated, use YAML or PHP instead.'); |
There was a problem hiding this comment.
shouldn't we rewrite this to use another format instead?
There was a problem hiding this comment.
This test looks like it’s XML specific: #24491
| @@ -32,6 +33,8 @@ | |||
|
|
|||
| class XmlDumperTest extends TestCase | |||
There was a problem hiding this comment.
side question: do we want to keep the xml dumper?
it's used by some external tools AFAIK, so likely yes
There was a problem hiding this comment.
We also have commands loading such XML dumps, like debug:container. Not sure what we should do about that.
There was a problem hiding this comment.
FTR I gave a try on other formats: PHP dumper cannot dump a non-compiled container, and YAML dumper suffers from #60573
src/Symfony/Component/DependencyInjection/Tests/Loader/FileLoaderTest.php
Outdated
Show resolved
Hide resolved
src/Symfony/Component/DependencyInjection/Tests/Loader/YamlFileLoaderTest.php
Outdated
Show resolved
Hide resolved
d725030 to
d1a9099
Compare
d1a9099 to
ded957c
Compare
From #60560 (comment)