Merge branch '3.3' into 3.4

* 3.3: (23 commits)
  Tests and fix for issue in array model data in EntityType field with multiple=true
  [Form] Fixed PercentToLocalizedStringTransformer to accept both comma and dot as decimal separator, if possible
  removed useless PHPDoc
  [Form] Fix FormInterface::submit() annotation
  [PHPUnitBridge] don't remove when set to  empty string
  PdoSessionHandler: fix advisory lock for pgsql when session.sid_bits_per_character > 4
  HttpCache does not consider ESI resources in HEAD requests
  Fix translation for "This field was not expected"
  [Routing] Enhance Route(Collection) docblocks
  Added improvement for accuracy in MoneyToLocalizedStringTransformer.
  Removed unused private property
  Use correct verb form in the pull request template
  Use PHP_MAXPATHLEN in Filesystem.
  Added null as explicit return type (?TokenInterface)
  [FrameworkBundle] Fix Routing\DelegatingLoader
  Render all line breaks according to the exception message
  [Form] Fix phpdoc
  [DI] remove confusing code
  [Form] Fixed GroupSequence with "constraints" option
  [Validator] Clarify UUID validator behavior
  ...

Author: nicolas-grekas

CacheWarmer/RouterCacheWarmer.php

@@ -29,8 +29,6 @@ class RouterCacheWarmer implements CacheWarmerInterface, ServiceSubscriberInterf
     protected $router;
 
     /**
-     * Constructor.
-     *
      * @param ContainerInterface $container
      */
     public function __construct($container)

CacheWarmer/TemplateFinder.php

@@ -30,8 +30,6 @@ class TemplateFinder implements TemplateFinderInterface
     private $templates;
 
     /**
-     * Constructor.
-     *
      * @param KernelInterface             $kernel  A KernelInterface instance
      * @param TemplateNameParserInterface $parser  A TemplateNameParserInterface instance
      * @param string                      $rootDir The directory where global templates can be stored

CacheWarmer/TemplatePathsCacheWarmer.php

@@ -26,8 +26,6 @@ class TemplatePathsCacheWarmer extends CacheWarmer
     protected $locator;
 
     /**
-     * Constructor.
-     *
      * @param TemplateFinderInterface $finder  A template finder
      * @param TemplateLocator         $locator The template locator
      */

Console/Application.php

@@ -25,8 +25,6 @@
 use Symfony\Component\HttpKernel\Bundle\Bundle;
 
 /**
- * Application.
- *
  * @author Fabien Potencier <[email protected]>
  */
 class Application extends BaseApplication
@@ -36,8 +34,6 @@ class Application extends BaseApplication
     private $registrationErrors = array();
 
     /**
-     * Constructor.
-     *
      * @param KernelInterface $kernel A KernelInterface instance
      */
     public function __construct(KernelInterface $kernel)

Console/Helper/DescriptorHelper.php

@@ -24,9 +24,6 @@
  */
 class DescriptorHelper extends BaseDescriptorHelper
 {
-    /**
-     * Constructor.
-     */
     public function __construct()
     {
         $this

Controller/ControllerNameParser.php

@@ -25,8 +25,6 @@ class ControllerNameParser
     protected $kernel;
 
     /**
-     * Constructor.
-     *
      * @param KernelInterface $kernel A KernelInterface instance
      */
     public function __construct(KernelInterface $kernel)

Controller/ControllerResolver.php

@@ -17,17 +17,13 @@
 use Symfony\Component\HttpKernel\Controller\ContainerControllerResolver;
 
 /**
- * ControllerResolver.
- *
  * @author Fabien Potencier <[email protected]>
  */
 class ControllerResolver extends ContainerControllerResolver
 {
     protected $parser;
 
     /**
-     * Constructor.
-     *
      * @param ContainerInterface   $container A ContainerInterface instance
      * @param ControllerNameParser $parser    A ControllerNameParser instance
      * @param LoggerInterface      $logger    A LoggerInterface instance

HttpCache/HttpCache.php

@@ -29,8 +29,6 @@ abstract class HttpCache extends BaseHttpCache
     protected $kernel;
 
     /**
-     * Constructor.
-     *
      * @param HttpKernelInterface $kernel   An HttpKernelInterface instance
      * @param string              $cacheDir The cache directory (default used if null)
      */

Routing/DelegatingLoader.php

@@ -30,8 +30,6 @@ class DelegatingLoader extends BaseDelegatingLoader
     private $loading = false;
 
     /**
-     * Constructor.
-     *
      * @param ControllerNameParser    $parser   A ControllerNameParser instance
      * @param LoaderResolverInterface $resolver A LoaderResolverInterface instance
      */
@@ -75,7 +73,7 @@ public function load($resource, $type = null)
         }
 
         foreach ($collection->all() as $route) {
-            if (!$controller = $route->getDefault('_controller')) {
+            if (!is_string($controller = $route->getDefault('_controller')) || !$controller) {
                 continue;
             }
 

Routing/Router.php

@@ -33,8 +33,6 @@ class Router extends BaseRouter implements WarmableInterface, ServiceSubscriberI
     private $collectedParameters = array();
 
     /**
-     * Constructor.
-     *
      * @param ContainerInterface $container A ContainerInterface instance
      * @param mixed              $resource  The main resource to load
      * @param array              $options   An array of options