Add prev and next button to navigate through different pages

[bookstack] / app / Exceptions / Handler.php

diff --git a/app/Exceptions/Handler.php b/app/Exceptions/Handler.php
index a979072e23822f5fcc54b633696ef4115a86c5bf..57078522b748bbbf905b1836705a4d24c0695bbb 100644 (file)
--- a/app/Exceptions/Handler.php
+++ b/app/Exceptions/Handler.php
@@ -3,14 +3,14 @@
 namespace BookStack\Exceptions;
 
 use Exception;
 namespace BookStack\Exceptions;
 
 use Exception;

+use Illuminate\Auth\Access\AuthorizationException;

 use Illuminate\Auth\AuthenticationException;
 use Illuminate\Auth\AuthenticationException;

+use Illuminate\Database\Eloquent\ModelNotFoundException;
+use Illuminate\Foundation\Exceptions\Handler as ExceptionHandler;
+use Illuminate\Http\JsonResponse;

 use Illuminate\Http\Request;
 use Illuminate\Http\Request;

-use Illuminate\Pipeline\Pipeline;

 use Illuminate\Validation\ValidationException;
 use Illuminate\Validation\ValidationException;

-use Illuminate\Database\Eloquent\ModelNotFoundException;

 use Symfony\Component\HttpKernel\Exception\HttpException;
 use Symfony\Component\HttpKernel\Exception\HttpException;

-use Illuminate\Foundation\Exceptions\Handler as ExceptionHandler;
-use Illuminate\Auth\Access\AuthorizationException;

 use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
 
 class Handler extends ExceptionHandler
 use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
 
 class Handler extends ExceptionHandler


@@ -25,6 +25,7 @@ class Handler extends ExceptionHandler
         HttpException::class,
         ModelNotFoundException::class,
         ValidationException::class,
         HttpException::class,
         ModelNotFoundException::class,
         ValidationException::class,

+        NotFoundException::class,

     ];
 
     /**
     ];
 
     /**


@@ -33,6 +34,7 @@ class Handler extends ExceptionHandler
      *
      * @param  \Exception $e
      * @return mixed
      *
      * @param  \Exception $e
      * @return mixed

+     * @throws Exception

      */
     public function report(Exception $e)
     {
      */
     public function report(Exception $e)
     {


@@ -48,10 +50,17 @@ class Handler extends ExceptionHandler
      */
     public function render($request, Exception $e)
     {
      */
     public function render($request, Exception $e)
     {

+        if ($this->isApiRequest($request)) {
+            return $this->renderApiException($e);
+        }
+

         // Handle notify exceptions which will redirect to the
         // specified location then show a notification message.
         if ($this->isExceptionType($e, NotifyException::class)) {
         // Handle notify exceptions which will redirect to the
         // specified location then show a notification message.
         if ($this->isExceptionType($e, NotifyException::class)) {

-            session()->flash('error', $this->getOriginalMessage($e));
+            $message = $this->getOriginalMessage($e);
+            if (!empty($message)) {
+                session()->flash('error', $message);
+            }

             return redirect($e->redirectLocation);
         }
 
             return redirect($e->redirectLocation);
         }
 


@@ -65,28 +74,45 @@ class Handler extends ExceptionHandler
 
         // Handle 404 errors with a loaded session to enable showing user-specific information
         if ($this->isExceptionType($e, NotFoundHttpException::class)) {
 
         // Handle 404 errors with a loaded session to enable showing user-specific information
         if ($this->isExceptionType($e, NotFoundHttpException::class)) {

-            return $this->loadErrorMiddleware($request, function ($request) use ($e) {
-                $message = $e->getMessage() ?: trans('errors.404_page_not_found');
-                return response()->view('errors/404', ['message' => $message], 404);
-            });
+            return \Route::respondWithRoute('fallback');

         }
 
         return parent::render($request, $e);
     }
 
     /**
         }
 
         return parent::render($request, $e);
     }
 
     /**

-     * Load the middleware required to show state/session-enabled error pages.
-     * @param Request $request
-     * @param $callback
-     * @return mixed
+     * Check if the given request is an API request.
+     */
+    protected function isApiRequest(Request $request): bool
+    {
+        return strpos($request->path(), 'api/') === 0;
+    }
+
+    /**
+     * Render an exception when the API is in use.

      */
      */

-    protected function loadErrorMiddleware(Request $request, $callback)
+    protected function renderApiException(Exception $e): JsonResponse

     {
     {

-        $middleware = (\Route::getMiddlewareGroups()['web_errors']);
-        return (new Pipeline($this->container))
-            ->send($request)
-            ->through($middleware)
-            ->then($callback);
+        $code = $e->getCode() === 0 ? 500 : $e->getCode();
+        $headers = [];
+        if ($e instanceof HttpException) {
+            $code = $e->getStatusCode();
+            $headers = $e->getHeaders();
+        }
+
+        $responseData = [
+            'error' => [
+                'message' => $e->getMessage(),
+            ]
+        ];
+
+        if ($e instanceof ValidationException) {
+            $responseData['error']['validation'] = $e->errors();
+            $code = $e->status;
+        }
+
+        $responseData['error']['code'] = $code;
+        return new JsonResponse($responseData, $code, $headers);

     }
 
     /**
     }
 
     /**


@@ -95,9 +121,12 @@ class Handler extends ExceptionHandler
      * @param $type
      * @return bool
      */
      * @param $type
      * @return bool
      */

-    protected function isExceptionType(Exception $e, $type) {
+    protected function isExceptionType(Exception $e, $type)
+    {

         do {
         do {

-            if (is_a($e, $type)) return true;
+            if (is_a($e, $type)) {
+                return true;
+            }

         } while ($e = $e->getPrevious());
         return false;
     }
         } while ($e = $e->getPrevious());
         return false;
     }


@@ -107,7 +136,8 @@ class Handler extends ExceptionHandler
      * @param Exception $e
      * @return string
      */
      * @param Exception $e
      * @return string
      */

-    protected function getOriginalMessage(Exception $e) {
+    protected function getOriginalMessage(Exception $e)
+    {

         do {
             $message = $e->getMessage();
         } while ($e = $e->getPrevious());
         do {
             $message = $e->getMessage();
         } while ($e = $e->getPrevious());