Merge branch '5.2' into 5.x

javiereguiluz · javiereguiluz · commit 6ae444f3516c · 2021-04-21T09:41:43.000+02:00
* 5.2:
  Fixed syntax errors on 4.4
diff --git a/components/console/helpers/formatterhelper.rst b/components/console/helpers/formatterhelper.rst
@@ -78,7 +78,9 @@ you can write::
     $truncatedMessage = $formatter->truncate($message, 7);
     $output->writeln($truncatedMessage);
 
-And the output will be::
+And the output will be:
+
+.. code-block:: text
 
     This is...
 
@@ -93,7 +95,9 @@ from the end of the string::
 
     $truncatedMessage = $formatter->truncate($message, -5);
 
-This will result in::
+This will result in:
+
+.. code-block:: text
 
     This is a very long message, which should be trun...
 
diff --git a/components/console/helpers/table.rst b/components/console/helpers/table.rst
@@ -72,8 +72,8 @@ You can add a table separator anywhere in the output by passing an instance of
 You can optionally display titles at the top and the bottom of the table::
 
     // ...
-    $table->setHeaderTitle('Books')
-    $table->setFooterTitle('Page 1/2')
+    $table->setHeaderTitle('Books');
+    $table->setFooterTitle('Page 1/2');
     $table->render();
 
 .. code-block:: terminal
diff --git a/components/dom_crawler.rst b/components/dom_crawler.rst
@@ -524,7 +524,9 @@ You can virtually set and get values on the form::
     // where "registration" is its own array
     $values = $form->getPhpValues();
 
-To work with multi-dimensional fields::
+To work with multi-dimensional fields:
+
+.. code-block:: html
 
     <form>
         <input name="multi[]"/>
diff --git a/components/event_dispatcher.rst b/components/event_dispatcher.rst
@@ -238,7 +238,7 @@ determine which instance is passed.
         $containerBuilder->addCompilerPass(new AddEventAliasesPass([
             \AcmeFooActionEvent::class => 'acme.foo.action',
         ]));
-        $containerBuilder->addCompilerPass(new RegisterListenersPass(), PassConfig::TYPE_BEFORE_REMOVING)
+        $containerBuilder->addCompilerPass(new RegisterListenersPass(), PassConfig::TYPE_BEFORE_REMOVING);
 
         $containerBuilder->register('event_dispatcher', EventDispatcher::class);
 
diff --git a/components/filesystem.rst b/components/filesystem.rst
@@ -252,7 +252,7 @@ absolute paths and returns the relative path from the second path to the first o
         '/var/lib/symfony/src/Symfony/Component'
     );
     // returns 'videos/'
-    $filesystem->makePathRelative('/tmp/videos', '/tmp')
+    $filesystem->makePathRelative('/tmp/videos', '/tmp');
 
 ``mirror``
 ~~~~~~~~~~
diff --git a/components/http_foundation.rst b/components/http_foundation.rst
@@ -585,7 +585,7 @@ represented by a PHP callable instead of a string::
     header in the response::
 
         // disables FastCGI buffering in nginx only for this response
-        $response->headers->set('X-Accel-Buffering', 'no')
+        $response->headers->set('X-Accel-Buffering', 'no');
 
 .. _component-http-foundation-serving-files:
 
diff --git a/components/intl.rst b/components/intl.rst
@@ -328,7 +328,7 @@ translate into any locale with the ``getName()`` method shown earlier::
 The reverse lookup is also possible thanks to the ``getCountryCode()`` method,
 which returns the code of the country where the given timezone ID belongs to::
 
-    $countryCode = Timezones::getCountryCode('America/Vancouver')
+    $countryCode = Timezones::getCountryCode('America/Vancouver');
     // => $countryCode = 'CA' (CA = Canada)
 
 The `UTC/GMT time offsets`_ of all timezones are provided by ``getRawOffset()``
diff --git a/components/lock.rst b/components/lock.rst
@@ -58,7 +58,7 @@ method can be safely called repeatedly, even if the lock is already acquired.
 
     Unlike other implementations, the Lock Component distinguishes lock
     instances even when they are created for the same resource. It means that for
-    a given scope and resource one lock instance can be acquired multiple times. 
+    a given scope and resource one lock instance can be acquired multiple times.
     If a lock has to be used by several services, they should share the same ``Lock``
     instance returned by the ``LockFactory::createLock`` method.
 
@@ -230,7 +230,7 @@ Lock will be released automatically as soon as one process finishes::
         sleep(30);
     } else {
         // Child process
-        echo 'The lock will be released now.'
+        echo 'The lock will be released now.';
         exit(0);
     }
     // ...
diff --git a/components/options_resolver.rst b/components/options_resolver.rst
@@ -507,7 +507,7 @@ the closure::
 
             $resolver->setDefault('host', function (Options $options, $previousValue) {
                 if ('ssl' === $options['encryption']) {
-                    return 'secure.example.org'
+                    return 'secure.example.org';
                 }
 
                 // Take default value configured in the base class
diff --git a/components/security/authorization.rst b/components/security/authorization.rst
@@ -207,7 +207,7 @@ expressions have access to a number of
 
     $expression = new Expression(
         '"ROLE_ADMIN" in role_names or (not is_anonymous() and user.isSuperAdmin())'
-    )
+    );
 
     $vote = $expressionVoter->vote($token, $object, [$expression]);
 
diff --git a/components/serializer.rst b/components/serializer.rst
@@ -1015,7 +1015,9 @@ For example, take an object normalized as following::
 
     ['foo' => [1, 2], 'bar' => true];
 
-The ``XmlEncoder`` will encode this object like that::
+The ``XmlEncoder`` will encode this object like that:
+
+.. code-block:: xml
 
     <?xml version="1.0" encoding="UTF-8" ?>
     <response>
diff --git a/components/validator/resources.rst b/components/validator/resources.rst
@@ -157,7 +157,7 @@ implement the PSR-6 interface :class:`Psr\\Cache\\CacheItemPoolInterface`)::
 
     $validator = Validation::createValidatorBuilder()
         // ... add loaders
-        ->setMappingCache(new SomePsr6Cache());
+        ->setMappingCache(new SomePsr6Cache())
         ->getValidator();
 
 .. note::
diff --git a/components/workflow.rst b/components/workflow.rst
@@ -68,8 +68,8 @@ are trying to use it with::
     use Symfony\Component\Workflow\Registry;
     use Symfony\Component\Workflow\SupportStrategy\InstanceOfSupportStrategy;
 
-    $blogPostWorkflow = ...
-    $newsletterWorkflow = ...
+    $blogPostWorkflow = ...;
+    $newsletterWorkflow = ...;
 
     $registry = new Registry();
     $registry->addWorkflow($blogPostWorkflow, new InstanceOfSupportStrategy(BlogPost::class));
diff --git a/configuration/override_dir_structure.rst b/configuration/override_dir_structure.rst
@@ -238,7 +238,7 @@ option in your ``composer.json`` file like this:
         "config": {
             "bin-dir": "bin",
             "vendor-dir": "/some/dir/vendor"
-        },
+        }
     }
 
 .. tip::
diff --git a/configuration/using_parameters_in_dic.rst b/configuration/using_parameters_in_dic.rst
@@ -85,8 +85,8 @@ Now, examine the results to see this closely:
                 // true/false (depends on 2nd parameter of Kernel),
                 // as expected, because %kernel.debug% inside configuration
                 // gets evaluated before being passed to the extension
-            )
-        ];
+            ]
+        );
 
         $container->loadFromExtension('my_bundle');
         // passes the string "%kernel.debug%".
diff --git a/contributing/code/conventions.rst b/contributing/code/conventions.rst
@@ -183,15 +183,19 @@ after the use declarations, like in this example from `ServiceRouterLoader`_::
 
 .. _`ServiceRouterLoader`: https://github.com/symfony/symfony/blob/4.4/src/Symfony/Component/Routing/Loader/DependencyInjection/ServiceRouterLoader.php
 
-The deprecation must be added to the ``CHANGELOG.md`` file of the impacted component::
+The deprecation must be added to the ``CHANGELOG.md`` file of the impacted component:
+
+.. code-block:: markdown
 
     4.4
     ---
 
     * Deprecate the `Deprecated` class, use `Replacement` instead
 
 It must also be added to the ``UPGRADE.md`` file of the targeted minor version
-(``UPGRADE-4.4.md`` in our example)::
+(``UPGRADE-4.4.md`` in our example):
+
+.. code-block:: markdown
 
     DependencyInjection
     -------------------
diff --git a/controller/soap_web_service.rst b/controller/soap_web_service.rst
@@ -39,7 +39,7 @@ In this case, the SOAP service will allow the client to call a method called
         public function hello($name)
         {
 
-            $message = new \Swift_Message('Hello Service')
+            $message = (new \Swift_Message('Hello Service'))
                 ->setTo('me@example.com')
                 ->setBody($name.' says hi!');
 
diff --git a/create_framework/front_controller.rst b/create_framework/front_controller.rst
@@ -185,7 +185,9 @@ the ``setContent()`` directly from the front controller script::
 
     // ...
 
-And the ``hello.php`` script can now be converted to a template::
+And the ``hello.php`` script can now be converted to a template:
+
+.. code-block:: html+php
 
     <!-- example.com/src/pages/hello.php -->
     <?php $name = $request->get('name', 'World') ?>
diff --git a/create_framework/routing.rst b/create_framework/routing.rst
@@ -30,7 +30,9 @@ framework just a little to make templates even more readable::
     $response->send();
 
 As we now extract the request query parameters, simplify the ``hello.php``
-template as follows::
+template as follows:
+
+.. code-block:: html+php
 
     <!-- example.com/src/pages/hello.php -->
     Hello <?= htmlspecialchars(isset($name) ? $name : 'World', ENT_QUOTES, 'UTF-8') ?>
@@ -161,7 +163,9 @@ There are a few new things in the code:
 
 * ``500`` errors are now managed correctly;
 
-* Request attributes are extracted to keep our templates simple::
+* Request attributes are extracted to keep our templates simple:
+
+.. code-block:: html+php
 
     // example.com/src/pages/hello.php
     Hello <?= htmlspecialchars($name, ENT_QUOTES, 'UTF-8') ?>
diff --git a/doctrine/custom_dql_functions.rst b/doctrine/custom_dql_functions.rst
@@ -131,16 +131,14 @@ In Symfony, you can register your custom DQL functions as follows:
             use App\DQL\DatetimeFunction;
 
             $container->loadFromExtension('doctrine', [
-                'doctrine' => [
-                    'orm' => [
-                        // ...
-                        'entity_managers' => [
-                            'example_manager' => [
-                                // place your functions here
-                                'dql' => [
-                                    'datetime_functions' => [
-                                        'test_datetime' => DatetimeFunction::class,
-                                    ],
+                'orm' => [
+                    // ...
+                    'entity_managers' => [
+                        'example_manager' => [
+                            // place your functions here
+                            'dql' => [
+                                'datetime_functions' => [
+                                    'test_datetime' => DatetimeFunction::class,
                                 ],
                             ],
                         ],
diff --git a/form/create_form_type_extension.rst b/form/create_form_type_extension.rst
@@ -191,14 +191,10 @@ Specifically, you need to override the ``file_widget`` block:
     {% extends 'form_div_layout.html.twig' %}
 
     {% block file_widget %}
-        {% spaceless %}
-
         {{ block('form_widget') }}
         {% if image_url is not null %}
             <img src="{{ asset(image_url) }}"/>
         {% endif %}
-
-        {% endspaceless %}
     {% endblock %}
 
 Be sure to :ref:`configure this form theme template <forms-theming-global>` so that
diff --git a/form/data_transformers.rst b/form/data_transformers.rst
@@ -208,7 +208,7 @@ to and from the issue number and the ``Issue`` object::
         {
             // no issue number? It's optional, so that's ok
             if (!$issueNumber) {
-                return;
+                return null;
             }
 
             $issue = $this->entityManager
diff --git a/form/type_guesser.rst b/form/type_guesser.rst
@@ -101,7 +101,7 @@ With this knowledge, you can implement the ``guessType()`` method of the
             $annotations = $this->readPhpDocAnnotations($class, $property);
 
             if (!isset($annotations['var'])) {
-                return; // guess nothing if the @var annotation is not available
+                return null; // guess nothing if the @var annotation is not available
             }
 
             // otherwise, base the type on the @var annotation
diff --git a/form/validation_group_service_resolver.rst b/form/validation_group_service_resolver.rst
@@ -39,7 +39,7 @@ Then in your form, inject the resolver and set it as the ``validation_groups``::
     namespace App\Form;
 
     use App\Validator\ValidationGroupResolver;
-    use Symfony\Component\Form\AbstractType
+    use Symfony\Component\Form\AbstractType;
     use Symfony\Component\OptionsResolver\OptionsResolver;
 
     class MyClassType extends AbstractType
diff --git a/forms.rst b/forms.rst
@@ -655,7 +655,7 @@ argument of ``createForm()``::
         {
             $task = new Task();
             // use some PHP logic to decide if this form field is required or not
-            $dueDateIsRequired = ...
+            $dueDateIsRequired = ...;
 
             $form = $this->createForm(TaskType::class, $task, [
                 'require_due_date' => $dueDateIsRequired,
diff --git a/http_client.rst b/http_client.rst
@@ -979,7 +979,7 @@ To abort a request (e.g. because it didn't complete in due time, or you want to
 fetch only the first bytes of the response, etc.), you can either use the
 ``cancel()`` method of ``ResponseInterface``::
 
-    $response->cancel()
+    $response->cancel();
 
 Or throw an exception from a progress callback::
 
diff --git a/introduction/from_flat_php_to_symfony.rst b/introduction/from_flat_php_to_symfony.rst
@@ -580,7 +580,7 @@ and uses Twig:
 
 .. code-block:: html+twig
 
-    <!-- templates/blog/list.html.twig -->
+    {# templates/blog/list.html.twig #}
     {% extends 'base.html.twig' %}
 
     {% block title %}List of Posts{% endblock %}
diff --git a/mercure.rst b/mercure.rst
@@ -623,7 +623,7 @@ You can instead make use of the `MockHub`::
 
             $controller = new MessageController($hub);
 
-            ...
+            // ...
         }
     }
 
@@ -646,13 +646,14 @@ During functional testing you can instead decorate the Hub::
     }
 
 HubStub decorates the default hub service so no updates are actually
-sent. Here is the HubStub implementation::
+sent. Here is the HubStub implementation:
+
+.. code-block:: yaml
 
     # config/services_test.yaml
     App\Tests\Functional\Fixtures\HubStub:
         decorates: mercure.hub.default
 
-
 Debugging
 ---------
 
diff --git a/migration.rst b/migration.rst
@@ -316,11 +316,11 @@ somewhat like this::
 
     class LegacyBridge
     {
-        public static function prepareLegacyScript(Request $request, Response $response, string $publicDirectory): string
+        public static function prepareLegacyScript(Request $request, Response $response, string $publicDirectory): ?string
         {
             // If Symfony successfully handled the route, you do not have to do anything.
             if (false === $response->isNotFound()) {
-                return;
+                return null;
             }
 
             // Figure out how to map to the needed script file
diff --git a/reference/configuration/framework.rst b/reference/configuration/framework.rst
@@ -462,7 +462,9 @@ Since every developer uses a different IDE, the recommended way to enable this
 feature is to configure it on a system level. This can be done by setting the
 ``xdebug.file_link_format`` option in your ``php.ini`` configuration file. The
 format to use is the same as for the ``framework.ide`` option, but without the
-need to escape the percent signs (``%``) by doubling them::
+need to escape the percent signs (``%``) by doubling them:
+
+.. code-block:: ini
 
     // example for PhpStorm
     xdebug.file_link_format="phpstorm://open?file=%f&line=%l"
@@ -485,7 +487,9 @@ need to escape the percent signs (``%``) by doubling them::
     When running your app in a container or in a virtual machine, you can tell
     Symfony to map files from the guest to the host by changing their prefix.
     This map should be specified at the end of the URL template, using ``&`` and
-    ``>`` as guest-to-host separators::
+    ``>`` as guest-to-host separators:
+
+    .. code-block:: text
 
         // /path/to/guest/.../file will be opened
         // as /path/to/host/.../file on the host
diff --git a/service_container.rst b/service_container.rst
diff --git a/setup/homestead.rst b/setup/homestead.rst
diff --git a/templating/hinclude.rst b/templating/hinclude.rst
diff --git a/testing.rst b/testing.rst
diff --git a/translation.rst b/translation.rst

Original file line number	Diff line number	Diff line change
`@@ -507,7 +507,7 @@ the closure::`
`507`	`507`
`508`	`508`	`$resolver->setDefault('host', function (Options $options, $previousValue) {`
`509`	`509`	`if ('ssl' === $options['encryption']) {`
`510`		`- return 'secure.example.org'`
	`510`	`+ return 'secure.example.org';`
`511`	`511`	`}`
`512`	`512`
`513`	`513`	`// Take default value configured in the base class`
Original file line number	Diff line number	Diff line change
@@ -238,7 +238,7 @@ option in your ``composer.json`` file like this:
`238`	`238`	`"config": {`
`239`	`239`	`"bin-dir": "bin",`
`240`	`240`	`"vendor-dir": "/some/dir/vendor"`
`241`		`- },`
	`241`	`+ }`
`242`	`242`	`}`
`243`	`243`
`244`	`244`	`.. tip::`
Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ In this case, the SOAP service will allow the client to call a method called`
`39`	`39`	`public function hello($name)`
`40`	`40`	`{`
`41`	`41`
`42`		`- $message = new \Swift_Message('Hello Service')`
	`42`	`+ $message = (new \Swift_Message('Hello Service'))`
`43`	`43`	`->setTo('[email protected]')`
`44`	`44`	`->setBody($name.' says hi!');`
`45`	`45`
Original file line number	Diff line number	Diff line change
@@ -208,7 +208,7 @@ to and from the issue number and the ``Issue`` object::
`208`	`208`	`{`
`209`	`209`	`// no issue number? It's optional, so that's ok`
`210`	`210`	`if (!$issueNumber) {`
`211`		`- return;`
	`211`	`+ return null;`
`212`	`212`	`}`
`213`	`213`
`214`	`214`	`$issue = $this->entityManager`
Original file line number	Diff line number	Diff line change
@@ -101,7 +101,7 @@ With this knowledge, you can implement the ``guessType()`` method of the
`101`	`101`	`$annotations = $this->readPhpDocAnnotations($class, $property);`
`102`	`102`
`103`	`103`	`if (!isset($annotations['var'])) {`
`104`		`- return; // guess nothing if the @var annotation is not available`
	`104`	`+ return null; // guess nothing if the @var annotation is not available`
`105`	`105`	`}`
`106`	`106`
`107`	`107`	`// otherwise, base the type on the @var annotation`
Original file line number	Diff line number	Diff line change
@@ -655,7 +655,7 @@ argument of ``createForm()``::
`655`	`655`	`{`
`656`	`656`	`$task = new Task();`
`657`	`657`	`// use some PHP logic to decide if this form field is required or not`
`658`		`- $dueDateIsRequired = ...`
	`658`	`+ $dueDateIsRequired = ...;`
`659`	`659`
`660`	`660`	`$form = $this->createForm(TaskType::class, $task, [`
`661`	`661`	`'require_due_date' => $dueDateIsRequired,`
Original file line number	Diff line number	Diff line change
@@ -623,7 +623,7 @@ You can instead make use of the `MockHub`::
`623`	`623`
`624`	`624`	`$controller = new MessageController($hub);`
`625`	`625`
`626`		`- ...`
	`626`	`+ // ...`
`627`	`627`	`}`
`628`	`628`	`}`
`629`	`629`
`@@ -646,13 +646,14 @@ During functional testing you can instead decorate the Hub::`
`646`	`646`	`}`
`647`	`647`
`648`	`648`	`HubStub decorates the default hub service so no updates are actually`
`649`		`-sent. Here is the HubStub implementation::`
	`649`	`+sent. Here is the HubStub implementation:`
	`650`	`+`
	`651`	`+.. code-block:: yaml`
`650`	`652`
`651`	`653`	`# config/services_test.yaml`
`652`	`654`	`App\Tests\Functional\Fixtures\HubStub:`
`653`	`655`	`decorates: mercure.hub.default`
`654`	`656`
`655`		`-`
`656`	`657`	`Debugging`
`657`	`658`	`---------`
`658`	`659`
Original file line number	Diff line number	Diff line change
`@@ -316,11 +316,11 @@ somewhat like this::`
`316`	`316`
`317`	`317`	`class LegacyBridge`
`318`	`318`	`{`
`319`		`- public static function prepareLegacyScript(Request $request, Response $response, string $publicDirectory): string`
	`319`	`+ public static function prepareLegacyScript(Request $request, Response $response, string $publicDirectory): ?string`
`320`	`320`	`{`
`321`	`321`	`// If Symfony successfully handled the route, you do not have to do anything.`
`322`	`322`	`if (false === $response->isNotFound()) {`
`323`		`- return;`
	`323`	`+ return null;`
`324`	`324`	`}`
`325`	`325`
`326`	`326`	`// Figure out how to map to the needed script file`