From ad7b8c3eda960a22959761d1157e84948bbc514d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?P=C5=91cze=20Bence?= <bence@pocze.ch>
Date: Tue, 18 Apr 2023 22:40:01 +0200
Subject: [PATCH] handle errors and exceptions in controllers

---
 src/Response/HttpResponse.php | 30 +++++++++++++++++++++++++++++-
 1 file changed, 29 insertions(+), 1 deletion(-)

diff --git a/src/Response/HttpResponse.php b/src/Response/HttpResponse.php
index a3496d4..9a0aed8 100644
--- a/src/Response/HttpResponse.php
+++ b/src/Response/HttpResponse.php
@@ -1,5 +1,7 @@
 <?php namespace SokoWeb\Response;
 
+use ErrorException;
+use Exception;
 use SokoWeb\Interfaces\Response\IRedirect;
 use SokoWeb\Interfaces\Response\IContent;
 use SokoWeb\Interfaces\Authentication\IAuthenticationRequired;
@@ -35,6 +37,8 @@ class HttpResponse
         string $requestMethod,
         string $requestUrl
     ) {
+        set_error_handler([$this, 'exceptionsErrorHandler']);
+
         $this->request = $request;
         $this->dbConnection = $dbConnection;
         $this->routeCollection = $routeCollection;
@@ -44,6 +48,11 @@ class HttpResponse
         $this->rawUrl = $requestUrl;
     }
 
+    public function exceptionsErrorHandler($severity, $message, $filename, $lineno)
+    {
+        throw new ErrorException($message, 0, $severity, $filename, $lineno);
+    }
+
     public function render(): void
     {
         $match = $this->routeCollection->match($this->method, $this->parsedUrl['path']);
@@ -81,7 +90,13 @@ class HttpResponse
         }
 
         $this->dbConnection->startTransaction();
-        $response = call_user_func([$controller, $handler[1]]);
+        try {
+            $response = call_user_func([$controller, $handler[1]]);
+        } catch (Exception $exception) {
+            $this->dbConnection->rollback();
+            $this->render500($exception);
+            return;
+        }
         $this->dbConnection->commit();
 
         if ($response instanceof IContent) {
@@ -114,4 +129,17 @@ class HttpResponse
         header('Content-Type: text/html; charset=UTF-8', true, 404);
         $content->render();
     }
+
+    private function render500(Exception $exception): void
+    {
+        if (empty($_ENV['DEV'])) {
+            $exceptionToPrint = null;
+        } else {
+            $exceptionToPrint = (string)$exception;
+        }
+
+        $content = new HtmlContent($this->appConfig['error500View'], ['exceptionToPrint' => $exceptionToPrint]);
+        header('Content-Type: text/html; charset=UTF-8', true, 500);
+        $content->render();
+    }
 }