luisgulo/BookStack
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Change JsonDebugException to Responsable interface

Browse source 
In all other exceptions, when a Response is supposed to be returned,
the Responsable interface is used instead of render.
This commit is contained in:
Thomas Kuschan 2023-06-14 14:30:14 +02:00
parent ec775aec02
commit 7249d947ec
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
app/Exceptions/JsonDebugException.php
View file

@ -4,8 +4,9 @@ namespace BookStack\Exceptions;
use Exception; use Exception;
use Illuminate\Http\JsonResponse; use Illuminate\Http\JsonResponse;
use Illuminate\Contracts\Support\Responsable;
class JsonDebugException extends Exception class JsonDebugException extends Exception implements Responsable
{ {
protected array $data; protected array $data;
@ -22,7 +23,7 @@ class JsonDebugException extends Exception
* Convert this exception into a response. * Convert this exception into a response.
* We add a manual data conversion to UTF8 to ensure any binary data is presentable as a JSON string. * We add a manual data conversion to UTF8 to ensure any binary data is presentable as a JSON string.
*/ */
public function render(): JsonResponse public function toResponse($request): JsonResponse
{ {
$cleaned = mb_convert_encoding($this->data, 'UTF-8'); $cleaned = mb_convert_encoding($this->data, 'UTF-8');