From 7249d947ecf829ab12686bcdb85cde1f174379e9 Mon Sep 17 00:00:00 2001 From: Thomas Kuschan Date: Wed, 14 Jun 2023 14:30:14 +0200 Subject: [PATCH] Change JsonDebugException to Responsable interface In all other exceptions, when a Response is supposed to be returned, the Responsable interface is used instead of render. --- app/Exceptions/JsonDebugException.php | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/app/Exceptions/JsonDebugException.php b/app/Exceptions/JsonDebugException.php index 8acc19778..1fa52c4a2 100644 --- a/app/Exceptions/JsonDebugException.php +++ b/app/Exceptions/JsonDebugException.php @@ -4,8 +4,9 @@ namespace BookStack\Exceptions; use Exception; use Illuminate\Http\JsonResponse; +use Illuminate\Contracts\Support\Responsable; -class JsonDebugException extends Exception +class JsonDebugException extends Exception implements Responsable { protected array $data; @@ -22,7 +23,7 @@ class JsonDebugException extends Exception * 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. */ - public function render(): JsonResponse + public function toResponse($request): JsonResponse { $cleaned = mb_convert_encoding($this->data, 'UTF-8');