minor #45810 [HttpFoundation] Propagate SessionNotFoundException in Request::getSession docBlock (a.dmitryuk)

fabpot · fabpot · commit f549e7a0cd1e · 2022-03-25T14:02:05.000+01:00
This PR was squashed before being merged into the 6.1 branch. Discussion ---------- [HttpFoundation] Propagate SessionNotFoundException in Request::getSession docBlock | Q | A | ------------- | --- | Branch? | 6.1 for features / 4.4, 5.4 or 6.0 for bug fixes  | Bug fix? | yes | New feature? | no | Deprecations? | no | License | MIT | Doc PR | Looks like [RequestStack::getSession](https://github.com/symfony/symfony/blob/6.1/src/Symfony/Component/HttpFoundation/RequestStack.php#L99) throws `SessionNotFoundException` exception in Docblock , but [Request](https://github.com/symfony/symfony/blob/6.1/src/Symfony/Component/HttpFoundation/Request.php#L694) don't. The PR adds missing Docblock to Request to solve misleadings. Commits ------- b887ff234b [HttpFoundation] Propagate SessionNotFoundException in Request::getSession docBlock
diff --git a/Request.php b/Request.php
@@ -690,6 +690,8 @@ public function get(string $key, mixed $default = null): mixed
 
     /**
      * Gets the Session.
+     *
+     * @throws SessionNotFoundException When session is not set properly
      */
     public function getSession(): SessionInterface
     {

Original file line number	Diff line number	Diff line change
`@@ -690,6 +690,8 @@ public function get(string $key, mixed $default = null): mixed`
`690`	`690`
`691`	`691`	`/**`
`692`	`692`	`* Gets the Session.`
	`693`	`+ *`
	`694`	`+ * @throws SessionNotFoundException When session is not set properly`
`693`	`695`	`*/`
`694`	`696`	`public function getSession(): SessionInterface`
`695`	`697`	`{`