minor #7737 lots of minor tweaks and fixes (atailouloute, Ahmed TAILOULOUTE)

xabbuh · xabbuh · commit 18a2ad4f50f0 · 2017-04-01T08:41:33.000+02:00
This PR was squashed before being merged into the 2.7 branch (closes #7737). Discussion ---------- lots of minor tweaks and fixes This PR contains all the commits from the following pull requests: * #7641 * #7652 * #7653 * #7666 * #7667 * #7669 * #7673 * #7674 * #7675 * #7677 * #7680 * #7681 * #7710 * #7711 * #7712 * #7713 * #7714 * #7716 * #7717 * #7718 * #7721 * #7722 * #7736 Commits ------- 3d61281 lots of minor tweaks and fixes
diff --git a/assetic/asset_management.rst b/assetic/asset_management.rst
@@ -328,7 +328,7 @@ configuration under the ``assetic`` section. Read more in the
                     ),
                 ),
             ),
-        );
+        ));
 
 After you have defined the named assets, you can reference them in your templates
 with the ``@named_asset`` notation:
diff --git a/assetic/jpeg_optimize.rst b/assetic/jpeg_optimize.rst
@@ -232,7 +232,6 @@ following configuration:
             ),
             'twig' => array(
                 'functions' => array('jpegoptim'),
-                ),
             ),
         ));
 
@@ -294,7 +293,7 @@ file:
             'twig' => array(
                 'functions' => array(
                     'jpegoptim' => array(
-                        output => 'images/*.jpg'
+                        'output' => 'images/*.jpg'
                     ),
                 ),
             ),
diff --git a/assetic/uglifyjs.rst b/assetic/uglifyjs.rst
@@ -168,11 +168,11 @@ can configure its location using the ``node`` key:
 
         // app/config/config.php
         $container->loadFromExtension('assetic', array(
-            'node' => '/usr/bin/nodejs',
+            'node'      => '/usr/bin/nodejs',
             'uglifyjs2' => array(
-                    // the path to the uglifyjs executable
-                    'bin' => '/usr/local/bin/uglifyjs',
-                ),
+                // the path to the uglifyjs executable
+                'bin' => '/usr/local/bin/uglifyjs',
+            ),
         ));
 
 Minify your Assets
diff --git a/components/routing.rst b/components/routing.rst
@@ -206,6 +206,9 @@ to find a route that fits the given request you can also build a URL from
 a certain route::
 
     use Symfony\Component\Routing\Generator\UrlGenerator;
+    use Symfony\Component\Routing\RequestContext;
+    use Symfony\Component\Routing\Route;
+    use Symfony\Component\Routing\RouteCollection;
 
     $routes = new RouteCollection();
     $routes->add('show_post', new Route('/show/{slug}'));
@@ -321,7 +324,7 @@ a path to the main route definition and some other settings::
         $resource,
         array $options = array(),
         RequestContext $context = null,
-        array $defaults = array()
+        LoggerInterface $logger = null
     );
 
 With the ``cache_dir`` option you can enable route caching (if you provide a
diff --git a/components/security/authentication.rst b/components/security/authentication.rst
@@ -76,6 +76,7 @@ The default authentication manager is an instance of
 :class:`Symfony\\Component\\Security\\Core\\Authentication\\AuthenticationProviderManager`::
 
     use Symfony\Component\Security\Core\Authentication\AuthenticationProviderManager;
+    use Symfony\Component\Security\Core\Exception\AuthenticationException;
 
     // instances of Symfony\Component\Security\Core\Authentication\Provider\AuthenticationProviderInterface
     $providers = array(...);
@@ -234,6 +235,7 @@ own, it just needs to follow these rules:
                }
 
                // ...
+           }
        }
 
 Using Password Encoders
diff --git a/configuration.rst b/configuration.rst
@@ -316,7 +316,7 @@ configure DoctrineBundle and other parts of Symfony:
     .. code-block:: php
 
         // app/config/config.php
-        $configuration->loadFromExtension('doctrine', array(
+        $container->loadFromExtension('doctrine', array(
             'dbal' => array(
                 'driver'   => 'pdo_mysql',
                 // ...
diff --git a/configuration/environments.rst b/configuration/environments.rst
@@ -294,7 +294,7 @@ The best way to accomplish this is via a new environment called, for example,
     .. code-block:: php
 
         // app/config/config_benchmark.php
-        $loader->import('config_prod.php')
+        $loader->import('config_prod.php');
 
         $container->loadFromExtension('framework', array(
             'profiler' => array('only_exceptions' => false),
diff --git a/controller/service.rst b/controller/service.rst
@@ -335,7 +335,7 @@ controller:
         $templating = $this->templating;
         $callback = function () use ($templating, $view, $parameters) {
             $templating->stream($view, $parameters);
-        }
+        };
 
         return new StreamedResponse($callback);
 
diff --git a/doctrine.rst b/doctrine.rst
@@ -95,7 +95,7 @@ information. By convention, this information is usually configured in an
         .. code-block:: php
 
             // app/config/config.php
-            $configuration->loadFromExtension('doctrine', array(
+            $container->loadFromExtension('doctrine', array(
                 'dbal' => array(
                     'driver'   => 'pdo_mysql',
                     'host'     => '%database_host%',
diff --git a/form/action_method.rst b/form/action_method.rst
@@ -47,7 +47,7 @@ to the ``form()`` or the ``form_start()`` helper functions:
 
     .. code-block:: html+php
 
-        <!-- app/Resources/views/default/newAction.html.php -->
+        <!-- app/Resources/views/default/new.html.php -->
         <?php echo $view['form']->start($form, array(
             'action' => $view['router']->generate('target_route'),
             'method' => 'GET',
diff --git a/form/create_form_type_extension.rst b/form/create_form_type_extension.rst
@@ -120,7 +120,7 @@ tag:
 
         $container
             ->register('app.image_type_extension', ImageTypeExtension::class)
-            ->addTag('form.type_extension', array('alias' => 'file'));
+            ->addTag('form.type_extension', array('alias' => 'file'))
         ;
 
 The ``alias`` key of the tag is the type of field that this extension should
diff --git a/form/data_transformers.rst b/form/data_transformers.rst
@@ -35,7 +35,7 @@ Suppose you have a Task form with a tags ``text`` type::
     {
         public function buildForm(FormBuilderInterface $builder, array $options)
         {
-            $builder->add('tags', 'text')
+            $builder->add('tags', 'text');
         }
 
         public function configureOptions(OptionsResolver $resolver)
diff --git a/form/without_class.rst b/form/without_class.rst
@@ -104,4 +104,4 @@ but here's a short example:
     
     .. code-block:: php
 
-        new NotBlank(array('groups' => array('create', 'update')))
+        new NotBlank(array('groups' => array('create', 'update')));
diff --git a/http_cache/cache_invalidation.rst b/http_cache/cache_invalidation.rst
@@ -74,9 +74,9 @@ Here is how you can configure the Symfony reverse proxy to support the
 
             $response = new Response();
             if ($this->getStore()->purge($request->getUri())) {
-                $response->setStatusCode(200, 'Purged');
+                $response->setStatusCode(Response::HTTP_OK, 'Purged');
             } else {
-                $response->setStatusCode(404, 'Not found');
+                $response->setStatusCode(Response::HTTP_NOT_FOUND, 'Not found');
             }
 
             return $response;
diff --git a/http_cache/esi.rst b/http_cache/esi.rst
@@ -137,9 +137,10 @@ matter), Symfony uses the standard ``render`` helper to configure ESI tags:
     .. code-block:: html+php
 
         <!-- app/Resources/views/static/about.html.php -->
-
+        <?php
         // you can use a controller reference
         use Symfony\Component\HttpKernel\Controller\ControllerReference;
+        ?>
         <?php echo $view['actions']->render(
             new ControllerReference(
                 'AppBundle:News:latest',
@@ -148,15 +149,17 @@ matter), Symfony uses the standard ``render`` helper to configure ESI tags:
             array('strategy' => 'esi')
         ) ?>
 
+        <?php
         // ... or a URL
         use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
+        ?>
         <?php echo $view['actions']->render(
             $view['router']->generate(
                 'latest_news',
                 array('maxPerPage' => 5),
                 UrlGeneratorInterface::ABSOLUTE_URL
             ),
-            array('strategy' => 'esi'),
+            array('strategy' => 'esi')
         ) ?>
 
 By using the ``esi`` renderer (via the ``render_esi()`` Twig function), you
@@ -234,7 +237,7 @@ that must be enabled in your configuration:
         <?xml version="1.0" encoding="UTF-8" ?>
         <container xmlns="http://symfony.com/schema/dic/services"
             xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-            xmlns:doctrine="http://symfony.com/schema/dic/framework"
+            xmlns:framework="http://symfony.com/schema/dic/framework"
             xsi:schemaLocation="http://symfony.com/schema/dic/services
                 http://symfony.com/schema/dic/services/services-1.0.xsd
                 http://symfony.com/schema/dic/symfony
diff --git a/http_cache/expiration.rst b/http_cache/expiration.rst
@@ -36,7 +36,7 @@ Most of the time, you will use the ``Cache-Control`` header. Recall that the
 ``Cache-Control`` header is used to specify many different cache directives::
 
     // Sets the number of seconds after which the response
-    // should no longer be considered fresh
+    // should no longer be considered fresh by shared caches
     $response->setSharedMaxAge(600);
 
 The ``Cache-Control`` header would take on the following format (it may have
diff --git a/reference/constraints/CardScheme.rst b/reference/constraints/CardScheme.rst
@@ -28,7 +28,7 @@ on an object that will contain a credit card number.
     .. code-block:: php-annotations
 
         // src/AppBundle/Entity/Transaction.php
-        namespace AppBundle\Entity\Transaction;
+        namespace AppBundle\Entity;
 
         use Symfony\Component\Validator\Constraints as Assert;
 
@@ -76,7 +76,7 @@ on an object that will contain a credit card number.
     .. code-block:: php
 
         // src/AppBundle/Entity/Transaction.php
-        namespace AppBundle\Entity\Transaction;
+        namespace AppBundle\Entity;
 
         use Symfony\Component\Validator\Mapping\ClassMetadata;
         use Symfony\Component\Validator\Constraints as Assert;
diff --git a/reference/constraints/UniqueEntity.rst b/reference/constraints/UniqueEntity.rst
@@ -89,7 +89,7 @@ your user table:
 
     .. code-block:: php
 
-        // src/AppBundle/Entity/User.php
+        // src/AppBundle/Entity/Author.php
         namespace AppBundle\Entity;
 
         use Symfony\Component\Validator\Constraints as Assert;
diff --git a/reference/dic_tags.rst b/reference/dic_tags.rst
@@ -1439,7 +1439,7 @@ If you do need to use this tag, just make a new class that implements the
 :class:`Symfony\\Component\\Validator\\ObjectInitializerInterface` interface.
 Then, tag it with the ``validator.initializer`` tag (it has no options).
 
-For an example, see the ``EntityInitializer`` class inside the Doctrine
+For an example, see the ``DoctrineInitializer`` class inside the Doctrine
 Bridge.
 
 .. _`Twig's documentation`: http://twig.sensiolabs.org/doc/advanced.html#creating-an-extension
diff --git a/routing/hostname_pattern.rst b/routing/hostname_pattern.rst
@@ -193,7 +193,8 @@ instance, if you want to match both ``m.example.com`` and
             <?xml version="1.0" encoding="UTF-8" ?>
             <routes xmlns="http://symfony.com/schema/routing"
                 xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-                xsi:schemaLocation="http://symfony.com/schema/routing http://symfony.com/schema/routing/routing-1.0.xsd">
+                xsi:schemaLocation="http://symfony.com/schema/routing
+                    http://symfony.com/schema/routing/routing-1.0.xsd">
 
                 <route id="mobile_homepage" path="/" host="m.{domain}">
                     <default key="_controller">AcmeDemoBundle:Main:mobileHomepage</default>
@@ -251,7 +252,8 @@ You can also set the host option on imported routes:
         <?xml version="1.0" encoding="UTF-8" ?>
         <routes xmlns="http://symfony.com/schema/routing"
             xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-            xsi:schemaLocation="http://symfony.com/schema/routing http://symfony.com/schema/routing/routing-1.0.xsd">
+            xsi:schemaLocation="http://symfony.com/schema/routing
+                http://symfony.com/schema/routing/routing-1.0.xsd">
 
             <import resource="@AcmeHelloBundle/Resources/config/routing.xml" host="hello.example.com" />
         </routes>
diff --git a/routing/requirements.rst b/routing/requirements.rst
@@ -184,7 +184,7 @@ accomplished with the following route configuration:
 
     .. code-block:: php-annotations
 
-        // src/AppBundle/Controller/MainController.php
+        // src/AppBundle/Controller/BlogApiController.php
         namespace AppBundle\Controller;
 
         use Sensio\Bundle\FrameworkExtraBundle\Configuration\Method;
diff --git a/security.rst b/security.rst
@@ -86,11 +86,11 @@ configuration looks like this:
             ),
             'firewalls' => array(
                 'dev' => array(
-                    'pattern'    => '^/(_(profiler|wdt)|css|images|js)/',
-                    'security'   => false,
+                    'pattern'   => '^/(_(profiler|wdt)|css|images|js)/',
+                    'security'  => false,
                 ),
                 'default' => array(
-                    'anonymous'  => null,
+                    'anonymous' => null,
                 ),
             ),
         ));
@@ -257,7 +257,7 @@ user to be logged in to access this URL:
             ),
            'access_control' => array(
                // require ROLE_ADMIN for /admin*
-                array('path' => '^/admin', 'role' => 'ROLE_ADMIN'),
+                array('path' => '^/admin', 'roles' => 'ROLE_ADMIN'),
             ),
         ));
 
diff --git a/security/acl.rst b/security/acl.rst
@@ -77,10 +77,12 @@ First, you need to configure the connection the ACL system is supposed to use:
     .. code-block:: php
 
         // app/config/security.php
-        $container->loadFromExtension('security', 'acl', array(
+        $container->loadFromExtension('security', array(
             // ...
-
-            'connection' => 'default',
+            
+            'acl' => array(
+                'connection' => 'default',            
+            ),
         ));
 
 .. note::
diff --git a/security/impersonating_user.rst b/security/impersonating_user.rst
@@ -97,7 +97,7 @@ to show a link to exit impersonation:
             <a
                 href="<?php echo $view['router']->generate('homepage', array(
                     '_switch_user' => '_exit',
-                ) ?>"
+                )) ?>"
             >
                 Exit impersonation
             </a>
diff --git a/security/pre_authenticated.rst b/security/pre_authenticated.rst
@@ -142,7 +142,7 @@ corresponding firewall in your security configuration:
         $container->loadFromExtension('security', array(
             'firewalls' => array(
                 'secured_area' => array(
-                    'pattern'     => '^/'
+                    'pattern'     => '^/',
                     'remote_user' => array(
                         'provider' => 'your_user_provider',
                     ),
diff --git a/security/voters.rst b/security/voters.rst
@@ -279,7 +279,7 @@ security configuration:
         >
 
             <config>
-                <access-decision-manager strategy="unanimous">
+                <access-decision-manager strategy="unanimous" />
             </config>
         </srv:container>
 
diff --git a/service_container.rst b/service_container.rst
@@ -444,7 +444,7 @@ Injecting the dependency by the setter method just needs a change of syntax:
 
         $container->setDefinition('app.mailer', ...);
 
-        $definition = new Definition(NewsletterManager::class)
+        $definition = new Definition(NewsletterManager::class);
         $definition->addMethodCall('setMailer', array(
             new Reference('app.mailer'),
         ));
diff --git a/session/php_bridge.rst b/session/php_bridge.rst
@@ -42,6 +42,7 @@ for the ``handler_id``:
             'session' => array(
                 'storage_id' => 'session.storage.php_bridge',
                 'handler_id' => null,
+            ),
         ));
 
 Otherwise, if the problem is simply that you cannot avoid the application
@@ -77,6 +78,7 @@ the example below:
             'session' => array(
                 'storage_id' => 'session.storage.php_bridge',
                 'handler_id' => 'session.storage.native_file',
+            ),
         ));
 
 .. note::
diff --git a/templating/namespaced_paths.rst b/templating/namespaced_paths.rst
@@ -68,7 +68,7 @@ directory:
         $container->loadFromExtension('twig', array(
             'paths' => array(
                 '%kernel.root_dir%/../vendor/acme/foo-bar/templates' => 'foo_bar',
-            );
+            )
         ));
 
 .. caution::
diff --git a/testing/database.rst b/testing/database.rst
@@ -145,7 +145,7 @@ configuration:
     .. code-block:: php
 
         // app/config/config_test.php
-        $configuration->loadFromExtension('doctrine', array(
+        $container->loadFromExtension('doctrine', array(
             'dbal' => array(
                 'host'     => 'localhost',
                 'dbname'   => 'testdb',
diff --git a/validation.rst b/validation.rst
@@ -155,7 +155,7 @@ message:
 
 .. code-block:: text
 
-    AppBundle\Author.name:
+    AppBundle\Entity\Author.name:
         This value should not be blank
 
 If you insert a value into the ``name`` property, the happy success message
diff --git a/validation/sequence_provider.rst b/validation/sequence_provider.rst

Original file line number	Diff line number	Diff line change
`@@ -76,6 +76,7 @@ The default authentication manager is an instance of`
`76`	`76`	:class:`Symfony\\Component\\Security\\Core\\Authentication\\AuthenticationProviderManager`::
`77`	`77`
`78`	`78`	`use Symfony\Component\Security\Core\Authentication\AuthenticationProviderManager;`
	`79`	`+ use Symfony\Component\Security\Core\Exception\AuthenticationException;`
`79`	`80`
`80`	`81`	`// instances of Symfony\Component\Security\Core\Authentication\Provider\AuthenticationProviderInterface`
`81`	`82`	`$providers = array(...);`
`@@ -234,6 +235,7 @@ own, it just needs to follow these rules:`
`234`	`235`	`}`
`235`	`236`
`236`	`237`	`// ...`
	`238`	`+ }`
`237`	`239`	`}`
`238`	`240`
`239`	`241`	`Using Password Encoders`
Original file line number	Diff line number	Diff line change
`@@ -120,7 +120,7 @@ tag:`
`120`	`120`
`121`	`121`	`$container`
`122`	`122`	`->register('app.image_type_extension', ImageTypeExtension::class)`
`123`		`- ->addTag('form.type_extension', array('alias' => 'file'));`
	`123`	`+ ->addTag('form.type_extension', array('alias' => 'file'))`
`124`	`124`	`;`
`125`	`125`
`126`	`126`	The ``alias`` key of the tag is the type of field that this extension should
Original file line number	Diff line number	Diff line change
@@ -35,7 +35,7 @@ Suppose you have a Task form with a tags ``text`` type::
`35`	`35`	`{`
`36`	`36`	`public function buildForm(FormBuilderInterface $builder, array $options)`
`37`	`37`	`{`
`38`		`- $builder->add('tags', 'text')`
	`38`	`+ $builder->add('tags', 'text');`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`public function configureOptions(OptionsResolver $resolver)`
Original file line number	Diff line number	Diff line change
`@@ -104,4 +104,4 @@ but here's a short example:`
`104`	`104`
`105`	`105`	`.. code-block:: php`
`106`	`106`
`107`		`- new NotBlank(array('groups' => array('create', 'update')))`
	`107`	`+ new NotBlank(array('groups' => array('create', 'update')));`
Original file line number	Diff line number	Diff line change
`@@ -97,7 +97,7 @@ to show a link to exit impersonation:`
`97`	`97`	`<a`
`98`	`98`	`href="<?php echo $view['router']->generate('homepage', array(`
`99`	`99`	`'_switch_user' => '_exit',`
`100`		`- ) ?>"`
	`100`	`+ )) ?>"`
`101`	`101`	`>`
`102`	`102`	`Exit impersonation`
`103`	`103`	`</a>`
Original file line number	Diff line number	Diff line change
`@@ -279,7 +279,7 @@ security configuration:`
`279`	`279`	`>`
`280`	`280`
`281`	`281`	`<config>`
`282`		`- <access-decision-manager strategy="unanimous">`
	`282`	`+ <access-decision-manager strategy="unanimous" />`
`283`	`283`	`</config>`
`284`	`284`	`</srv:container>`
`285`	`285`