mirror/php-flarum
1
0
Fork 0
mirror of https://github.com/flarum/core.git synced 2025-07-21 08:41:17 +02:00
Code Issues Releases Wiki Activity

Support multiple error reporters

Browse Source 
The error handling middleware now expects an array of reporters.
Extensions can register new reporters in the container like this:

    use Flarum\Foundation\ErrorHandling\Reporter;

    $container->tag(NewReporter::class, Reporter::class);

Note that this is just an implementation detail and will be hidden
behind an extender.
This commit is contained in:
Franz Liedke
2019-08-10 11:03:44 +02:00
parent f73a39d3f4
commit 01c77b8e2a
6 changed files with 12 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
framework/core/src/Http/Middleware/HandleErrors.php
View File

@@ -13,7 +13,6 @@ namespace Flarum\Http\Middleware;
use Flarum\Foundation\ErrorHandling\Formatter;
use Flarum\Foundation\ErrorHandling\Registry;
use Flarum\Foundation\ErrorHandling\Reporter;
use Psr\Http\Message\ResponseInterface as Response;
use Psr\Http\Message\ServerRequestInterface as Request;
use Psr\Http\Server\MiddlewareInterface as Middleware;
@@ -33,15 +32,15 @@ class HandleErrors implements Middleware
protected $formatter;
/**
* @var Reporter
* @var \Flarum\Foundation\ErrorHandling\Reporter[]
*/
protected $reporter;
protected $reporters;
public function __construct(Registry $registry, Formatter $formatter, Reporter $reporter)
public function __construct(Registry $registry, Formatter $formatter, array $reporters)
{
$this->registry = $registry;
$this->formatter = $formatter;
$this->reporter = $reporter;
$this->reporters = $reporters;
}
/**
@@ -55,7 +54,9 @@ class HandleErrors implements Middleware
$error = $this->registry->handle($e);
if ($error->shouldBeReported()) {
$this->reporter->report($error);
foreach ($this->reporters as $reporter) {
$reporter->report($error);
}
}
return $this->formatter->format($error, $request);