ByTIC\Models\SmartProperties\Definitions\Definition\HasPropertiesNamespaces

Complexity

Total Complexity

14

Size/Duplication

Total Lines	88
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	initPropertiesNamespaces()	0	3	1
A	generatePropertyDirectory()	0	3	1
A	getPropertyItemsRootNamespace()	0	14	3
A	generateItemsDirectory()	0	13	3
A	generateManagerDirectory()	0	5	1
A	getPropertiesNamespaces()	0	7	2
A	generatePropertiesNamespaces()	0	12	3

<?php

namespace ByTIC\Models\SmartProperties\Definitions\Definition;

use Nip\Utility\Str;

/**
 * Trait HasPropertiesNamespaces
 * @package ByTIC\Models\SmartProperties\Definitions\Definition
 */
trait HasPropertiesNamespaces
{
    protected $propertiesNamespaces = null;

    /**
     * @return null
     */
    public function getPropertiesNamespaces()
    {
        if ($this->propertiesNamespaces === null) {
            $this->initPropertiesNamespaces();
        }

        return $this->propertiesNamespaces;
    }

    protected function initPropertiesNamespaces()
    {
        $this->propertiesNamespaces = $this->generatePropertiesNamespaces();
    }

    protected function generatePropertiesNamespaces()
    {
        $directories = (array)$this->generateItemsDirectory();
        $rootNamespace = $this->getPropertyItemsRootNamespace();

        $propertiesNamespaces = [];
        foreach ($directories as $namespace => $directory) {
            $namespace = Str::contains($namespace, '\\') ? $namespace : $rootNamespace;
            $namespace = trim($namespace, '\\');
            $propertiesNamespaces[$namespace.'\\'] = $directory;
        }
        return $propertiesNamespaces;
    }

    /**
     * @return string
     */
    protected function generateItemsDirectory()
    {
        $methodName = 'get' . $this->getName() . 'ItemsDirectory';

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

        if (method_exists($this->manager, $methodName)) {
            return $this->manager->$methodName();
        }

        $methodName = 'get' . Str::plural($this->getName()) . 'Directory';
        if (method_exists($this->manager, $methodName)) {
            return $this->manager->$methodName();
        }

        return $this->generateManagerDirectory() . DIRECTORY_SEPARATOR . $this->generatePropertyDirectory();
    }

    /**
     * @return string
     */
    protected function generateManagerDirectory()
    {
        $reflector = new \ReflectionObject($this->manager);

        return dirname($reflector->getFileName());
    }

    /**
     * @return string
     */
    protected function generatePropertyDirectory()
    {
        return $this->getLabel();

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

    }

    /**
     * @return string
     */
    protected function getPropertyItemsRootNamespace(): string
    {
        $manager = $this->getManager();

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

        $method = 'get' . $this->getName() . 'ItemsRootNamespace';
        if (method_exists($manager, $method)) {
            return $manager->{$method}();
        }

        $method = 'get' . $this->getName() . 'Namespace';
        if (method_exists($manager, $method)) {
            return $manager->{$method}();
        }

        return $manager->getModelNamespace() . $this->getLabel() . '\\';
    }
}