8000 Clarify deprecations for framework.templating · symfony/symfony@704c610 · GitHub
[go: up one dir, main page]
Skip to content

Commit 704c610

Browse files
linaorilinaori
committed
Clarify deprecations for framework.templating
1 parent 950306a commit 704c610

File tree

4 files changed

+4
-4
lines changed

4 files changed

+4
-4
lines changed

‎UPGRADE-4.3.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ Form
7777
FrameworkBundle
7878
---------------
7979

80-
* Deprecated the `framework.templating` option, use Twig instead.
80+
* Deprecated the `framework.templating` option, configure the twig bundle instead. If twig is already configured, you can safely remove this option.
8181
* Not passing the project directory to the constructor of the `AssetsInstallCommand` is deprecated. This argument will
8282
be mandatory in 5.0.
8383
* Deprecated the "Psr\SimpleCache\CacheInterface" / "cache.app.simple" service, use "Symfony\Contracts\Cache\CacheInterface" / "cache.app" instead.

‎UPGRADE-5.0.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -169,7 +169,7 @@ Form
169169
FrameworkBundle
170170
---------------
171171

172-
* Removed the `framework.templating` option, use Twig instead.
172+
* Removed the `framework.templating` option, configure the twig bundle instead.
173173
* The project dir argument of the constructor of `AssetsInstallCommand` is required.
174174
* Removed support for `bundle:controller:action` syntax to reference controllers. Use `serviceOrFqcn::method`
175175
instead where `serviceOrFqcn` is either the service ID when using controllers as services or the FQCN of the controller.

‎src/Symfony/Bundle/FrameworkBundle/CHANGELOG.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ CHANGELOG
44
4.3.0
55
-----
66

7-
* Deprecated the `framework.templating` option, use Twig instead.
7+
* Deprecated the `framework.templating` option, configure the twig bundle instead. If twig is already configured, you can safely remove this option.
88
* Added `WebTestAssertionsTrait` (included by default in `WebTestCase`)
99
* Renamed `Client` to `KernelBrowser`
1010
* Not passing the project directory to the constructor of the `AssetsInstallCommand` is deprecated. This argument will

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -608,7 +608,7 @@ private function addTemplatingSection(ArrayNodeDefinition $rootNode)
608608
->arrayNode('templating')
609609
->info('templating configuration')
610610
->canBeEnabled()
611-
->setDeprecated('The "%path%.%node%" configuration is deprecated since Symfony 4.3. Use the "twig" service directly instead.')
611+
->setDeprecated('The "%path%.%node%" configuration is deprecated since Symfony 4.3. Configure the "twig" section provided by the Twig Bundle instead.')
612612
->beforeNormalization()
613613
->ifTrue(function ($v) { return false === $v || \is_array($v) && false === $v['enabled']; })
614614
->then(function () { return ['enabled' => false, 'engines' => false]; })

0 commit comments

Comments
 (0)
0