kalanis\kw_mapper\Storage\File\FilesSingleton

Complexity

Total Complexity

7

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	1

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	2	1
A	getFilesAccessor()	0	6	2
A	setFileAccessor()	0	3	1
A	getInstance()	0	6	2
A	__clone()	0	2	1

<?php

namespace kalanis\kw_mapper\Storage\File;


use kalanis\kw_files\Interfaces\IProcessFiles;
use kalanis\kw_mapper\MapperException;


/**
 * Class FilesSingleton
 * @package kalanis\kw_mapper\Storage\File
 * Singleton to access files across the mappers
 */
class FilesSingleton
{
    protected static ?FilesSingleton $instance = null;
    private ?IProcessFiles $filesAccessor = null;

    public static function getInstance(): self
    {
        if (empty(static::$instance)) {
            static::$instance = new self();
        }
        return static::$instance;

The expression return static::instance could return the type null which is incompatible with the type-hinted return kalanis\kw_mapper\Storage\File\FilesSingleton. Consider adding an additional type-check to rule them out.

    }

    protected function __construct()
    {
    }

    /**
     * @codeCoverageIgnore why someone would run that?!
     */
    private function __clone()
    {
    }

    public function setFileAccessor(?IProcessFiles $files): void
    {
        $this->filesAccessor = $files;
    }

    /**
     * @throws MapperException
     * @return IProcessFiles
     */
    public function getFilesAccessor(): IProcessFiles
    {
        if (empty($this->filesAccessor)) {
            throw new MapperException('You must set the files accessor - instance of *IProcessFiles* - first!');
        }
        return $this->filesAccessor;
    }
}