jidaikobo-shibata / kontiki-framework

src/Handlers/TableHandler.php

<?php

namespace Jidaikobo\Kontiki\Handlers;

use Jidaikobo\Kontiki\Models\ModelInterface;
use Jidaikobo\Kontiki\Utils\MessageUtils;

class TableHandler
{
    private ?ModelInterface $model = null;
    private string $tableHtml = '';

    public function setModel(ModelInterface $model): void
    {
        $this->model = $model;
    }
    public function setHtml(string $html): void
    {
        $this->tableHtml = $html;
    }

    public function addErrors(array $errors): void
    {
        if (empty($errors)) {
            return;
        }
        $this->tableHtml = MessageUtils::errorHtml($errors, $this->model) . $this->tableHtml;

It seems like $this->model can also be of type null; however, parameter $model of Jidaikobo\Kontiki\Utils\MessageUtils::errorHtml() does only seem to accept Jidaikobo\Kontiki\Models\ModelInterface, maybe add an additional type check?

    }

    public function addSuccessMessages(array $successMessages): void
    {
        if (empty($successMessages)) {
            return;
        }
        $successMessage = join($successMessages);

The call to join() has too few arguments starting with array.

        $this->tableHtml = MessageUtils::alertHtml($successMessage) . $this->tableHtml;
    }

    public function getHtml(): string
    {
        return $this->tableHtml;
    }
}