fix: Explicit @psalm-param handling

[provokateurin]

Fixes #32

Both @param and @psalm-param were already parsed, but if both are present only the first was used. Now it considers both if both are present.

[nickvergessen]

This breaks Talk again:

PHP Fatal error:  Uncaught Exception: Panic: BreakoutRoom#configureBreakoutRooms: Unable to resolve OpenAPI type for type 'PHPStan\PhpDocParser\Ast\Type\ConstTypeNode'
 in /home/nickv/Nextcloud/Repositories/openapi-extractor/src/Logger.php:40
Stack trace:
#0 /home/nickv/Nextcloud/Repositories/openapi-extractor/src/OpenApiType.php(211): OpenAPIExtractor\Logger::panic()
#1 /home/nickv/Nextcloud/Repositories/openapi-extractor/src/OpenApiType.php(98): OpenAPIExtractor\OpenApiType::resolve()
#2 /home/nickv/Nextcloud/Repositories/openapi-extractor/src/ControllerMethodParameter.php(21): OpenAPIExtractor\OpenApiType::resolve()
#3 /home/nickv/Nextcloud/Repositories/openapi-extractor/src/ControllerMethod.php(143): OpenAPIExtractor\ControllerMethodParameter->__construct()
#4 /home/nickv/Nextcloud/Repositories/openapi-extractor/generate-spec(351): OpenAPIExtractor\ControllerMethod::parse()
#5 {main}
  thrown in /home/nickv/Nextcloud/Repositories/openapi-extractor/src/Logger.php on line 40

I moved the "unsupported const" to psalm types to not break the openapi definition while the extractor does not support it and listed the plain list of integers on the normal type:
https://github.com/nextcloud/spreed/blob/4c5128dc22af05217df9e9d044515dbf921e05dc/lib/Controller/BreakoutRoomController.php#L62-L63

Can we catch the exception and ignore it if another type was given, until constants are understood?

[provokateurin]

Not great, but I guess the only way right now. Let's still print it as a warning though.

[nickvergessen]

Not great, but I guess the only way right now.

The problem of being in the middle of 2 evolutions :)

Let's still print it as a warning though.

Sounds okay for me atm

[provokateurin]

Testing this with Talk now shows some unimplemented types like non-negative-int 🎉

[nickvergessen]

So time to do the fixups and merge?

[provokateurin]

Ah too bad I forgot to remove the stacktrace from the log

[nickvergessen]

Ah too bad I forgot to remove the stacktrace from the log

We can remove it when we add the verbose flag