Merge branch '5.0'

nicolas-grekas · nicolas-grekas · commit 332178281e96 · 2019-11-21T08:02:52.000+01:00
* 5.0: [Routing] fix tests [DI] minor cleanup [Form] group constraints when calling the validator Remove wrong @group legacy annotations [DependencyInjection] Fix dumping multiple deprecated aliases allow button names to start with uppercase letter Allow PHP ^7.2.5 States that the HttpClient provides a Http Async implementation [Routing] Fix ContainerLoader and ObjectLoaderTest [HttpKernel] Make ErrorListener::onKernelException()'s dispatcher argument explicit [HttpKernel] Drop deprecated ExceptionListener Removed extra whitespace [Security] Fix best encoder not wired using migrate_from
diff --git a/Loader/ContainerLoader.php b/Loader/ContainerLoader.php
@@ -30,7 +30,7 @@ public function __construct(ContainerInterface $container)
     /**
      * {@inheritdoc}
      */
-    public function supports($resource, $type = null)
+    public function supports($resource, string $type = null)
     {
         return 'service' === $type;
     }
diff --git a/Loader/ObjectLoader.php b/Loader/ObjectLoader.php
@@ -52,7 +52,7 @@ public function load($resource, string $type = null)
         $loaderObject = $this->getObject($parts[0]);
 
         if (!\is_object($loaderObject)) {
-            throw new \LogicException(sprintf('%s:getObject() must return an object: %s returned', \get_class($this), \gettype($loaderObject)));
+            throw new \TypeError(sprintf('%s:getObject() must return an object: %s returned', \get_class($this), \gettype($loaderObject)));
         }
 
         if (!\is_callable([$loaderObject, $method])) {
diff --git a/Tests/Loader/ObjectLoaderTest.php b/Tests/Loader/ObjectLoaderTest.php
@@ -62,12 +62,9 @@ public function getBadResourceStrings()
         ];
     }
 
-    /**
-     * @group legacy
-     */
     public function testExceptionOnNoObjectReturned()
     {
-        $this->expectException('LogicException');
+        $this->expectException(\TypeError::class);
         $loader = new TestObjectLoader();
         $loader->loaderMap = ['my_service' => 'NOT_AN_OBJECT'];
         $loader->load('my_service::method');
diff --git a/composer.json b/composer.json
@@ -16,7 +16,7 @@
         }
     ],
     "require": {
-        "php": "^7.2.9"
+        "php": "^7.2.5"
     },
     "require-dev": {
         "symfony/config": "^5.0",

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ public function __construct(ContainerInterface $container)`
`30`	`30`	`/**`
`31`	`31`	`* {@inheritdoc}`
`32`	`32`	`*/`
`33`		`- public function supports($resource, $type = null)`
	`33`	`+ public function supports($resource, string $type = null)`
`34`	`34`	`{`
`35`	`35`	`return 'service' === $type;`
`36`	`36`	`}`
Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ public function load($resource, string $type = null)`
`52`	`52`	`$loaderObject = $this->getObject($parts[0]);`
`53`	`53`
`54`	`54`	`if (!\is_object($loaderObject)) {`
`55`		`- throw new \LogicException(sprintf('%s:getObject() must return an object: %s returned', \get_class($this), \gettype($loaderObject)));`
	`55`	`+ throw new \TypeError(sprintf('%s:getObject() must return an object: %s returned', \get_class($this), \gettype($loaderObject)));`
`56`	`56`	`}`
`57`	`57`
`58`	`58`	`if (!\is_callable([$loaderObject, $method])) {`
Original file line number	Diff line number	Diff line change
`@@ -62,12 +62,9 @@ public function getBadResourceStrings()`
`62`	`62`	`];`
`63`	`63`	`}`
`64`	`64`
`65`		`- /**`
`66`		`- * @group legacy`
`67`		`- */`
`68`	`65`	`public function testExceptionOnNoObjectReturned()`
`69`	`66`	`{`
`70`		`- $this->expectException('LogicException');`
	`67`	`+ $this->expectException(\TypeError::class);`
`71`	`68`	`$loader = new TestObjectLoader();`
`72`	`69`	`$loader->loaderMap = ['my_service' => 'NOT_AN_OBJECT'];`
`73`	`70`	`$loader->load('my_service::method');`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@`
`16`	`16`	`}`
`17`	`17`	`],`
`18`	`18`	`"require": {`
`19`		`- "php": "^7.2.9"`
	`19`	`+ "php": "^7.2.5"`
`20`	`20`	`},`
`21`	`21`	`"require-dev": {`
`22`	`22`	`"symfony/config": "^5.0",`