nextcloud / server

lib/private/AppFramework/Http/Dispatcher.php

Indentation +224 added lines, -224 removed lines

@@ -26,228 +26,228 @@
  * Class to dispatch the request to the middleware dispatcher
  */
 class Dispatcher {
-	/** @var MiddlewareDispatcher */
-	private $middlewareDispatcher;
-
-	/** @var Http */
-	private $protocol;
-
-	/** @var ControllerMethodReflector */
-	private $reflector;
-
-	/** @var IRequest */
-	private $request;
-
-	/** @var IConfig */
-	private $config;
-
-	/** @var ConnectionAdapter */
-	private $connection;
-
-	/** @var LoggerInterface */
-	private $logger;
-
-	/** @var IEventLogger */
-	private $eventLogger;
-
-	private ContainerInterface $appContainer;
-
-	/**
-	 * @param Http $protocol the http protocol with contains all status headers
-	 * @param MiddlewareDispatcher $middlewareDispatcher the dispatcher which
-	 *                                                   runs the middleware
-	 * @param ControllerMethodReflector $reflector the reflector that is used to inject
-	 *                                             the arguments for the controller
-	 * @param IRequest $request the incoming request
-	 * @param IConfig $config
-	 * @param ConnectionAdapter $connection
-	 * @param LoggerInterface $logger
-	 * @param IEventLogger $eventLogger
-	 */
-	public function __construct(
-		Http $protocol,
-		MiddlewareDispatcher $middlewareDispatcher,
-		ControllerMethodReflector $reflector,
-		IRequest $request,
-		IConfig $config,
-		ConnectionAdapter $connection,
-		LoggerInterface $logger,
-		IEventLogger $eventLogger,
-		ContainerInterface $appContainer,
-	) {
-		$this->protocol = $protocol;
-		$this->middlewareDispatcher = $middlewareDispatcher;
-		$this->reflector = $reflector;
-		$this->request = $request;
-		$this->config = $config;
-		$this->connection = $connection;
-		$this->logger = $logger;
-		$this->eventLogger = $eventLogger;
-		$this->appContainer = $appContainer;
-	}
-
-
-	/**
-	 * Handles a request and calls the dispatcher on the controller
-	 * @param Controller $controller the controller which will be called
-	 * @param string $methodName the method name which will be called on
-	 *                           the controller
-	 * @return array $array[0] contains the http status header as a string,
-	 *               $array[1] contains response headers as an array,
-	 *               $array[2] contains response cookies as an array,
-	 *               $array[3] contains the response output as a string,
-	 *               $array[4] contains the response object
-	 * @throws \Exception
-	 */
-	public function dispatch(Controller $controller, string $methodName): array {
-		$out = [null, [], null];
-
-		try {
-			// prefill reflector with everything that's needed for the
-			// middlewares
-			$this->reflector->reflect($controller, $methodName);
-
-			$this->middlewareDispatcher->beforeController($controller,
-				$methodName);
-
-			$databaseStatsBefore = [];
-			if ($this->config->getSystemValueBool('debug', false)) {
-				$databaseStatsBefore = $this->connection->getInner()->getStats();
-			}
-
-			$response = $this->executeController($controller, $methodName);
-
-			if (!empty($databaseStatsBefore)) {
-				$databaseStatsAfter = $this->connection->getInner()->getStats();
-				$numBuilt = $databaseStatsAfter['built'] - $databaseStatsBefore['built'];
-				$numExecuted = $databaseStatsAfter['executed'] - $databaseStatsBefore['executed'];
-
-				if ($numBuilt > 50) {
-					$this->logger->debug('Controller {class}::{method} created {count} QueryBuilder objects, please check if they are created inside a loop by accident.', [
-						'class' => get_class($controller),
-						'method' => $methodName,
-						'count' => $numBuilt,
-					]);
-				}
-
-				if ($numExecuted > 100) {
-					$this->logger->warning('Controller {class}::{method} executed {count} queries.', [
-						'class' => get_class($controller),
-						'method' => $methodName,
-						'count' => $numExecuted,
-					]);
-				}
-			}
-
-			// if an exception appears, the middleware checks if it can handle the
-			// exception and creates a response. If no response is created, it is
-			// assumed that there's no middleware who can handle it and the error is
-			// thrown again
-		} catch (\Exception $exception) {
-			$response = $this->middlewareDispatcher->afterException(
-				$controller, $methodName, $exception);
-		} catch (\Throwable $throwable) {
-			$exception = new \Exception($throwable->getMessage() . ' in file \'' . $throwable->getFile() . '\' line ' . $throwable->getLine(), $throwable->getCode(), $throwable);
-			$response = $this->middlewareDispatcher->afterException(
-				$controller, $methodName, $exception);
-		}
-
-		$response = $this->middlewareDispatcher->afterController(
-			$controller, $methodName, $response);
-
-		// depending on the cache object the headers need to be changed
-		$out[0] = $this->protocol->getStatusHeader($response->getStatus());
-		$out[1] = array_merge($response->getHeaders());
-		$out[2] = $response->getCookies();
-		$out[3] = $this->middlewareDispatcher->beforeOutput(
-			$controller, $methodName, $response->render()
-		);
-		$out[4] = $response;
-
-		return $out;
-	}
-
-
-	/**
-	 * Uses the reflected parameters, types and request parameters to execute
-	 * the controller
-	 * @param Controller $controller the controller to be executed
-	 * @param string $methodName the method on the controller that should be executed
-	 * @return Response
-	 */
-	private function executeController(Controller $controller, string $methodName): Response {
-		$arguments = [];
-
-		// valid types that will be cast
-		$types = ['int', 'integer', 'bool', 'boolean', 'float', 'double'];
-
-		foreach ($this->reflector->getParameters() as $param => $default) {
-			// try to get the parameter from the request object and cast
-			// it to the type annotated in the @param annotation
-			$value = $this->request->getParam($param, $default);
-			$type = $this->reflector->getType($param);
-
-			// Converted the string `'false'` to false when the controller wants a boolean
-			if ($value === 'false' && ($type === 'bool' || $type === 'boolean')) {
-				$value = false;
-			} elseif ($value !== null && \in_array($type, $types, true)) {
-				settype($value, $type);
-				$this->ensureParameterValueSatisfiesRange($param, $value);
-			} elseif ($value === null && $type !== null && $this->appContainer->has($type)) {
-				$value = $this->appContainer->get($type);
-			}
-
-			$arguments[] = $value;
-		}
-
-		$this->eventLogger->start('controller:' . get_class($controller) . '::' . $methodName, 'App framework controller execution');
-		try {
-			$response = \call_user_func_array([$controller, $methodName], $arguments);
-		} catch (\TypeError $e) {
-			// Only intercept TypeErrors occuring on the first line, meaning that the invocation of the controller method failed.
-			// Any other TypeError happens inside the controller method logic and should be logged as normal.
-			if ($e->getFile() === $this->reflector->getFile() && $e->getLine() === $this->reflector->getStartLine()) {
-				$this->logger->debug('Failed to call controller method: ' . $e->getMessage(), ['exception' => $e]);
-				return new Response(Http::STATUS_BAD_REQUEST);
-			}
-
-			throw $e;
-		}
-		$this->eventLogger->end('controller:' . get_class($controller) . '::' . $methodName);
-
-		if (!($response instanceof Response)) {
-			$this->logger->debug($controller::class . '::' . $methodName . ' returned raw data. Please wrap it in a Response or one of it\'s inheritors.');
-		}
-
-		// format response
-		if ($response instanceof DataResponse || !($response instanceof Response)) {
-			$format = $this->request->getFormat();
-
-			if ($format !== null) {
-				$response = $controller->buildResponse($response, $format);
-			} else {
-				$response = $controller->buildResponse($response);
-			}
-		}
-
-		return $response;
-	}
-
-	/**
-	 * @psalm-param mixed $value
-	 * @throws ParameterOutOfRangeException
-	 */
-	private function ensureParameterValueSatisfiesRange(string $param, $value): void {
-		$rangeInfo = $this->reflector->getRange($param);
-		if ($rangeInfo) {
-			if ($value < $rangeInfo['min'] || $value > $rangeInfo['max']) {
-				throw new ParameterOutOfRangeException(
-					$param,
-					$value,
-					$rangeInfo['min'],
-					$rangeInfo['max'],
-				);
-			}
-		}
-	}
+    /** @var MiddlewareDispatcher */
+    private $middlewareDispatcher;
+
+    /** @var Http */
+    private $protocol;
+
+    /** @var ControllerMethodReflector */
+    private $reflector;
+
+    /** @var IRequest */
+    private $request;
+
+    /** @var IConfig */
+    private $config;
+
+    /** @var ConnectionAdapter */
+    private $connection;
+
+    /** @var LoggerInterface */
+    private $logger;
+
+    /** @var IEventLogger */
+    private $eventLogger;
+
+    private ContainerInterface $appContainer;
+
+    /**
+     * @param Http $protocol the http protocol with contains all status headers
+     * @param MiddlewareDispatcher $middlewareDispatcher the dispatcher which
+     *                                                   runs the middleware
+     * @param ControllerMethodReflector $reflector the reflector that is used to inject
+     *                                             the arguments for the controller
+     * @param IRequest $request the incoming request
+     * @param IConfig $config
+     * @param ConnectionAdapter $connection
+     * @param LoggerInterface $logger
+     * @param IEventLogger $eventLogger
+     */
+    public function __construct(
+        Http $protocol,
+        MiddlewareDispatcher $middlewareDispatcher,
+        ControllerMethodReflector $reflector,
+        IRequest $request,
+        IConfig $config,
+        ConnectionAdapter $connection,
+        LoggerInterface $logger,
+        IEventLogger $eventLogger,
+        ContainerInterface $appContainer,
+    ) {
+        $this->protocol = $protocol;
+        $this->middlewareDispatcher = $middlewareDispatcher;
+        $this->reflector = $reflector;
+        $this->request = $request;
+        $this->config = $config;
+        $this->connection = $connection;
+        $this->logger = $logger;
+        $this->eventLogger = $eventLogger;
+        $this->appContainer = $appContainer;
+    }
+
+
+    /**
+     * Handles a request and calls the dispatcher on the controller
+     * @param Controller $controller the controller which will be called
+     * @param string $methodName the method name which will be called on
+     *                           the controller
+     * @return array $array[0] contains the http status header as a string,
+     *               $array[1] contains response headers as an array,
+     *               $array[2] contains response cookies as an array,
+     *               $array[3] contains the response output as a string,
+     *               $array[4] contains the response object
+     * @throws \Exception
+     */
+    public function dispatch(Controller $controller, string $methodName): array {
+        $out = [null, [], null];
+
+        try {
+            // prefill reflector with everything that's needed for the
+            // middlewares
+            $this->reflector->reflect($controller, $methodName);
+
+            $this->middlewareDispatcher->beforeController($controller,
+                $methodName);
+
+            $databaseStatsBefore = [];
+            if ($this->config->getSystemValueBool('debug', false)) {
+                $databaseStatsBefore = $this->connection->getInner()->getStats();
+            }
+
+            $response = $this->executeController($controller, $methodName);
+
+            if (!empty($databaseStatsBefore)) {
+                $databaseStatsAfter = $this->connection->getInner()->getStats();
+                $numBuilt = $databaseStatsAfter['built'] - $databaseStatsBefore['built'];
+                $numExecuted = $databaseStatsAfter['executed'] - $databaseStatsBefore['executed'];
+
+                if ($numBuilt > 50) {
+                    $this->logger->debug('Controller {class}::{method} created {count} QueryBuilder objects, please check if they are created inside a loop by accident.', [
+                        'class' => get_class($controller),
+                        'method' => $methodName,
+                        'count' => $numBuilt,
+                    ]);
+                }
+
+                if ($numExecuted > 100) {
+                    $this->logger->warning('Controller {class}::{method} executed {count} queries.', [
+                        'class' => get_class($controller),
+                        'method' => $methodName,
+                        'count' => $numExecuted,
+                    ]);
+                }
+            }
+
+            // if an exception appears, the middleware checks if it can handle the
+            // exception and creates a response. If no response is created, it is
+            // assumed that there's no middleware who can handle it and the error is
+            // thrown again
+        } catch (\Exception $exception) {
+            $response = $this->middlewareDispatcher->afterException(
+                $controller, $methodName, $exception);
+        } catch (\Throwable $throwable) {
+            $exception = new \Exception($throwable->getMessage() . ' in file \'' . $throwable->getFile() . '\' line ' . $throwable->getLine(), $throwable->getCode(), $throwable);
+            $response = $this->middlewareDispatcher->afterException(
+                $controller, $methodName, $exception);
+        }
+
+        $response = $this->middlewareDispatcher->afterController(
+            $controller, $methodName, $response);
+
+        // depending on the cache object the headers need to be changed
+        $out[0] = $this->protocol->getStatusHeader($response->getStatus());
+        $out[1] = array_merge($response->getHeaders());
+        $out[2] = $response->getCookies();
+        $out[3] = $this->middlewareDispatcher->beforeOutput(
+            $controller, $methodName, $response->render()
+        );
+        $out[4] = $response;
+
+        return $out;
+    }
+
+
+    /**
+     * Uses the reflected parameters, types and request parameters to execute
+     * the controller
+     * @param Controller $controller the controller to be executed
+     * @param string $methodName the method on the controller that should be executed
+     * @return Response
+     */
+    private function executeController(Controller $controller, string $methodName): Response {
+        $arguments = [];
+
+        // valid types that will be cast
+        $types = ['int', 'integer', 'bool', 'boolean', 'float', 'double'];
+
+        foreach ($this->reflector->getParameters() as $param => $default) {
+            // try to get the parameter from the request object and cast
+            // it to the type annotated in the @param annotation
+            $value = $this->request->getParam($param, $default);
+            $type = $this->reflector->getType($param);
+
+            // Converted the string `'false'` to false when the controller wants a boolean
+            if ($value === 'false' && ($type === 'bool' || $type === 'boolean')) {
+                $value = false;
+            } elseif ($value !== null && \in_array($type, $types, true)) {
+                settype($value, $type);
+                $this->ensureParameterValueSatisfiesRange($param, $value);
+            } elseif ($value === null && $type !== null && $this->appContainer->has($type)) {
+                $value = $this->appContainer->get($type);
+            }
+
+            $arguments[] = $value;
+        }
+
+        $this->eventLogger->start('controller:' . get_class($controller) . '::' . $methodName, 'App framework controller execution');
+        try {
+            $response = \call_user_func_array([$controller, $methodName], $arguments);
+        } catch (\TypeError $e) {
+            // Only intercept TypeErrors occuring on the first line, meaning that the invocation of the controller method failed.
+            // Any other TypeError happens inside the controller method logic and should be logged as normal.
+            if ($e->getFile() === $this->reflector->getFile() && $e->getLine() === $this->reflector->getStartLine()) {
+                $this->logger->debug('Failed to call controller method: ' . $e->getMessage(), ['exception' => $e]);
+                return new Response(Http::STATUS_BAD_REQUEST);
+            }
+
+            throw $e;
+        }
+        $this->eventLogger->end('controller:' . get_class($controller) . '::' . $methodName);
+
+        if (!($response instanceof Response)) {
+            $this->logger->debug($controller::class . '::' . $methodName . ' returned raw data. Please wrap it in a Response or one of it\'s inheritors.');
+        }
+
+        // format response
+        if ($response instanceof DataResponse || !($response instanceof Response)) {
+            $format = $this->request->getFormat();
+
+            if ($format !== null) {
+                $response = $controller->buildResponse($response, $format);
+            } else {
+                $response = $controller->buildResponse($response);
+            }
+        }
+
+        return $response;
+    }
+
+    /**
+     * @psalm-param mixed $value
+     * @throws ParameterOutOfRangeException
+     */
+    private function ensureParameterValueSatisfiesRange(string $param, $value): void {
+        $rangeInfo = $this->reflector->getRange($param);
+        if ($rangeInfo) {
+            if ($value < $rangeInfo['min'] || $value > $rangeInfo['max']) {
+                throw new ParameterOutOfRangeException(
+                    $param,
+                    $value,
+                    $rangeInfo['min'],
+                    $rangeInfo['max'],
+                );
+            }
+        }
+    }
 }

Spacing +5 added lines, -5 removed lines

@@ -147,7 +147,7 @@ 
 			$response = $this->middlewareDispatcher->afterException(
 				$controller, $methodName, $exception);
 		} catch (\Throwable $throwable) {
-			$exception = new \Exception($throwable->getMessage() . ' in file \'' . $throwable->getFile() . '\' line ' . $throwable->getLine(), $throwable->getCode(), $throwable);
+			$exception = new \Exception($throwable->getMessage().' in file \''.$throwable->getFile().'\' line '.$throwable->getLine(), $throwable->getCode(), $throwable);
 			$response = $this->middlewareDispatcher->afterException(
 				$controller, $methodName, $exception);
 		}
@@ -200,23 +200,23 @@ 
 			$arguments[] = $value;
 		}
 
-		$this->eventLogger->start('controller:' . get_class($controller) . '::' . $methodName, 'App framework controller execution');
+		$this->eventLogger->start('controller:'.get_class($controller).'::'.$methodName, 'App framework controller execution');
 		try {
 			$response = \call_user_func_array([$controller, $methodName], $arguments);
 		} catch (\TypeError $e) {
 			// Only intercept TypeErrors occuring on the first line, meaning that the invocation of the controller method failed.
 			// Any other TypeError happens inside the controller method logic and should be logged as normal.
 			if ($e->getFile() === $this->reflector->getFile() && $e->getLine() === $this->reflector->getStartLine()) {
-				$this->logger->debug('Failed to call controller method: ' . $e->getMessage(), ['exception' => $e]);
+				$this->logger->debug('Failed to call controller method: '.$e->getMessage(), ['exception' => $e]);
 				return new Response(Http::STATUS_BAD_REQUEST);
 			}
 
 			throw $e;
 		}
-		$this->eventLogger->end('controller:' . get_class($controller) . '::' . $methodName);
+		$this->eventLogger->end('controller:'.get_class($controller).'::'.$methodName);
 
 		if (!($response instanceof Response)) {
-			$this->logger->debug($controller::class . '::' . $methodName . ' returned raw data. Please wrap it in a Response or one of it\'s inheritors.');
+			$this->logger->debug($controller::class.'::'.$methodName.' returned raw data. Please wrap it in a Response or one of it\'s inheritors.');
 		}
 
 		// format response

lib/private/AppFramework/Utility/ControllerMethodReflector.php

Indentation +133 added lines, -133 removed lines

@@ -14,137 +14,137 @@
  * Reads and parses annotations from doc comments
  */
 class ControllerMethodReflector implements IControllerMethodReflector {
-	public $annotations = [];
-	private $types = [];
-	private $parameters = [];
-	private array $ranges = [];
-	private int $startLine = 0;
-	private string $file = '';
-
-	/**
-	 * @param object $object an object or classname
-	 * @param string $method the method which we want to inspect
-	 */
-	public function reflect($object, string $method) {
-		$reflection = new \ReflectionMethod($object, $method);
-		$this->startLine = $reflection->getStartLine();
-		$this->file = $reflection->getFileName();
-
-		$docs = $reflection->getDocComment();
-
-		if ($docs !== false) {
-			// extract everything prefixed by @ and first letter uppercase
-			preg_match_all('/^\h+\*\h+@(?P<annotation>[A-Z]\w+)((?P<parameter>.*))?$/m', $docs, $matches);
-			foreach ($matches['annotation'] as $key => $annotation) {
-				$annotation = strtolower($annotation);
-				$annotationValue = $matches['parameter'][$key];
-				if (str_starts_with($annotationValue, '(') && str_ends_with($annotationValue, ')')) {
-					$cutString = substr($annotationValue, 1, -1);
-					$cutString = str_replace(' ', '', $cutString);
-					$splitArray = explode(',', $cutString);
-					foreach ($splitArray as $annotationValues) {
-						[$key, $value] = explode('=', $annotationValues);
-						$this->annotations[$annotation][$key] = $value;
-					}
-					continue;
-				}
-
-				$this->annotations[$annotation] = [$annotationValue];
-			}
-
-			// extract type parameter information
-			preg_match_all('/@param\h+(?P<type>\w+)\h+\$(?P<var>\w+)/', $docs, $matches);
-			$this->types = array_combine($matches['var'], $matches['type']);
-			preg_match_all('/@psalm-param\h+(\?)?(?P<type>\w+)<(?P<rangeMin>(-?\d+|min)),\h*(?P<rangeMax>(-?\d+|max))>(\|null)?\h+\$(?P<var>\w+)/', $docs, $matches);
-			foreach ($matches['var'] as $index => $varName) {
-				if ($matches['type'][$index] !== 'int') {
-					// only int ranges are possible at the moment
-					// @see https://psalm.dev/docs/annotating_code/type_syntax/scalar_types
-					continue;
-				}
-				$this->ranges[$varName] = [
-					'min' => $matches['rangeMin'][$index] === 'min' ? PHP_INT_MIN : (int)$matches['rangeMin'][$index],
-					'max' => $matches['rangeMax'][$index] === 'max' ? PHP_INT_MAX : (int)$matches['rangeMax'][$index],
-				];
-			}
-		}
-
-		foreach ($reflection->getParameters() as $param) {
-			// extract type information from PHP 7 scalar types and prefer them over phpdoc annotations
-			$type = $param->getType();
-			if ($type instanceof \ReflectionNamedType) {
-				$this->types[$param->getName()] = $type->getName();
-			}
-
-			$default = null;
-			if ($param->isOptional()) {
-				$default = $param->getDefaultValue();
-			}
-			$this->parameters[$param->name] = $default;
-		}
-	}
-
-	/**
-	 * Inspects the PHPDoc parameters for types
-	 * @param string $parameter the parameter whose type comments should be
-	 *                          parsed
-	 * @return string|null type in the type parameters (@param int $something)
-	 *                     would return int or null if not existing
-	 */
-	public function getType(string $parameter) {
-		if (array_key_exists($parameter, $this->types)) {
-			return $this->types[$parameter];
-		}
-
-		return null;
-	}
-
-	public function getRange(string $parameter): ?array {
-		if (array_key_exists($parameter, $this->ranges)) {
-			return $this->ranges[$parameter];
-		}
-
-		return null;
-	}
-
-	/**
-	 * @return array the arguments of the method with key => default value
-	 */
-	public function getParameters(): array {
-		return $this->parameters;
-	}
-
-	/**
-	 * Check if a method contains an annotation
-	 * @param string $name the name of the annotation
-	 * @return bool true if the annotation is found
-	 */
-	public function hasAnnotation(string $name): bool {
-		$name = strtolower($name);
-		return array_key_exists($name, $this->annotations);
-	}
-
-	/**
-	 * Get optional annotation parameter by key
-	 *
-	 * @param string $name the name of the annotation
-	 * @param string $key the string of the annotation
-	 * @return string
-	 */
-	public function getAnnotationParameter(string $name, string $key): string {
-		$name = strtolower($name);
-		if (isset($this->annotations[$name][$key])) {
-			return $this->annotations[$name][$key];
-		}
-
-		return '';
-	}
-
-	public function getStartLine(): int {
-		return $this->startLine;
-	}
-
-	public function getFile(): string {
-		return $this->file;
-	}
+    public $annotations = [];
+    private $types = [];
+    private $parameters = [];
+    private array $ranges = [];
+    private int $startLine = 0;
+    private string $file = '';
+
+    /**
+     * @param object $object an object or classname
+     * @param string $method the method which we want to inspect
+     */
+    public function reflect($object, string $method) {
+        $reflection = new \ReflectionMethod($object, $method);
+        $this->startLine = $reflection->getStartLine();
+        $this->file = $reflection->getFileName();
+
+        $docs = $reflection->getDocComment();
+
+        if ($docs !== false) {
+            // extract everything prefixed by @ and first letter uppercase
+            preg_match_all('/^\h+\*\h+@(?P<annotation>[A-Z]\w+)((?P<parameter>.*))?$/m', $docs, $matches);
+            foreach ($matches['annotation'] as $key => $annotation) {
+                $annotation = strtolower($annotation);
+                $annotationValue = $matches['parameter'][$key];
+                if (str_starts_with($annotationValue, '(') && str_ends_with($annotationValue, ')')) {
+                    $cutString = substr($annotationValue, 1, -1);
+                    $cutString = str_replace(' ', '', $cutString);
+                    $splitArray = explode(',', $cutString);
+                    foreach ($splitArray as $annotationValues) {
+                        [$key, $value] = explode('=', $annotationValues);
+                        $this->annotations[$annotation][$key] = $value;
+                    }
+                    continue;
+                }
+
+                $this->annotations[$annotation] = [$annotationValue];
+            }
+
+            // extract type parameter information
+            preg_match_all('/@param\h+(?P<type>\w+)\h+\$(?P<var>\w+)/', $docs, $matches);
+            $this->types = array_combine($matches['var'], $matches['type']);
+            preg_match_all('/@psalm-param\h+(\?)?(?P<type>\w+)<(?P<rangeMin>(-?\d+|min)),\h*(?P<rangeMax>(-?\d+|max))>(\|null)?\h+\$(?P<var>\w+)/', $docs, $matches);
+            foreach ($matches['var'] as $index => $varName) {
+                if ($matches['type'][$index] !== 'int') {
+                    // only int ranges are possible at the moment
+                    // @see https://psalm.dev/docs/annotating_code/type_syntax/scalar_types
+                    continue;
+                }
+                $this->ranges[$varName] = [
+                    'min' => $matches['rangeMin'][$index] === 'min' ? PHP_INT_MIN : (int)$matches['rangeMin'][$index],
+                    'max' => $matches['rangeMax'][$index] === 'max' ? PHP_INT_MAX : (int)$matches['rangeMax'][$index],
+                ];
+            }
+        }
+
+        foreach ($reflection->getParameters() as $param) {
+            // extract type information from PHP 7 scalar types and prefer them over phpdoc annotations
+            $type = $param->getType();
+            if ($type instanceof \ReflectionNamedType) {
+                $this->types[$param->getName()] = $type->getName();
+            }
+
+            $default = null;
+            if ($param->isOptional()) {
+                $default = $param->getDefaultValue();
+            }
+            $this->parameters[$param->name] = $default;
+        }
+    }
+
+    /**
+     * Inspects the PHPDoc parameters for types
+     * @param string $parameter the parameter whose type comments should be
+     *                          parsed
+     * @return string|null type in the type parameters (@param int $something)
+     *                     would return int or null if not existing
+     */
+    public function getType(string $parameter) {
+        if (array_key_exists($parameter, $this->types)) {
+            return $this->types[$parameter];
+        }
+
+        return null;
+    }
+
+    public function getRange(string $parameter): ?array {
+        if (array_key_exists($parameter, $this->ranges)) {
+            return $this->ranges[$parameter];
+        }
+
+        return null;
+    }
+
+    /**
+     * @return array the arguments of the method with key => default value
+     */
+    public function getParameters(): array {
+        return $this->parameters;
+    }
+
+    /**
+     * Check if a method contains an annotation
+     * @param string $name the name of the annotation
+     * @return bool true if the annotation is found
+     */
+    public function hasAnnotation(string $name): bool {
+        $name = strtolower($name);
+        return array_key_exists($name, $this->annotations);
+    }
+
+    /**
+     * Get optional annotation parameter by key
+     *
+     * @param string $name the name of the annotation
+     * @param string $key the string of the annotation
+     * @return string
+     */
+    public function getAnnotationParameter(string $name, string $key): string {
+        $name = strtolower($name);
+        if (isset($this->annotations[$name][$key])) {
+            return $this->annotations[$name][$key];
+        }
+
+        return '';
+    }
+
+    public function getStartLine(): int {
+        return $this->startLine;
+    }
+
+    public function getFile(): string {
+        return $this->file;
+    }
 }