Merge pull request 'feature/database-transactions' (#9) from feature/database-transactions into master
All checks were successful
soko-web/pipeline/head This commit looks good
All checks were successful
soko-web/pipeline/head This commit looks good
Reviewed-on: #9
This commit is contained in:
commit
0b047b0bcc
@ -1,9 +1,12 @@
|
||||
<?php namespace SokoWeb\Response;
|
||||
|
||||
use ErrorException;
|
||||
use Exception;
|
||||
use SokoWeb\Interfaces\Response\IRedirect;
|
||||
use SokoWeb\Interfaces\Response\IContent;
|
||||
use SokoWeb\Interfaces\Authentication\IAuthenticationRequired;
|
||||
use SokoWeb\Interfaces\Authorization\ISecured;
|
||||
use SokoWeb\Interfaces\Database\IConnection;
|
||||
use SokoWeb\Interfaces\Request\IRequest;
|
||||
use SokoWeb\Response\Redirect;
|
||||
use SokoWeb\Response\HtmlContent;
|
||||
@ -14,6 +17,8 @@ class HttpResponse
|
||||
{
|
||||
private IRequest $request;
|
||||
|
||||
private IConnection $dbConnection;
|
||||
|
||||
private RouteCollection $routeCollection;
|
||||
|
||||
private array $appConfig;
|
||||
@ -26,12 +31,16 @@ class HttpResponse
|
||||
|
||||
public function __construct(
|
||||
IRequest $request,
|
||||
IConnection $dbConnection,
|
||||
RouteCollection $routeCollection,
|
||||
array $appConfig,
|
||||
string $requestMethod,
|
||||
string $requestUrl
|
||||
) {
|
||||
set_error_handler([$this, 'exceptionsErrorHandler']);
|
||||
|
||||
$this->request = $request;
|
||||
$this->dbConnection = $dbConnection;
|
||||
$this->routeCollection = $routeCollection;
|
||||
$this->appConfig = $appConfig;
|
||||
$this->method = strtolower($requestMethod);
|
||||
@ -39,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']);
|
||||
@ -75,7 +89,16 @@ class HttpResponse
|
||||
return;
|
||||
}
|
||||
|
||||
$response = call_user_func([$controller, $handler[1]]);
|
||||
$this->dbConnection->startTransaction();
|
||||
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) {
|
||||
header('Content-Type: ' . $response->getContentType() . '; charset=UTF-8');
|
||||
$response->render();
|
||||
@ -106,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();
|
||||
}
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user