matecat / simple-s3

src/Commands/CommandHandler.php

<?php
/**
 *  This file is part of the Simple S3 package.
 *
 * (c) Mauro Cassani<https://github.com/mauretto78>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 */

namespace Matecat\SimpleS3\Commands;

use Matecat\SimpleS3\Client;
use Matecat\SimpleS3\Components\Logger\CommandHandlerLogger;

abstract class CommandHandler implements CommandHandlerInterface
{
    /**
     * @var Client
     */
    protected Client $client;

    /**
     * @var CommandHandlerLogger|null
     */
    protected ?CommandHandlerLogger $commandHandlerLogger = null;

    /**
     * CommandHandler constructor.
     *
     * @param Client $client
     */
    public function __construct(Client $client)
    {
        $this->client = $client;
        if ($this->client->hasLogger()) {
            $this->commandHandlerLogger = new CommandHandlerLogger($client->getLogger());

It seems like $client->getLogger() can also be of type null; however, parameter $logger of Matecat\SimpleS3\Components\Logger\CommandHandlerLogger::__construct() does only seem to accept Psr\Log\LoggerInterface, maybe add an additional type check?

        }
    }
}