GarbuzIvan\LaravelGeneratorPackage\Builder\Builder

Complexity

Total Complexity

15

Size/Duplication

Total Lines	90
Duplicated Lines	0 %

Test Coverage

Coverage

84.38%

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getPackageName()	0	3	1
A	initFilter()	0	14	4
A	isIgnore()	0	9	5
A	getPackageVendor()	0	3	1
A	init()	0	12	3
A	__construct()	0	3	1

<?php

declare(strict_types=1);

namespace GarbuzIvan\LaravelGeneratorPackage\Builder;

use GarbuzIvan\LaravelGeneratorPackage\Configuration;

class Builder
{
    /**
     * @var Configuration
     */
    private Configuration $config;

    /**
     * @var string|null
     */
    private ?string $packageVendor = null;
    private ?string $packageName = null;

    /**
     * Builder constructor.
     * @param Configuration $config
     */
    public function __construct(Configuration $config)
    {
        $this->config = $config;
    }

    /**
     * @param string|null $package
     * @return bool
     */
    public function init(?string $package = null): bool
    {
        $this->initFilter($package);
        $configGenerator = $this->config->getGenerator();
        foreach ($configGenerator as $package) {
            if ($this->isIgnore($package['vendor'], $package['package'])) {
                continue;
            }
            /* Generation */
            var_dump($this->packageVendor, $this->packageName);

var_dump($this->packageVendor, $this->packageName) looks like debug code. Are you sure you do not want to remove it?

        }
        return true;
    }

    /**
     * @param string $vandor
     * @param string $name
     * @return bool
     */
    public function isIgnore(string $vandor, string $name): bool
    {
        if ($this->getPackageVendor() != null && $this->getPackageVendor() != $vandor) {

It seems like you are loosely comparing $this->getPackageVendor() of type null|string against null; this is ambiguous if the string can be empty. Consider using a strict comparison !== instead.

            return true;
        }
        if ($this->getPackageName() != null && $this->getPackageName() != $name) {

It seems like you are loosely comparing $this->getPackageName() of type null|string against null; this is ambiguous if the string can be empty. Consider using a strict comparison !== instead.

            return true;
        }
        return false;
    }

    /**
     * @param string|null $package
     * @return bool
     */
    public function initFilter(?string $package = null): bool
    {
        if (is_null($package)) {
            return true;
        }
        $package = str_replace('\\', '/', $package);
        if (!mb_stripos($package, '/')) {
            $this->packageVendor = $package;
            return true;
        }
        $match = explode('/', $package, 2);
        $this->packageVendor = $match[0];
        $this->packageName = mb_strlen(trim($match[1])) > 0 ? $match[1] : null;
        return true;
    }

    /**
     * @return string|null
     */
    public function getPackageVendor(): ?string
    {
        return $this->packageVendor;
    }

    /**
     * @return string|null
     */
    public function getPackageName(): ?string
    {
        return $this->packageName;
    }
}