ProblemDetailsMiddleware::attachListener does also allow object&callable

[boesing]

Bug Report

Q	A
Version(s)	1.11.0

Summary

ProblemDetailsMiddleware::attachListener does also allow object&callable

Current behavior

Argument 1 of Mezzio\ProblemDetails\ProblemDetailsMiddleware::attachListener expects callable(Throwable, Psr\Http\Message\RequestInterface, Psr\Http\Message\ResponseInterface):void, but parent type Application\ProblemDetails\NonProblemResponseListener provided

How to reproduce

use Mezzio\ProblemDetails\Exception\ProblemDetailsExceptionInterface;
use Psr\Http\Message\ResponseInterface;
use Psr\Http\Message\ServerRequestInterface;
use Throwable;

final class Foo
{
   public function __invoke(Throwable $error, ServerRequestInterface $request, ResponseInterface $response): void
   {}
}

/** @var Mezzio\ProblemDetails\ProblemDetailsMiddleware $middleware */
$middleware = null;
$middleware->attachListener(new Foo());

Expected behavior

No psalm error.

[boesing]

With one of the more frequent versions of psalm I've added support for object&callable.

I'd recommend moving the callable type to a @psalm-type which allows you to re-use it like:

/** @psalm-type ListenerCallableType = callable(Throwable, RequestInterface, ResponseInterface):void */
class Middleware
{
    /**
     * @param ListenerCallableType|(object&ListenerCallableType) $listener
     */
    public function attachListener(callable $listener): void