Konekt\Concord\Concerns\ReadsManifestFile

Complexity

Total Complexity

2

Size/Duplication

Total Lines	11
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	manifest()	0	7	2

<?php

declare(strict_types=1);

/**
 * Contains the ReadsManifestFile trait.
 *
 * @copyright   Copyright (c) 2021 Attila Fulop
 * @author      Attila Fulop
 * @license     MIT
 * @since       2021-11-18
 *
 */

namespace Konekt\Concord\Concerns;

use Konekt\Concord\Contracts\Convention;
use Konekt\Concord\Module\ManifestFile;

trait ReadsManifestFile
{
    private ?ManifestFile $manifestFile = null;

    protected function manifest(Convention $convention): ManifestFile
    {
        if (null === $this->manifestFile) {
            $this->manifestFile = ManifestFile::read($this->getBasePath() . '/' . $convention->manifestFile());

It seems like getBasePath() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        return $this->manifestFile;
    }
}