TASK: Make SpecialResponsesSupport static to avoid accidentally acc…

…essing any state

While looking where the `$this->request` and  `$this->response` variables are used i found that they leak also into other objects. So a lot of things are stateful:

- $this->request
- $this->response
- $this->controllerContext
- $this->uriBuilder
- $this->arguments

with that many properties, its just a question of time until one of our utility traits like `SpecialResponsesSupport` accesses - accidentally e.g. in a bugfix - any of them and were doomed.

We should take reinforcements like making the trait's methods static

Neos.Flow/Classes/Mvc/Controller/SpecialResponsesSupport.php

@@ -21,7 +21,7 @@ trait SpecialResponsesSupport
      * @return never
      * @throws StopActionException
      */
-    final protected function responseThrowsStatus(int $statusCode, string $content = '', ?ActionResponse $response =  null): never
+    final protected static function responseThrowsStatus(int $statusCode, string $content = '', ?ActionResponse $response =  null): never
     {
         $response = $response ?? new ActionResponse;
 
@@ -30,7 +30,7 @@ final protected function responseThrowsStatus(int $statusCode, string $content =
             $response->setContent($content);
         }
 
-        $this->throwStopActionWithResponse($response, $content);
+        self::throwStopActionWithResponse($response, $content);
     }
 
     /**
@@ -43,13 +43,13 @@ final protected function responseThrowsStatus(int $statusCode, string $content =
      * @return ActionResponse
      * @throws StopActionException
      */
-    final protected function responseRedirectsToUri(UriInterface $uri, int $delay = 0, int $statusCode = 303, ?ActionResponse $response = null): ActionResponse
+    final protected static function responseRedirectsToUri(UriInterface $uri, int $delay = 0, int $statusCode = 303, ?ActionResponse $response = null): ActionResponse
     {
         $nextResponse = $response !== null ? clone $response : new ActionResponse();
 
         if ($delay < 1) {
             $nextResponse->setRedirectUri($uri, $statusCode);
-            $this->throwStopActionWithResponse($nextResponse, '');
+            self::throwStopActionWithResponse($nextResponse, '');
         }
 
         $nextResponse->setStatusCode($statusCode);
@@ -64,7 +64,7 @@ final protected function responseRedirectsToUri(UriInterface $uri, int $delay =
      * @return never
      * @throws StopActionException
      */
-    final protected function throwStopActionWithResponse(ActionResponse $response, string $details = ''): never
+    final protected static function throwStopActionWithResponse(ActionResponse $response, string $details = ''): never
     {
         throw StopActionException::createForResponse($response, $details);
     }
@@ -80,7 +80,7 @@ final protected function throwStopActionWithResponse(ActionResponse $response, s
      * @return never
      * @throws ForwardException
      */
-    final protected function forwardToRequest(ActionRequest $request): never
+    final protected static function forwardToRequest(ActionRequest $request): never
     {
         $nextRequest = clone $request;
         throw ForwardException::createForNextRequest($nextRequest, '');

Neos.FluidAdaptor/Classes/Core/Widget/AbstractWidgetController.php

@@ -46,7 +46,7 @@ abstract class AbstractWidgetController extends ActionController
      * Handles a request. The result output is returned by altering the given response.
      *
      * @param ActionRequest $request The request object
-     * @return ActionResponse $response The response, modified by this handler
+     * @return ActionResponse The response, modified by this handler
      * @throws WidgetContextNotFoundException
      * @throws InvalidActionVisibilityException
      * @throws InvalidArgumentTypeException