Chinstrap\Core\Http\Handlers\PageHandler

Complexity

Total Complexity

5

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	__invoke()	0	14	4

<?php

declare(strict_types=1);

namespace Chinstrap\Core\Http\Handlers;

use Chinstrap\Core\Contracts\Sources\Source;
use Chinstrap\Core\Contracts\Views\Renderer;
use League\Route\Http\Exception\NotFoundException;
use Psr\Http\Message\ResponseInterface;
use Psr\Http\Message\ServerRequestInterface;

final class PageHandler
{
    private ResponseInterface $response;
    private Source $source;
    private Renderer $view;

    public function __construct(
        ResponseInterface $response,
        Source $source,
        Renderer $view
    ) {
        $this->response = $response;
        $this->source = $source;
        $this->view = $view;
    }

    /**
     * GET request to content page
     *
     * @param ServerRequestInterface $request
     * @param array{name: ?string} $args
     */
    public function __invoke(ServerRequestInterface $request, array $args): ResponseInterface
    {
        $name = isset($args['name']) ? $args['name'] : 'index';
        if (!$document = $this->source->find($name)) {

It seems like $name can also be of type null; however, parameter $name of Chinstrap\Core\Contracts\Sources\Source::find() does only seem to accept string, maybe add an additional type check?

            throw new NotFoundException('Page not found');
        }
        $data = $document->getFields();
        $data['content'] = $document->getContent();
        $layout = isset($data['layout']) ? $data['layout'] . '.html' : 'default.html';
        $response = $this->response->withAddedHeader(
            'Last-Modified',
            $document->getUpdatedAt()->format('D, d M Y H:i:s') . ' GMT'
        );
        return $this->view->render($response, $layout, $data);
    }
}