Merge pull request #2328 from timonf/2.3

dunglas · web-flow · commit 84887fce3dd6 · 2018-11-16T16:43:16.000+01:00
(Very small improvement) Improves exception handling in ReadListener
diff --git a/src/EventListener/ReadListener.php b/src/EventListener/ReadListener.php
@@ -101,7 +101,7 @@ public function onKernelRequest(GetResponseEvent $event)
                 $data = $this->getSubresourceData($identifiers, $attributes, $context);
             }
         } catch (InvalidIdentifierException $e) {
-            $data = null;
+            throw new NotFoundHttpException('Not found, because of an invalid identifier configuration', $e);
         }
 
         if (null === $data) {

Original file line number	Diff line number	Diff line change
`@@ -101,7 +101,7 @@ public function onKernelRequest(GetResponseEvent $event)`
`101`	`101`	`$data = $this->getSubresourceData($identifiers, $attributes, $context);`
`102`	`102`	`}`
`103`	`103`	`} catch (InvalidIdentifierException $e) {`
`104`		`- $data = null;`
	`104`	`+ throw new NotFoundHttpException('Not found, because of an invalid identifier configuration', $e);`
`105`	`105`	`}`
`106`	`106`
`107`	`107`	`if (null === $data) {`